Ticket #5589 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

[Indirect] - Python Algorithm Unit Tests

Reported by: Peter Parker Owned by: Dereck Kachere
Priority: critical Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

As per Owen's email of 19 June 2012, we're lacking unit tests for many of the Indirect Inelastic Python Algorithms. This needs to be fixed.

Change History

comment:1 Changed 8 years ago by Peter Parker

Refs #5589 #5293 - RetrieveRunInfo modification + unit tests.

Modifications made to how the aglogrithm works.

Unit tests added.

Changeset: bed93dc9123eb7515e6266e9bea96710bcfcb519

comment:2 Changed 8 years ago by Peter Parker

Refs #5589 - Import DeleteWorkspace in unit test.

Changeset: 1e4af6d772b892a58adcd78a727d10f9813509e4

comment:3 Changed 8 years ago by Peter Parker

Refs #5589 #5293 - RetrieveRunInfo modification + unit tests.

Modifications made to how the aglogrithm works.

Unit tests added.

Changeset: bed93dc9123eb7515e6266e9bea96710bcfcb519

comment:4 Changed 8 years ago by Peter Parker

Refs #5589 - Import DeleteWorkspace in unit test.

Changeset: 1e4af6d772b892a58adcd78a727d10f9813509e4

comment:5 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:6 Changed 8 years ago by Peter Parker

  • Summary changed from Python Algorithm Unit Tests to [Indirect] - Python Algorithm Unit Tests

Update:

Most of the Indirect Python code is woefully lacking in unit tests, mainly as a result of the ever-changing nature of a lot of the code and the fact that the development of this stuff has mainly been driven outside the Mantid team by Spencer. As more and more of it settles down we need to think about backing it up with unit tests, as per the other Python Algorithms.

comment:7 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.3 to Release 2.4

Moved to milestone 2.4

comment:8 Changed 8 years ago by Dereck Kachere

  • Owner changed from Peter Parker to Dereck Kachere
  • Status changed from new to accepted

comment:9 Changed 8 years ago by Dereck Kachere

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

The tests added for ISIS indirect bayes referred to in #6282 and #6283 address the comment in the 'Update:' section.

comment:10 Changed 8 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:11 Changed 8 years ago by Martyn Gigg

  • Status changed from verifying to closed

Confirmed this has moved to other tickets.

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6435

Note: See TracTickets for help on using tickets.