sinonjs/sinon

Sinon fails in comparing call order when running tests in parallel

Fechada

#2.472 aberto em 17 de set. de 2022

Ver no GitHubGuia do colaborador
 (11 comentários) (0 reação) (0 responsável)JavaScript (764 forks)batch import
BugDifficulty: EasyHelp wantedImprovementgood first issue

Métricas do repositório

Stars
 (9.548 estrelas)
Métricas de merge de PR
 (Mesclagem média 6d 16h) (15 fundiu PRs em 30d)

Description

Describe the bug

spy1.calledImmediatelyBefore(spy2) fails when used in a node-tap test suite that is being executed in parallel. In serial mode it works without issue. calledBefore continues to work as normal.

I've even tried using sandboxes for each test but it did not fix the issue.

To Reproduce

Repo: https://github.com/Ethan-Arrowood/sinon-spy-parallel-bug-reproduction

git clone https://github.com/Ethan-Arrowood/sinon-spy-parallel-bug-reproduction
cd sinon-spy-parallel-bug-reproduction
pnpm install
pnpm run test

To see tests passing comment out line 27 in test/index.ts:

+ // tap.jobs = 2;
- tap.jobs = 2;

Expected behavior

I'd expect my spy assertions to pass in both serial and parallel mode

Context (please complete the following information):

  • Library version: 14.0.0
  • Environment: Node.js 18
  • Example URL:
  • Other libraries you are using: node-tap, typescript

Guia do colaborador

Direção de pesquisa
Investigue como o Sinon rastreia a ordem das chamadas entre espiões quando os testes são executados em paralelo. O problema provavelmente é devido ao estado global compartilhado nos registros de chamadas internos do Sinon. Verifique se o uso de sandboxes resolve o problema e, se não, considere um rastreador de ordem de chamadas por teste.
Pilha de tecnologia
javascripttypescriptnodejs
Domain
testing
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
GitNode.jsJavaScript
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65