-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[BUGFIX] Do not try to wire up Testem unless a test framework is dete… #10314
Conversation
…cted -- this is for legacy compat -- modern libraries, such as ember-qunit, call hookIntoTestFramework themselves, because it has better knowledge of when Testem needs to be configured Update test to reflect changes to internal default-packager files
looking like floating deps (DT) caused the failures? |
Failure in CI is apparently caused by #10301 |
The discussion on the meeting was to skip that failing test in Node 14 👍 |
tests/acceptance/new-test.js
Outdated
@@ -625,7 +625,7 @@ describe('Acceptance: ember new', function () { | |||
checkFileWithEmberCLIVersionReplacement(fixturePath, 'tests/dummy/config/ember-cli-update.json'); | |||
}); | |||
|
|||
it('app + typescript', async function () { | |||
it.skip('app + typescript', async function () { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The intention was to skip them in node 14, not skip them in every node.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added a node check in the latest commit 🥳
(I had tried checking NODE_VERSION on the env, but our test infra doesn't expose that, and I didn't think it made sense to mess with the test infra rn)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
…cted -- this is for legacy compat -- modern libraries, such as ember-qunit, call hookIntoTestFramework themselves, because it has better knowledge of when Testem needs to be configured
Update test to reflect changes to internal default-packager files
Backport of #10300