Skip to content

Commit f4854b6

Browse files
abrichrclaude
andcommitted
fix: update verify_apps tests to use lightweight=False
Tests expected verify_apps to be called, but lightweight mode (now default) skips it. Tests now explicitly set lightweight=False. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent b017b6e commit f4854b6

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

tests/test_setup_handlers.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -385,9 +385,11 @@ def test_verify_apps_prepended(self):
385385
"""related_apps in raw_config causes verify_apps step to be prepended."""
386386
from openadapt_evals.adapters import WAALiveAdapter, WAALiveConfig
387387

388+
# lightweight=False to test verify_apps injection (default skips it)
388389
adapter = WAALiveAdapter(WAALiveConfig(
389390
server_url="http://test:5000",
390391
evaluate_url="http://test:5050",
392+
lightweight=False,
391393
))
392394

393395
raw_config = {
@@ -446,12 +448,13 @@ def test_no_verify_without_related_apps(self):
446448
assert "Popen" in cmd # launch command
447449

448450
def test_empty_config_with_related_apps(self):
449-
"""related_apps with empty config still injects verify_apps."""
451+
"""related_apps with empty config still injects verify_apps (non-lightweight)."""
450452
from openadapt_evals.adapters import WAALiveAdapter, WAALiveConfig
451453

452454
adapter = WAALiveAdapter(WAALiveConfig(
453455
server_url="http://test:5000",
454456
evaluate_url="http://test:5050",
457+
lightweight=False,
455458
))
456459

457460
raw_config = {

0 commit comments

Comments
 (0)