[Kiev-pm] Devel::Cover & t/

Mishin Nikolay nikolay.mishin at gmail.com
Tue Oct 15 06:33:17 PDT 2013


мне кажется тесты можно и не проверять, а то я тут perlcritic на 5.18
зупустил
он мне там такое понаписал
при https://github.com/mishin/gists/blob/master/.perlcriticrc
и проверке файла
https://github.com/mishin/gists/blob/master/google_translate.pl
на v5.14.2
>perlcritic google_translate.pl
google_translate.pl source OK
v5.18.0
он выдал
Use Emacs file variables to declare coding style at line 1, column 1.
Emacs can read per-file settings.  (Severity: 2)
Missing Perl version at line 1, column 1.  Add "use 5.006" or similar.
(Severity: 1)
Code found in comment at line 65, column 1.  Commented-out code found can
be confusing.  (Severity: 2)
Found use of croak. Use an exception instead at line 78, column 29.
Exception objects should be used instead of the standard Perl error
mechanism.  (Severity:4)
Found use of croak. Use an exception instead at line 94, column 5.
Exception objects should be used instead of the standard Perl error
mechanism.  (Severity: 4)
Use only ASCII code at line 104, column 25.  Put any non-ASCII in separate
files.  (Severity: 1)
Found use of croak. Use an exception instead at line 154, column 5.
Exception objects should be used instead of the standard Perl error
mechanism.  (Severity:4)

раньше ругались на die - требовали croak (use Carp)
теперь ругаются на croak - требуют Error.pm (даже не Try::Tiny!)

в общем всему надо знать меру и вовремя останавливаться,
это я к тому что тесты на покрытие документацией можно и не проверять,
ну только, если уже совсем нечего делать


8 октября 2013 г., 15:50 пользователь Maxim Vuets
<maxim.vuets � gmail.com>написал:

> 2013/10/7 Yaroslav Korshak <ykorshak � gmail.com>:
> > Если coverage  >= 50% и есть инструменты и согласие на нотификации
> > приведенного ниже вида, то стоит.
> >
> ...
> > - "В следствии коммита #666 тест t/FooBar.t выполняется только на 90%"
>
> Тест-плана для этого достаточно, разве нет?
> _______________________________________________
> Kiev-pm mailing list
> Kiev-pm � pm.org
> http://mail.pm.org/mailman/listinfo/kiev-pm
>



-- 
С уважением,
Николай Мишин

моб.: +7(965) 117-26-01
----------- ???? ???-----------
??? ?????HTML ?? ?????hellip;
URL: <http://mail.pm.org/pipermail/kiev-pm/attachments/20131015/4cd7e830/attachment.html>


More information about the Kiev-pm mailing list