Skip to content

chore(tests): updated timedMatch test to account v2.9.1 test coverage changes - #107

Merged
petruki merged 1 commit into
masterfrom
staging
Jul 26, 2026
Merged

chore(tests): updated timedMatch test to account v2.9.1 test coverage changes#107
petruki merged 1 commit into
masterfrom
staging

Conversation

@petruki

@petruki petruki commented Jul 26, 2026

Copy link
Copy Markdown
Member

This pull request focuses on updating the Deno version used across the project and improving the reliability of the TimedMatch test suite. The most significant changes are grouped below.

Deno version upgrades:

  • Updated the base Deno image in .devcontainer/Dockerfile from 2.8.1 to 2.9.4 to ensure the development container uses the latest compatible version.
  • Updated Deno setup steps and matrix versions in GitHub Actions workflows (.github/workflows/master.yml and .github/workflows/sonar.yml) to use v2.9.4 instead of v2.8.1, ensuring consistency in CI environments. [1] [2] [3]
  • Bumped the delay import in tests/deps.ts from @std/async@1.4.0 to @std/async@1.5.0 for up-to-date dependencies.

Test suite reliability improvements:

  • Enhanced tests/utils/timed-match.test.ts to introduce a warmUpWorker() function that preps the regex worker before each test, reducing flakiness due to JIT/module-instrumentation costs, especially under coverage runs. Also added a mechanism to track and restore the intended maxTimeLimit for tests.
  • Modified test setup (beforeEach) and certain test cases to use setMaxTimeLimit and warmUpWorker, ensuring consistent timing and worker state across tests. [1] [2]

@petruki petruki added this to the v2.5.1 milestone Jul 26, 2026
@petruki petruki self-assigned this Jul 26, 2026
@sonarqubecloud

Copy link
Copy Markdown

@petruki
petruki merged commit 47b240c into master Jul 26, 2026
7 checks passed
@petruki
petruki deleted the staging branch July 26, 2026 23:26
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