Skip to content

Commit 97dfe9d

Browse files
committed
Fix stale AI-extras install-hint test to match actual command
The reviewer emits a quoted, checkout-relative install command; assert the actual _ai_extras_install_cmd() appears twice instead of a hardcoded string that never matched (editable installs log a path).
1 parent 855a5ff commit 97dfe9d

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

tests/ai_reviewer/test_reviewer.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
AIReviewer,
77
AIReviewerProtocol,
88
UnavailableAIReviewer,
9+
_ai_extras_install_cmd,
910
build_ai_reviewer,
1011
)
1112
from jumper_extension.monitor.common import UnavailablePerformanceMonitor
@@ -37,7 +38,7 @@ def test_unavailable_ai_reviewer_logs_install_hint_for_review_and_resume(caplog)
3738
reviewer.review(shell=Mock())
3839
reviewer.resume(shell=Mock(), run_id="abc123", select=1)
3940

40-
assert caplog.text.count("pip install jumper-extension[ai]") == 2
41+
assert caplog.text.count(_ai_extras_install_cmd()) == 2
4142

4243

4344
def test_ai_reviewer_starts_unattached_and_binds_monitor_via_attach():

0 commit comments

Comments
 (0)