<div>;-)</div><div><br></div><div>SKIP: {</div><div> skip "Author tests aren't required to install this module", 1</div><div> unless $ENV{RUSSOZ_IS_IN_THE_HOUSE};</div><div><br></div><div> diag("Who's in the house? Russoz is in the house! Uh Uh!");</div>
<div><br></div><div> eval 'require Test::Perl::Critic';</div><div> if ($@) {</div><div> Test::More::plan(</div><div> skip_all => "Test::Perl::Critic required for testing PBP compliance"</div>
<div> );</div><div> }</div><div> Test::Perl::Critic::all_critic_ok();</div><div>}</div><div><br></div><br><div class="gmail_quote">2010/3/8 Luis Motta Campos <span dir="ltr"><<a href="mailto:luismottacampos@yahoo.co.uk">luismottacampos@yahoo.co.uk</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="im">Alexei Znamensky wrote:<br>
> 1) Porque o PerlPentelho tá reclamando disso? Não estou acessando um objeto.<br>
> Não quero que seja um objeto, não preciso que seja um objeto ali. Esse<br>
> $config é apenas, e tẽo somente, e nada mais que, uma estrutura de dados.<br>
> There is no code. Reclamar que estou quebrando o encapsulamento é o ó do<br>
> bórógódó. Para ser gentil.<br>
<br>
</div>Eu não recomendo que você use Perl::Critic nos seus testes do CPAN. O<br>
pessoal tradiciionalmente divide os testes em duas categorias: testes de<br>
usuário, e testes de autor. Os testes de usuário você deveria rodar<br>
antes de instalar o módulo, para se certificar que ele funciona com a<br>
sua arquitetura / plataforma / módulos disponíveis / etc. Os testes de<br>
autor são apenas para você saber o que acontece: POD, Perl::Tidy,<br>
Perl::Critic, são todos módulos muito bons para ajudar a escrever<br>
código, mas que eu não quero que explodam quando o SmokeTest estiver<br>
rodando na máquina do pessoal de QA.<br>
<br>
Para resolver esta, eu implemento os testes de autor dependendo de uma<br>
variável de ambiente que eu publico na minha máquina:<br>
<br>
# t/perl-critic.t<br>
SKIP: {<br>
skip "Author tests aren't required to install this module"<br>
unless $ENV{I_KNOW_WHAT_IM_DOING};<br>
# teste do Perl::Critic aqui...<br>
<div class="im">}<br>
<br>
> 2) Vocês recomendam usar o perlcritic nos testes do módulo, ou é melhor<br>
> tirar essa tchonga logo de uma vez? Eu gosto da idéia de ter um teste desses<br>
> no módulo, mas se vai ser pentelho nesse nível, jisuis, eu estou<br>
> considerando tirar: muito bonito na teoria, mas na prática não está sendo<br>
> tão legal assim.<br>
<br>
</div>SIM, eu recomendo fortemente que você rode o Perl::Critic, mas não como<br>
parte dos testes de usuário. Arranja para você rodar apenas durante os<br>
testes de autor. O Perl::Critic é excelente para manter o seu Perl na<br>
linha. E ele é linha-dura. ;)<br>
<div class="im"><br>
> Funnily enough, instalei o módulo aqui no trabalho, e o Perl::Critic pegou<br>
> uns erros diferentes. Lá em casa ele passa nos testes. Ha.. vou me divertir<br>
> um bom tanto esta semana, ao que parece.<br>
<br>
</div>O Perl::Critic depende de configuração local (i.e., um smoke-tester pode<br>
introduzir os defaults dele) e também depende de versão: dependendo da<br>
versõa do Perl::Critic que você tem, ele pode ter defaults diferentes.<br>
<br>
Sim, isso dá bastante divertimento. Especialmente quando você descobrir<br>
que seus testes podem obter e manipular a configuração do Perl::Critic<br>
antes de ele criticar... ;)<br>
<div><div></div><div class="h5"><br>
Putamplexos!<br>
--<br>
Luis Motta Campos is a DBA,<br>
Perl Programmer, foodie and photographer.<br>
_______________________________________________<br>
SaoPaulo-pm mailing list<br>
<a href="mailto:SaoPaulo-pm@pm.org">SaoPaulo-pm@pm.org</a><br>
<a href="http://mail.pm.org/mailman/listinfo/saopaulo-pm" target="_blank">http://mail.pm.org/mailman/listinfo/saopaulo-pm</a></div></div></blockquote></div><br><br clear="all"><br>-- <br>Alexei Znamensky [russoz_gmail_com] [<a href="http://russoz.wordpress.com">russoz.wordpress.com</a>] [<a href="http://www.flickr.com/photos/alexeiz">www.flickr.com/photos/alexeiz</a>]<br>
"Though we live in trying times, we're the ones who have to try"<br>