Attachment 164285 Details for Bug 36493 – Bug 36493: Make cashups test more robust

[patch] Bug 36493: Make cashups test more robust

Bug-36493-Make-cashups-test-more-robust.patch (text/plain), 1.47 KB, created by Tomás Cohen Arazi (tcohen) on 2024-04-02 15:42:29 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tomás Cohen Arazi (tcohen)

Created: 2024-04-02 15:42:29 UTC

Size: 1.47 KB

patch

obsolete

>From dc19e9584fac58330793a9aa9ed28a4e4b17422a Mon Sep 17 00:00:00 2001
>From: Tomas Cohen Arazi <tomascohen@theke.io>
>Date: Tue, 2 Apr 2024 12:39:02 -0300
>Subject: [PATCH] Bug 36493: Make cashups test more robust
>
>This patch makes the test independent from existing cash registers in
>the system.
>
>To test:
>1. Enable UseCashRegisters on a fresh KTD
>2. Add a cash register
>3. Run:
>   $ ktd --shell
>  k$ prove t/db_dependent/api/v1/cashups.t
>=> FAIL: Tests fail!
>4. Apply this patch
>5. Repeat 3
>=> SUCCESS: Tests pass!
>6. Sign off :-D
>
>Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
>---
> t/db_dependent/api/v1/cashups.t | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/api/v1/cashups.t b/t/db_dependent/api/v1/cashups.t
>index 4dcb4ae6a6a..ab7646d0f96 100755
>--- a/t/db_dependent/api/v1/cashups.t
>+++ b/t/db_dependent/api/v1/cashups.t
>@@ -63,7 +63,11 @@ subtest 'list() tests' => sub {
> 
>     ## Authorized user tests
>     # No cash register, so 404 should be returned
>-    $t->get_ok("//$userid:$password@/api/v1/cash_registers/1/cashups")
>+    my $cash_register_to_delete = $builder->build_object( { class => 'Koha::Cash::Registers' } );
>+    my $non_existent_cr_id      = $cash_register_to_delete->id;
>+    $cash_register_to_delete->delete;
>+
>+    $t->get_ok("//$userid:$password@/api/v1/cash_registers/$non_existent_cr_id/cashups")
>       ->status_is(404)->json_is( '/error' => 'Register not found' );
> 
>     my $register = $builder->build_object(
>-- 
>2.44.0

Actions: View | Diff | Splinter Review

Attachments on bug 36493: 164285 | 164288 | 164788