-
Notifications
You must be signed in to change notification settings - Fork 108
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
example/unit-tests-ts is not producing coverage for unit-test files #435
Comments
@bahmutov Can u please provide a solution to this issue? Of all the work we did to replace testing framework from Jest and Selenium to Cypress last year, this is the only limitation we are stuck now. Thanks. |
Same here, can't find the root of the problem. |
I was able to get this working by using webpack instead of browserify and I also included and got it working in my next.js project. Hopefully this might be useful for someone else. This ticket might be related: #361 node -v
v15.9.0 /// <reference types="cypress" />
const webpack = require('@cypress/webpack-preprocessor')
const commonOptions = {
module: {
rules: [{
test: /ts$/,
use: [
{
loader: 'babel-loader',
options: {
presets: [
'@babel/preset-typescript'
]
}
}
]
}]
},
resolve: {
extensions: ['.tsx', '.ts', '.js', '.css']
}
}
/**
* @type {Cypress.PluginConfig}
*/
module.exports = (on, config) => {
const options = {
webpackOptions: commonOptions,
watchOptions: {}
}
on('file:preprocessor', webpack(options))
require('../../../../task')(on, config)
return config
} I also cleared the presets in the babel config.
|
@emiljohansson with your nextjs app you can better use babel.config.js else your production app ends up with istanbul wrappers :) Just pass a NODE_ENV test when executing your script, hope it helps!
|
Logs and screenshots
Versions
[email protected]
Describe the bug
We are looking hard to make the cypress test-coverage for unit-tests for functionality written in TS and after trying the example/unit-tests-ts in this repo, we see that this example is still not working. Can someone work on this and make it work?
The text was updated successfully, but these errors were encountered: