Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#2425 closed defect (fixed)

[PATCH] Resolve always failing test_performance_DISABLED() test

Reported by: Echelon9 Owned by: mimo
Priority: Should Have Milestone: Alpha 16
Component: Build & Packages Keywords:
Cc: Patch:

Description

Whilst this test must be manually enabled (not in the default set) at present it always fails due to a data map file move subsequent to its original coding.

What happens:

$ ./test -test TestCmpPathfinder
Running 2 tests
In TestCmpPathfinder::test_performance_DISABLED:
../../../source/simulation2/components/tests/test_Pathfinder.h:60: Error: Test failed: File_OpenFailed
.
Failed 1 of 2 tests
Success rate: 50%

What should happen

$ ./test -test TestCmpPathfinder
Running 2 tests[2.268849]..OK!

Attachments (1)

fix-2425.patch (784 bytes ) - added by Echelon9 10 years ago.
Resolves always failing test

Download all attachments as: .zip

Change History (5)

by Echelon9, 10 years ago

Attachment: fix-2425.patch added

Resolves always failing test

comment:1 by mimo, 10 years ago

Thanks for reporting this failed test, but for the fix I think it would make more sense to rename the median Oasis maps 01, 02 and 03 instead of the present 04, 06 and 07.

comment:2 by mimo, 10 years ago

Owner: set to mimo
Resolution: fixed
Status: newclosed

In 14740:

rename Median Oasis maps, fixes #2425

comment:3 by mimo, 10 years ago

Keywords: review patch removed

comment:4 by Echelon9, 10 years ago

Many thanks, the prudent resolution alternative in r14740 works for me:

$ svn info
...
Revision: 14741
Node Kind: directory
...

$ ./test -test TestCmpPathfinder
Running 2 tests[2.263204]..OK!
Note: See TracTickets for help on using tickets.