globalizejs/globalize

Code cleanup: Can we remove the nested functions pattern of util.assertFns?

Open

#361 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#r21450932

util.assertCurrencyParameter( assert, "currency", function( invalidValue ) {
 return function() {
    Globalize.currencyFormatter( invalidValue );
  };
});

@jzaefferer

I guess we've discussed this before as well, still doesn't make much sense: Why does this test have to return two nested functions?

@rxaviers

Yeap, we have discussed about this and it was clear this was needed. Although, I also believe this could be simplified.

Guida contributor

Direzione di ricerca
Esegui il refactoring del pattern di funzioni annidate in util.assertFns semplificando le funzioni restituite in una singola funzione o utilizzando un approccio diverso, mantenendo lo stesso comportamento del test.
Tech stack
javascript
Dominio
frontend
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.
Attiva
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