fix: add CREATE TABLE to SQL validation test #172

Merged

fuzzy merged 1 commit from fix/149-integration-create-table into main

2026-07-06 09:18:33 +00:00

automation commented

2026-07-06 09:16:39 +00:00

Owner

Prepends a CREATE TABLE action (...) statement before INSERT statements in the SQLite3 syntax validation test. This creates the target table so the INSERT statements can actually execute against :memory: database.

Closes #149

Prepends a `CREATE TABLE action (...)` statement before INSERT statements in the SQLite3 syntax validation test. This creates the target table so the INSERT statements can actually execute against `:memory:` database. Closes #149

automation added 1 commit

2026-07-06 09:16:39 +00:00

fix(tests): add CREATE TABLE to SQL validation test

CI / lint-and-test (pull_request) Successful in 1m24s

Details

c93fcb9227

Prepend CREATE TABLE statement before INSERT statements
when validating generated SQL against SQLite3. This makes
the test actually run the SQL instead of skipping.

Closes #149

fuzzy scheduled this pull request to auto merge when all checks succeed

2026-07-06 09:17:34 +00:00

the.auditor reviewed

2026-07-06 09:17:50 +00:00

the.auditor left a comment

Audit Review — PR #172

Clean fix for #149 (Priority/High). CREATE TABLE action (...) with all 6 columns prepended to SQL before sqlite3 :memory: .read validation. Test now actually validates INSERTs against a real schema.

No suggestions.

Summary

Result: COMMENT — no issues found.

## Audit Review — PR #172 Clean fix for #149 (Priority/High). `CREATE TABLE action (...)` with all 6 columns prepended to SQL before `sqlite3 :memory: .read` validation. Test now actually validates INSERTs against a real schema. No suggestions. ## Summary **Result:** COMMENT — no issues found.

the.auditor approved these changes

2026-07-06 09:17:53 +00:00

the.auditor left a comment

APPROVED. Test now actually validates SQL, closes #149. Merge when ready.