-
-
Notifications
You must be signed in to change notification settings - Fork 722
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
feat(timeout): Added timeouts for steps #2518
feat(timeout): Added timeouts for steps #2518
Conversation
c4bf0cb
to
5bdb5dd
Compare
5bdb5dd
to
05a4c5c
Compare
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.
Ok, we need to add documentation for this. I think we have some mentions on timeouts.
Can you add it there?
lib/actor.js
Outdated
@@ -116,7 +117,16 @@ function recordStep(step, args) { | |||
event.emit(event.step.started, step); | |||
step.startTime = Date.now(); | |||
} | |||
return val = step.run(...args); | |||
console.log(step); |
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.
console.log(step); |
const config_run_config = (config, grep) => `${codecept_run} --config ${codecept_dir}/${config} ${grep ? `--grep "${grep}"` : ''}`; | ||
|
||
describe('CodeceptJS Steps', () => { | ||
it('should stop test, when step exceeded', (done) => { |
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"m just curious if those tests can be written in async/await approach?
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.
yes, it can be.
Motivation/Description of the PR
Applicable helpers:
Applicable plugins:
Type of change
Checklist:
npm run docs
)npm run lint
)npm test
)