Skip to content
Snippets Groups Projects

TrigHLTJetHypoUnitTests: Disable thread-safety checking for gtest tests.

Merged TrigHLTJetHypoUnitTests: Disable thread-safety checking for gtest tests.
ssnyder/athena:thread.TrigHLTJetHypoUnitTests-20240131 into main
Merged Scott Snyder requested to merge ssnyder/athena:thread.TrigHLTJetHypoUnitTests-20240131 into main

Disable thread-safety checking for unit tests that use googletest. Fixes warnings seen when (disabled by default) warnings about calls to unchecked code are enabled.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • ATLAS Robot
    ATLAS Robot @atlasbot ·
    Developer

    This merge request affects 1 package:

    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests

    This merge request affects 20 files:

    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/CMakeLists.txt
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/CombinationsGeneratorTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/ConditionFilterTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/JetGroupProductFactoryTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/JetGroupProductTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/JetGroupReducerTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/JetGroupRegisterTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/JetGroupSingleCliqueTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/JetGroupUnionTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/JetStreamerTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/JetTrigTimerTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/MultiFilterTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/PassThroughFilterTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/ProductGenTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/RangeFilterTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/TLorentzVectorFactoryTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/TreeTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/all_tests.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/make_jetstreamTest.cxx
    • Trigger/TrigHypothesis/TrigHLTJetHypoUnitTests/tests/xAODJetCollectorTest.cxx

    Adding @cantel ,@ayana ,@peter ,@valentem ,@miochoa as watchers

  • ATLAS Robot added JetEtmiss Trigger TriggerJet main review-pending-level-1 labels

    added JetEtmiss Trigger TriggerJet main review-pending-level-1 labels

  • ATLAS Robot
    ATLAS Robot @atlasbot ·
    Developer

    :white_check_mark: CI Result SUCCESS (hash bbdbfaac)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3866]

  • James Beacham
    James Beacham @jbeacham ·
    Developer

    The changes are straightforward and there are no issues with the CI. Approving from L1.

  • James Beacham added review-approved label and removed review-pending-level-1 label

    added review-approved label and removed review-pending-level-1 label

  • Adam Edward Barton merged

    merged

  • Adam Edward Barton mentioned in commit 120156d9

    mentioned in commit 120156d9

Please register or sign in to reply