Attachment 156199 Details for Bug 34911 – Bug 34911: Test files from HEAD instead of 'master'

[patch] Bug 34911: Test files from HEAD instead of 'master'

Bug-34911-Test-files-from-HEAD-instead-of-master.patch (text/plain), 1.55 KB, created by Jonathan Druart on 2023-09-26 08:28:56 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Jonathan Druart

Created: 2023-09-26 08:28:56 UTC

Size: 1.55 KB

patch

obsolete

>From d1bc8841e454447f3d2a4e992230b043b7311f59 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Tue, 26 Sep 2023 10:27:57 +0200
>Subject: [PATCH] Bug 34911: Test files from HEAD instead of 'master'
>
>Since bug 34303 the test suite (when ran on jenkins) is no longer testing Perl::Critic
>
>koha_1       | fatal: Not a valid object name master
>koha_1       | [14:58:14] t/00-testcritic.t
>
>We are not cloning the whole repo and "master" is not available. Additionally we do not want to test master's files on stable branches anyway...
>---
> t/00-testcritic.t   | 2 +-
> xt/check_makefile.t | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/t/00-testcritic.t b/t/00-testcritic.t
>index debd8b2f18d..66458df7073 100755
>--- a/t/00-testcritic.t
>+++ b/t/00-testcritic.t
>@@ -14,5 +14,5 @@ if ( $EVAL_ERROR ) {
> }
> 
> Test::Perl::Critic->import( -profile => '.perlcriticrc' );
>-my @files = map { chomp; $_ } `git ls-tree -r master --name-only`;    # only files part of git
>+my @files = map { chomp; $_ } `git ls-tree -r HEAD --name-only`;    # only files part of git
> all_critic_ok(@files);
>diff --git a/xt/check_makefile.t b/xt/check_makefile.t
>index 69b0f24891d..91235bfa147 100755
>--- a/xt/check_makefile.t
>+++ b/xt/check_makefile.t
>@@ -26,7 +26,7 @@ use File::Slurp qw( read_file );
> use Data::Dumper;
> 
> my $curdir = File::Spec->curdir();
>-my @dirs = `git ls-tree -d --name-only master`;
>+my @dirs = `git ls-tree -d --name-only HEAD`;
> my $makefile = read_file("$curdir/Makefile.PL");
> my @missing;
> for my $d ( sort @dirs ) {
>-- 
>2.25.1

Actions: View | Diff | Splinter Review

Attachments on bug 34911: 156199 | 156213