Ticket #1915 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Eliminate the use of TOPAZ_900.nxs in unit tests

Reported by: Russell Taylor Owned by: Vickie Lynch
Priority: major Milestone: Iteration 27
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

This is part of the drive to reduce the size of our subversion repository and elimate the reading of data files in unit tests unless absolutely necessary.

At 27MB, TOPAZ_900.nxs is the eighth largest file currently used in unit tests.

Their use features in:

  • LoadInstrumentFromSNSNexusTest
  • LoadLogsFromSNSNexusTest
  • LoadSNSNexusTest

Tests that have to load a file (i.e. those testing file loading algorithms) should switch to using a much smaller file. Others should create fictional data by using routines like WorkspaceCreationHelper.hh. If an algorithm under test requires the instrument geometry in any way then you may have to wait for completion of ticket #1899.

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 26 to Iteration 27

Bulk move of tickets to iteration 27, if your ticket is essential for Iteration 26 then move it back.

comment:2 Changed 10 years ago by Vickie Lynch

  • Status changed from new to accepted

comment:3 Changed 10 years ago by Vickie Lynch

(In [8715]) Refs #1915 Changed TOPAZ_900 to CNCS_7860 to eliminate large NeXus file

comment:4 Changed 10 years ago by Vickie Lynch

(In [8716]) Refs #1915 Changed TOPAZ_900 to CNCS_7860 to eliminate large NeXus file

comment:5 Changed 10 years ago by Vickie Lynch

(In [8717]) Refs #1915 Delete TOPAZ_900.nxs stored twice since no longer needed for tests

comment:6 Changed 10 years ago by Vickie Lynch

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

comment:7 Changed 10 years ago by Michael Whitty

  • Status changed from verify to verifying
  • Tester set to Michael Whitty

comment:8 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

successfully removed from unit tests

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2762

Note: See TracTickets for help on using tickets.