Skip to content
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(jest): new listener events #3626

Merged
merged 3 commits into from
Oct 11, 2022
Merged

Conversation

noomorph
Copy link
Collaborator

@noomorph noomorph commented Oct 10, 2022

Description

Unblocks my further work on Allure integration.

In this pull request, I have:

  • renamed initTimeout to setupTimeout in testRunner.jest config;
  • added teardownTimeoutto testRunner.jest config (30 seconds by default, should not be changed often);
  • added bail option to testRunner config — to control precisely whether the "already retried (via jest.retryTimes) and still failing tests" should cancel the consequent retries;
  • added environment_setup_failure, environment_teardown_failure events for the custom event listeners.
  • refactored custom Detox Jest environment class
  • changed reportFailedTests to reportTestResults in Detox Internals API
  • fixed error reporting issues for early test suite errors by moving the error reporting to DetoxReporter again

For features/enhancements:

  • I have added/updated the relevant references in the documentation files.

For API changes:

  • I have made the necessary changes in the types index file.

@noomorph noomorph changed the base branch from master to next October 10, 2022 12:45
@noomorph noomorph marked this pull request as ready for review October 11, 2022 13:19
@noomorph noomorph requested a review from d4vidi as a code owner October 11, 2022 13:19
@noomorph noomorph merged commit 716cf30 into next Oct 11, 2022
@noomorph noomorph deleted the feat/jest-environment-advanced branch October 11, 2022 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant