From 5ef97539838ede202bcf620b7ed99246c70c13ec Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Thu, 9 May 2024 08:58:46 +0100 Subject: [PATCH] Bug 36687: (RM follow-up) Fix more unit tests Use currency.active to test `int(1) DEFAULT NULL` now that itemtypes.notforloan is `NOT NULL DEFAULT '0'` Signed-off-by: Martin Renvoize (cherry picked from commit 26e7ed29af9ce233066d980136ee10ca0c5b8609) Signed-off-by: Fridolin Somers --- t/db_dependent/Koha/Object.t | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Koha/Object.t b/t/db_dependent/Koha/Object.t index 92763c5699..28ac6e5e94 100755 --- a/t/db_dependent/Koha/Object.t +++ b/t/db_dependent/Koha/Object.t @@ -1001,9 +1001,17 @@ subtest 'store() tests' => sub { }; is( $@, '', 'No error should be raised by ->store if empty strings are passed' ); is( $itemtype->rentalcharge, undef, 'decimal DEFAULT NULL should default to null'); - is( $itemtype->notforloan, undef, 'int DEFAULT NULL should default to null'); is( $itemtype->hideinopac, 0, 'int NOT NULL DEFAULT 0 should default to 0'); + my $currency = eval { + Koha::Acquisition::Currency->new( + { + active => 'IT4test', + } + )->store; + }; + is( $currency->active, undef, 'int DEFAULT NULL should default to null'); + subtest 'Bad value tests' => sub { plan tests => 3; -- 2.39.5