Skip to content

Add dockerized test coverage and CI workflow#12

Merged
merval merged 3 commits into
mainfrom
codex/add-test-coverage-and-github-actions
May 31, 2026
Merged

Add dockerized test coverage and CI workflow#12
merval merged 3 commits into
mainfrom
codex/add-test-coverage-and-github-actions

Conversation

@merval

@merval merval commented May 31, 2026

Copy link
Copy Markdown
Member

Motivation

  • Provide broad automated unit test coverage for client, config, routes, and service logic to catch regressions early.
  • Enable running the same test suite inside a Docker container so CI can validate behavior in a hermetic image.

Description

  • Added many new and expanded unit tests under tests/ covering client helpers (tests/test_client.py), config loading (tests/test_config.py), routes (tests/test_routes.py), and services (tests/test_services.py).
  • Updated Dockerfile to introduce a test build stage that copies the test suite and runs python -m unittest discover -s tests -v, and kept a separate runtime stage for the app server.
  • Added a GitHub Actions workflow .github/workflows/tests.yml that builds the Docker test target and runs the test container on pushes to main, pull requests, and manual dispatch.
  • Documented test commands in README.md and added .dockerignore to keep Docker contexts small.

Testing

  • Ran python -m unittest discover -s tests -v locally which executed 49 tests and all passed (OK).
  • Ran python -m unittest discover -v locally which also executed 49 tests and all passed (OK).
  • Verified the CI flow by adding a workflow that builds and runs the Docker test image, and attempted a local Docker build/run but it could not be executed here because Docker is not available in this environment (docker: command not found).

Codex Task

@merval merval merged commit 1e524ac into main May 31, 2026
1 check passed
@merval merval deleted the codex/add-test-coverage-and-github-actions branch May 31, 2026 03:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant