test_integration.py: generated SQL test needs CREATE TABLE to validate #149

New issue

Closed

opened 2026-07-06 07:07:41 +00:00 by the.auditor · 0 comments

the.auditor commented

2026-07-06 07:07:41 +00:00

Owner

tests/test_integration.py:121-141 — test_generated_sql_is_valid_sqlite3 runs sqlite3 :memory: .read on generated SQL, but there's no CREATE TABLE action (...) statement. The INSERT will fail with "no such table: action" if sqlite3 is installed.

The test is silently skipped on systems without sqlite3, only manifesting as a failure when sqlite3 IS present.

Fix: either create the table in the SQL before validating, or add a CREATE TABLE statement to the generated SQL being tested.

Ref: PR #148

`tests/test_integration.py:121-141` — `test_generated_sql_is_valid_sqlite3` runs `sqlite3 :memory: .read` on generated SQL, but there's no `CREATE TABLE action (...)` statement. The INSERT will fail with "no such table: action" if sqlite3 is installed. The test is silently skipped on systems without sqlite3, only manifesting as a failure when sqlite3 IS present. Fix: either create the table in the SQL before validating, or add a CREATE TABLE statement to the generated SQL being tested. Ref: PR #148

the.auditor added the

Kind/Testing

Priority

High

labels

2026-07-06 07:07:44 +00:00