globalizejs/globalize

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

Open

#359 ouverte le 8 déc. 2014

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (645 forks)batch import
help wantedquick change

Métriques du dépôt

Stars
 (4 759 stars)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez les fonctions utilitaires de test dans le dépôt, en particulier celles qui acceptent 'assert' comme paramètre. Refactorez les pour attacher les fonctions utilitaires directement à l'objet assert afin de simplifier les appels. Suivez le modèle suggéré dans les commentaires de l'issue.
Stack technique
javascript
Domaine
testing
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
JavaScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65