Attachment 105917 Details for Bug 23092 – Bug 23092: Add unit tests

[patch] Bug 23092: Add unit tests

Bug-23092-Add-unit-tests.patch (text/plain), 2.30 KB, created by Martin Renvoize (ashimema) on 2020-06-16 13:21:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2020-06-16 13:21:14 UTC

Size: 2.30 KB

patch

obsolete

>From 700694c41d339b0d2e2a1ade41834e218fe558a9 Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Tue, 16 Jun 2020 13:51:00 +0100
>Subject: [PATCH] Bug 23092: Add unit tests
>
>---
> t/db_dependent/Koha/Item/Transfers.t | 43 +++++++++++++++++++++++++++-
> 1 file changed, 42 insertions(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/Koha/Item/Transfers.t b/t/db_dependent/Koha/Item/Transfers.t
>index b49f78091a..28e070671f 100644
>--- a/t/db_dependent/Koha/Item/Transfers.t
>+++ b/t/db_dependent/Koha/Item/Transfers.t
>@@ -19,7 +19,7 @@
> 
> use Modern::Perl;
> 
>-use Test::More tests => 2;
>+use Test::More tests => 3;
> 
> use Koha::Item::Transfer;
> use Koha::Item::Transfers;
>@@ -27,6 +27,7 @@ use Koha::Database;
> use Koha::DateUtils;
> 
> use t::lib::TestBuilder;
>+use t::lib::Dates;
> 
> my $schema = Koha::Database->new->schema;
> $schema->storage->txn_begin;
>@@ -66,3 +67,43 @@ is( $retrieved_transfer_1->itemnumber, $new_transfer_1->itemnumber, 'Find a tran
> 
> $schema->storage->txn_rollback;
> 
>+subtest 'daterequested tests' => sub {
>+
>+    plan tests => 3;
>+
>+    $schema->storage->txn_begin;
>+    my $library_from = $builder->build( { source => 'Branch' } );
>+    my $library_to   = $builder->build( { source => 'Branch' } );
>+    my $item         = $builder->build(
>+        {
>+            source => 'Item',
>+            value  => {
>+                holding_branch => $library_from->{branchcode},
>+                homebranch     => $library_to->{branchcode}
>+            }
>+        }
>+    );
>+
>+    my $now = dt_from_string;
>+    my $transfer = Koha::Item::Transfer->new(
>+        {
>+            itemnumber => $item->{itemnumber},
>+            frombranch => $library_from->{branchcode},
>+            tobranch   => $library_to->{branchcode}
>+        }
>+    )->store;
>+    $transfer->discard_changes;
>+
>+    ok( $transfer->daterequested, 'daterequested set on creation' );
>+    is( t::lib::Dates::compare( $transfer->daterequested, $now ),
>+        0, 'daterequested was set correctly' );
>+
>+    my $new_date = $now->clone->add( hours => 1 );
>+    $transfer->set({ datesent => $new_date })->store;
>+    $transfer->discard_changes;
>+
>+    is( t::lib::Dates::compare( $transfer->daterequested, $now ),
>+        0, 'daterequested is not updated when other fields are updated' );
>+
>+    $schema->storage->txn_rollback;
>+};
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 23092: 97512 | 97542 | 100631 | 101093 | 105912 | 105913 | 105916 | 105917 | 106609 | 106610 | 107238 | 107239