Ticket #5606 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Stress test framework needs assertEqual

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: minor Milestone: Release 2.2
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Jose Borreguero

Description

Comparisons can currently be made for equality using assertTrue, or assertDelta, but they are a little problematic.

Firstly, writing self.assertTrue(a == b) will not print out the expected result when the test fails. This can lead to developers wasting a lot of time debugging scripts. Secondly, when you're performing a an equal comparison using assertDelta, you're forced to supply a delta argument.

It would just be easier to add an assertEqual function. This would also make coding tests using the stress tests framework similar to our python unit test framework, which already has an assertEqual method.

Change History

comment:1 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Owen Arnold

refs #5606. new assert implemented

Changeset: c65bec21e74cdebc9117f72445c08a995d28a675

comment:3 Changed 8 years ago by Owen Arnold

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

Tester: This is being used in the LoadAndCheckBase.py file in the SystemTest/Analysis directory.

comment:4 Changed 8 years ago by Jose Borreguero

  • Status changed from verify to verifying
  • Tester set to Jose Borreguero

comment:5 Changed 8 years ago by Jose Borreguero

  • Status changed from verifying to closed

comment:6 Changed 8 years ago by Michael Reuter

Refs #5453 and #5606. Handling detcal file.

Changeset: 1323ed40e413b94ba696988c34ccdce4ea0e667c

comment:7 Changed 8 years ago by Michael Reuter

Refs #5453 and #5606. Handling detcal file.

Changeset: 1323ed40e413b94ba696988c34ccdce4ea0e667c

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6452

Note: See TracTickets for help on using tickets.