Attachment 168884 Details for Bug 36641 – Bug 36641: Unit tests for effective=0

[patch] Bug 36641: Unit tests for effective=0

Bug-36641-Unit-tests-for-effective0.patch (text/plain), 3.60 KB, created by Martin Renvoize (ashimema) on 2024-07-12 10:27:43 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Martin Renvoize (ashimema)

Created: 2024-07-12 10:27:43 UTC

Size: 3.60 KB

patch

obsolete

>From bb080f14026e154988a7e665535c930dd860426c Mon Sep 17 00:00:00 2001
>From: Martin Renvoize <martin.renvoize@ptfs-europe.com>
>Date: Fri, 12 Jul 2024 10:16:08 +0100
>Subject: [PATCH] Bug 36641: Unit tests for effective=0
>
>---
> t/db_dependent/api/v1/circulation_rules.t | 69 ++++++++++++++++++++++-
> 1 file changed, 68 insertions(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/api/v1/circulation_rules.t b/t/db_dependent/api/v1/circulation_rules.t
>index f1ec45a56ec..712c9d459b4 100755
>--- a/t/db_dependent/api/v1/circulation_rules.t
>+++ b/t/db_dependent/api/v1/circulation_rules.t
>@@ -33,7 +33,7 @@ my $t = Test::Mojo->new('Koha::REST::V1');
> t::lib::Mocks::mock_preference( 'RESTBasicAuth', 1 );
> 
> subtest 'list_rules() tests' => sub {
>-    plan tests => 32;
>+    plan tests => 33;
> 
>     $schema->storage->txn_begin;
> 
>@@ -202,5 +202,72 @@ subtest 'list_rules() tests' => sub {
>     # Unauthorized access
>     $t->get_ok("//$unauth_userid:$password@/api/v1/circulation_rules")->status_is(403);
> 
>+    subtest 'effective=false tests' => sub {
>+
>+        my $expected_keys = [ keys %{ Koha::CirculationRules->rule_kinds } ];
>+        my $count         = scalar( @{$expected_keys} );
>+
>+        plan tests => ( $count * 2 ) + $count + 10;
>+
>+        # All rules
>+        $t->get_ok("//$userid:$password@/api/v1/circulation_rules?effective=0")->status_is(200);
>+
>+        # Extract and decode the JSON response
>+        my $json = $t->tx->res->json;
>+
>+        # Check if the response is an array
>+        is( ref $json,          'ARRAY', 'Response is an array' );
>+        is( scalar( @{$json} ), 2,       'Response contains 2 rule sets' );
>+
>+        # Iterate over each hash in the array
>+        my $index = 0;
>+        foreach my $hash ( @{$json} ) {
>+            my $pointer = Mojo::JSON::Pointer->new($hash);
>+
>+            # First rule set should march default, default, default
>+            if ( $index == 0 ) {
>+                ok(        $pointer->get('/branchcode') eq "*"
>+                        && $pointer->get('/itemtype') eq '*'
>+                        && $pointer->get('/categorycode') eq '*', "Default rules returned first" );
>+            }
>+
>+            # Iterate over the list of expected keys for each hash
>+            foreach my $key ( @{$expected_keys} ) {
>+                ok( $pointer->contains( '/' . $key ), "Hash contains key '$key'" );
>+            }
>+
>+            $index++;
>+        }
>+
>+        # Filter on library
>+        $t->get_ok("//$userid:$password@/api/v1/circulation_rules?effective=0&library_id=$branchcode")->status_is(200);
>+
>+        # Extract and decode the JSON response
>+        $json = $t->tx->res->json;
>+
>+        # Check if the response is an array
>+        is( ref $json,          'ARRAY', 'Response is an array' );
>+        is( scalar( @{$json} ), 1,       'Filtered response contains 1 rule set' );
>+
>+        $index = 0;
>+        foreach my $hash ( @{$json} ) {
>+            my $pointer = Mojo::JSON::Pointer->new($hash);
>+
>+            # First (and only) rule set should match branchcode, default, default.
>+            if ( $index == 0 ) {
>+                ok(        $pointer->get('/branchcode') eq $branchcode
>+                        && $pointer->get('/itemtype') eq '*'
>+                        && $pointer->get('/categorycode') eq '*', "Branchcode rule set returned when filtered" );
>+            }
>+
>+            # Iterate over the list of expected keys for each hash
>+            foreach my $key ( @{$expected_keys} ) {
>+                ok( $pointer->contains( '/' . $key ), "Hash contains key '$key'" );
>+            }
>+
>+            $index++;
>+        }
>+
>+    };
>     $schema->storage->txn_rollback;
> };
>-- 
>2.45.2

Actions: View | Diff | Splinter Review

Attachments on bug 36641: 165086 | 167937 | 167938 | 167946 | 167947 | 168018 | 168019 | 168020 | 168023 | 168024 | 168025 | 168365 | 168366 | 168367 | 168368 | 168369 | 168370 | 168378 | 168379 | 168395 | 168396 | 168397 | 168398 | 168399 | 168400 | 168401 | 168841 | 168874 | 168875 | 168876 | 168877 | 168878 | 168879 | 168880 | 168881 | 168882 | 168883 | 168884 | 168905 | 168906 | 168907 | 168908 | 168909 | 168910 | 168911 | 168912 | 168913 | 168914 | 168915 | 168916 | 168926 | 168927 | 168928 | 168929 | 168930 | 168931 | 168932 | 168933 | 168934 | 168935 | 168936 | 168937 | 168938 | 168940 | 169013 | 169014 | 169015 | 169016 | 169017 | 169018 | 169019 | 169020 | 169021 | 169022 | 169023 | 169024 | 169025 | 169026