From 42a1b3ffef7f8aae05718aa3a8676452364b923e Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Wed, 6 Oct 2021 15:45:17 +0200 Subject: [PATCH] Bug 27526: Improve robustness of tests * Could fail if not using the original sample data: build the AV we need instead of relying on what is in the DB --- t/db_dependent/Koha/Item.t | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/t/db_dependent/Koha/Item.t b/t/db_dependent/Koha/Item.t index b791646b40a..1625e0132c6 100755 --- a/t/db_dependent/Koha/Item.t +++ b/t/db_dependent/Koha/Item.t @@ -1103,7 +1103,7 @@ subtest 'columns_to_str' => sub { my $biblio = $builder->build_sample_biblio({ frameworkcode => '' }); my $item = $builder->build_sample_item({ biblionumber => $biblio->biblionumber }); - my $itemlost = Koha::AuthorisedValues->search({ category => 'LOST' })->next->authorised_value; + my $lost_av = $builder->build_object({ class => 'Koha::AuthorisedValues', value => { category => 'LOST', authorised_value => '42' }}); my $dateaccessioned = '2020-12-15'; my $library = Koha::Libraries->search->next; my $branchcode = $library->branchcode; @@ -1126,7 +1126,7 @@ subtest 'columns_to_str' => sub { $item->update( { - itemlost => $itemlost, + itemlost => $lost_av->authorised_value, dateaccessioned => $dateaccessioned, more_subfields_xml => $some_marc_xml, } @@ -1135,9 +1135,9 @@ subtest 'columns_to_str' => sub { $item = $item->get_from_storage; my $s = $item->columns_to_str; - is( $s->{itemlost}, 'Lost' ); - is( $s->{dateaccessioned}, '2020-12-15'); - is( $s->{'é'}, 'value é'); + is( $s->{itemlost}, $lost_av->lib, 'Attributes linked with AV replaced with description' ); + is( $s->{dateaccessioned}, '2020-12-15', 'Date attributes iso formatted'); + is( $s->{'é'}, 'value é', 'subfield ok with more than a-Z'); is( $s->{'è'}, $library->branchname ); $cache->clear_from_cache("MarcStructure-0-"); -- 2.25.1