From cc3678f0336cb0cb0be949393618774e40008e1f Mon Sep 17 00:00:00 2001 From: Greg Cobb and Tim Jarratt Date: Wed, 27 Aug 2014 13:50:56 -0700 Subject: [PATCH] Refactor suite to share reportExpectationFailure func This was shared between our suite factory and the suite factory. Sweet. --- src/core/Env.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/core/Env.js b/src/core/Env.js index 965c9ab2..a6d61e00 100644 --- a/src/core/Env.js +++ b/src/core/Env.js @@ -183,9 +183,7 @@ getJasmineRequireObj().Env = function(j$) { description: 'Jasmine__TopLevel__Suite', queueRunner: queueRunnerFactory, resultCallback: function() {}, // TODO - hook this up - reportExpectationFailure: function(message) { - reporter.afterAllEvent('Expectation failed: '+ message); - } + reportExpectationFailure: reportExpectationFailure }); runnableLookupTable[topSuite.id] = topSuite; defaultResourcesForRunnable(topSuite.id); @@ -242,9 +240,7 @@ getJasmineRequireObj().Env = function(j$) { } reporter.suiteDone(attrs); }, - reportExpectationFailure: function(message) { - reporter.afterAllEvent('Expectation failed: '+ message); - } + reportExpectationFailure: reportExpectationFailure }); runnableLookupTable[suite.id] = suite; @@ -365,6 +361,10 @@ getJasmineRequireObj().Env = function(j$) { this.pending = function() { throw j$.Spec.pendingSpecExceptionMessage; }; + + function reportExpectationFailure(message) { + reporter.afterAllEvent('Expectation failed: '+ message); + } } return Env;