Skip to content

test: migrate context tests to vitest spies#4810

Merged
JoviDeCroock merged 2 commits intopreactjs:mainfrom
43081j:vitest-sinon-browser-context
Jun 25, 2025
Merged

test: migrate context tests to vitest spies#4810
JoviDeCroock merged 2 commits intopreactjs:mainfrom
43081j:vitest-sinon-browser-context

Conversation

@43081j
Copy link
Copy Markdown
Contributor

@43081j 43081j commented Jun 24, 2025

Migrates to using vitest spies.

Depends on #4807.

much of it used the codemod here:
https://github.com/43081j/codemods/blob/main/codemods/sinon-to-vitest.yml

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jun 24, 2025

📊 Tachometer Benchmark Results

Summary

A summary of the benchmark results will show here once they finish.

Results

The full results of your benchmarks will show here once they finish.

tachometer-reporter-action v2 for CI

@43081j 43081j force-pushed the vitest-sinon-browser-context branch from 588e101 to ae995e2 Compare June 24, 2025 19:53
@43081j 43081j force-pushed the vitest-sinon-browser-context branch from ae995e2 to 4ed553c Compare June 25, 2025 10:24
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 25, 2025

Coverage Status

coverage: 94.727%. remained the same
when pulling f14a260 on 43081j:vitest-sinon-browser-context
into 443e688 on preactjs:main.

@JoviDeCroock JoviDeCroock merged commit 2732718 into preactjs:main Jun 25, 2025
4 checks passed
@43081j 43081j deleted the vitest-sinon-browser-context branch June 25, 2025 15:27
@JoviDeCroock JoviDeCroock mentioned this pull request Aug 19, 2025
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.

3 participants