-
Notifications
You must be signed in to change notification settings - Fork 3
Add option to mark integration tests CTAO use case and requirements. #1405
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
Change how markers are attached to parametrized tests
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
A minor note, as I don't know if this is relevant to you, in general, the use case to test mapping is many-to-many. So you might want to make the id fields lists instead of a single string and add a loop to add the markers. |
@maxnoe - thanks for this. This will be easy to extend - I however can't think a case right now where this would be the case. I suggest to review this PR and add the extension to list when needed. |
|
||
|
||
@pytest.fixture(scope="session", params=test_configs) | ||
def test_config(request): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe consider adding a short docstring.
Allow to mark integration tests in their use case config with a corresponding CTAO use case or requirement ID, see pytest-requirements package description.
application_workflow.metaschema
:TEST_REQUIREMENT
andTEST_USE_CASE
test_applications_from_config
to mark the tests.TEST_USE_CASE
marker (this PR is about adding the functionality; so don't worry that this is a random use case ID).