Ticket #6432 (closed: fixed)
Convert unit test for IntegrateEllipsoids to a system test.
Reported by: | Dennis Mikkelson | Owned by: | Ruth Mikkelson |
---|---|---|---|
Priority: | major | Milestone: | Release 2.6 |
Component: | Diffraction | Keywords: | |
Cc: | petersonpf@…, mikkelsonr@… | Blocked By: | |
Blocking: | Tester: | Mathieu Doucet |
Description
The unit test for the IntegrateEllipsoids requires a 4 MB NeXus file and runs in about 10 sec. 4 MB exceeds the size limit for unit test data, and 10 sec is a relatively long running unit test. Russell suggested that this should be made a system test, instead of a unit test.
Change History
comment:2 Changed 7 years ago by Dennis Mikkelson
- Status changed from new to assigned
- Owner changed from Dennis Mikkelson to Ruth Mikkelson
comment:4 Changed 7 years ago by Ruth Mikkelson
Refs #6432 Added test and data
Changeset: f553f337277423ba4d50b23c578123e76c11232f
comment:5 Changed 7 years ago by Ruth Mikkelson
- Status changed from accepted to verify
- Resolution set to fixed
comment:6 Changed 7 years ago by Mathieu Doucet
- Status changed from verify to verifying
- Tester set to Mathieu Doucet
comment:7 Changed 7 years ago by Mathieu Doucet
- Status changed from verifying to reopened
- Resolution fixed deleted
The old unit test code should be removed.
comment:9 Changed 7 years ago by Ruth Mikkelson
- Status changed from accepted to verify
- Resolution set to fixed
The old unit test is in mantid. This code is in a branch off of systemtests. The change to mantid will be done with another ticket in a branch off of mantid.
comment:10 Changed 7 years ago by Mathieu Doucet
- Status changed from verify to verifying
Sound good. The new ticket is #7296.
comment:14 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 7278
Moved to r2.6 at the end of r2.5