Skip to content

Migrate to new testing APIs#39

Merged
andreasschacht merged 1 commit intomasterfrom
new-testing-api
Mar 26, 2018
Merged

Migrate to new testing APIs#39
andreasschacht merged 1 commit intomasterfrom
new-testing-api

Conversation

@simonihmig
Copy link
Owner

This is still WIP, as the support for the new testing APIs is not yet released in ember-mocha (using the master branch for now). So don't merge yet!

But all green it seems, so 🎉 /cc @Turbo87

Copy link

@Turbo87 Turbo87 left a comment

Choose a reason for hiding this comment

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

❤️

package.json Outdated
"ember-cli-htmlbars-inline-precompile": "^1.0.0",
"ember-cli-inject-live-reload": "^1.4.1",
"ember-cli-mocha": "^0.14.4",
"ember-cli-mocha": "emberjs/ember-mocha#master",
Copy link

Choose a reason for hiding this comment

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

interesting that this works... ember-cli-mocha vs ember-mocha 😉

Copy link
Owner Author

Choose a reason for hiding this comment

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

Uh, good catch! 😬

Copy link
Owner Author

Choose a reason for hiding this comment

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

Btw, do I still need ember-cli-mocha actually? Seems it only contains the blueprints nowadays, right?

Copy link

Choose a reason for hiding this comment

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

ember-cli/ember-cli-eslint#225 and code like that is the problem and the reason why we're keeping the "wrapper" for now

@simonihmig simonihmig changed the title [WIP] Migrate to new testing APIs Migrate to new testing APIs Mar 24, 2018
@simonihmig
Copy link
Owner Author

@andreasschacht this is now updated to the real release, and should be ready to be merged!

@andreasschacht andreasschacht merged commit 470e2c2 into master Mar 26, 2018
@simonihmig simonihmig deleted the new-testing-api branch January 8, 2021 23:48
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.

3 participants