keithamus/observable-polyfill

tearDown order needs to be upstream->downstream on consumer-initiated unsubscription

Open

#30 ouverte le 30 juil. 2025

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)JavaScript (5 forks)github user discovery
bughelp wanted

Métriques du dépôt

Stars
 (16 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

About

  • wpt test "Teardowns are called in upstream->downstream order on consumer-initiated unsubscription" fails

Expected behavior

  • tearDown order:
    • "upstream abort handler"
    • "upstream teardown. reason: Abort!"
    • "middle abort handler"
    • "middle teardown. reason: Abort!"
    • "downstream abort handler"
    • "downstream teardown. reason: Abort!"

Current behavior

  • tearDown order:
    • "downstream abort handler"
    • "middle abort handler"
    • "upstream abort handler"
    • "upstream teardown. reason: Abort!"
    • "middle teardown. reason: Abort!"
    • "downstream teardown. reason: Abort!"

How to reproduce

  • npm run test-serve and inspect wpt test results in the browser

Additional context

https://github.com/keithamus/observable-polyfill/blob/5af5df1b6bb87abc44a773701ebcf64c1d1f25c7/test/wpt/observable-constructor.any.js#L728-L763

Guide contributeur

Direction de recherche
Tracez la logique de l'ordre de démontage dans le polyfill et comparez avec la spécification WICG. Assurez vous que lors de la désinscription initiée par le consommateur, les callbacks de démontage upstream sont appelés en premier, puis ceux downstream.
Stack technique
javascript
Domaine
api
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Gitnpm
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70