Bug 14408: Add tests to get_template_and_user
authorJonathan Druart <jonathan.druart@koha-community.org>
Fri, 19 Jun 2015 08:25:30 +0000 (10:25 +0200)
committerTomas Cohen Arazi <tomascohen@theke.io>
Mon, 22 Jun 2015 20:44:58 +0000 (17:44 -0300)
Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

t/db_dependent/Auth.t

index 2cb1d5f..8513240 100644 (file)
@@ -8,7 +8,8 @@ use Modern::Perl;
 use CGI qw ( -utf8 );
 use Test::MockModule;
 use List::MoreUtils qw/all any none/;
-use Test::More tests => 8;
+use Test::More tests => 11;
+use Test::Warn;
 use C4::Members;
 use Koha::AuthUtils qw/hash_password/;
 
@@ -105,6 +106,27 @@ $dbh->{RaiseError} = 1;
 
     ok( ( any { $_->name eq 'KohaOpacLanguage' and $_->value eq 'en' } @$cookies ),
         'BZ9735: invalid language, then default to en');
+
+    for my $template_name (
+        qw(
+            ../../../../../../../../../../../../../../../etc/passwd
+            test/../../../../../../../../../../../../../../etc/passwd
+            /etc/passwd
+        )
+    ) {
+        eval {
+            ( $template, $loggedinuser, $cookies ) = get_template_and_user(
+                {
+                    template_name   => $template_name,
+                    query           => $query,
+                    type            => "intranet",
+                    authnotrequired => 1,
+                    flagsrequired   => { catalogue => 1 },
+                }
+            );
+        };
+        like ( $@, qr(^bad template path), 'The file $template_name should not be accessible' );
+    }
 }
 
 # Check that there is always an OPACBaseURL set.