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
[SCRAM] Update to support generating multiple tests using single command #7056
Conversation
test parameters:
|
A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch IB/CMSSW_12_0_X/master. @cmsbuild, @smuzaffar, @mrodozov, @iarspider can you please review it and eventually sign? Thanks. |
please test |
please test |
Pull request #7056 was updated. |
please test |
Pull request #7056 was updated. |
please test |
Pull request #7056 was updated. |
-1 Failed Tests: Build The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: BuildI found compilation error when building: >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-06-23-1100/src/FWCore/Framework/test/test_catch2_main.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-06-23-1100/src/FWCore/Framework/test/test_catch2notTP_ESRecordsToProxyIndices.cc >> Compiling /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-06-23-1100/src/FWCore/Framework/test/test_catch2notTP_MergeableRunProductMetadata.cc In file included from /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-06-23-1100/src/FWCore/Framework/test/test_catch2notTP_ESRecordsToProxyIndices.cc:1: /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-06-23-1100/src/FWCore/Framework/test/test_catch2notTP_ESRecordsToProxyIndices.cc: In function 'void ____C_A_T_C_H____T_E_S_T____0()': /data/cmsbld/jenkins/workspace/ib-run-pr-tests/CMSSW_12_0_X_2021-06-23-1100/src/FWCore/Framework/test/test_catch2notTP_ESRecordsToProxyIndices.cc:87:23: error: comparison of integer expressions of different signedness: 'unsigned int' and 'edm::ESProxyIndex::Value_t' {aka 'int'} [-Werror=sign-compare] 87 | REQUIRE(index == r2pi.indexInRecord(pr.first, dk).value()); | ~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ cc1plus: some warnings being treated as errors gmake: *** [tmp/slc7_amd64_gcc900/src/FWCore/Framework/test/TestFWCoreFrameworkCatch2notTP/test_catch2notTP_ESRecordsToProxyIndices.cc.o] Error 1 >> Building binary TestFWCoreFrameworkCatch2notTP |
please test with cms-sw/cmssw#34230 for CMSSW_12_0_X |
please test with cms-sw/cmssw#34230 |
Pull request #7056 was updated. |
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_12_0_X/master IBs after it passes the integration tests. This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
-1 Failed Tests: UnitTests Unit TestsI found errors in the following unit tests: ---> test test_PixelBaryCentreTool had ERRORS Comparison SummaryThe workflows 140.53 have different files in step1_dasquery.log than the ones found in the baseline. You may want to check and retrigger the tests if necessary. You can check it in the "files" directory in the results of the comparisons Summary:
|
SCRAMV3 now supports to generate multiple unit tests using single command. The format to define multiple tests is
e.g.