Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Mocha and Sinon framework & spy support #11

Merged
merged 5 commits into from
Mar 2, 2019
Merged

Conversation

JoshuaKGoldberg
Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg commented Mar 1, 2019

Types of frameworks & environments that need to be verified with this change:

  • Mocha without Sinon
  • Mocha with Sinon
  • Jest

Also to do:

  • Update documentation

This is a rather large change. Instead of directly calling afterEach and so on by the detected environment, the environment is treated as a set of "hooks" used by the cft driver. This is necessary because different hooks might need to be called by different environments: before/beforeEach vs. beforeAll/before, for example.

The spy library in use is now also decoupled from the test framework. Mocha sometimes is run with nothing and sometimes is run with Sinon.

Fixes #4. Opens #10.

Fixes #4.

This is a rather large change. Instead of directly calling `afterEach` and so on by the detected environment, the environment is treated as a set of "hooks" used by the `cft` driver. This is necessary because different hooks might need to be called by different environments: `before`/`beforeEach` vs. `beforeAll`/`before`, for example.

The spy library in use is now also decoupled from the test framework. Mocha sometimes is run with nothing and sometimes is run with Sinon.
Josh Goldberg added 2 commits March 1, 2019 15:39
Console methods can't be both library spies and cft registrations. There should be an object of them outside of the methods.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant