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

adds $.mockjax.clear() to the mixin's teardown method. #30

Merged
merged 1 commit into from
Nov 1, 2014

Conversation

yratanov
Copy link
Contributor

@yratanov yratanov commented Nov 1, 2014

  • I believe it is expected behavior by user (at least it was expected by me) that library clean up after itself. It is especially important in tests to keep them isolated.
  • tests clean up. avoid of using third-party libraries' code in the repo. Every dependency should be resolved using bower/npm.
  • using same mockjax version in tests as in dist.

* I believe it is expected behavior by user (at least it was expected by me) that library clean after work. It is especially important in tests to keep them isolated.
* tests clean up. avoid of using third-party libraries' code in the repo. Every dependency should be resolved using bower/npm.
* using same mockjax version in tests as in dist.
danielspaniel added a commit that referenced this pull request Nov 1, 2014
adds $.mockjax.clear() to the mixin's teardown method.
@danielspaniel danielspaniel merged commit aadada6 into adopted-ember-addons:master Nov 1, 2014
@danielspaniel
Copy link
Collaborator

@yratanov .. thanks for that by the way .. you were right about all 3 points. :)

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.

2 participants