Ticket #8610 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

System test for MuonLoad

Reported by: Arturs Bekasovs Owned by: Arturs Bekasovs
Priority: major Milestone: Release 3.2
Component: Muon Keywords: Maintenance
Cc: Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Some of it's features are currently tested using unit tests, but a proper system test will help to make sure it does what it should.

Change History

comment:1 Changed 7 years ago by Arturs Bekasovs

  • Summary changed from [Muon] System test for MuonLoad to System test for MuonLoad
  • Milestone changed from Backlog to Release 3.2

comment:2 Changed 7 years ago by Arturs Bekasovs

  • Blocking 8985 added

comment:3 Changed 7 years ago by Arturs Bekasovs

  • Blocking 8702 added

(In #8702) Should probably wait for system test to be added be fore modifying MuonLoad.

comment:4 Changed 7 years ago by Arturs Bekasovs

  • Status changed from new to inprogress

Refs #8610. Add the system test.

Changeset: 3812365ef000a940178532194d40b69f6dd3da4f

comment:5 Changed 7 years ago by Arturs Bekasovs

Tester:

The system test is quite simple. I just run the MuonLoad setting all the possible parameters, and compare it to the workspace produced by MuonAnalysis interface using the same parameters. It is assumed that MuonAnalysis produces the right results.

So check that the test script is sensible and is cleaning after itself. Check that it is passing all the test builds of the develop branch.

If passed, merge the branch in systemtests repository only.

comment:6 Changed 7 years ago by Arturs Bekasovs

  • Blocking 8702 removed

(In #8702) The system test for MuonLoad should now be in the develop branch of the systemtests repository, so as an additional check please verify that it is passing on test_*_develop builds.

comment:7 Changed 7 years ago by Arturs Bekasovs

  • Blocking 8985 removed

(In #8985) The system test is in the develop, which is run against the current develop of main repo. No need to wait until it gets to master.

comment:6 Changed 7 years ago by Arturs Bekasovs

  • Status changed from inprogress to verify
  • Resolution set to fixed
  • Blocking 8702, 8985 added

comment:7 Changed 7 years ago by Arturs Bekasovs

  • Blocking 8985 removed

(In #8985) Trac.

comment:8 Changed 7 years ago by Arturs Bekasovs

  • Blocking 8702 removed

(In #8702) Trac.

comment:9 Changed 7 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:12 Changed 7 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8610_muon_load_system_test'

Full changeset: ​0f9e4e4178369b9df0b600b1d78d89d7c98ec2d8

comment:13 Changed 7 years ago by Anders Markvardsen

I should mention all test builds passed and systemtests merged

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​9454

Note: See TracTickets for help on using tickets.