Ticket #2185 (closed: worksforme)

Opened 10 years ago

Last modified 5 years ago

Datasearch paths not considered when FileProperty set via setProperty

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: minor Milestone: Release 3.2
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Parker

Description

If the value of a FileProperty is set via the setProperty method as opposed to the setPropertyValue method, and you are relying on the values in the datasearch.directories config property then the setting will fail (an exception will be thrown).

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner set to Roman Tolchenov

comment:2 Changed 10 years ago by Nick Draper

  • Milestone changed from Iteration 27 to Iteration 28

Bulk move of tickets at the end of iteration 27

comment:3 Changed 10 years ago by Roman Tolchenov

Cannot reproduce the error

comment:4 Changed 10 years ago by Russell Taylor

Change line 45 of LoadNexusProcessedTest from alg.setPropertyValue("Filename", testFile); to alg.setProperty("Filename", testFile);.

The test will then fail as follows: 

1/1 Test #210: NexusTest_LoadNexusProcessedTest ...***Failed    0.10 sec
Running 10 teststerminate called after throwing an instance of 'std::invalid_argument'
  what():  File "GEM38370_Focussed_Legacy.nxs" not found

comment:5 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 28 to Iteration 29

Bulk move of tickets at the end of iteration 28

comment:6 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 29 to Iteration 30

Accepted and assigned tickets moved at iteration 29 code freeze

comment:7 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:8 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 32 to Iteration 33

Moved to iteration 33 at iteration 32 code freeze

comment:9 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.1 to Release 2.2

Moved at end of release 2.1

comment:10 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.3

Moved at the end of release 2.2

comment:11 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.3 to Release 2.4

Moved to milestone 2.4

comment:12 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.4 to Release 2.5

Moved at the code freeze for release 2.4

comment:13 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:14 Changed 7 years ago by Nick Draper

  • Status changed from assigned to new

comment:15 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:16 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

Moved to backlog at the code freeze for R2.6

comment:17 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

bulk move to assigned at the into of the triage step

comment:18 Changed 7 years ago by Russell Taylor

  • Owner changed from Roman Tolchenov to Russell Taylor
  • Status changed from assigned to verify
  • Resolution set to worksforme

This doesn't seem to be a problem any more - I followed the steps in comment:4 and the test still passed.

comment:19 Changed 7 years ago by Peter Parker

  • Status changed from verify to verifying
  • Tester set to Peter Parker

comment:20 Changed 7 years ago by Peter Parker

  • Status changed from verifying to closed

Same for me.

comment:21 Changed 7 years ago by Nick Draper

  • Milestone changed from Backlog to Release 3.2

comment:22 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​3032

Note: See TracTickets for help on using tickets.