Ticket #4089 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Improve test coverage for Python operators

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Nick Draper

Description

MatrixWorkspace has no coverage in unit tests for operator overloads (or much of anything). Very bad (I broke this in my build, found out when I manually tested after all the tests passed!)

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Janik Zikovsky

Refs #4089 unit test for operator overloads

... on MatrixWorkspace in python. Renamed the relevant test.

Changeset: 3d60097e440afc55d3ee5de48169ec68e5e1a2f6

comment:3 Changed 9 years ago by Janik Zikovsky

Binary operations for MDHistoWorkspaces

Refs #4085, refs #4089, REfs #4083. Added tests for workspace groups in binary operations. The +-*/ operations are all overloaded and tested for matrix, group, and MD types.

Changeset: b61d875ec32d08b4b77b47c41730b53f97cff61d

comment:4 Changed 9 years ago by Janik Zikovsky

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

There are now unit tests under VanillaPythonTests.

comment:5 Changed 9 years ago by Janik Zikovsky

Refs #4089 unit test for operator overloads

... on MatrixWorkspace in python. Renamed the relevant test.

Changeset: 3d60097e440afc55d3ee5de48169ec68e5e1a2f6

comment:6 Changed 9 years ago by Janik Zikovsky

Binary operations for MDHistoWorkspaces

Refs #4085, refs #4089, REfs #4083. Added tests for workspace groups in binary operations. The +-*/ operations are all overloaded and tested for matrix, group, and MD types.

Changeset: b61d875ec32d08b4b77b47c41730b53f97cff61d

comment:7 Changed 9 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:8 Changed 9 years ago by Nick Draper

  • Status changed from verifying to closed

verified in both old and new api

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4936

Note: See TracTickets for help on using tickets.