-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
run tests in separate tests #41055
run tests in separate tests #41055
Conversation
please test |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-41055/34636
|
A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for master. It involves the following packages:
@pmandrik, @emanueleusai, @tvami, @saumyaphor4252, @syuvivida, @rvenditti, @micsucmed, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-a53bc2/31280/summary.html Comparison SummarySummary:
|
@perrotta @rappoccio, as this is just a technical change so can we get this in 11h00 IB? |
+alca |
I have merged cms-sw/cmsdist#8385 ( to enable all unit tests to use separate test dir) so this PR is not needed any more. Once new IB based on cms-sw/cmsdist#8385 is available then I will clode the PR |
IB based on cms-sw/cmsdist#8383 is available and unit tests passed. So this PR is not needed any more |
This is a quick fix for unittests failing in IBs. Both tests generate same output files name [a] which also conflict with
out
directory generated by another test. For now I propose to run these tests in their own separate directory. Once we enableUSE_UNITTEST_DIR
for cmssw then these changes can be reverted.[a]