Ticket #7296 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Clean up unused unit test

Reported by: Mathieu Doucet Owned by: Dennis Mikkelson
Priority: major Milestone: Release 2.6
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Mathieu Doucet

Description

To follow up on #6432, the old unit tests in the mantid repo should be removed. The unused unit test code is still there and the test is only commented out in CMakeList.txt.

Change History

comment:1 Changed 7 years ago by Dennis Mikkelson

  • Status changed from new to accepted

comment:2 Changed 7 years ago by Dennis Mikkelson

RE 7296_Clean_up_unused_unit_test

Removed unit test IntegrateEllipsoidsTest.h, which was not allowed since it required a data file that was too large. This was converted to a system test under ticket 6432.

refs #7296

Changeset: 4c0fae84fba370566acadd9f6be81cdc0a937a58

comment:3 Changed 7 years ago by Dennis Mikkelson

  • Status changed from accepted to verify
  • Resolution set to fixed

Note to tester:

The branch name is: feature/7296_Clean_up_unused_unit_test

Not much to test, just verify that the file Code/Mantid/Framework/MDEvents/test/IntegrateEllipsoidsTest.h is not there and that the CMakeLists.txt file no longer has a reference to that test, commented out.

comment:4 Changed 7 years ago by Mathieu Doucet

  • Status changed from verify to verifying
  • Tester set to Mathieu Doucet

comment:5 Changed 7 years ago by Mathieu Doucet

  • Status changed from verifying to closed

comment:6 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​8142

Note: See TracTickets for help on using tickets.