Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Cetralise test setup - Closes #589 #613

Merged
merged 9 commits into from
Aug 28, 2017
Merged

Conversation

slaweet
Copy link
Contributor

@slaweet slaweet commented Aug 17, 2017

Closes #589

@slaweet slaweet self-assigned this Aug 17, 2017
@slaweet slaweet changed the title 589 cetralise test setup Cetralise test setup - Closes #589 Aug 17, 2017
@slaweet slaweet changed the base branch from 548-eslint-in-jenkins to development August 17, 2017 15:12
@slaweet slaweet requested a review from yasharAyari August 18, 2017 07:10
@slaweet slaweet force-pushed the 589-cetralise-test-setup branch from 389c0e0 to 26a90d7 Compare August 21, 2017 13:58
@slaweet slaweet force-pushed the 589-cetralise-test-setup branch from 26a90d7 to 10a2277 Compare August 21, 2017 14:39
chai.use(chaiEnzyme());
chai.use(chaiAsPromised);
sinonStubPromise(sinon);

// load all tests into one bundle
const testsContext = require.context('.', true, /\.test\.js$/);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use es6 import instead of require.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it is possible to use import for this task. If you know how, please tell me.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I looked at dir-loader, but it is still using require, so I don't see how does it help.

I don't see what is the problem with the current solution.

@slaweet slaweet merged commit b70c570 into development Aug 28, 2017
@slaweet slaweet deleted the 589-cetralise-test-setup branch August 28, 2017 15:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants