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
Upgrade tests to use Babel 7 #2094
Conversation
@@ -0,0 +1,24 @@ | |||
module.exports = function(api) { |
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.
babel-loader
in the karma.config.js
wasn't picking up the configuration in .babelrc
but did work when I changed it to a babel.config.js
. 🤷♂
spyAll(Receiver); | ||
spyAll(Receiver.prototype); |
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.
This tests failed unless I swapped the order here. Perhaps some new class transform broke the old form? 🤷♂
} | ||
] | ||
] | ||
? [['istanbul', { include: '**/src/**/*.js' }]] |
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.
That's much better 👍
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.
Really happy to see this PR 🎉 I attempted to switch to babel7 when it just came out and ran into many problems. Glad to see that they've been resolved 💯
I'm exploring writing a babel plugin to convert property names to enable our tests to run against the minified builds. In the process of doing that I realized that our tests still used Babel 6. This PR upgrades them to Babel 7.