Attachment 117672 Details for Bug 26636 – Bug 26636: Regression tests for undef case

[patch] Bug 26636: Regression tests for undef case

Bug-26636-Regression-tests-for-undef-case.patch (text/plain), 1.45 KB, created by Martin Renvoize (ashimema) on 2021-03-04 10:34:20 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2021-03-04 10:34:20 UTC

Size: 1.45 KB

patch

obsolete

>From 975dfa6105df316cdebf0b9ea11025f2b3492952 Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Tue, 2 Feb 2021 16:28:07 -0300
>Subject: [PATCH] Bug 26636: Regression tests for undef case
>
>This patch adds tests for the case in which there's no object identified
>by the supplied $id. It should return undef.
>
>Sponsored-by: Virginia Polytechnic Institute and State University
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>Signed-off-by: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
>Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>---
> t/db_dependent/Koha/REST/Plugin/Objects.t | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/Koha/REST/Plugin/Objects.t b/t/db_dependent/Koha/REST/Plugin/Objects.t
>index 2f4937f89d..e85a57f78d 100755
>--- a/t/db_dependent/Koha/REST/Plugin/Objects.t
>+++ b/t/db_dependent/Koha/REST/Plugin/Objects.t
>@@ -470,7 +470,7 @@ subtest 'object.search helper order by embedded columns' => sub {
> 
> subtest 'objects.find helper' => sub {
> 
>-    plan tests => 6;
>+    plan tests => 9;
> 
>     my $t = Test::Mojo->new;
> 
>@@ -487,6 +487,13 @@ subtest 'objects.find helper' => sub {
>       ->status_is(200)
>       ->json_is( $city_2->to_api );
> 
>+    # Remove the city
>+    my $city_2_id = $city_2->id;
>+    $city_2->delete;
>+    $t->get_ok( '/cities/' . $city_2_id )
>+      ->status_is(200)
>+      ->json_is( undef );
>+
>     $schema->storage->txn_rollback;
> };
> 
>-- 
>2.20.1

Actions: View | Diff | Splinter Review

Attachments on bug 26636: 111418 | 111419 | 116222 | 116223 | 116224 | 116225 | 116226 | 116227 | 116228 | 116229 | 116230 | 116231 | 116594 | 116595 | 116596 | 116597 | 116598 | 117669 | 117670 | 117671 | 117672 | 117673 | 127528 | 127529 | 127530 | 127531 | 127532