Skip to content

[codex] remove fixed test count gates - #538

Merged
kachofugetsu09 merged 1 commit into
mainfrom
feature/test-cleanup-followup
Sep 4, 2026
Merged

[codex] remove fixed test count gates#538
kachofugetsu09 merged 1 commit into
mainfrom
feature/test-cleanup-followup

Conversation

@kachofugetsu09

Copy link
Copy Markdown
Owner

Summary

  • remove the hard-coded 1080 Python test budget, retained-file manifest, and 62 Web test count assertion
  • keep CI running the repository's actual Python tests and make the Web runner discover .test.mjs files from the source roots
  • fix three ESLint errors in kaomoji-markdown without changing matching behavior
  • document the current test-selection contract and recovery point

Why

The fixed counts turned a historical cleanup snapshot into a false CI contract. The current Web suite has 56 tests, so the old 62-test assertion rejected the real suite. Test retention now follows observable regressions, invariants, boundaries, and concrete bugs.

Checks

  • npm ci --no-audit --no-fund
  • npm run test:web (56 passed)
  • npm run typecheck
  • npm run lint (0 errors; 42 existing warnings)
  • .venv/bin/pytest -q tests/ (1075 passed, 5 skipped)
  • SDK pyright and pytest (11 passed)
  • Python pyright and control schema check
  • change-impact Gate passed: docker/debug/reports/change-gate/20260904-121135-04ec73d2

@kachofugetsu09
kachofugetsu09 marked this pull request as ready for review September 4, 2026 13:20
@kachofugetsu09
kachofugetsu09 merged commit 6a15444 into main Sep 4, 2026
2 checks passed
@kachofugetsu09
kachofugetsu09 deleted the feature/test-cleanup-followup branch September 4, 2026 13:20
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