Fixed several unit test main() functions that were returning 0 instead of the result of runAllTests. This caused unit tests to print failures but not count as failures in the pass/fail counts. In CMake projects, since test output is logged to a file instead of printed, this caused no indication of failures when they occured! All tests still pass with no other modifications after fixing this.

cpp/testCalibratedCamera.cpp

@@ -110,7 +110,7 @@ TEST( CalibratedCamera, Dproject_point_pose)
 }
 
 /* ************************************************************************* */
-int main() { TestResult tr; TestRegistry::runAllTests(tr); return 0; }
+int main() { TestResult tr; return TestRegistry::runAllTests(tr); }
 /* ************************************************************************* */
 
 

cpp/testSimpleCamera.cpp

@@ -91,7 +91,7 @@ TEST( SimpleCamera, Dproject_point_pose)
 }
 
 /* ************************************************************************* */
-int main() { TestResult tr; TestRegistry::runAllTests(tr); return 0; }
+int main() { TestResult tr; return TestRegistry::runAllTests(tr); }
 /* ************************************************************************* */
 
 

cpp/testVSLAMFactor.cpp

@@ -92,10 +92,6 @@ TEST( ProjectionFactor, equals )
 }
 
 /* ************************************************************************* */
-int main() {
-	TestResult tr;
-	TestRegistry::runAllTests(tr);
-	return 0;
-}
+int main() { TestResult tr; return TestRegistry::runAllTests(tr); }
 /* ************************************************************************* */
 

cpp/testVSLAMGraph.cpp

@@ -195,5 +195,5 @@ TEST( Graph, CHECK_ORDERING)
 }
 
 /* ************************************************************************* */
-int main() { TestResult tr; TestRegistry::runAllTests(tr); return 0;}
+int main() { TestResult tr; return TestRegistry::runAllTests(tr); }
 /* ************************************************************************* */

cpp/timeGaussianFactorGraph.cpp

@@ -237,9 +237,5 @@ TEST(timeGaussianFactorGraph, planar_join_new)
 
 
 /* ************************************************************************* */
-int main() {
-	TestResult tr;
-	TestRegistry::runAllTests(tr);
-	return 0;
-}
+int main() { TestResult tr; return TestRegistry::runAllTests(tr); }
 /* ************************************************************************* */

wrap/testSpirit.cpp

@@ -90,5 +90,5 @@ TEST( spirit, constMethod_p ) {
 }
 
 /* ************************************************************************* */
-int main() { TestResult tr; TestRegistry::runAllTests(tr); return 0; }
+int main() { TestResult tr; return TestRegistry::runAllTests(tr); }
 /* ************************************************************************* */

wrap/testWrap.cpp

@@ -87,5 +87,5 @@ TEST( wrap, matlab_code ) {
 }
 
 /* ************************************************************************* */
-int main() { TestResult tr; TestRegistry::runAllTests(tr); return 0; }
+int main() { TestResult tr; return TestRegistry::runAllTests(tr); }
 /* ************************************************************************* */