мне кажется тесты можно и не проверять, а то я тут 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
<[email protected]>написал:

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



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

моб.: +7(965) 117-26-01
_______________________________________________
Kiev-pm mailing list
[email protected]
http://mail.pm.org/mailman/listinfo/kiev-pm

Ответить