From faa468dfc966ae1ad03ec010037b70f54d223758 Mon Sep 17 00:00:00 2001 From: Tomas Cohen Arazi Date: Thu, 25 Mar 2021 09:48:35 -0300 Subject: [PATCH] Bug 28031: Regression tests Signed-off-by: Martin Renvoize --- t/db_dependent/Koha/Patron/Attribute.t | 34 +++++++++++++++++++++++++- 1 file changed, 33 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Koha/Patron/Attribute.t b/t/db_dependent/Koha/Patron/Attribute.t index b46d811374..670bc87cbf 100755 --- a/t/db_dependent/Koha/Patron/Attribute.t +++ b/t/db_dependent/Koha/Patron/Attribute.t @@ -33,7 +33,7 @@ my $builder = t::lib::TestBuilder->new; subtest 'store() tests' => sub { - plan tests => 3; + plan tests => 4; subtest 'repeatable attributes tests' => sub { @@ -215,6 +215,38 @@ subtest 'store() tests' => sub { $schema->storage->txn_rollback; }; + + subtest 'Edit attribute tests for non-repeatable tests (Bug 28031)' => sub { + + plan tests => 1; + + my $patron = $builder->build_object({ class => 'Koha::Patrons' }); + my $non_repeatable_type = $builder->build_object( + { + class => 'Koha::Patron::Attribute::Types', + value => { + mandatory => 0, + repeatable => 0, + unique_id => 1, + category_code => undef + } + } + ); + + # Here we test the case of editing an already stored attribute + my $non_repeatable_attr = $patron->add_extended_attribute( + { + code => $non_repeatable_type->code, + attribute => 'WOW' + } + ); + + $non_repeatable_attr->set({ attribute => 'HEY' }) + ->store + ->discard_changes; + + is( $non_repeatable_attr->attribute, 'HEY', 'Value stored correctly' ); + }; }; subtest 'type() tests' => sub { -- 2.20.1