From cb189bacb10f76af751f77e0a419268c18d246e8 Mon Sep 17 00:00:00 2001 From: lrnselfreliance <33263708+lrnselfreliance@users.noreply.github.com> Date: Thu, 8 Oct 2026 17:01:59 -0600 Subject: [PATCH] Give the repos ownership tests a fixed git identity (#790) test_ownership.py's git() helper did not use TEST_GIT_ENV, so its commits read the machine's git identity. CI runners have none and cannot guess one, so seven tests failed there on every run. Co-authored-by: Claude Opus 5.5 --- modules/repos/test/test_ownership.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/modules/repos/test/test_ownership.py b/modules/repos/test/test_ownership.py index 914debd1..0ee5d871 100644 --- a/modules/repos/test/test_ownership.py +++ b/modules/repos/test/test_ownership.py @@ -1,14 +1,16 @@ +import os import subprocess import pytest from modules.repos import lib +from modules.repos.conftest import TEST_GIT_ENV from wrolpi.downloader import Download, DownloadStatus def git(directory, *args) -> str: return subprocess.run(['git', '-c', 'safe.directory=*', *args], cwd=directory, check=True, - capture_output=True).stdout.decode().strip() + env=dict(os.environ, **TEST_GIT_ENV), capture_output=True).stdout.decode().strip() async def update(test_session, repos_download_manager):