-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Fix #4347 | Ensure root level hooks are called when running in watch mode #4382
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 tasks
indieisaconcept
force-pushed
the
issue/4347
branch
from
July 26, 2020 02:34
c4db914
to
4ef171d
Compare
## Rationale Tests with root level hooks are ignored when running in watch mode. This occurs on initial & subsequent runs. ## Changes - updated `lib/cli/run-watch` to re-initialise rootHooks As we have swapped out the root suite it is necessary to re-initialize root level hooks again. - Extended integration tests for watch to take into consideration hooks ## Refs - mochajs/issues/4347
indieisaconcept
force-pushed
the
issue/4347
branch
from
July 26, 2020 12:56
4ef171d
to
88fd011
Compare
indieisaconcept
changed the title
Ensure root level hooks are called when running in watch mode
Fix 4347 | Ensure root level hooks are called when running in watch mode
Jul 27, 2020
indieisaconcept
changed the title
Fix 4347 | Ensure root level hooks are called when running in watch mode
Fix #4347 | Ensure root level hooks are called when running in watch mode
Jul 27, 2020
boneskull
added
type: bug
a defect, confirmed by a maintainer
semver-patch
implementation requires increase of "patch" version number; "bug fixes"
labels
Jul 29, 2020
2 tasks
boneskull
approved these changes
Jul 29, 2020
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.
Thanks, LGTM. I'll see about cutting a release...hopefully.
4 tasks
Awesome. Thanks @boneskull |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
semver-patch
implementation requires increase of "patch" version number; "bug fixes"
type: bug
a defect, confirmed by a maintainer
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rationale
Tests with root level hooks are ignored when running in watch mode.
This occurs on initial & subsequent runs.
Description of the Change
lib/cli/run-watch
to re-initialise rootHooksAs we have swapped out the root suite it is necessary to re-initialize
root level hooks again.
hooks
Refs