Skip to content

Drop 3 low-value Issues regression tests - #8556

Merged
TomasVotruba merged 1 commit into
mainfrom
tv-drop-redundant-issue-tests
Oct 8, 2026
Merged

TomasVotruba merged 1 commit into
mainfrom
tv-drop-redundant-issue-tests

Conversation

@TomasVotruba

Copy link
Copy Markdown
Member

Reviewed all ~60 tests/Issues/ dirs. The set is tight - each dir guards a distinct rule interaction, parser edge, infinite-loop, printer-crash, scope or config-API case, with no true duplicates. Only 3 earn removal:

  • RenameString - the two registered rules do not actually interact; the fixture shows only RenameStringRector acting, so the pairing is contrived.
  • EmptyLongArraySyntax - SensitiveConstantNameRector is unrelated noise; the real check (param default copied from parent prints right) is narrow.
  • GetValueMagicDir - custom test-rule for __DIR__ value resolution, an engine feature already covered by core unit tests.

Everything else - including sibling-package rules (TestWith/TestDoc/Downgrade) whose only local coverage is their Issues dir - is kept.

@TomasVotruba
TomasVotruba merged commit d97ad6c into main Oct 8, 2026
50 checks passed
@TomasVotruba
TomasVotruba deleted the tv-drop-redundant-issue-tests branch October 8, 2026 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant