admin/jasmine
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Move afterAll reporting into Suite since that's where they're being tracked

Browse Source
This commit is contained in:
slackersoft
2014-08-27 08:13:17 -07:00
parent 97867b2bf5
commit f8bccabf03
3 changed files with 16 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/core/Env.js
View File

@@ -173,9 +173,6 @@ getJasmineRequireObj().Env = function(j$) {
reporter.afterAllEvent('Error thrown: '+ (e.message || e.description));
}
};
options.reportExpectationFailure = function(message) {
reporter.afterAllEvent('Expectation failed: '+ message);
};
new j$.QueueRunner(options).execute();
};
@@ -185,7 +182,10 @@ getJasmineRequireObj().Env = function(j$) {
id: getNextSuiteId(),
description: 'Jasmine__TopLevel__Suite',
queueRunner: queueRunnerFactory,
resultCallback: function() {} // TODO - hook this up
resultCallback: function() {}, // TODO - hook this up
reportExpectationFailure: function(message) {
reporter.afterAllEvent('Expectation failed: '+ message);
}
});
runnableLookupTable[topSuite.id] = topSuite;
defaultResourcesForRunnable(topSuite.id);
@@ -241,6 +241,9 @@ getJasmineRequireObj().Env = function(j$) {
currentlyExecutingSuites.pop();
}
reporter.suiteDone(attrs);
},
reportExpectationFailure: function(message) {
reporter.afterAllEvent('Expectation failed: '+ message);
}
});