Skip to content

feat: Add HTTP response status code matching support#1790

Draft
Copilot wants to merge 18 commits intomasterfrom
copilot/add-status-code-matching-support
Draft

feat: Add HTTP response status code matching support#1790
Copilot wants to merge 18 commits intomasterfrom
copilot/add-status-code-matching-support

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 6, 2026

  • Rebase branch onto latest origin/master
  • Resolve rebase conflicts in status matcher files
  • Reconcile branch divergence with existing remote branch history
  • Review compatibility with current repository test stack (Vitest, no sinon)
  • Update status matcher test to use Vitest (vi.fn) instead of sinon/chai-sinon
  • Run npm run check:fix
  • Run targeted tests for changed areas
  • Run full test suite (npm test)

Copilot AI linked an issue May 6, 2026 that may be closed by this pull request
2 tasks
Copilot AI and others added 4 commits May 6, 2026 04:50
Since we have consolidated to vitest, the jest and mocha regression
suites are actually not currently functional.

If we want, we can create dedicated tests for these later, but deleting
for now as I don't think they serve much benefit.

Signed-off-by: JP-Ellis <josh@jpellis.me>
…, StatusCodeMatcher)

Agent-Logs-Url: https://github.com/pact-foundation/pact-js/sessions/5ef516cc-a979-4b12-9ecf-06f58a7622be

Co-authored-by: mefellows <53900+mefellows@users.noreply.github.com>
Copilot AI changed the title [WIP] Add support for status code matching feat: Add HTTP response status code matching support May 6, 2026
Copilot AI requested a review from mefellows May 6, 2026 04:59
JP-Ellis and others added 7 commits May 6, 2026 18:48
Replace mocha with vitest for faster test running, and native TS
support. Also reduce the number of dependencies by switching to vitest's
own assertion library.

Signed-off-by: JP-Ellis <josh@jpellis.me>
Signed-off-by: JP-Ellis <josh@jpellis.me>
Not sure why this wasn't covered in CI before

Signed-off-by: JP-Ellis <josh@jpellis.me>
In particular, consolidate the x86 and ARM tests into one job now that
we have ARM runners available.

Signed-off-by: JP-Ellis <josh@jpellis.me>
Helps with cross-platform support, in case `bash` is not available or
easily invoked (e.g., CI in Windows running in pwsh).

Signed-off-by: JP-Ellis <josh@jpellis.me>
Signed-off-by: JP-Ellis <josh@jpellis.me>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Copilot AI and others added 6 commits May 6, 2026 23:48
Co-authored-by: mefellows <53900+mefellows@users.noreply.github.com>
…, StatusCodeMatcher)

Agent-Logs-Url: https://github.com/pact-foundation/pact-js/sessions/5ef516cc-a979-4b12-9ecf-06f58a7622be

Co-authored-by: mefellows <53900+mefellows@users.noreply.github.com>
…est test

Co-authored-by: mefellows <53900+mefellows@users.noreply.github.com>
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.

Adding support for status code matching

3 participants