Skip to content

feat: add contract-driven test fixtures - #760

Closed
kvlonge wants to merge 1 commit into
mainfrom
feat/contract-driven-null-fixtures
Closed

kvlonge wants to merge 1 commit into
mainfrom
feat/contract-driven-null-fixtures

Conversation

@kvlonge

@kvlonge kvlonge commented Sep 21, 2026

Copy link
Copy Markdown
Contributor

Why

SQL tests currently repeat explicit typed-null projections even when relation metadata already provides authoritative column types. Zero-row fixtures also require authors to restate every column solely to preserve schema shape.

Changes

  • infer bare NULL AS column types for expected outputs as well as mocked relations
  • add SELECT * FROM __empty_fixture() for schema-known zero-row input and expected fixtures
  • let sqb format remove redundant typed-null mock columns when planner completion proves the rewrite safe
  • preserve authored SQL for set operations, macro-expanded fixtures, disabled SQL analysis, and ambiguous source mappings

Verification

  • focused compiler, planner, lint, integration, and DuckDB E2E tests: 102 passed
  • make check-ci
  • public repository hygiene scan

Tracking: CHI-321

@kvlonge
kvlonge enabled auto-merge (squash) September 21, 2026 03:53
@kvlonge
kvlonge disabled auto-merge September 21, 2026 03:54
@kvlonge kvlonge closed this Sep 21, 2026
@kvlonge
kvlonge deleted the feat/contract-driven-null-fixtures branch September 21, 2026 03:56
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