Add unit test for FEDRawDataCollection format#41494
Conversation
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-41494/35362
|
|
A new Pull Request was created by @wddgit (W. David Dagenhart) for master. It involves the following packages:
@cmsbuild, @emeschi, @smorovic can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
please test FYI @makortel |
|
abort test |
|
please test with https://github.com/cms-data/DataFormats-FEDRawData#1 |
|
please test with cms-data/DataFormats-FEDRawData#1 |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-311e19/32323/summary.html 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: Comparison SummarySummary:
|
|
+1 |
|
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
|
+1
|
PR description:
Add a new unit test for the FEDRawDataCollection data format. This generates a data file containing a FEDRawDataCollection object with known content. Then it reads it. It verifies that when we read it we obtain values that match the known written values for all the data fields in the object. In particular all containers have content so all contained types are also read.
It also reads the old files in the DataFormats/FEDRawData data repository which are listed in the shell script. The plan is that each time the data format of FEDRawDataCollection is modified a file will added.
PR validation:
This only adds a unit test in DataFormats/FEDRawData which passes. It shouldn't affect anything else.