Skip to content

fix: only trim snapshots after complete test runs - #382

Open
15r10nk wants to merge 1 commit into
mainfrom
fix/trim-only-complete-runs
Open

fix: only trim snapshots after complete test runs#382
15r10nk wants to merge 1 commit into
mainfrom
fix/trim-only-complete-runs

Conversation

@15r10nk

@15r10nk 15r10nk commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • apply trim changes only when no test call failed
  • keep trim enabled for skipped tests and fixture teardown errors
  • suppress source and external trim changes for -k selections, file selections, and node IDs
  • update existing expectations and add regression coverage

Fixes #147

Tests

  • 42 affected plugin/config/external tests passed
  • Python 3.10 low/high help-message test passed
  • all pre-commit hooks passed

@15r10nk
15r10nk force-pushed the fix/trim-only-complete-runs branch 2 times, most recently from 6c562bf to 5c9427f Compare July 19, 2026 13:55
@15r10nk
15r10nk force-pushed the fix/trim-only-complete-runs branch from 5c9427f to b9dd7b5 Compare July 19, 2026 17:23
@15r10nk
15r10nk force-pushed the fix/trim-only-complete-runs branch from b9dd7b5 to bb6dd78 Compare July 20, 2026 08:52
@15r10nk
15r10nk force-pushed the fix/trim-only-complete-runs branch from bb6dd78 to 11950f4 Compare July 24, 2026 21:07
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.

trim should only remove things if all tests where executed successfully

1 participant