frintjs/frint

Possible race condition issue in `streamProps`

Open

#382 aperta il 28 nov 2017

Vedi su GitHub
 (0 commenti) (2 reazioni) (0 assegnatari)JavaScript (37 fork)batch import
buggood first issuehelp wanted

Metriche repository

Star
 (746 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Current situation

We ship streamProps helper function from frint-component-utils package, which allows combining values from various sources and then generate a single observable of props.

It uses concatMap operator from RxJS internally.

Possible bug

There is a possibility that with concatMap, we will have some race condition issue.

It is advised to use switchMap instead.

What to do

  • Write unit tests proving there is a race condition issue with streamProps
  • If there is, replace concatMap usage with switchMap without breaking the API

Guida contributor

Direzione di ricerca
Scrivi test unitari per dimostrare la race condition in streamProps dovuta a concatMap, quindi sostituisci con switchMap se necessario.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptRxJS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75