Skip to content

Commit

Permalink
Fix ctkVTKConnectionTest1 removing connection time ratio test
Browse files Browse the repository at this point in the history
As discussed in [1], the test should check that the code works
as expected and not set a bar that we cannot yet attain.

[1] #378 (comment)
  • Loading branch information
jcfr committed May 27, 2014
1 parent 1d9648c commit 949d26c
Showing 1 changed file with 0 additions and 12 deletions.
12 changes: 0 additions & 12 deletions Libs/Visualization/VTK/Core/Testing/Cpp/ctkVTKConnectionTest1.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -244,17 +244,5 @@ int ctkVTKConnectionTest1( int argc, char * argv [] )
obj4->Delete();
obj5->Delete();

{
const char* measurementName = "time_connection2-over-time_connection1";
double ratio = time_connection2 / time_connection1;
displayDartMeasurement(measurementName, ratio);
#ifdef QT_NO_DEBUG // In Debug mode, the ratio can be over 2 !
// Ideally a ratio ~= 1.
if (ratio > 1.2)
{
return EXIT_FAILURE;
}
#endif
}
return EXIT_SUCCESS;
}

0 comments on commit 949d26c

Please sign in to comment.