Convert yield-based tests to nose2 @params decorator#6162
Draft
JanuszL wants to merge 7 commits intoNVIDIA:mainfrom
Draft
Convert yield-based tests to nose2 @params decorator#6162JanuszL wants to merge 7 commits intoNVIDIA:mainfrom
JanuszL wants to merge 7 commits intoNVIDIA:mainfrom
Conversation
Contributor
|
Skipped: This PR changes more files than the configured file change limit: ( |
Contributor
Author
|
!build |
Collaborator
|
CI MESSAGE: [41833773]: BUILD STARTED |
- Replaces generator-based test patterns (yield test_func, arg1, arg2) with nose2's @params decorator for parameterized testing. This makes tests compatible with nose2 test runner. - Converts yield statements to @params decorated test functions - Moves parameter generation to module-level lists/functions - Handles @raises decorator compatibility with @params - Updates QA test scripts to use nose2 instead of nosetests Signed-off-by: Janusz Lisiecki <jlisiecki@nvidia.com>
JanuszL
force-pushed
the
rework_nose2_renamed
branch
from
a62e0ad to
c181caf
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| import nose.tools as tools | ||
| from unittest import SkipTest # noqa: F401 | ||
| import unittest |
Check notice
Code scanning / CodeQL
Module is imported with 'import' and 'import from' Note test
Contributor
Author
|
!build |
Collaborator
|
CI MESSAGE: [41864947]: BUILD STARTED |
JanuszL
force-pushed
the
rework_nose2_renamed
branch
from
c181caf to
eb280d4
Compare
JanuszL
changed the title
JanuszL
force-pushed
the
rework_nose2_renamed
branch
from
eb280d4 to
b9608bb
Compare
Contributor
Author
|
!build |
Collaborator
|
CI MESSAGE: [41878636]: BUILD STARTED |
Collaborator
|
CI MESSAGE: [41864947]: BUILD FAILED |
Collaborator
|
CI MESSAGE: [41878636]: BUILD FAILED |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
with nose2's @params decorator for parameterized testing. This makes
tests compatible with nose2 test runner.
Category:
Other (e.g. Documentation, Tests, Configuration)
Description:
with nose2's @params decorator for parameterized testing. This makes
tests compatible with nose2 test runner.
Additional information:
Affected modules and functionalities:
Key points relevant for the review:
Tests:
Checklist
Documentation
DALI team only
Requirements
REQ IDs: N/A
JIRA TASK: N/A