XCTest Nightly: text-input probe test reuses a commandId the runner journal now answers from its record

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#4,212 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
90/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
ios, typescript
Lĩnh vực
mobile-dev, testing

Hướng nghiên cứu

Đọc apple/runner/AgentDeviceRunner/AgentDeviceRunnerUITests/UnitTests/RunnerTests+TextInputProbeTests.swift quanh dòng 239–240 và đặt commandId duy nhất cho lệnh type thứ hai. Sau đó chạy toàn bộ bộ kiểm thử iPhone và pnpm check:xctest-selection; hoàn tất khi cả hai đều đạt và bản đồ lựa chọn định tuyến các thay đổi của RunnerCommandJournal đến các kiểm thử thăm dò này nếu thay đổi đó không tốn nhiều công sức.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

bug ready-for-agent

Purpose

XCTest Nightly has failed twice in a row (runs 41578854330 on 2026-11-03 and 41773968778 on 2026-11-04), after green nights through 2026-11-02 at 343b7fcf41. Both fail in one test.

Current behavior

RunnerTests.testFreshCoordinateTapContainsUnavailableTextInputProbe (apple/runner/AgentDeviceRunner/AgentDeviceRunnerUITests/UnitTests/RunnerTests+TextInputProbeTests.swift:239-240) sends the same type command twice, both with commandId: "type-after-unavailable-probe". The first send is expected to fail with TEXT_INPUT_NOT_FOCUSED; the second, after a tap that focuses the field, is expected to type.

Since #4140 (e4034a714d), the runner's command journal answers a repeated commandId from its recorded outcome. So the second send returns the first send's TEXT_INPUT_NOT_FOCUSED, and the field stays empty:

RunnerTests+TextInputProbeTests.swift:239: error: XCTAssertTrue failed
RunnerTests+TextInputProbeTests.swift:240: error: XCTAssertEqual failed: ("Optional("")") is not equal to ("Optional("must-not-type")")

The journal behavior is intended (ADR 0005); the test reuses an id it must not reuse.

Required behavior

  • The second type in this test sends its own commandId.
  • XCTest Nightly passes the iPhone full suite.
  • The PR lane's XCTest selection did not run this test for #4140's journal change. If the selection map can cheaply route RunnerCommandJournal changes to the text-input probe tests, it does.

Completion conditions

  • The test passes in the iPhone full suite.
  • pnpm check:xctest-selection passes.
Ngôn ngữ chính
TypeScript
Star
5.3k
Fork
377
Merge trung bình
13 giờ 49 phút
Pull request đã merge (30 ngày)
650

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của callstack/agent-device

Tất cả issue của callstack/agent-device

Issue tương tự

Thêm issue về TypeScript

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.