Skip to content

Add scenario SQL smoke runner#814

Merged
bill-ph merged 1 commit into
mainfrom
codex/scenario-runner-pr3
Jun 25, 2026
Merged

Add scenario SQL smoke runner#814
bill-ph merged 1 commit into
mainfrom
codex/scenario-runner-pr3

Conversation

@bill-ph

@bill-ph bill-ph commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add tests/scenario/sql for PGWire SQL and inline SQL catalog scenario steps
  • add SNI/hostaddr DSN generation, transient startup retry handling, and SQL result recording
  • wire scripts/scenario_run.sh, just scenario-smoke, just scenario, and just test-scenario
  • replace the fake provision smoke with provision -> wait -> SELECT 1 -> deprovision
  • add a scenario runner runbook with dev cleanup guidance

Tests

  • go test -count=1 ./tests/scenario/...
  • just test-scenario
  • just lint

Review

  • Ran one review pass and fixed the findings around Go test timeout, run-unique smoke org/database names, shipped YAML validation, and docs for connect timeout.

@github-actions

Copy link
Copy Markdown

Test Impact Plan

Deterministic summary of how this PR changes tests, CI runners, and coverage-risk signals.

Summary

Area Added Changed Deleted
Test files 8 1 0
E2E/journey files 0 0 0
Workflow files 0 0 0

Signals

  • Test cases: +12 / -0
  • Assertions: +48 / -0
  • Skips or known failures added: 1
  • Workflow continue-on-error added: 0
  • Workflow path filters added: 0
  • Test commands removed from justfile: 0
  • E2E/journey retry lines added: 0

Coverage risk: likely reduced

Warnings

  • New skips or known failures (likely reduced)
    • 1 skip/allowlist line(s) added

@bill-ph bill-ph merged commit ea8d081 into main Jun 25, 2026
27 checks passed
@bill-ph bill-ph deleted the codex/scenario-runner-pr3 branch June 25, 2026 20:03
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