Revirtualised IResultCapture methods

Didn't really impact runtime anyway, but will need to use interface for threading support.

include/internal/catch_assertionhandler.cpp

@@ -56,7 +56,7 @@ namespace Catch {
             StringRef capturedExpression,
             ResultDisposition::Flags resultDisposition )
     :   m_assertionInfo{ macroName, lineInfo, capturedExpression, resultDisposition },
-        m_resultCapture( static_cast<RunContext&>( getResultCapture() ) )
+        m_resultCapture( getResultCapture() )
     {}
 
     AssertionHandler::~AssertionHandler() {

include/internal/catch_assertionhandler.h

@@ -44,7 +44,7 @@ namespace Catch {
         AssertionInfo m_assertionInfo;
         AssertionReaction m_reaction;
         bool m_completed = false;
-        RunContext& m_resultCapture;
+        IResultCapture& m_resultCapture;
 
     public:
         AssertionHandler

include/internal/catch_interfaces_capture.h

@@ -11,6 +11,7 @@
 #include <string>
 
 #include "catch_stringref.h"
+#include "catch_result_type.h"
 
 namespace Catch {
 
@@ -22,6 +23,9 @@ namespace Catch {
     struct Counts;
     struct BenchmarkInfo;
     struct BenchmarkStats;
+    struct AssertionReaction;
+
+    struct ITransientExpression;
 
     struct IResultCapture {
 
@@ -40,6 +44,31 @@ namespace Catch {
 
         virtual void handleFatalErrorCondition( StringRef message ) = 0;
 
+        virtual void handleExpr
+                (   AssertionInfo const& info,
+                    ITransientExpression const& expr,
+                    AssertionReaction& reaction ) = 0;
+        virtual void handleMessage
+                (   AssertionInfo const& info,
+                    ResultWas::OfType resultType,
+                    StringRef const& message,
+                    AssertionReaction& reaction ) = 0;
+        virtual void handleUnexpectedExceptionNotThrown
+                (   AssertionInfo const& info,
+                    AssertionReaction& reaction ) = 0;
+        virtual void handleUnexpectedInflightException
+                (   AssertionInfo const& info,
+                    std::string const& message,
+                    AssertionReaction& reaction ) = 0;
+        virtual void handleIncomplete
+                (   AssertionInfo const& info ) = 0;
+        virtual void handleNonExpr
+                (   AssertionInfo const &info,
+                    ResultWas::OfType resultType,
+                    AssertionReaction &reaction ) = 0;
+
+
+
         virtual bool lastAssertionPassed() = 0;
         virtual void assertionPassed() = 0;
 

include/internal/catch_run_context.h

@@ -75,44 +75,33 @@ namespace Catch {
         IConfigPtr config() const;
         IStreamingReporter& reporter() const;
 
+    public: // IResultCapture
+
         // Assertion handlers
         void handleExpr
                 (   AssertionInfo const& info,
                     ITransientExpression const& expr,
-                    AssertionReaction& reaction );
+                    AssertionReaction& reaction ) override;
         void handleMessage
                 (   AssertionInfo const& info,
                     ResultWas::OfType resultType,
                     StringRef const& message,
-                    AssertionReaction& reaction );
+                    AssertionReaction& reaction ) override;
         void handleUnexpectedExceptionNotThrown
                 (   AssertionInfo const& info,
-                    AssertionReaction& reaction );
+                    AssertionReaction& reaction ) override;
         void handleUnexpectedInflightException
                 (   AssertionInfo const& info,
                     std::string const& message,
-                    AssertionReaction& reaction );
+                    AssertionReaction& reaction ) override;
         void handleIncomplete
-                (   AssertionInfo const& info );
+                (   AssertionInfo const& info ) override;
         void handleNonExpr
                 (   AssertionInfo const &info,
                     ResultWas::OfType resultType,
-                    AssertionReaction &reaction );
+                    AssertionReaction &reaction ) override;
-
-        void reportExpr
-                (AssertionInfo const &info,
-                 ResultWas::OfType resultType,
-                 ITransientExpression const *expr,
-                 bool negated );
-
-        void populateReaction( AssertionReaction& reaction );
-
-    public: // IResultCapture
-
-        void assertionEnded(AssertionResult const& result);
 
         bool sectionStarted( SectionInfo const& sectionInfo, Counts& assertions ) override;
-        bool testForMissingAssertions(Counts& assertions);
 
         void sectionEnded( SectionEndInfo const& endInfo ) override;
         void sectionEndedEarly( SectionEndInfo const& endInfo ) override;
@@ -145,6 +134,16 @@ namespace Catch {
         void invokeActiveTestCase();
 
         void resetAssertionInfo();
+        bool testForMissingAssertions( Counts& assertions );
+
+        void assertionEnded( AssertionResult const& result );
+        void reportExpr
+                (   AssertionInfo const &info,
+                    ResultWas::OfType resultType,
+                    ITransientExpression const *expr,
+                    bool negated );
+
+        void populateReaction( AssertionReaction& reaction );
 
     private: