Ticket #3795 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Clean up old code, especially disabled test code

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: minor Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Jean Bilheux

Description

Search for my xtest() methods in tests and see which can be removed.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

Describe any of my disabled test methods that do not already have a justification for being disabled.

comment:2 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:3 Changed 9 years ago by Janik Zikovsky

In [14826]:

Refs #3795: Re-enabled a test in Dspacemaptocal

comment:4 Changed 9 years ago by Janik Zikovsky

In [14828]:

Refs #3795: Removed old memory leak test. Re-enabled a test that might fail on windows?

comment:5 Changed 9 years ago by Janik Zikovsky

In [14829]:

Refs #3795: Tests removed/re-enabled

comment:6 Changed 9 years ago by Janik Zikovsky

In [14830]:

Refs #3795: Cleaning up tests

comment:7 Changed 9 years ago by Janik Zikovsky

In [14831]:

Refs #3795

comment:8 Changed 9 years ago by Janik Zikovsky

In [14832]:

Refs #3795

comment:9 Changed 9 years ago by Janik Zikovsky

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

In [14868]:

Fixes #3795: Cleaned up some old test code. For testers: I just added comments to disabled tests (void xtest) and removed the really useless ones.

comment:10 Changed 9 years ago by Jean Bilheux

  • Status changed from verify to verifying
  • Tester set to Jean Bilheux

comment:11 Changed 9 years ago by Jean Bilheux

  • Status changed from verifying to closed

ok

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4642

Note: See TracTickets for help on using tickets.