Skip to content

feat: improve native test diagnostics - #616

Merged
kvlonge merged 2 commits into
mainfrom
feat/chi-237-native-test-diagnostics
Sep 10, 2026
Merged

kvlonge merged 2 commits into
mainfrom
feat/chi-237-native-test-diagnostics

Conversation

@kvlonge

@kvlonge kvlonge commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Why

Native SQL tests exposed actionable-diagnostics gaps during large fixture migrations: missing columns failed one at a time in warehouses, collection type conflicts lacked fixture context, parameter cases could not run independently, and multi-model mock boundaries were difficult to inspect. Complex strings in VALUES also triggered a planner hang because expected CTEs were re-parsed with a nested-parenthesis regex.

Changes

  • align changed column values when exactly one safely sampled row differs each way
  • add sqb test --case NAME with parent/model selection-aware errors
  • add offline sqb test --inspect for mocks, replacement boundaries, real chains, expected models, and unsatisfied leaves
  • statically aggregate provable missing-column and collection/scalar fixture errors before connection
  • keep ambiguous star projections and transformed types conservative
  • carry expected CTEs structurally instead of regex re-parsing
  • preserve complex comma/bracket/parenthesis strings in DuckDB execution and Snowflake compilation
  • document all behavior and regenerate the packaged SQLBuild skill

Verification

  • 60 focused unit, integration, and real CLI tests passed with xdist
  • DuckDB execution and Snowflake-dialect compile fixture regressions
  • exact cache-focused SQL test coverage passed
  • uv sync --all-extras
  • make check-ci
  • Fensu: zero faults and generated skills current
  • one bounded independent review plus focused follow-up
  • public-repository hygiene scan and git diff --check
  • docs PR docs: explain native test diagnostics sqlbuild-docs#44 merged

@kvlonge
kvlonge enabled auto-merge (squash) September 10, 2026 17:54
@blacksmith-sh

This comment has been minimized.

@kvlonge
kvlonge merged commit 44231e7 into main Sep 10, 2026
18 checks passed
@kvlonge
kvlonge deleted the feat/chi-237-native-test-diagnostics branch September 10, 2026 18:18
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