-
Notifications
You must be signed in to change notification settings - Fork 392
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
Re-enable omrport tests on all platforms #6533
Conversation
jenkins build all |
jenkins build all |
Failing tests are disabled and tracked via: - eclipse#6542 Signed-off-by: Babneet Singh <sbabneet@ca.ibm.com>
jenkins build all |
jenkins build zos |
@dsouzai Can you help with reviewing/merging this PR? |
set(EXCLUDE_LIST "${EXCLUDE_LIST}-PortFileTest2.file_test20") | ||
set(EXCLUDE_LIST "${EXCLUDE_LIST}:PortFileTest2.file_test21") | ||
set(EXCLUDE_LIST "${EXCLUDE_LIST}:PortFileTest2.file_test22") | ||
set(EXCLUDE_LIST "${EXCLUDE_LIST}:PortFileTest2.file_test23") | ||
set(EXCLUDE_LIST "${EXCLUDE_LIST}:PortFileTest2.file_test24") |
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.
Each if/elseif
block corresponds to a platform/spec in order to keep the exclude-list easy to read. This leads to duplicates. Avoiding the duplicates will lead to extra logic, which will make it difficult to comprehend the exclude-list.
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.
LGTM; all tests pass and I see the excluded tests in the console output.
Failing tests are disabled, and they are tracked via: #6542.
Signed-off-by: Babneet Singh sbabneet@ca.ibm.com