Attachment 113378 Details for Bug 23916 – Bug 23916: (follow-up) Add unit tests

[patch] Bug 23916: (follow-up) Add unit tests

Bug-23916-follow-up-Add-unit-tests.patch (text/plain), 1.04 KB, created by Tomás Cohen Arazi (tcohen) on 2020-11-10 12:00:39 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2020-11-10 12:00:39 UTC

Size: 1.04 KB

patch

obsolete

>From 745da971721aaec87664c5cd42c15a90140ab163 Mon Sep 17 00:00:00 2001
>From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
>Date: Mon, 9 Nov 2020 14:24:22 +0100
>Subject: [PATCH] Bug 23916: (follow-up) Add unit tests
>
>Fix perlcritic error
>Subroutine "new" called using indirect syntax at line 128, column 14.  See page 349 of PBP.  (Severity: 4)
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>---
> t/db_dependent/Circulation.t | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/Circulation.t b/t/db_dependent/Circulation.t
>index 41b10d23f6c..819c6c1f446 100755
>--- a/t/db_dependent/Circulation.t
>+++ b/t/db_dependent/Circulation.t
>@@ -125,7 +125,7 @@ $dbh->do('DELETE FROM borrowers');
> # Disable recording of the staff who checked out an item until we're ready for it
> t::lib::Mocks::mock_preference('RecordStaffUserOnCheckout', 0);
> 
>-my $module = new Test::MockModule('C4::Context');
>+my $module = Test::MockModule->new('C4::Context');
> 
> my $library = $builder->build({
>     source => 'Branch',
>-- 
>2.29.2

Actions: View | Diff | Splinter Review

Attachments on bug 23916: 94914 | 94915 | 94916 | 94917 | 94918 | 95503 | 95504 | 95505 | 95506 | 95507 | 95508 | 95509 | 97719 | 97720 | 97721 | 97722 | 97723 | 97724 | 98586 | 98587 | 98588 | 98589 | 98590 | 98591 | 98594 | 99886 | 99887 | 99888 | 99889 | 99890 | 99891 | 110649 | 110650 | 110651 | 110652 | 110653 | 110654 | 110655 | 111485 | 111486 | 111487 | 111488 | 111489 | 111490 | 111491 | 111492 | 111966 | 111967 | 111968 | 111969 | 111970 | 111971 | 111972 | 111973 | 111974 | 112248 | 112249 | 112250 | 112251 | 112252 | 112253 | 112254 | 112255 | 112256 | 112257 | 112363 | 112364 | 112365 | 112366 | 112367 | 112368 | 112369 | 112370 | 112371 | 112372 | 112373 | 113035 | 113036 | 113037 | 113038 | 113039 | 113040 | 113041 | 113042 | 113043 | 113044 | 113045 | 113316 | 113317 | 113318 | 113319 | 113320 | 113321 | 113322 | 113323 | 113324 | 113325 | 113326 | 113327 | 113367 | 113368 | 113369 | 113370 | 113371 | 113372 | 113373 | 113374 | 113375 | 113376 | 113377 | 113378 | 113379 | 113380 | 113381 | 113382 | 113396