frintjs/frint

Possible race condition issue in `streamProps`

Open

#382 aberto em 28 de nov. de 2017

Ver no GitHub
 (0 comments) (2 reactions) (0 assignees)JavaScript (37 forks)batch import
buggood first issuehelp wanted

Métricas do repositório

Stars
 (746 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Escreva testes unitários para demonstrar a condição de corrida no streamProps devido ao concatMap e, se necessário, substitua por switchMap.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptRxJS
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75