Skip to content

fix(contracts): align official backtest request fixture - #71

Merged
hjcud merged 2 commits into
developfrom
feature/3-4-official-backtest-fixture
Aug 8, 2026
Merged

fix(contracts): align official backtest request fixture#71
hjcud merged 2 commits into
developfrom
feature/3-4-official-backtest-fixture

Conversation

@hjcud

@hjcud hjcud commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

What changed

  • vendored the backend-owned OFFICIAL_BACKTEST_REQUESTED valid fixture byte-for-byte
  • required all nine provider runtime fields in the consumer schema and contract tests
  • made BASIC dispatch verify the request period, dataset hash, and feature materialization pins against the provider-created run
  • aligned lifecycle/API test harness identities with the provider-registered run and execution policy

Why

Launch-readiness task 3.4 requires the backtest consumer fixture to match the backend producer fixture exactly. The consumer copy was missing nine runtime fields, and BASIC dispatch was substituting stored values for several message fields instead of checking provider/consumer run identity.

Impact

The backtest engine now rejects incomplete official requests and detects BASIC run identity drift before publishing a job. This unblocks the root parity gate and downstream INT01 once this PR is merged and the root submodule pointer is advanced by the root owner.

Validation

  • python -m pytest -q — passed (3 skipped)
  • ruff check . — passed
  • producer and consumer fixture Git blob hashes both 1b754da1a507d64f0479361588dd0636c23d4fec
  • git diff --cached --check — passed

@hjcud
hjcud marked this pull request as ready for review August 8, 2026 12:32
@hjcud
hjcud merged commit e950bee into develop Aug 8, 2026
9 checks passed
@hjcud
hjcud deleted the feature/3-4-official-backtest-fixture branch August 8, 2026 12:33
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