Feature/integration tests 7 (#21)

* feat: add integration tests with docker-compose (#7) - wait-for, render, fetch, exec, seed PostgreSQL/MySQL with cross-table refs, idempotency, reset, create database/schema, create-if-missing tests

* fix: use inline escaped values in postgres insert_row/row_exists to fix TEXT-to-INTEGER coercion

* fix: reverse seed set order during reset to respect foreign key constraints

* fix: separate reset (reverse) and seed (forward) passes for FK-safe reset mode

* refactor: remove placeholder seed_sets from test specs and move inline seed specs to files

* Update .github/workflows/integration.yml

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* fix: PostgreSQL insert_row fails for tables with text primary keys (#22)

The insert_row function always used RETURNING COALESCE(CAST(id AS BIGINT), 0)
even when no auto_id_column was specified, defaulting to the id column.
For tables with text primary keys (like NetBird's accounts, users, groups),
CAST to BIGINT fails with a generic db error.
Fix: Only use RETURNING clause when auto_id_column is explicitly set.
When no auto_id is needed, use a simple INSERT without RETURNING.

* fix: address PR #21 review comments - remove unused CI client install, add cfg feature guards, fix README test name, document escaped literals rationale, reword CHANGELOG

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: mikkeldamsgaard

.github/workflows/integration.yml

@@ -51,10 +51,6 @@ jobs:
       - uses: actions/checkout@v4
       - uses: dtolnay/rust-toolchain@stable
       - uses: Swatinem/rust-cache@v2
-      - name: Install psql and mysql clients
-        run: |
-          sudo apt-get update -qq
-          sudo apt-get install -y -qq postgresql-client default-mysql-client
       - name: Run integration tests
         env:
           INTEGRATION: "1"

CHANGELOG.md

@@ -7,11 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+- Clarified that seed phases with only `create_if_missing` can omit the `seed_sets` field entirely (`seed_sets` defaults to empty via `#[serde(default)]`); updated integration test YAML specs accordingly
+
 ### Added
 - Integration tests with docker-compose for end-to-end testing against real Postgres 16, MySQL 8.0, and nginx services (`tests/integration_test.rs`): wait-for TCP/HTTP/timeout/multiple targets, render template, fetch HTTP, exec command, seed PostgreSQL and MySQL with cross-table reference verification, create database/schema, idempotency, and reset mode
 - Additional create-if-missing integration tests: 2 PostgreSQL and 2 MySQL tests using known non-existing database names (`initium_noexist_alpha`, `initium_noexist_beta`) to verify database creation, existence checks, and idempotent re-runs
 - `tests/docker-compose.yml` with Postgres, MySQL, and HTTP health-check server definitions
-- `tests/fixtures/` with seed spec files and template for integration tests
+- `tests/input/` with seed spec files and template for integration tests
 - Separate GitHub Actions workflow (`.github/workflows/integration.yml`) for integration tests with service containers
 - Helm chart unit tests using helm-unittest plugin (`charts/initium/tests/deployment_test.yaml`) covering deployment rendering, securityContext enforcement, disabled sampleDeployment, multiple initContainers, extraVolumes/extraVolumeMounts, image configuration, workdir mount, and labels
 - `helm unittest` step added to CI helm-lint job with automatic plugin installation

src/seed/db.rs

@@ -307,6 +307,11 @@ impl Database for PostgresDb {
         Ok(())
     }
 
+    // Seed values are untyped strings that may target columns of any type
+    // (INTEGER, TEXT, etc.). Parameterized queries with $N send values as
+    // TEXT, which postgres cannot implicitly cast to other types. String
+    // literals have type `unknown` and auto-cast to the target column type,
+    // so we use escaped literals here. Identifiers are still sanitized.
     fn insert_row(
         &mut self,
         table: &str,

tests/README.md

@@ -30,7 +30,7 @@ docker compose -f tests/docker-compose.yml down
 | `test_waitfor_tcp_postgres`           | wait-for TCP against Postgres succeeds                   |
 | `test_waitfor_tcp_mysql`              | wait-for TCP against MySQL succeeds                      |
 | `test_waitfor_http_server`            | wait-for HTTP against nginx returns 200                  |
-| `test_waitfor_nonexistent_service`    | wait-for against closed port fails with exit code 1      |
+| `test_waitfor_nonexistent_service_timeout` | wait-for against closed port fails with exit code 1 |
 | `test_waitfor_multiple_targets`       | wait-for with Postgres + MySQL + HTTP all reachable      |
 | `test_render_template`                | render envsubst template produces correct output         |
 | `test_fetch_from_http_server`         | fetch from nginx writes HTML to file                     |

tests/input/create-db-mysql.yaml

@@ -0,0 +1,15 @@
+database:
+  driver: mysql
+  url_env: MYSQL_URL
+  tracking_table: initium_seed
+
+phases:
+  - name: create_db
+    order: 1
+    database: initium_created_db
+    create_if_missing: true
+    seed_sets:
+      - name: placeholder
+        tables:
+          - table: products
+            rows: []

tests/input/create-db-postgres.yaml

@@ -0,0 +1,15 @@
+database:
+  driver: postgres
+  url_env: POSTGRES_URL
+  tracking_table: initium_seed
+
+phases:
+  - name: create_db
+    order: 1
+    database: initium_created_db
+    create_if_missing: true
+    seed_sets:
+      - name: placeholder
+        tables:
+          - table: departments
+            rows: []

tests/input/create-nonexistent-db-alpha-mysql.yaml

@@ -0,0 +1,10 @@
+database:
+  driver: mysql
+  url_env: MYSQL_URL
+  tracking_table: initium_seed
+
+phases:
+  - name: create_alpha
+    order: 1
+    database: initium_noexist_alpha
+    create_if_missing: true

tests/input/create-nonexistent-db-alpha-postgres.yaml

@@ -0,0 +1,10 @@
+database:
+  driver: postgres
+  url_env: POSTGRES_URL
+  tracking_table: initium_seed
+
+phases:
+  - name: create_alpha
+    order: 1
+    database: initium_noexist_alpha
+    create_if_missing: true

tests/input/create-nonexistent-db-beta-mysql.yaml

@@ -0,0 +1,10 @@
+database:
+  driver: mysql
+  url_env: MYSQL_URL
+  tracking_table: initium_seed
+
+phases:
+  - name: create_beta
+    order: 1
+    database: initium_noexist_beta
+    create_if_missing: true

tests/input/create-nonexistent-db-beta-postgres.yaml

@@ -0,0 +1,10 @@
+database:
+  driver: postgres
+  url_env: POSTGRES_URL
+  tracking_table: initium_seed
+
+phases:
+  - name: create_beta
+    order: 1
+    database: initium_noexist_beta
+    create_if_missing: true