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

Aperta Adatta ai principianti
#4,212 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
90/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
ios, typescript

Direzione di ricerca

Leggi apple/runner/AgentDeviceRunner/AgentDeviceRunnerUITests/UnitTests/RunnerTests+TextInputProbeTests.swift intorno alle righe 239–240 e assegna un commandId univoco al secondo comando type. Poi esegui l’intera suite iPhone e pnpm check:xctest-selection; il lavoro è terminato quando entrambi passano e la mappa di selezione indirizza le modifiche a RunnerCommandJournal verso questi test di probe, se la modifica richiede poco lavoro.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.
Lingua principale
TypeScript
Stelle
5.3k
Fork
377
Merge medio
13h 49m
PR unite (30g)
650

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di callstack/agent-device

Tutte le issue di callstack/agent-device

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.