Disable CrashingTests... in debug.

pjohnmeyer · pjohnmeyer · commit 967593c44cb9 · 2015-04-04T00:56:40.000-05:00
After some consideration I decided to just remove this test in debug configurations. Many debuggers will behave as XCode does and try to 'catch' this by default. The test still works in release configs. Fixes #45.
diff --git a/tests/TestTest.cpp b/tests/TestTest.cpp
@@ -73,6 +73,8 @@ TEST(ThrowingTestsAreReportedAsFailures)
 }
 
 #if !defined(UNITTEST_MINGW) && !defined(UNITTEST_WIN32)
+// Skip this test in debug because some debuggers don't like it.
+#if defined(NDEBUG)
 TEST(CrashingTestsAreReportedAsFailures)
 {
     class CrashingTest : public Test
@@ -96,6 +98,7 @@ TEST(CrashingTestsAreReportedAsFailures)
 }
 #endif
 #endif
+#endif
 
 TEST(TestWithUnspecifiedSuiteGetsDefaultSuite)
 {

Original file line number	Diff line number	Diff line change
`@@ -73,6 +73,8 @@ TEST(ThrowingTestsAreReportedAsFailures)`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`#if !defined(UNITTEST_MINGW) && !defined(UNITTEST_WIN32)`
	`76`	`+// Skip this test in debug because some debuggers don't like it.`
	`77`	`+#if defined(NDEBUG)`
`76`	`78`	`TEST(CrashingTestsAreReportedAsFailures)`
`77`	`79`	`{`
`78`	`80`	`class CrashingTest : public Test`
`@@ -96,6 +98,7 @@ TEST(CrashingTestsAreReportedAsFailures)`
`96`	`98`	`}`
`97`	`99`	`#endif`
`98`	`100`	`#endif`
	`101`	`+#endif`
`99`	`102`
`100`	`103`	`TEST(TestWithUnspecifiedSuiteGetsDefaultSuite)`
`101`	`104`	`{`