From 177d6d9df0b5d9df134877859939fa940d399e20 Mon Sep 17 00:00:00 2001 From: Jonny Spicer Date: Sat, 15 Aug 2026 09:41:00 -0700 Subject: [PATCH] ci: run the config-merger test suite (86cb4jfk7) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 19 tests have existed in this repo and never run in CI. That is how test_actual_config_files came to assert San Francisco coordinates for five months after 64ff0c9 deliberately replaced them with generic placeholders. The repo has no Python packaging, so the config-merger's dependencies are named directly in the workflow. They mirror config-merger/Dockerfile, which installs the same two unpinned — those two lines need to stay in step. Tests live under config-merger/test/ rather than tests/, so the path is explicit. Verified that a wrong path exits non-zero, so a broken step fails rather than silently collecting nothing. --- .github/workflows/lint.yml | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index 4b0bd85..0b6ed04 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -24,3 +24,14 @@ jobs: - name: Dead code run: pre-commit run --all-files --show-diff-on-failure + + # This repo has no Python packaging, so the config-merger's dependencies + # are named here directly. They mirror config-merger/Dockerfile, which + # installs the same two unpinned — keep the two in step. + - run: pip install pyyaml mergedeep 'pytest>=8.0' + + # Tests live under config-merger/test/, not tests/, so the path is + # explicit. This suite had never run in CI until 86cb4jfk7; the failure + # it now protects against had gone unnoticed for five months. + - name: Tests + run: pytest config-merger/test/