Ticket #8704 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Add tests to check that default INST_Parameters.xml gets loaded

Reported by: Anders Markvardsen Owned by: Anders Markvardsen
Priority: major Milestone: Release 3.1
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Andrei Savici

Description (last modified by Anders Markvardsen) (diff)

When loading raw data INST_Definition.xml is loaded. If exist also INST_Parameters.xml is loaded.

Add tests that checks this is done for

ISIS histogram .nxs and .raw files ISIS/SNS event data

Change History

comment:1 Changed 7 years ago by Anders Markvardsen

  • Status changed from new to inprogress

cover tests to check default params loaded. re #8704

Changeset: a663702cdf94bac9d95981b35ca10b2b70124a8b

comment:2 Changed 7 years ago by Anders Markvardsen

  • Description modified (diff)

comment:3 Changed 7 years ago by Anders Markvardsen

To tester:

Add tests that checks this is done for

ISIS histogram .nxs where inst+param are not in nexus file

ISIS histogram .raw file

event data where inst+param are not in nexus file (note event data where inst+param are in nexus file was already covered)

Test that these passes.

Note I did not cover the case:

ISIS histogram .nxs where inst+param are in nexus file

This will covered as part of an already existing ticket: #8214

comment:4 Changed 7 years ago by Anders Markvardsen

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

comment:5 Changed 7 years ago by Andrei Savici

  • Status changed from verify to verifying
  • Tester set to Andrei Savici

comment:6 Changed 7 years ago by Andrei Savici

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8704_testINSTdefaultParam'

Full changeset: 37624fb966943783baad15edd058f80b84135ba2

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9548

Note: See TracTickets for help on using tickets.