fixes potential crash under gatekeeper

[wxWidgets.git] / tests / test.cpp

diff --git a/tests/test.cpp b/tests/test.cpp
index 1e59f598067c3ac4e6e17fc12c89adeadb4e3ad4..cdb57c535ad55c5278c513bbc71d3ef7ff406594 100644 (file)
--- a/tests/test.cpp
+++ b/tests/test.cpp
@@ -34,6 +34,7 @@
 #include <cppunit/Test.h>
 #include <cppunit/TestResult.h>
 #include <cppunit/TestFailure.h>
 #include <cppunit/Test.h>
 #include <cppunit/TestResult.h>
 #include <cppunit/TestFailure.h>

+#include <cppunit/TestResultCollector.h>

 
 #ifdef __VISUALC__
     #pragma warning(default:4100)
 
 #ifdef __VISUALC__
     #pragma warning(default:4100)


@@ -41,9 +42,10 @@
 #include "wx/afterstd.h"
 
 #include "wx/cmdline.h"
 #include "wx/afterstd.h"
 
 #include "wx/cmdline.h"

+#include <exception>

 #include <iostream>
 
 #include <iostream>
 

-#ifdef __WXMSW__
+#ifdef __WINDOWS__

     #include "wx/msw/msvcrt.h"
 #endif
 
     #include "wx/msw/msvcrt.h"
 #endif
 


@@ -85,6 +87,9 @@ struct CrtAssertFailure
 
 #if wxDEBUG_LEVEL
 
 
 #if wxDEBUG_LEVEL
 

+// Information about the last not yet handled assertion.
+static wxString s_lastAssertMessage;
+

 static wxString FormatAssertMessage(const wxString& file,
                                     int line,
                                     const wxString& func,
 static wxString FormatAssertMessage(const wxString& file,
                                     int line,
                                     const wxString& func,


@@ -104,16 +109,48 @@ static void TestAssertHandler(const wxString& file,
                               const wxString& cond,
                               const wxString& msg)
 {
                               const wxString& cond,
                               const wxString& msg)
 {

-    // can't throw from other threads, die immediately
+    // Determine whether we can safely throw an exception to just make the test
+    // fail or whether we need to abort (in this case "msg" will contain the
+    // explanation why did we decide to do it).
+    wxString abortReason;
+
+    const wxString
+        assertMessage = FormatAssertMessage(file, line, func, cond, msg);
+

     if ( !wxIsMainThread() )
     {
     if ( !wxIsMainThread() )
     {

-        wxPrintf("%s in a worker thread -- aborting.",
-                 FormatAssertMessage(file, line, func, cond, msg));
-        fflush(stdout);
-        _exit(-1);
+        // Exceptions thrown from worker threads are not caught currently and
+        // so we'd just die without any useful information -- abort instead.
+        abortReason << assertMessage << "in a worker thread.";
+    }
+    else if ( uncaught_exception() )
+    {
+        // Throwing while already handling an exception would result in
+        // terminate() being called and we wouldn't get any useful information
+        // about why the test failed then.
+        if ( s_lastAssertMessage.empty() )
+        {
+            abortReason << assertMessage << "while handling an exception";
+        }
+        else // In this case the exception is due to a previous assert.
+        {
+            abortReason << s_lastAssertMessage << "\n  and another "
+                        << assertMessage << " while handling it.";
+        }
+    }
+    else // Can "safely" throw from here.
+    {
+        // Remember this in case another assert happens while handling this
+        // exception: we want to show the original assert as it's usually more
+        // useful to determine the real root of the problem.
+        s_lastAssertMessage = assertMessage;
+
+        throw TestAssertFailure(file, line, func, cond, msg);

     }
 
     }
 

-    throw TestAssertFailure(file, line, func, cond, msg);
+    wxFputs(abortReason, stderr);
+    fflush(stderr);
+    _exit(-1);

 }
 
 #endif // wxDEBUG_LEVEL
 }
 
 #endif // wxDEBUG_LEVEL


@@ -128,10 +165,10 @@ static string GetExceptionMessage()
         throw;
     }
 #if wxDEBUG_LEVEL
         throw;
     }
 #if wxDEBUG_LEVEL

