test_sim reports SUCCEEDED with 0 tests when per-call projectPath is ignored
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 52/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- typescript
- Lĩnh vực
- devtools, mobile-dev, testing-qa
Hướng nghiên cứu
Bắt đầu từ công cụ test_sim và cách công cụ này xử lý projectPath theo từng lần gọi cùng với session_set_defaults. Tái hiện báo cáo bằng các đường dẫn worktree riêng biệt và bộ lọc -only-testing, sau đó xác minh rằng project được yêu cầu được sử dụng hoặc một lần chạy được lọc còn 0 test sẽ thất bại thay vì trả về SUCCEEDED.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Summary
test_sim appears to ignore a per-call projectPath and use the session default instead. When the two differ, it reported SUCCEEDED having run zero tests.
What happened
I had several agents working concurrently in separate git worktrees of one repo. Session defaults were set once, pointing projectPath at the main checkout:
session_set_defaults
projectPath: <repo>/src/PinPoint.xcodeproj
scheme: PinPoint
simulatorName: iPhone 17 Pro
An agent working in <repo>/.claude/worktrees/agent-XXXX/ called test_sim with its own worktree's projectPath and an -only-testing filter for a suite that exists only in that worktree. The call returned SUCCEEDED with 0 tests passed and 0 failed.
The suite it was asked to run did not exist in the project it actually built, so there was nothing to run and nothing to fail. The agent's only clue was the zero count; the status field said success.
Why this is worth fixing above its size
A test tool that returns SUCCEEDED without running the requested tests is the worst available failure mode — it is indistinguishable from a pass at the point where a caller decides whether work is done. In an agent workflow nothing reads the count; it reads the status. Two of my agents would have reported tickets complete on the strength of it. Both worked around it by driving xcodebuild directly with their own -derivedDataPath, which is the right answer but only because they noticed.
Suggested
Either honour a per-call projectPath over the session default, or fail loudly when one is passed and cannot be honoured. Failing that, a zero-test run should not report as SUCCEEDED — an -only-testing filter matching nothing is an error, not a pass.
Environment
Multiple concurrent git worktrees of one project, one shared MCP session, iOS simulator destination. Repointing the shared session defaults was not an option, since a concurrent agent was relying on them.
- Ngôn ngữ chính
- TypeScript
- Star
- 6.4k
- Fork
- 320
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của getsentry/XcodeBuildMCP
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
getsentry/XcodeBuildMCP#520 ·
-
Warden weekly sweep Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
getsentry/XcodeBuildMCP#495 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
getsentry/XcodeBuildMCP#537 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 45/100
getsentry/XcodeBuildMCP#535 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 28/100
getsentry/XcodeBuildMCP#534 ·
Tất cả issue của getsentry/XcodeBuildMCP
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
mksglu/context-mode#1200 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
jaegertracing/jaeger-ui#4506 ·
-
area:desktop area:ui bug platform:macos
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
anthropics/claude-code#96687 ·
-
good first issue
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 95/100
AOSSIE-Org/DebateAI#582 · 2 bình luận ·