sinonjs/sinon

sinon not throwing the error that been thrown when the error is undefined and also fail to assert

Open

#2471 aperta il 11 set 2022

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (764 fork)batch import
Difficulty: EasyHelp wantedwontfix

Metriche repository

Star
 (9548 star)
Metriche merge PR
 (Merge medio 6g 16h) (15 PR mergiate in 30 g)

Descrizione

Describe the bug If the function throw undefined then:

  1. calling the function does not throw
  2. sinon.assert.threw(a); fail

To Reproduce ReplIt Link

const a = sinon.spy(() => {
  throw undefined;
});

try {
  a();
} catch (e) {
   // 1. this won't be logged
  console.log('Error has being thrown');
}

// 2. This will throw
// Uncaught AssertError: spy did not throw exception
sinon.assert.threw(a);

Expected behavior

  1. throw the thrown error
  2. the assertion should pass

As said in the docs about the 2nd thing:

sinon.assert.threw(spyOrSpyCall, exception);

Passes if spy threw the given exception.

The exception can be a String denoting its type, or an actual object.

If only one argument is provided, the assertion passes if spy ever threw any exception.

Context (please complete the following information):

  • Library version: 14.0.0
  • Environment: node

Guida contributor

Direzione di ricerca
Indaga su come sinon catturi i valori lanciati nelle chiamate spy (probabilmente in lib/spy.js). Il problema è che il lancio di undefined non viene gestito correttamente, forse perché il codice verifica null o tratta undefined come nessun lancio. Esamina anche sinon.assert.threw in lib/assert.js per vedere come verifica le eccezioni lanciate. La correzione dovrebbe garantire che qualsiasi valore lanciato, incluso undefined, venga registrato e verificato correttamente.
Tech stack
javascript
Dominio
testing
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
JavaScriptNode.jsSinon basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75