mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-07-31 19:16:29 +00:00
3e89edf830
Add a module-scoped `shared_container` fixture to tests/docker/conftest.py that boots one `sleep infinity` container per test module and tears it down at module exit. Convert read-only tests that previously used `docker run --rm --entrypoint sh/cat/test/su` (bypassing s6 to check static image properties) or `docker run -d` + `docker exec` (starting identical containers per test) to use `docker exec` on the shared container instead. Converted files: test_immutable_install_permissions.py — 2 throwaway runs → 2 execs test_license_file_present.py — 1 throwaway run → 1 exec test_tini_compat_shim.py — 1 throwaway run → 1 exec test_tui_prebuilt_bundle.py — 2 throwaway runs → 2 execs test_dump_build_sha.py — 2 throwaway runs → 2 execs test_immutable_install.py — 3 detached runs → 1 shared + 1 isolated test_dashboard.py — 2 detached runs → 0 (use shared) Local profiling shows docker run calls in these 7 files dropped from ~25 to 7 (the 7 are shared_container boots per module + the one test that needs a restart). Each eliminated `docker run` was paying 1-9s of s6 cont-init startup; the replacement `docker exec` calls average 0.10s — an ~50x speedup per operation. Tests that mutate state (restarts, config changes, gateway starts) still use their own containers via `container_name` + `start_container`.
22 lines
793 B
Python
22 lines
793 B
Python
"""Runtime smoke test for Docker image license-file presence.
|
|
|
|
Build the real image and verify the LICENSE file is present inside the
|
|
container (PEP 639 license-files metadata must resolve inside the
|
|
Docker image).
|
|
"""
|
|
from __future__ import annotations
|
|
|
|
from tests.docker.conftest import docker_exec
|
|
|
|
|
|
def test_docker_image_contains_license_file(shared_container: str) -> None:
|
|
"""The LICENSE file must be present inside the built Docker image.
|
|
|
|
PEP 639 license-files metadata references LICENSE, and the Docker
|
|
build context must not exclude it.
|
|
"""
|
|
r = docker_exec(shared_container, "test", "-f", "/opt/hermes/LICENSE")
|
|
assert r.returncode == 0, (
|
|
f"LICENSE file not found at /opt/hermes/LICENSE inside the Docker "
|
|
f"image: {r.stderr[-500:]}"
|
|
)
|