Skip to content

Commit

Permalink
Merge pull request #193 from Turbo87/chains
Browse files Browse the repository at this point in the history
Simplify `chainHooks()` function
  • Loading branch information
Turbo87 authored Feb 21, 2018
2 parents 2a7a378 + 6e5c52d commit af25add
Showing 1 changed file with 5 additions and 9 deletions.
14 changes: 5 additions & 9 deletions addon-test-support/ember-mocha/setup-container-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,8 @@ import { resolve } from 'rsvp';

const _assign = assign || merge;

function chainHooks(hooks, context, promise = resolve()) {
return hooks.reduce((promise, fn) => promise.then(fn.bind(context)), promise);
function chainHooks(hooks, context) {
return hooks.reduce((promise, fn) => promise.then(fn.bind(context)), resolve());
}

export default function setupUnitTest(options) {
Expand All @@ -23,23 +23,19 @@ export default function setupUnitTest(options) {
beforeEach(function() {
originalContext = _assign({}, this);

let promise = setupContext(this, options).then(() => {
return setupContext(this, options).then(() => {

let originalPauseTest = this.pauseTest;
this.pauseTest = function Mocha_pauseTest() {
this.timeout(0); // prevent the test from timing out

return originalPauseTest.call(this);
};
});

return chainHooks(beforeEachHooks, this, promise);
}).then(() => chainHooks(beforeEachHooks, this));
});

afterEach(function() {
let promise = chainHooks(afterEachHooks, this);

return promise
return chainHooks(afterEachHooks, this)
.then(() => teardownContext(this))
.then(() => {
// delete any extraneous properties
Expand Down

0 comments on commit af25add

Please sign in to comment.