Ticket #5125 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

AlgorithmProxy isRunning test fails randomly

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 2.4
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description

Every so often the AlgorithmProxyTest fails. Most probably due to timer resolution but it could do with being fixed properly. 

Stacktrace

Assertion failed: alg->isRunning()

Standard Output

In AlgorithmProxyTest::testRunning:
C:/jenkins/workspace/isis_windowsXP_32bit/Code/Mantid/Framework/API/test/AlgorithmProxyTest.h:159: Error: Assertion failed: alg->isRunning()

Change History

comment:1 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:2 Changed 8 years ago by Nick Draper

  • Owner set to Martyn Gigg
  • Status changed from new to assigned

comment:3 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:4 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.3 to Release 2.4

Moved to milestone 2.4

comment:5 Changed 8 years ago by Martyn Gigg

It has been agreed by Russell, Nick & myself that we should disable the test and leave a comment as to why it is disabled.

comment:6 Changed 8 years ago by Martyn Gigg

Disable AlgorithmProxy::isRunning test. Refs #5125

Changeset: 5240026410936dde203ec5e8fa8dba8f928acc7c

comment:7 Changed 8 years ago by Martyn Gigg

  • Status changed from assigned to accepted

comment:8 Changed 8 years ago by Martyn Gigg

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

comment:9 Changed 8 years ago by Martyn Gigg

Disable AlgorithmProxy::isRunning test. Refs #5125

Changeset: 5240026410936dde203ec5e8fa8dba8f928acc7c

comment:10 Changed 8 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:11 Changed 8 years ago by Russell Taylor

  • Status changed from verifying to closed

This has been done in the manner agreed in comment:5

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5971

Note: See TracTickets for help on using tickets.