search

codigourbano / cep-scraper

A Javascript client to search Brazilian postcodes
http://codigourbano.org/raspador-de-dados-facilita-acesso-para-consultas-ao-cep/
13 stars 3 forks source link

Added some tests and quality tools for the library #2

Closed PauloLuan closed 9 years ago

PauloLuan commented 9 years ago

Added Travis-CI (as a continuous integration tool), Bithound (Code quality, maintainability, and stability) and Coveralls (Test coverage analyser).

PauloLuan commented 9 years ago

@miguelpeixe e @vgeorge adicionei alguns testes de integração e algumas ferramentas de análise de qualidade de código, não fiz nenhuma refatoração, fiquem a vontade para analisar o código.

Parabéns pelo trabalho. Abraços.

miguelpeixe commented 9 years ago

@PauloLuan obrigado pela contribuição. O npm test não parece estar funcionando:

> cep-scraper@0.2.0 test /home/miguelpeixe/devel/cep-scraper
> script/test

sh: 1: script/test: Permission denied
npm ERR! Test failed.  See above for more details.

Tentei alterar as permissões de execução do script/test mas não mudou nada.

PauloLuan commented 9 years ago

Tenta instalar o mocha globalmente npm install -g mocha e executar mocha na pasta raíz, é que o script npm test está configurado pra mandar a cobertura dos testes pro serviço do coveralls, ai isto está sendo feito direto pelo travis.

Isso vai solucionar o problema, apesar de eu não saber pq está dando Permission denied, alternativamente, tente dar um sudo chown -R $(whoami) . que provavelmente vai solucionar o caso.

vgeorge commented 9 years ago

Paulo, aceitei parcialmente as alterações. Removi o corretor/beautify porque penso ser melhor usar um plugin do atom. Mudei o local do script de cobertura do coveralls porque só assim consegui fazer rodar. Em outros projetos usamos conseguimos rodar o Mocha sem ter que instalar globalmente, seria legal manter assim, se possível. Os bagdes quebraram, alterei os links que estavam pro seu repositório, mas não deu certo.

PauloLuan commented 9 years ago

É que vc tem que pegar sua key lá no coveralls e colocar como variavel de ambiente no travis, o coveralls vai pegar e toda vez que fizer o build no travis vai mandar o update pro coveralls mandando a key pra lá automaticamente (e sem expor esta informação publicamente),

Você tem que adicionar as variáveis (aqui): service_name: travis-pro repo_token: <SUA_KEY_DO_COVERALLS>

Por isso que o npm test fazia o procedimento:

node node_modules/.bin/istanbul cover _mocha -- -R spec --recursive && cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js && rm -rf ./coverage

colocando isto no script test lá no package.json ele vai funcionar, ou podemos adicionar isto no próprio .travis.yml como um after_script para atualizar o coveralls (tipo como eu fiz aqui )