#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
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.
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.