fix(test): add root_plan_id to raw SQL in plan_phase_migration constraint tests #10806

Merged
HAL9000 merged 4 commits from bugfix/m3-plan-phase-migration-raw-sql-root-plan-id into master 2026-05-03 23:11:40 +00:00

4 Commits

Author SHA1 Message Date
HAL9000 54385d5b42 ci: retrigger CI after infrastructure failure (attempt 2)
CI / lint (pull_request) Successful in 44s
CI / quality (pull_request) Successful in 52s
CI / build (pull_request) Successful in 35s
CI / helm (pull_request) Successful in 31s
CI / push-validation (pull_request) Successful in 38s
CI / security (pull_request) Successful in 1m42s
CI / typecheck (pull_request) Successful in 1m53s
CI / e2e_tests (pull_request) Successful in 3m41s
CI / integration_tests (pull_request) Successful in 4m31s
CI / unit_tests (pull_request) Successful in 5m52s
CI / docker (pull_request) Successful in 1m29s
CI / coverage (pull_request) Successful in 11m14s
CI / status-check (pull_request) Successful in 3s
2026-05-02 23:52:48 +00:00
HAL9000 70612e0585 ci: retrigger CI for bugfix/m3 branch after infrastructure failure
CI / lint (pull_request) Failing after 1s
CI / quality (pull_request) Failing after 1s
CI / push-validation (pull_request) Successful in 27s
CI / helm (pull_request) Successful in 31s
CI / build (pull_request) Successful in 57s
CI / security (pull_request) Successful in 1m32s
CI / typecheck (pull_request) Successful in 1m51s
CI / coverage (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 4m20s
CI / integration_tests (pull_request) Successful in 5m41s
CI / unit_tests (pull_request) Successful in 6m10s
CI / docker (pull_request) Has been skipped
CI / status-check (pull_request) Failing after 3s
2026-04-25 00:16:18 +00:00
HAL9000 be8cd9b814 fix(test): update a2a SDK TDD test to use Client instead of A2AClient
CI / typecheck (pull_request) Failing after 0s
CI / lint (pull_request) Failing after 0s
CI / security (pull_request) Failing after 0s
CI / quality (pull_request) Failing after 0s
CI / unit_tests (pull_request) Failing after 0s
CI / integration_tests (pull_request) Failing after 0s
CI / e2e_tests (pull_request) Failing after 0s
CI / coverage (pull_request) Has been skipped
CI / docker (pull_request) Has been skipped
CI / build (pull_request) Failing after 0s
CI / helm (pull_request) Failing after 0s
CI / push-validation (pull_request) Failing after 0s
CI / status-check (pull_request) Failing after 0s
2026-04-24 11:53:14 +00:00
HAL9000 a0fae4535e fix(test): add root_plan_id to raw SQL in plan_phase_migration constraint tests
CI / push-validation (pull_request) Successful in 27s
CI / helm (pull_request) Successful in 37s
CI / build (pull_request) Successful in 3m50s
CI / lint (pull_request) Successful in 3m57s
CI / quality (pull_request) Successful in 4m23s
CI / typecheck (pull_request) Successful in 4m40s
CI / security (pull_request) Successful in 4m50s
CI / unit_tests (pull_request) Failing after 5m51s
CI / docker (pull_request) Has been skipped
CI / e2e_tests (pull_request) Successful in 7m14s
CI / integration_tests (pull_request) Successful in 7m51s
CI / coverage (pull_request) Successful in 15m0s
CI / status-check (pull_request) Failing after 3s
The step_try_insert_plan_with_phase_and_state function was using raw SQL to
insert a plan with an invalid phase value to test the CHECK constraint. However,
the raw SQL was missing the root_plan_id column, which is NOT NULL in the schema.
This caused the insert to fail with a NOT NULL violation instead of the intended
CHECK constraint violation on the phase column.

This fix adds root_plan_id to the raw SQL INSERT statement, setting it to the
same value as plan_id (for a root plan). This allows the test to properly
exercise the CHECK constraint on the phase column, ensuring the test fails for
the correct reason.

ISSUES CLOSED: #9411
2026-04-15 16:38:29 +00:00