Attachment 141342 Details for Bug 22321 – Bug 22321: Add unit tests

[patch] Bug 22321: Add unit tests

Bug-22321-Add-unit-tests.patch (text/plain), 2.86 KB, created by Martin Renvoize (ashimema) on 2022-10-05 10:44:57 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2022-10-05 10:44:57 UTC

Size: 2.86 KB

patch

obsolete

>From 7086ad28c2502083186820327325a65846993aa5 Mon Sep 17 00:00:00 2001
>From: Lari Taskula <lari.taskula@hypernova.fi>
>Date: Mon, 14 Feb 2022 11:14:36 +0000
>Subject: [PATCH] Bug 22321: Add unit tests
>
>To test:
>1. prove t/db_dependent/Illrequests.t
>2. Observe test failure
>
>    1..3
>    ok 1 - before change, original borrowernumber found
>    not ok 2 - after change, changed borrowernumber found in holds
>    #   Failed test 'after change, changed borrowernumber found in holds'
>    #   at t/db_dependent/Illrequests.t line 167.
>    #          got: '3786'
>    #     expected: '3787'
>    ok 3 - after change, changed borrowernumber found in illrequests
>    # Looks like you failed 1 test of 3.
>not ok 4 - store borrowernumber change also updates holds
>
>Signed-off-by: Magnus Enger <magnus@libriotech.no>
>Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> t/db_dependent/Illrequests.t | 49 ++++++++++++++++++++++++++++++++++++
> 1 file changed, 49 insertions(+)
>
>diff --git a/t/db_dependent/Illrequests.t b/t/db_dependent/Illrequests.t
>index 1d5d8c07ec..1bc9c5116f 100755
>--- a/t/db_dependent/Illrequests.t
>+++ b/t/db_dependent/Illrequests.t
>@@ -125,6 +125,55 @@ subtest 'Basic object tests' => sub {
>     $schema->storage->txn_rollback;
> };
> 
>+subtest 'store borrowernumber change also updates holds' => sub {
>+    plan tests => 3;
>+
>+    $schema->storage->txn_begin;
>+
>+    Koha::Illrequests->search->delete;
>+
>+    my $patron = $builder->build_object({ class => 'Koha::Patrons' });
>+    my $other_patron = $builder->build_object({ class => 'Koha::Patrons' });
>+    my $biblio = $builder->build_object({ class => 'Koha::Biblios' });
>+
>+    my $request = $builder->build_object({
>+        class => 'Koha::Illrequests',
>+        value => {
>+            borrowernumber => $patron->borrowernumber,
>+            biblio_id => $biblio->biblionumber,
>+        }
>+    });
>+    $builder->build({
>+        source => 'Reserve',
>+        value => {
>+            borrowernumber => $patron->borrowernumber,
>+            biblionumber => $request->biblio_id
>+        }
>+    });
>+
>+    my $hold = Koha::Holds->find({
>+        biblionumber => $request->biblio_id,
>+        borrowernumber => $request->borrowernumber,
>+    });
>+
>+    is( $hold->borrowernumber, $request->borrowernumber,
>+       'before change, original borrowernumber found' );
>+
>+    $request->borrowernumber( $other_patron->borrowernumber )->store;
>+
>+    # reload changes
>+    $hold->discard_changes;
>+
>+    is( $hold->borrowernumber, $other_patron->borrowernumber,
>+       'after change, changed borrowernumber found in holds' );
>+
>+    is( $request->borrowernumber, $other_patron->borrowernumber,
>+       'after change, changed borrowernumber found in illrequests' );
>+
>+    $schema->storage->txn_rollback;
>+
>+};
>+
> subtest 'Working with related objects' => sub {
> 
>     plan tests => 7;
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 22321: 130544 | 130545 | 130546 | 131012 | 131013 | 131055 | 131056 | 131057 | 131058 | 131059 | 138873 | 138874 | 138875 | 138876 | 138877 | 138878 | 141342 | 141343 | 141344 | 141345 | 141346 | 141347