Skip to content

test: isolate price proxy export coverage - #51

Merged
Pigbibi merged 2 commits into
mainfrom
refactor/extract-price-proxy-tests
Oct 31, 2026
Merged

Pigbibi merged 2 commits into
mainfrom
refactor/extract-price-proxy-tests

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Oct 31, 2026

Copy link
Copy Markdown
Contributor

Problem

Price-proxy research export and consumed-input manifest checks are embedded in a large BTC cycle test module. This makes the exporter contract harder to locate and review independently.

Change

Move the five existing exporter and manifest tests into tests/test_price_proxy_research_export.py, with their names, signatures, decorators, bodies and assertions unchanged. Remove only the two imports no longer used by the original module. The small hash helper remains local to each test module.

Validation

  • Compared the complete source text and AST of all five moved functions with the baseline; they are identical.
  • Confirmed all 69 remaining functions retain their AST and checked for module-level fixtures, marks and references to the old test paths.
  • Compared collection identities: all 304 test names and parameter sets remain; only five expected file-path prefixes change, with each test collected once.
  • Ran the full suite: 286 passed, 18 skipped. The 18 existing skips still require an actual QSP v2 module path; their conditions are unchanged.
  • Independent review repeated the full suite and collection comparison with the same result and zero external network attempts.
  • git diff --check and the repository documentation/style check pass.

Production code, dependencies, lock files, configuration and workflows are unchanged. The candidate tree matches the independently reviewed local tree.

Rollback

Revert this test-only commit. No runtime or data migration is involved.

@Pigbibi
Pigbibi marked this pull request as ready for review October 31, 2026 12:00
@Pigbibi
Pigbibi merged commit 3741bb8 into main Oct 31, 2026
3 checks passed
@Pigbibi
Pigbibi deleted the refactor/extract-price-proxy-tests branch October 31, 2026 12:02
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