AthenaCommon: Update for py 3.11.
In py 3.11, the format that unittest uses to print test names changes. Update the reference files accordingly. If we're using an older python version, monkey-patch unittest to do what 3.11 does.
Merge request reports
Activity
This merge request affects 1 package:
- Control/AthenaCommon
This merge request affects 10 files:
- Control/AthenaCommon/share/AthAppMgrUnitTests.ref
- Control/AthenaCommon/share/ConfigurableUnitTests.ref
- Control/AthenaCommon/share/JobOptionsUnitTests.ref
- Control/AthenaCommon/share/JobPropertiesUnitTests.ref
- Control/AthenaCommon/share/KeyStoreUnitTests.ref
- Control/AthenaCommon/test/AthAppMgrUnitTests.py
- Control/AthenaCommon/test/ConfigurableUnitTests.py
- Control/AthenaCommon/test/JobOptionsUnitTests.py
- Control/AthenaCommon/test/JobPropertiesUnitTests.py
- Control/AthenaCommon/test/KeyStoreUnitTests.py
added Core analysis-review-required main review-pending-level-1 labels
CI Result SUCCESS (hash d655efa9)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 75814]It was only this package.
However, i realize now that that was because the tests in this package (as opposed to others) had their verbosity levels increased from the default. Otherwise, unittest doesn't trace each individual test.
So now i'm thinking that a better approach is to just turn off the extra verbosity. Will close this and open a separate MR with that change.
Changes look good from L1. However there's still a thread open. Maybe do you comment on it, @ssnyder ? Or should we directly resolve it?
Thanks a lot!
Francisco Sili (L1)
added review-user-action-required label and removed review-pending-level-1 label