globalizejs/globalize

Code cleanup: Simplify tests util functions that passes assert as a parameter

Open

#359 aperta il 8 dic 2014

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (645 fork)batch import
help wantedquick change

Metriche repository

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

Descrizione

Ref: https://github.com/jquery/globalize/pull/351/files#r21450909

util.assertParameterPresence( assert, "currency", function() {
  Globalize.currencyFormatter();
});

@jzaefferer

I brought this up before, might be worth another try. Can this be changed to the below?

assert.parameterPresence( "currency", ... );

The same applies to any other calls that pass assert as a parameter

@rxaviers

Sorry if I missed it. But, I don't remember such comment. If you say it's better overloading assert, it shall be. Will be tracked as a separate issue #359

Guida contributor

Direzione di ricerca
Esamina le funzioni di utilità dei test nel repository, in particolare quelle che accettano 'assert' come parametro. Refactor per allegare le funzioni di utilità direttamente all'oggetto assert per semplificare le chiamate. Segui lo schema suggerito nei commenti dell'issue.
Tech stack
javascript
Dominio
testing
Tipo issue
Refactoring
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65