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

Merge pull request #908 from mcamac/master

Browse Source 
Reverse suite afterEach behavior to match semantics?
This commit is contained in:
Gregg Van Hove
2015-09-01 21:33:52 -07:00
parent 6028a5d92c f3a7396845
commit 825dab32ff
3 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/jasmine-core/jasmine.js
+1 -1
View File
@@ -558,7 +558,7 @@ getJasmineRequireObj().Env = function(j$) {
while(suite) { while(suite) {
befores = befores.concat(suite.beforeFns); befores = befores.concat(suite.beforeFns);
afters = afters.concat(suite.afterFns); afters = afters.concat(suite.afterFns.reverse());
suite = suite.parentSuite; suite = suite.parentSuite;
} }
spec/core/integration/SpecRunningSpec.js
+3 -3
View File
@@ -217,10 +217,10 @@ describe("jasmine spec running", function () {
"beforeEach1", "beforeEach1",
"beforeEach2", "beforeEach2",
"outer it 1", "outer it 1",
"afterEach2",
"afterEach1", "afterEach1",
"runner afterEach2", "afterEach2",
"runner afterEach1" "runner afterEach1",
"runner afterEach2"
]; ];
expect(actions).toEqual(expected); expect(actions).toEqual(expected);
done(); done();
src/core/Env.js
+1 -1
View File
@@ -108,7 +108,7 @@ getJasmineRequireObj().Env = function(j$) {
while(suite) { while(suite) {
befores = befores.concat(suite.beforeFns); befores = befores.concat(suite.beforeFns);
afters = afters.concat(suite.afterFns); afters = afters.concat(suite.afterFns.reverse());
suite = suite.parentSuite; suite = suite.parentSuite;
} }