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

fix: Resolve issues with timeouts and target closures in test framework #981

Merged
merged 1 commit into from Sep 8, 2023

Conversation

AfolabiOlaoluwa
Copy link
Contributor

Description

This PR introduces a couple of important updates to our project's configuration files to address specific issues and improve the reliability of our testing and browser automation setup.

This commit addresses issues related to timeouts and target closures in our test framework. It includes improvements to handle timeouts more effectively and prevent unexpected target closures during test execution.

  • Playwright browser contexts now have increased timeout settings.
  • Web Test Runner configuration has been adjusted to handle timeouts gracefully.

This commit ensures the reliability and stability of our automated tests. Running yarn test now perfectly process test:unit && yarn test:browser effectively and proceeds to playwright test without any hassle.

Reason for Changes

The changes were made to address timeout issues encountered when running tests, specifically in Firefox. By extending the browser start timeout, we ensure that the browser has sufficient time to initialize and start the test page, reducing the likelihood of test failures due to timeouts.

These adjustments aim to improve the overall stability and reliability of our testing environment, making it more robust when running tests across different browsers.

Console Output Image

Screenshot 2023-09-07 at 18 43 54

Increase Playwright browser context timeouts, adjust Web Test Runner configuration for graceful timeout handling, and review test scripts for proper error handling and interactions.

This commit addresses issues related to timeouts and target closures in our test framework. It includes improvements to handle timeouts more effectively and prevent unexpected target closures during test execution.

- Playwright browser contexts now have increased timeout settings.
- Web Test Runner configuration has been adjusted to handle timeouts gracefully.
- Test scripts have been reviewed for proper error handling and interactions.

This commit ensures the reliability and stability of our automated tests.
@afcapel afcapel merged commit ae6329a into hotwired:main Sep 8, 2023
1 check passed
@afcapel
Copy link
Collaborator

afcapel commented Sep 8, 2023

Thanks @AfolabiOlaoluwa

shiftyp pushed a commit to shiftyp/ts-turbo that referenced this pull request Sep 8, 2023
…ixes

fix: Resolve issues with timeouts and target closures in test framework
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants