8000 [app] Test instable : TestCheckAllBoundsTheWholeBatchByTheTimeout · Issue #224 · arnakoff/api · GitHub
[go: up one dir, main page]

Skip to content

[app] Test instable : TestCheckAllBoundsTheWholeBatchByTheTimeout #224

Closed

Description

@EnO33

Constat

TestCheckAllBoundsTheWholeBatchByTheTimeout (internal/app/check_all_test.go) a échoué une fois sous go test -race -shuffle=on ./... (graine 1793701952115853000), le 03/11/2026, sur la branche feat/64-mcp-http, qui ne touche pas internal/app :

check_all_test.go:136: report 3: RDAP ok, want unavailable past the deadline

Relancé 200 fois seul (-race -count 200) puis avec la même graine : aucun échec. Le faux registre retient la requête jusqu'à la fin du contexte, puis un rapport est quand même compté « ok » : course entre l'échéance et la collecte des réponses, cause exacte « à vérifier ».

À faire

  • Reproduire (charge parallèle, -cpu, -count) et trouver si la course est dans le test ou dans CheckAll/collect.
  • Corriger ; un registre qui répond après l'échéance doit rester « indisponible ».

Activity

  1. EnO33 commented on Nov 4, 2026

    @EnO33
    ContributorAuthor

    Cause confirmée et corrigée par #269 : le faux registre (asked.record) rendait nil après ctx.Done(), et collect voyait alors la réponse et l'échéance prêtes en même temps ; select choisissait au hasard. Le faux rend désormais ctx.Err(), comme un vrai client : un registre qui répond après l'échéance reste « indisponible ». Pas de changement de production. Revu le 04/11/2026 sur main (run 41812633990).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    0