Cleanup empty unit test references
- delete unused (empty) unit test reference files
- alternatively where applicable use
nopost.sh
instead of providing an empty reference file
Merge request reports
Activity
This merge request affects 7 packages:
- Control/AthToolSupport/AsgTools
- Control/RootUtils
- Event/xAOD/xAODCore
- LArCalorimeter/LArRecUtils
- PhysicsAnalysis/D3PDTools/RootCoreUtils
- Trigger/TriggerCommon/TrigEDMConfig
- Trigger/TriggerCommon/TriggerMenuMT
Affected files list will not be printed in this case
Adding @sutt ,@hrussell ,@akraszna ,@krumnack ,@pavol ,@okuprash ,@ssnyder ,@bernius as watchers
- Resolved by Attila Krasznahorkay
Empty isn't the same as unused. That would only be the case if a test were ignoring everything (.*). We had some of those, but from a quick look here, that's not most of these, at least.
CI Result FAILURE (hash bfbe15ed)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional 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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 48163]added 47 commits
-
bfbe15ed...6536f454 - 44 commits from branch
atlas:master
- 4d204f79 - xAODCore: delete empty reference files and use nopost.sh
- 1d97ef10 - Delete unused empty unit test reference file
- e2c70553 - RootCoreUtils: delete empty test references
Toggle commit list-
bfbe15ed...6536f454 - 44 commits from branch
This merge request affects 6 packages:
- Control/AthToolSupport/AsgTools
- Control/RootUtils
- Event/xAOD/xAODCore
- LArCalorimeter/LArRecUtils
- PhysicsAnalysis/D3PDTools/RootCoreUtils
- Trigger/TriggerCommon/TrigEDMConfig
Affected files list will not be printed in this case
Adding @sutt ,@akraszna ,@krumnack ,@pavol ,@okuprash ,@ssnyder ,@bernius as watchers
CI Result SUCCESS (hash e2c70553)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional 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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 48192]added review-approved label and removed review-pending-level-1 label
added review-user-action-required label
added 44 commits
-
e2c70553...5e9af3c7 - 41 commits from branch
atlas:master
- d29d59db - xAODCore: delete empty reference files and use nopost.sh
- 97e0fdf9 - Delete unused empty unit test reference file
- 4896a147 - RootCoreUtils: delete empty test references
Toggle commit list-
e2c70553...5e9af3c7 - 41 commits from branch
This merge request affects 6 packages:
- Control/AthToolSupport/AsgTools
- Control/RootUtils
- Event/xAOD/xAODCore
- LArCalorimeter/LArRecUtils
- PhysicsAnalysis/D3PDTools/RootCoreUtils
- Trigger/TriggerCommon/TrigEDMConfig
Affected files list will not be printed in this case
Adding @sutt ,@akraszna ,@krumnack ,@pavol ,@okuprash ,@ssnyder ,@bernius as watchers
added review-pending-level-1 label and removed review-approved review-user-action-required labels
CI Result SUCCESS (hash 4896a147)Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon externals cmake make required tests optional 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
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 48247]added review-approved label and removed review-pending-level-1 label
mentioned in commit bf11fa24