Ticket #8227 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

SaveFocusedXYE unit test shouldn't load files.

Reported by: Samuel Jackson Owned by: Karl Palmen
Priority: major Milestone: Release 3.2
Component: Framework Keywords: Maintenance
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

To quote Russell:

New unit test should not load data files. It should be trivial to use create2DWorkspaceWithFullInstrument and knock the detector off a spectrum or two.

Change History

comment:1 Changed 7 years ago by Russell Taylor

  • Keywords Maintenance added

comment:2 Changed 7 years ago by Karl Palmen

  • Owner changed from Anyone to Karl Palmen
  • Status changed from new to inprogress

comment:3 Changed 7 years ago by Karl Palmen

The wiki description has line indents that make it appear in the wrong manner. I'll fix that too.

comment:4 Changed 7 years ago by Karl Palmen

Modify test_doesnt_fail_on_missing_detectors to not use file re #8227

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 717e5a9b3ecf8383c1149c799a67977e6c7e8ec9

comment:5 Changed 7 years ago by Karl Palmen

Remove indent of wiki description re #8227

Signed-off-by: Karl Palmen <karl.palmen@…>

Changeset: 7050941f3cffe775b4432564c77eb3ed73a25203

comment:6 Changed 7 years ago by Karl Palmen

To test check that unit test no longer loads any files and tests adequately.

Also check that change to wiki description is good.

comment:7 Changed 7 years ago by Karl Palmen

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

comment:8 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:9 Changed 7 years ago by Russell Taylor

  • Milestone changed from Backlog to Release 3.2

comment:10 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

Merge remote branch 'origin/feature/8227_no_file_read_load'

Full changeset: 55107b220692aceaa0be823ee192e42dd4f5ff39

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9072

Note: See TracTickets for help on using tickets.