Ticket #6432 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

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:1 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:2 Changed 7 years ago by Dennis Mikkelson

  • Status changed from new to assigned
  • Owner changed from Dennis Mikkelson to Ruth Mikkelson

comment:3 Changed 7 years ago by Ruth Mikkelson

  • Status changed from assigned to accepted

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:8 Changed 7 years ago by Ruth Mikkelson

  • Status changed from reopened to accepted

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:11 Changed 7 years ago by Mathieu Doucet

  • Status changed from verifying to closed

comment:12 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:13 Changed 7 years ago by Nick Draper

  • Component changed from Framework to Diffraction

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7278

Note: See TracTickets for help on using tickets.