Skip to content

chore: consolidate test directories — delete stale root tests/ and src/#143

Merged
longieirl merged 9 commits intomainfrom
ci/notify-downstream-on-core-release
Apr 12, 2026
Merged

chore: consolidate test directories — delete stale root tests/ and src/#143
longieirl merged 9 commits intomainfrom
ci/notify-downstream-on-core-release

Conversation

@longieirl
Copy link
Copy Markdown
Owner

Summary

  • Ports valuable unique tests from root tests/ to packages/parser-core/tests/ (test_output_strategies, test_recursive_scan_entitlements_integration)
  • Deletes stale root tests/ (gitignored, never tracked) and src/ directories; removes their .gitignore entries
  • Redirects all Makefile test targets (test, test-unit, test-integration, test-fast, test-watch, coverage, ci-test) to packages/parser-core/tests/ with 91% coverage gate
  • Adds CI guardrail job (check-test-structure) that fails if tests/ or src/ reappear at repo root

Test plan

  • make test exits 0 with ≥ 91% coverage (currently 94.34%)
  • packages/parser-core/tests/ passes 1580 tests, 0 failures
  • tests/ and src/ directories are absent from the repo
  • CI check-test-structure job passes on this PR

@longieirl longieirl self-assigned this Apr 11, 2026
@github-actions github-actions bot added documentation Improvements or additions to documentation ci labels Apr 11, 2026
@longieirl longieirl merged commit ceddc64 into main Apr 12, 2026
12 checks passed
@longieirl longieirl deleted the ci/notify-downstream-on-core-release branch April 12, 2026 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants