Skip to content

Bring the test-data convention over from iLEAPP - #431

Merged
abrignoni merged 2 commits into
mainfrom
feat/test-data-convention
Aug 24, 2026
Merged

Bring the test-data convention over from iLEAPP#431
abrignoni merged 2 commits into
mainfrom
feat/test-data-convention

Conversation

@abrignoni

Copy link
Copy Markdown
Owner

Brings the test-data convention over from iLEAPP.

  • Fixture tooling: make_test_data.py and test_module.py, with the iOS version mock made conditional so one copy runs in every core.
  • Baseline gate: run_test_cases.py and its workflow compare committed cases against recorded snapshots; inert until the first case lands.
  • The bot: comments on external artifact PRs that include no test data. Without an image manifest here, every module gets the full ask.
  • Adds the PR template.

Unit tests for the ported logic run in the existing runtime-contract job.

abrignoni and others added 2 commits August 24, 2026 15:39
Adds the fixture tooling (make_test_data.py, test_module.py), the
baseline gate (run_test_cases.py plus workflow), and the bot that asks
external artifact PRs for test data when none is included. The gate is
inert until the first case is committed. The bot routes every module to
the full ask until this repo has an image manifest. Adds the PR
template.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Underscore-prefixes the unused mock and signature arguments so the file
lands warning-free in repos where it is new.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@abrignoni abrignoni closed this Aug 24, 2026
@abrignoni
abrignoni deleted the feat/test-data-convention branch August 24, 2026 20:51
@abrignoni
abrignoni restored the feat/test-data-convention branch August 24, 2026 20:52
@abrignoni abrignoni reopened this Aug 24, 2026
@abrignoni
abrignoni merged commit b3e4e83 into main Aug 24, 2026
16 checks passed
@abrignoni
abrignoni deleted the feat/test-data-convention branch August 24, 2026 20:54
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