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

Skip to content

[app] Test instable : TestCheckAllBoundsTheWholeBatchByTheTimeout #268

Closed as not planned

Description

@EnO33

Contexte

CI de main rouge le 04/11/2026 (run 41812633990, commit 7ac98ad, #267 sans rapport) :

--- FAIL: TestCheckAllBoundsTheWholeBatchByTheTimeout (0.10s)
    check_all_test.go:136: report 2: RDAP ok, want unavailable past the deadline

Le même test passe sur la CI de la PR et localement (0 échec sur 300 exécutions sans -race).

Cause

Le faux registre des tests (asked.record, internal/app/registries_fakes_test.go) attend la fin du contexte, puis répond a.err, nil : il répond « trouvé » juste après l'échéance. collect (internal/app/check.go) a alors deux cas prêts, la réponse et ctx.Done(), et select en choisit un au hasard. Un vrai registre ne fait pas cela : le client HTTP rend l'erreur du contexte une fois l'échéance passée, et la réponse est indisponible dans les deux cas.

Correctif attendu

Le faux registre rend ctx.Err() quand son contexte est fini après hold, comme un vrai client. Le code de production ne change pas.

Activity

  1. EnO33 commented on Nov 4, 2026

    @EnO33
    ContributorAuthor

    Doublon de #213 (et #224) : même test, même cause. Corrigé par #269.

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

    area:appZone : apptype:bugComportement incorrect

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    0