Skip to content

tests: Add skips for feature- or configuration-dependent tests#3005

Merged
EmilienM merged 2 commits intogophercloud:masterfrom
stephenfin:more-skips
May 8, 2024
Merged

tests: Add skips for feature- or configuration-dependent tests#3005
EmilienM merged 2 commits intogophercloud:masterfrom
stephenfin:more-skips

Conversation

@stephenfin
Copy link
Copy Markdown
Contributor

@stephenfin stephenfin commented Mar 21, 2024

Allow us to run all tests in a module and have those that require specific extensions or configuration be skipped.

This is broken out from #3001 to help minimize the diff from that (rather beefy) PR

@github-actions github-actions bot added the semver:patch No API change label Mar 21, 2024
@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels Mar 21, 2024
@coveralls
Copy link
Copy Markdown

coveralls commented Mar 21, 2024

Coverage Status

coverage: 79.017%. remained the same
when pulling 79b1324 on stephenfin:more-skips
into ef98774 on gophercloud:master.

@github-actions github-actions bot added semver:major Breaking change and removed semver:patch No API change labels Apr 3, 2024
@stephenfin stephenfin mentioned this pull request Apr 3, 2024
@EmilienM
Copy link
Copy Markdown
Contributor

EmilienM commented May 8, 2024

please rebase so it triggers new CI jobs. Thanks

stephenfin added 2 commits May 8, 2024 18:02
This is a non-default (and somewhat esoteric) configuration. Skip the
tests unless explicitly provided.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Skip neutron tests if the relevant extension is not available in our
environment. We migrate existing variations of this to the new helper
and add some for tests that depend on neutron's 'fwaas_v2' extension
(though not for fwaas v1, since those tests are all already skipped and
should be removed in a separate change).

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
@github-actions github-actions bot added semver:patch No API change and removed semver:major Breaking change labels May 8, 2024
@stephenfin
Copy link
Copy Markdown
Contributor Author

please rebase so it triggers new CI jobs. Thanks

Done

@EmilienM EmilienM merged commit 59af460 into gophercloud:master May 8, 2024
@stephenfin stephenfin deleted the more-skips branch May 10, 2024 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:patch No API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants