Opened 9 years ago

Closed 7 years ago

Last modified 5 years ago

#1140 closed defect (duplicate)

[trigger-sim] test coverage low

Reported by: Hans Dembinski Owned by: Alex Olivas
Priority: normal Milestone:
Component: combo simulation Keywords:
Cc:

Description

despite having a considerable number of tests, the coverage is low in trigger-sim, especially in

trigger-sim/private/trigger-sim/modules
trigger-sim/private/trigger-sim/algorithms/CylinderTriggerAlgorithm.cxx

Change History (4)

comment:1 Changed 8 years ago by Alex Olivas

  • Milestone set to IceSim 5.0

comment:2 Changed 8 years ago by Alex Olivas

  • Milestone changed from IceSim 5.0 to IceSim 5.1

The test coverage is considerably better now, but still not great, so I'm going to push to the next milestone. Also trigger-sim will likely go through a major re-factor and so would most tests I'd write today. Pushing to the next milestone.

comment:3 Changed 7 years ago by Alex Olivas

  • Resolution set to duplicate
  • Status changed from new to closed

Folded into #1933.

comment:4 Changed 5 years ago by Alex Olivas

  • Milestone IceSim 5.1 deleted

Milestone IceSim? 5.1 deleted

Note: See TracTickets for help on using tickets.