Ticket #8227 (closed: fixed)
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: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: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