Attachment 166510 Details for Bug 35994 – Bug 35994: (QA follow-up) Fix Biblio.t

[patch] Bug 35994: (QA follow-up) Fix Biblio.t

Bug-35994-QA-follow-up-Fix-Bibliot.patch (text/plain), 1.32 KB, created by Marcel de Rooy on 2024-05-10 08:41:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2024-05-10 08:41:02 UTC

Size: 1.32 KB

patch

obsolete

>From c138ece6049460494a208fc52f2a59d76a3f4f9b Mon Sep 17 00:00:00 2001
>From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Date: Fri, 10 May 2024 08:37:32 +0000
>Subject: [PATCH] Bug 35994: (QA follow-up) Fix Biblio.t
>Content-Type: text/plain; charset=utf-8
>
>A follow-up changed orders->cancel a bit. This test assumed to
>cancel completed lines here. So we are doing it now with a
>direct update to have the original result.
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> t/db_dependent/Koha/Biblio.t | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/Koha/Biblio.t b/t/db_dependent/Koha/Biblio.t
>index 56c58f1535..9dda4f9f82 100755
>--- a/t/db_dependent/Koha/Biblio.t
>+++ b/t/db_dependent/Koha/Biblio.t
>@@ -999,7 +999,7 @@ subtest '->orders, ->uncancelled_orders and ->acq_status tests' => sub {
>     is( $biblio->acq_status, 'processing', 'Still processing for presence of new lines' );
>     $orders->filter_out_cancelled->update( { orderstatus => 'complete' } );
>     is( $biblio->acq_status, 'acquired', 'Acquired: some complete, rest cancelled' );
>-    $orders->cancel;
>+    $orders->update( { orderstatus => 'cancelled', datecancellationprinted => dt_from_string() } );
>     is( $biblio->acq_status, 'cancelled', 'Cancelled for only cancelled lines' );
> 
>     $schema->storage->txn_rollback;
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 35994: 162334 | 162335 | 165154 | 165155 | 165224 | 165225 | 166510 | 166537 | 166538 | 166539