search

ledgersmb / LedgerSMB

Double-entry accounting & ERP for the web
https://ledgersmb.org
Other
429 stars 152 forks source link

deps test does not run locally #1859

Closed einhverfr closed 8 years ago

einhverfr commented 8 years ago

Even though this passes on travis, it fails locally

t/01.2-deps.t .. 1/? 
#   Failed test 'requires('App::LedgerSMB::Admin::Database') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('CGI::Simple::Standard') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('Digest::SHA') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('HTTP::Exception') in Makefile.PL'
#   at t/01.2-deps.t line 26.
t/01.2-deps.t .. 23/? 
#   Failed test 'requires('Module::Runtime') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('Moose::Role') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('Moose::Util::TypeConstraints') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('MooseX::NonMoose') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('OpenOffice::OODoc::Styles') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('Path::Class') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('TeX::Encode::charmap') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('Template::Latex') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('Template::Parser') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('Template::Provider') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('Try::Tiny') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('X12::Parser') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'requires('feature') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('File::Util') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Module::CPANfile') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Perl::Critic') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Selenium::Remote::Driver') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Selenium::Remote::WDKeys') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Test::BDD::Cucumber::Extension') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Test::BDD::Cucumber::Harness::TestBuilder') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Test::BDD::Cucumber::Loader') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Test::BDD::Cucumber::Model::TagSpec') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Test::BDD::Cucumber::StepFile') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Test::Dependencies') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Weasel::FindExpanders') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Weasel::FindExpanders::Dojo') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Weasel::FindExpanders::HTML') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Weasel::WidgetHandlers') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Weasel::Widgets::Dojo') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('Weasel::Widgets::HTML') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('YAML::Syck') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'build_requires('the') in Makefile.PL'
#   at t/01.2-deps.t line 26.

#   Failed test 'App::LedgerSMB::Admin is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'Carp::Always is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'FileHandle is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'Getopt::Long is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'HTML::Entities is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'IO::File is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'IO::Scalar is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'Image::Size is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'PGObject::Simple::Role is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'Parse::RecDescent is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'Plack::Builder is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'Plack::Middleware::Static is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'Starman is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'Template::Plugin::Latex is not a run-time dependency'
#   at t/01.2-deps.t line 26.

#   Failed test 'ExtUtils::MakeMaker is not a build-time dependency'
#   at t/01.2-deps.t line 26.
# Looks like you failed 51 tests of 105.
t/01.2-deps.t .. Dubious, test returned 51 (wstat 13056, 0x3300)
Failed 51/105 subtests 

Test Summary Report
-------------------
t/01.2-deps.t (Wstat: 13056 Tests: 105 Failed: 51)
  Failed tests:  2, 5, 16, 21, 32, 34-36, 39, 45, 47, 49-51
                53-54, 59, 68-78, 82-88, 90-105
  Non-zero exit status: 51
Files=1, Tests=105,  3 wallclock secs ( 0.03 usr  0.00 sys +  3.34 cusr  0.01 csys =  3.38 CPU)
Result: FAIL
ehuelsmann commented 8 years ago

You have an old dep installed. Please upgrade Test::Dependencies

einhverfr commented 8 years ago

Given that I installed it from the Fedora 22 repos, this will likely affect any user that tries to do a make test, so I think the first step should be disable the test unless either it is running on Travis OR the dependencies are met.

Do you know which version is minimally required? Otherwise I will set at current released version from CPAN.

On Tue, Jul 26, 2016 at 7:43 AM, Erik Huelsmann notifications@github.com wrote:

You have an old dep installed. Please upgrade Test::Dependencies

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ledgersmb/LedgerSMB/issues/1859#issuecomment-235168577, or mute the thread https://github.com/notifications/unsubscribe-auth/AAU0l0fAKdi6zROZjazL1fwDG4VwhzkWks5qZZ6egaJpZM4JUyml .

Best Wishes, Chris Travers

Efficito: Hosted Accounting and ERP. Robust and Flexible. No vendor lock-in. http://www.efficito.com/learn_more

ehuelsmann commented 8 years ago

Version 0.20 is minimally required.

We do declare that dependency in our cpanfile...

einhverfr commented 8 years ago

ok, I think we should make all the on develop modules optional for a basic make test run so that users can make test without errors. Will use that as a basis, and expect to have that in for rc2

einhverfr commented 8 years ago

running travis on fix to skip when dependency not installed or up to date.