-    catch ( TestAssertFailure& e )
+    catch ( TestAssertFailure& )

     {
     {

-        msg << FormatAssertMessage(e.m_file, e.m_line, e.m_func,
-                                   e.m_cond, e.m_msg);
+        msg = s_lastAssertMessage;
+        s_lastAssertMessage.clear();

     }
 #endif // wxDEBUG_LEVEL
 #ifdef wxUSE_VC_CRTDBG
     }
 #endif // wxDEBUG_LEVEL
 #ifdef wxUSE_VC_CRTDBG


@@ -194,7 +231,7 @@ public:
 
     virtual void startTest(CppUnit::Test *test)
     {
 
     virtual void startTest(CppUnit::Test *test)
     {

-        wxPrintf("  %-60s  ", test->getName());
+        printf("  %-60s  ", test->getName().c_str());

         m_result = RESULT_OK;
         m_watch.Start();
     }
         m_result = RESULT_OK;
         m_watch.Start();
     }


@@ -207,10 +244,10 @@ public:
     virtual void endTest(CppUnit::Test * WXUNUSED(test))
     {
         m_watch.Pause();
     virtual void endTest(CppUnit::Test * WXUNUSED(test))
     {
         m_watch.Pause();

-        wxPrintf(GetResultStr(m_result));
+        printf("%s", GetResultStr(m_result));

         if (m_timing)
         if (m_timing)

-            wxPrintf("  %6ld ms", m_watch.Time());
-        wxPrintf("\n");
+            printf("  %6ld ms", m_watch.Time());
+        printf("\n");

     }
 
 protected :
     }
 
 protected :


@@ -222,7 +259,7 @@ protected :
         RESULT_MAX
     };
 
         RESULT_MAX
     };
 

-    wxString GetResultStr(ResultType type) const
+    const char* GetResultStr(ResultType type) const

     {
         static const char *resultTypeNames[] =
         {
     {
         static const char *resultTypeNames[] =
         {


@@ -375,6 +412,25 @@ extern bool IsNetworkAvailable()
     return online;
 }
 
     return online;
 }
 

+extern bool IsAutomaticTest()
+{
+    static int s_isAutomatic = -1;
+    if ( s_isAutomatic == -1 )
+    {
+        // Allow setting an environment variable to emulate buildslave user for
+        // testing.
+        wxString username;
+        if ( !wxGetEnv("WX_TEST_USER", &username) )
+            username = wxGetUserId();
+
+        username.MakeLower();
+        s_isAutomatic = username.Matches("buildslave*") ||
+                            username.Matches("sandbox*");
+    }
+
+    return s_isAutomatic == 1;
+}
+

 // helper of OnRun(): gets the test with the given name, returning NULL (and
 // not an empty test suite) if there is no such test
 static Test *GetTestByName(const wxString& name)
 // helper of OnRun(): gets the test with the given name, returning NULL (and
 // not an empty test suite) if there is no such test
 static Test *GetTestByName(const wxString& name)


@@ -423,7 +479,16 @@ bool TestApp::OnInit()
 #else
     cout << "Test program for wxWidgets non-GUI features\n"
 #endif
 #else
     cout << "Test program for wxWidgets non-GUI features\n"
 #endif

-         << "build: " << WX_BUILD_OPTIONS_SIGNATURE << std::endl;
+         << "build: " << WX_BUILD_OPTIONS_SIGNATURE << "\n"
+         << "running under " << wxGetOsDescription()
+         << " as " << wxGetUserId() << std::endl;
+
+    if ( m_detail )
+    {
+        // Output some important information about the test environment.
+        cout << "Running under " << wxGetOsDescription() << ", "
+                "locale is " << setlocale(LC_ALL, NULL) << std::endl;
+    }

 
 #if wxUSE_GUI
     // create a hidden parent window to be used as parent for the GUI controls
 
 #if wxUSE_GUI
     // create a hidden parent window to be used as parent for the GUI controls


@@ -599,7 +664,9 @@ int TestApp::OnRun()
     runner.eventManager().pushProtector(new wxUnitTestProtector);
 
     bool printProgress = !(verbose || m_detail || m_timing);
     runner.eventManager().pushProtector(new wxUnitTestProtector);
 
     bool printProgress = !(verbose || m_detail || m_timing);

-    return runner.run("", false, true, printProgress) ? EXIT_SUCCESS : EXIT_FAILURE;
+    runner.run("", false, true, printProgress);
+
+    return runner.result().testFailures() == 0 ? EXIT_SUCCESS : EXIT_FAILURE;

 }
 
 int TestApp::OnExit()
 }
 
 int TestApp::OnExit()