From 52b8a4fc03022d12fd6a5fc62ccbc0cd66974437 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 25 Nov 2022 09:07:34 +0100 Subject: [PATCH] Bug 32349: Remove TEST_QA There is an env var that we are passing from koha-testing-docker, TEST_QA. It's used in Koha from a single test file, t/00-testcritic.t. If not set, no test is run. Do we really need this? libtest-perl-critic-perl is packaged and automatically installed. Signed-off-by: Mason James --- t/00-testcritic.t | 7 ------- 1 file changed, 7 deletions(-) diff --git a/t/00-testcritic.t b/t/00-testcritic.t index 4c7f44132ab..cd3a7af71ba 100755 --- a/t/00-testcritic.t +++ b/t/00-testcritic.t @@ -1,18 +1,11 @@ #!/usr/bin/env perl # This script can be used to run perlcritic on perl files in koha -# The script is purely optional requiring Test::Perl::Critic to be installed -# and the environment variable TEST_QA to be set use Modern::Perl; use Test::More; use English qw(-no_match_vars); -if ( not $ENV{TEST_QA} ) { - my $msg = 'Author test. Set $ENV{TEST_QA} to a true value to run'; - plan( skip_all => $msg ); -} - eval { require Test::Perl::Critic; }; if ( $EVAL_ERROR ) { -- 2.25.1