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

chore: enable strict mode in tests #8524

Merged
merged 1 commit into from Jun 15, 2022
Merged

chore: enable strict mode in tests #8524

merged 1 commit into from Jun 15, 2022

Conversation

jrandolf
Copy link
Contributor

@jrandolf jrandolf commented Jun 14, 2022

This PR does a few things:

  • The testing tsconfig.json inherits from the base TS config.
    • A lot of type assertions have been inserted...a lot.
  • All testing utilities have migrated to TS.
  • text-diff is being replaced with diff for TS compatibility.
  • ProtocolError has been added to PuppeteerErrors and PuppeteerErrors is no longer a record (it's been frozen).
  • Fixes a small bug where null was an allowable media type in emulation (should be undefined).

Issues: #6769

@jrandolf jrandolf requested a review from OrKoN June 14, 2022 23:30
@jrandolf jrandolf force-pushed the strict_tests branch 5 times, most recently from 977f530 to 5e8fc99 Compare June 15, 2022 00:14
@OrKoN OrKoN changed the base branch from main to composite1 June 15, 2022 05:21
Base automatically changed from composite1 to main June 15, 2022 10:05
@jrandolf jrandolf merged commit 570087e into main Jun 15, 2022
@jrandolf jrandolf deleted the strict_tests branch June 15, 2022 10:09
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.

None yet

2 participants