Two mobile markdown pipeline tests fail on release/v0.122.5 and CI does not run them

Chiusa Adatta ai principianti
  • Qualcosa non va?
#7,468 1 commento 1 reazione 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
76/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
react, typescript
Ambito
ci-cd, testing

Direzione di ricerca

Leggi web/mobile/tests/unit/assistantMarkdownPipeline.test.tsx insieme a web/mobile/src/features/chat/AssistantMarkdown.tsx per confermare quale resolver i test devono simulare. Aggiorna il mock affinché i due test dei collegamenti ai file verifichino il resolver usato dal componente. Poi esegui il comando Vitest fornito e controlla che tutti e sette i test passino. L’integrazione con CI è tracciata separatamente in #7326.

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

Descrizione

🤖 The AI agent says:

What happens

Two tests in web/mobile/tests/unit/assistantMarkdownPipeline.test.tsx fail on release/v0.122.5 (7ddc8e8ba1). The same two fail on every run, with any Node, timezone and locale. The CI workflow "17 - check mobile" is green because it never runs them.

Repro

git checkout --detach origin/release/v0.122.5
cd web && pnpm install --frozen-lockfile
cd mobile && pnpm exec vitest run tests/unit/assistantMarkdownPipeline.test.tsx

Result: 2 failed, 5 passed. The full mobile suite: 2 failed, 421 passed (68 files).

FAIL  ... > still sends the file link the platform prompt prescribes to the drive resolver
AssertionError: expected [] to include '/agent-files/report.md'   (line 94)
FAIL  ... > decodes a percent-encoded file name on the way to the resolver
AssertionError: expected [] to include '/agent-files/my report.md'  (line 100)

Cause

Commit 439f265abc ("preview images the agent names under the paragraph", 2026-10-08) changed web/mobile/src/features/chat/AssistantMarkdown.tsx to resolve links with chatReplyFileResolver. The test still mocks only chatFileResolver from @agenta/entity-ui/drive, so its renderCode spy never runs and resolved stays empty. The test is stale. The product code is not shown to be wrong.

Why CI did not catch it

17-check-mobile.yml runs only types:check and a Docker build with a smoke test. 12-check-unit-tests.yml runs run-tests.ts --layer unit, which calls pnpm -r --if-present test:unit. @agenta/mobile defines test, not test:unit, so it is skipped. This is the gap already tracked in #7326.

Fix direction

Update the mock to cover chatReplyFileResolver (or both), and wire the mobile suite into CI per #7326.

Lingua principale
TypeScript
Stelle
4.8k
Fork
690
Merge medio
23h 29m
PR unite (30g)
540

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 Agenta-AI/agenta

Tutte le issue di Agenta-AI/agenta

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.