Skip to content

test(google): guard DataOps contract test selection - #34

Merged
alexeygrigorev merged 1 commit into
mainfrom
wt-1d9cf506-harness-guard
Nov 3, 2026
Merged

alexeygrigorev merged 1 commit into
mainfrom
wt-1d9cf506-harness-guard

Conversation

@alexeygrigorev

Copy link
Copy Markdown
Member

This stacked follow-up hardens the synthetic DataOps contract harness after the pinned PR 307 consumer renamed its dry-run test.

Changes:

  • Match the pinned CSV importer dry-run contract preserves raw amount precision and formatted dates title.
  • Use Node's spec reporter and fail unless the expected test title appears in the executed output, so a zero-match filter cannot pass.
  • Add a synthetic regression for the zero-selected-tests guard and document it.

Validation on this branch:

  • PYTHONPATH=. pytest -q plugins/google/tests/test_google_data_pull.py: 5 passed, 1 skipped.
  • python3 .agents/skills/google-data-pull/run_dataops_contract.py: pinned DataOps Node dry-run 1 passed; Dapier exporter tests with the DataOps checkout 6 passed.
  • Pinned DataOps PR 307 head: 5de43419c7e1ce890f4e11e083c68a9cf76e9ec6.

Base this PR on Dapier PR #33's branch so this remains a separate reviewable follow-up. Do not merge or deploy; the parent PR remains open.

@alexeygrigorev
alexeygrigorev changed the base branch from wt-1d9cf506 to main November 3, 2026 17:32
@alexeygrigorev
alexeygrigorev force-pushed the wt-1d9cf506-harness-guard branch from 776c857 to 7c92d03 Compare November 3, 2026 17:32
@alexeygrigorev
alexeygrigorev merged commit 0f8c6ff into main Nov 3, 2026
13 checks passed
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