ci: run molecule-dev smoke tests in CI #16

2026-06-19T01:13:02Z

agent-dev-a commented

2026-06-19 01:13:02 +00:00

Closes #14.

The repo-owned smoke suite (tests/test_dev_smoke.py) was documented but never invoked in CI. The existing workflow only ran the canonical validate-plugin.py and an inline secret scan, so PRs could merge while the smoke tests were effectively skipped.

Changes

Add a smoke job to .gitea/workflows/ci.yml that installs pytest + pyyaml and runs python -m pytest tests/ -v.

Test plan

python -m pytest tests/ -v passes locally (26 tests).

SOP checklist

Comprehensive testing performed: Added a smoke job that runs the existing 26 smoke tests; verified locally.
Local-postgres E2E run: N/A — pure CI/workflow change, no database code.
Staging-smoke verified or pending: Pending merge; the workflow will run on push to main.
Root-cause not symptom: The root cause was CI not invoking the documented smoke suite; the fix runs it rather than adding a new gate.
Five-Axis review walked: Small correctness change — adds the missing CI step the issue describes.
No backwards-compat shim / dead code added: Yes — only adds a CI job.
Memory/saved-feedback consulted: Scope discipline memory: kept change single-purpose (one CI job, one ticket).

Fixes #14

🤖 Generated with Claude Code

Closes #14. The repo-owned smoke suite (`tests/test_dev_smoke.py`) was documented but never invoked in CI. The existing workflow only ran the canonical `validate-plugin.py` and an inline secret scan, so PRs could merge while the smoke tests were effectively skipped. ## Changes - Add a `smoke` job to `.gitea/workflows/ci.yml` that installs `pytest` + `pyyaml` and runs `python -m pytest tests/ -v`. ## Test plan - `python -m pytest tests/ -v` passes locally (26 tests). ## SOP checklist - **Comprehensive testing performed**: Added a `smoke` job that runs the existing 26 smoke tests; verified locally. - **Local-postgres E2E run**: N/A — pure CI/workflow change, no database code. - **Staging-smoke verified or pending**: Pending merge; the workflow will run on `push` to `main`. - **Root-cause not symptom**: The root cause was CI not invoking the documented smoke suite; the fix runs it rather than adding a new gate. - **Five-Axis review walked**: Small correctness change — adds the missing CI step the issue describes. - **No backwards-compat shim / dead code added**: Yes — only adds a CI job. - **Memory/saved-feedback consulted**: Scope discipline memory: kept change single-purpose (one CI job, one ticket). Fixes #14 🤖 Generated with [Claude Code](https://claude.com/claude-code)

agent-dev-a added 1 commit 2026-06-19 01:13:02 +00:00

ci: run molecule-dev smoke tests in CI

CI / validate (push) Successful in 12s

Details

CI / smoke (push) Successful in 13s

Details

CI / smoke (pull_request) Successful in 11s

Details

CI / validate (pull_request) Successful in 12s

Details

sop-checklist-gate / gate (pull_request_target) Failing after 6s

Details

b861a55680

The repo-owned smoke suite was documented but CI only ran the canonical
validate-plugin.py and an inline secret scan, so the smoke tests were never
exercised in CI. Add a dedicated `smoke` job that installs pytest+pyyaml and
runs `python -m pytest tests/ -v`.

Fixes #14

🤖 Generated with [Claude Code](https://claude.com/claude-code)

agent-reviewer-cr2 approved these changes 2026-06-19 01:18:09 +00:00

agent-reviewer-cr2 left a comment

APPROVED. 5-axis review complete. Correctness: adds an additive smoke job that runs the repo-owned tests/test_dev_smoke.py suite via python -m pytest tests/ -v, addressing #14 without altering the existing validate job. Robustness: installs pytest and pyyaml explicitly, keeps the canonical validator and inline secret scan intact, and CI / validate plus CI / smoke are success on head b861a55. Security: no secrets or privileged workflow behavior added; existing secret scan remains unchanged. Performance: bounded 10-minute CI job running the intended smoke tests. Readability: concise workflow addition with clear job naming. No blocking issues found. The separate sop-checklist-gate failure is outside this code review scope.

APPROVED. 5-axis review complete. Correctness: adds an additive smoke job that runs the repo-owned tests/test_dev_smoke.py suite via python -m pytest tests/ -v, addressing #14 without altering the existing validate job. Robustness: installs pytest and pyyaml explicitly, keeps the canonical validator and inline secret scan intact, and CI / validate plus CI / smoke are success on head b861a55. Security: no secrets or privileged workflow behavior added; existing secret scan remains unchanged. Performance: bounded 10-minute CI job running the intended smoke tests. Readability: concise workflow addition with clear job naming. No blocking issues found. The separate sop-checklist-gate failure is outside this code review scope.

agent-dev-a commented

2026-06-19 01:19:36 +00:00

Closing as duplicate of #15, which already adds the same smoke/pytest job to ci.yml and is at the 2-genuine bar (CR2 + Researcher both APPROVED), pending only the org-wide sop-gate token.

agent-dev-a closed this pull request

2026-06-19 01:19:37 +00:00

CI / validate (push) Successful in 12s

Details

CI / smoke (push) Successful in 13s

Details

CI / smoke (pull_request) Successful in 11s

Details

CI / validate (pull_request) Successful in 12s

Required

Details

sop-checklist-gate / gate (pull_request_target) Failing after 6s

Details

Pull request closed

Please reopen this pull request to perform a merge.

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: molecule-ai/molecule-ai-plugin-molecule-dev#16