globalizejs/globalize

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

Open

#359 aberto em 8 de dez. de 2014

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)JavaScript (645 forks)batch import
help wantedquick change

Métricas do repositório

Stars
 (4.759 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

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

Guia do colaborador

Direção de pesquisa
Examine as funções utilitárias de teste no repositório, especialmente aquelas que aceitam 'assert' como parâmetro. Refatore as para anexar as funções utilitárias diretamente ao objeto assert para simplificar as chamadas. Siga o padrão sugerido nos comentários da issue.
Pilha de tecnologia
javascript
Domain
testing
Tipo Issue
Refatoração
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65