Skip to content

Update fetch-mock-jest to @fetch-mock/jest#5136

Merged
t3chguy merged 22 commits intodevelopfrom
t3chguy/fetch-mock
Jan 14, 2026
Merged

Update fetch-mock-jest to @fetch-mock/jest#5136
t3chguy merged 22 commits intodevelopfrom
t3chguy/fetch-mock

Conversation

@t3chguy
Copy link
Member

@t3chguy t3chguy commented Jan 12, 2026

Plucked out from #5131
Depends on #5137

Any odd type-assertions are due to @fetch-mock/jest types being weak and kicking in on generics & unknown types, resulting in the available assertions going away. The types for extending assertions seem not to have this issue in vitest but given I was asked to split the jest->vitest PR here we are.

No clue how but this upgrade broke the comparisons between objects in the test & the unit under test contexts so had to temporarily patch deepCompare - this is reverted in #5131

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
This test failed when ran individually, same as after the clearAllMocks call

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
…ny other way

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
…ch-mock & fix typo

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy force-pushed the t3chguy/fetch-mock branch from ee46e52 to c58f6cc Compare January 12, 2026 16:41
@t3chguy t3chguy changed the base branch from develop to t3chguy/prepare-vitest January 12, 2026 16:41
@t3chguy t3chguy force-pushed the t3chguy/fetch-mock branch 2 times, most recently from c6e888f to dcf7bd9 Compare January 12, 2026 16:57
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy force-pushed the t3chguy/fetch-mock branch from dcf7bd9 to 758439b Compare January 12, 2026 17:47
@t3chguy t3chguy changed the title Update fetch-mock Update fetch-mock-jest to @fetch-mock/jest Jan 12, 2026
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy added the T-Task Tasks for the team like planning label Jan 12, 2026
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy requested review from a team as code owners January 13, 2026 08:55
@t3chguy t3chguy requested review from MidhunSureshR, andybalaam, hughns and uhoreg and removed request for a team January 13, 2026 08:55
@t3chguy t3chguy marked this pull request as draft January 13, 2026 09:29
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Copy link
Member

@andybalaam andybalaam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable and OK from a Crypto point of view.

Base automatically changed from t3chguy/prepare-vitest to develop January 13, 2026 15:40
…chguy/fetch-mock

# Conflicts:
#	spec/integ/crypto/cross-signing.spec.ts
#	spec/integ/crypto/state-events.spec.ts
#	spec/unit/oidc/authorize.spec.ts
#	spec/unit/rust-crypto/rust-crypto.spec.ts
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Copy link
Member

@robintown robintown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MatrixRTC files look fine

@t3chguy t3chguy added this pull request to the merge queue Jan 14, 2026
Merged via the queue into develop with commit da7c671 Jan 14, 2026
32 checks passed
@t3chguy t3chguy deleted the t3chguy/fetch-mock branch January 14, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-Task Tasks for the team like planning

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants