Skip to content

Comments

test(functional): use inline-snapshot#1464

Open
mkniewallner wants to merge 11 commits intomainfrom
test/use-inline-snapshot
Open

test(functional): use inline-snapshot#1464
mkniewallner wants to merge 11 commits intomainfrom
test/use-inline-snapshot

Conversation

@mkniewallner
Copy link
Collaborator

@mkniewallner mkniewallner commented Feb 15, 2026

Use inline-snapshot in functional tests to make it easier to maintain test outputs. Since the snapshots are now automatically updated, we can also use the text reporter in tests instead of JSON output.

Because Windows outputs paths differently, I've added an hidden argument that enforces POSIX paths.

I've also added a method in VirtualEnv to run deptry, which by default enforces POSIX paths, and disables ANSI characters to avoid colours in the test outputs.

@codecov
Copy link

codecov bot commented Feb 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.1%. Comparing base (610e4ed) to head (ac8d107).

Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1464     +/-   ##
=======================================
+ Coverage   93.6%   97.1%   +3.5%     
=======================================
  Files         38      38             
  Lines       1034    1041      +7     
  Branches     102     103      +1     
=======================================
+ Hits         968    1011     +43     
+ Misses        53      21     -32     
+ Partials      13       9      -4     

☔ 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.

@mkniewallner mkniewallner force-pushed the test/use-inline-snapshot branch 2 times, most recently from cf3949e to 285cbac Compare February 21, 2026 16:33
@mkniewallner mkniewallner force-pushed the test/use-inline-snapshot branch 10 times, most recently from c8b9b96 to 7ff52d3 Compare February 21, 2026 18:58
@mkniewallner mkniewallner force-pushed the test/use-inline-snapshot branch from 631c4b6 to 27a4182 Compare February 24, 2026 22:05
@mkniewallner mkniewallner force-pushed the test/use-inline-snapshot branch from 662b940 to f505c37 Compare February 24, 2026 23:27
@mkniewallner mkniewallner marked this pull request as ready for review February 24, 2026 23:50
@mkniewallner mkniewallner requested a review from fpgmaas February 24, 2026 23:50
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