Skip to content

Tests/rewrite to snapshots#1442

Open
zorkow wants to merge 11 commits intodevelopfrom
tests/rewrite_to_snapshots
Open

Tests/rewrite to snapshots#1442
zorkow wants to merge 11 commits intodevelopfrom
tests/rewrite_to_snapshots

Conversation

@zorkow
Copy link
Member

@zorkow zorkow commented Mar 12, 2026

Moves all tests that use our custom toXmlMatch method to use Jest's snapshot matching. The only exceptions are the tests in ConfigMacros. I need to think a bit about those.
Other tests that could probably be done with snapshot matching are those using toXmlArrayMatch.
Both are for a follow-up PR.

@codecov
Copy link

codecov bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.49%. Comparing base (5caee44) to head (71f6868).

Additional details and impacted files
@@            Coverage Diff             @@
##           develop    #1442     +/-   ##
==========================================
  Coverage    86.49%   86.49%             
==========================================
  Files          340      340             
  Lines        86044    86044             
  Branches      4833     3190   -1643     
==========================================
  Hits         74425    74425             
- Misses       11595    11619     +24     
+ Partials        24        0     -24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@zorkow zorkow marked this pull request as ready for review March 12, 2026 21:59
@zorkow zorkow requested a review from dpvc March 12, 2026 22:01
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