-
-
Notifications
You must be signed in to change notification settings - Fork 719
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
bump puppeteer version to v3.0 #2342
Conversation
test/helper/webapi.js
Outdated
@@ -648,7 +648,8 @@ module.exports.tests = function () { | |||
}); | |||
}); | |||
|
|||
describe('#attachFile', () => { | |||
describe.skip('#attachFile', () => { | |||
// Those tests are skipped due to bug https://github.com/puppeteer/puppeteer/issues/5543 |
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.
could you mark this as // TODO: ....
?
possibly, we can create issue with this problem, to see it
and type link on it // TODO: #xxxx bla bla....
or // TODO: https://github.com/Codeception/CodeceptJS/issues/xxxx bla bla...
test/helper/Puppeteer_test.js
Outdated
@@ -17,7 +17,9 @@ let page; | |||
let FS; | |||
const siteUrl = TestHelper.siteUrl(); | |||
|
|||
describe('Puppeteer - BasicAuth', () => { | |||
describe('Puppeteer - BasicAuth', function () { | |||
this.timeout(35000); |
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.
why so much?
test/helper/webapi.js
Outdated
@@ -648,7 +648,8 @@ module.exports.tests = function () { | |||
}); | |||
}); | |||
|
|||
describe('#attachFile', () => { | |||
describe.skip('#attachFile', () => { |
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.
ups, this is going to skip this test for ALL helpers!
please check webapi.js there are examples how to skip tests for exactly one helper
Closed in favor of #2355 |
Motivation/Description of the PR
Applicable helpers:
Type of change
Checklist:
npm run docs
)npm run lint
)npm test
)Notes: