X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=t%2FInstaller_PerlModules.t;h=dad968fed60616ea4a7b6ec74f31b6813db0d551;hb=865f9a165d5d5e025737324843eccb5c39a74efb;hp=f64ed85e28a2c4e32f9e1a22e03029a2fcd8ba80;hpb=84825be42adf4a50b77360cf016cd5bfee15c072;p=koha.git

diff --git a/t/Installer_PerlModules.t b/t/Installer_PerlModules.t
index f64ed85e28..dad968fed6 100755
--- a/t/Installer_PerlModules.t
+++ b/t/Installer_PerlModules.t
@@ -3,15 +3,32 @@
 # This Koha test module is a stub!  
 # Add more tests here!!!
 
-use strict;
-use warnings;
+use Modern::Perl;
 
-use Test::More tests => 15;
+use Test::More tests => 22;
 
 BEGIN {
         use_ok('C4::Installer::PerlModules');
 }
 
+$C4::Installer::PerlModules::PERL_DEPS->{'Local::Module::Upgraded'} = {
+    'required' => '1',
+    'min_ver' => '0.9.3',
+    'usage' => "Testing: make sure numbers are compared numerically and not lexicographically",
+};
+$Local::Module::Upgraded::VERSION = '0.9.13';
+$INC{"Local/Module/Upgraded.pm"} = 1;
+use_ok("Local::Module::Upgraded");
+
+$C4::Installer::PerlModules::PERL_DEPS->{'Local::Module::NotUpgraded'} = {
+    'required' => '1',
+    'min_ver' => '0.9.3',
+    'usage' => "Testing: make sure numbers are compared numerically and not lexicographically",
+};
+$Local::Module::NotUpgraded::VERSION = '0.9.1';
+$INC{"Local/Module/NotUpgraded.pm"} = 1;
+use_ok("Local::Module::NotUpgraded");
+
 my $modules;
 ok ($modules = C4::Installer::PerlModules->new(), 'Tests modules object');
 my $prereq_pm = $modules->prereq_pm();
@@ -27,11 +44,32 @@ my $optional = $modules->required('optional'=>1);
 %params = map { $_ => 1 } @$optional;
 ok (exists($params{"Test::Strict"}), 'test::strict optional for installer to run');
 is ($optional = $modules->required('spaghetti'=>1),-1, '-1 returned when parsing in unknown parameter');
-my $version_info = $modules->version_info('module'=>"DBI");
-ok (exists($version_info->{'DBI'}->{"required"}), 'required exists');
-ok (exists($version_info->{'DBI'}->{"upgrade"}), 'upgrade exists');
-is ($modules->version_info('module'=>"thisdoesn'texist"),-1, 'thisdoesntexist should return -1');
+my $version_info = $modules->version_info('DBI');
+ok (exists($version_info->{"required"}), 'required exists');
+ok (exists($version_info->{"upgrade"}), 'upgrade exists');
+is ($modules->version_info("thisdoesn'texist"),-1, 'thisdoesntexist should return -1');
 ok ($modules->module_count() >10 , 'count should be greater than 10');
 my @module_list = $modules->module_list;
 %params = map { $_ => 1 } @module_list;
 ok (exists($params{"DBI"}), 'DBI exists in array');
+is ($modules->required('module'=>"String::Random"),1, 'String::Random should return 1 since required');
+is ($modules->version_info(), -1, "Testing empty modules");
+
+is($modules->version_info("Local::Module::Upgraded")->{"upgrade"},0,"Version 0.9.13 is greater than 0.9.3, so no upgrade needed");
+is($modules->version_info("Local::Module::NotUpgraded")->{"upgrade"},1,"Version 0.9.1 is smaller than 0.9.1, so no upgrade needed");
+
+subtest 'versions_info' => sub {
+    plan tests => 4;
+    my $modules = C4::Installer::PerlModules->new;
+    $modules->versions_info;
+    ok( exists $modules->{missing_pm}, 'versions_info fills the missing_pm key' );
+    ok( exists $modules->{upgrade_pm}, 'versions_info fills the upgrade_pm key' );
+    ok( exists $modules->{current_pm}, 'versions_info fills the current_pm key' );
+    my $missing_modules = $modules->get_attr( 'missing_pm' );
+    my $upgrade_modules = $modules->get_attr( 'upgrade_pm' );
+    my $current_modules = $modules->get_attr( 'current_pm' );
+    my $dbi_is_missing = grep { exists $_->{DBI} ? 1 : () } @$missing_modules;
+    my $dbi_is_upgrade = grep { exists $_->{DBI} ? 1 : () } @$upgrade_modules;
+    my $dbi_is_current = grep { exists $_->{DBI} ? 1 : () } @$current_modules;
+    ok( $dbi_is_missing || $dbi_is_upgrade || $dbi_is_current, 'DBI should either be missing, upgrade or current' );
+};