From e0c0e53ae042ce253e2bb07466b742836ee3ca3b Mon Sep 17 00:00:00 2001 From: Gabriel Brown Date: Wed, 19 Aug 2026 12:22:38 -0400 Subject: [PATCH] Say what the sweep helper is, when run on its own A suite runner walking the tests directory executes it with no arguments and gets an IndexError, which reads as a failing test rather than a helper being used wrongly. Claude-Session: https://claude.ai/code/session_01BRvzt4H8XXLPVH5MyYdk9L --- tests/quickshell/settings_write_sweep.py | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/tests/quickshell/settings_write_sweep.py b/tests/quickshell/settings_write_sweep.py index f8b396c..9cb9f79 100644 --- a/tests/quickshell/settings_write_sweep.py +++ b/tests/quickshell/settings_write_sweep.py @@ -178,6 +178,14 @@ def ipc(config_home: str, harness: str, *arguments: str) -> subprocess.Completed def main() -> int: + # Driven by settings-write-sweep-contract.sh, which starts the harness this + # needs and restores the desktop afterwards. Run bare -- by a suite runner + # walking the directory, say -- it says so instead of failing on argv. + if len(sys.argv) < 3: + print("settings_write_sweep is driven by tests/quickshell/" + "settings-write-sweep-contract.sh; run that instead.") + return 0 + config_home = sys.argv[1] harness = sys.argv[2]