Skip to content

test(skills): expand utility behavior and boundary coverage - #5

Merged
sent-dm merged 1 commit into
mainfrom
codex/utility-behavior-tests
Aug 9, 2026
Merged

test(skills): expand utility behavior and boundary coverage#5
sent-dm merged 1 commit into
mainfrom
codex/utility-behavior-tests

Conversation

@sent-dm

@sent-dm sent-dm commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

What changed

  • Replaced the exit-code loop with named unittest.TestCase coverage.
  • Added direct validator tests for malformed roots, invalid records, and exact policy boundaries.
  • Added subprocess coverage for stdout/stderr diagnostics, missing files, malformed JSON, unsupported MDR extensions, and documented exit codes.
  • Normalized campaign and template input failures to their public 0/1 contracts and made MDR parser failures deterministic under 0/2/3.
  • Kept fixtures synthetic and privacy-safe, and narrowed contract tests to manifest/documentation conformance.

Why

The utility gates previously exercised only eight checked-in fixture exit codes. Parser tracebacks and inconsistent error streams were not protected by tests.

Validation

  • python3 scripts/generate_adapters.py --check
  • python3 scripts/validate.py
  • python3 scripts/test_fixtures.py
  • python3 scripts/test_contracts.py
  • python3 scripts/test_validation_gates.py
  • git diff --check

@sent-dm
sent-dm marked this pull request as ready for review August 9, 2026 22:21
@sent-dm
sent-dm merged commit 36e1ef1 into main Aug 9, 2026
2 checks passed
@sent-dm
sent-dm deleted the codex/utility-behavior-tests branch August 9, 2026 22:24
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