Bugzilla – Attachment 64170 Details for
Bug 18762
Some tests are noisy
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 18762: Catch noisy conditions in t/db_dependent/Search.t
Bug-18762-Catch-noisy-conditions-in-tdbdependentSe.patch (text/plain), 2.22 KB, created by
Mark Tompsett
on 2017-06-10 03:09:51 UTC
(
hide
)
Description:
Bug 18762: Catch noisy conditions in t/db_dependent/Search.t
Filename:
MIME Type:
Creator:
Mark Tompsett
Created:
2017-06-10 03:09:51 UTC
Size:
2.22 KB
patch
obsolete
>From ff719f8cd71fd2781a63cc9209607046126a4129 Mon Sep 17 00:00:00 2001 >From: Mark Tompsett <mtompset@hotmail.com> >Date: Fri, 9 Jun 2017 23:03:34 -0400 >Subject: [PATCH] Bug 18762: Catch noisy conditions in t/db_dependent/Search.t > >Seems that the grs1 case for ti:punctuation generates a warning, >while the dom case does not. This tweak catches both. > >before patch: noisy. >after patch: no unexpected output. >run koha qa test tools >--- > t/db_dependent/Search.t | 16 ++++++++++++---- > 1 file changed, 12 insertions(+), 4 deletions(-) > >diff --git a/t/db_dependent/Search.t b/t/db_dependent/Search.t >index c8cbc48..d376913 100644 >--- a/t/db_dependent/Search.t >+++ b/t/db_dependent/Search.t >@@ -661,8 +661,16 @@ if ( $indexing_mode eq 'dom' ) { > ok(!exists($newresults[0]->{norequests}), 'presence of a transit does not block hold request action (bug 10741)'); > > ## Regression test for bug 10684 >- ( undef, $results_hashref, $facets_loop ) = >- getRecords('ti:punctuation', 'punctuation', [], [ 'biblioserver' ], '19', 0, undef, \%branches, \%itemtypes, 'ccl', undef); >+ my $expected_warning = $indexing_mode eq 'grs1' ? >+ qr/^ERROR DECODING RECORD - Tag "50%" is not a valid tag/ : >+ undef; >+ my $expected_description = $indexing_mode eq 'grs1' ? >+ "Warning is raised correctly for ti:punctuation MARC::Record" : >+ "No warning is raised as expected for ti:punctuation MARC::Record"; >+ warning_like { ( undef, $results_hashref, $facets_loop ) = >+ getRecords('ti:punctuation', 'punctuation', [], [ 'biblioserver' ], '19', 0, undef, \%branches, \%itemtypes, 'ccl', undef) >+ } >+ $expected_warning, $expected_description; > is($results_hashref->{biblioserver}->{hits}, 1, "search for ti:punctuation returned expected number of records"); > warning_like { @newresults = searchResults('intranet', $query_desc, > $results_hashref->{'biblioserver'}->{'hits'}, 20, 0, 0, >@@ -966,12 +974,12 @@ sub run_unimarc_search_tests { > } > > subtest 'MARC21 + GRS-1' => sub { >- plan tests => 109; >+ plan tests => 110; > run_marc21_search_tests('grs1'); > }; > > subtest 'MARC21 + DOM' => sub { >- plan tests => 109; >+ plan tests => 110; > run_marc21_search_tests('dom'); > }; > >-- >2.1.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 18762
:
64114
|
64115
|
64116
|
64117
|
64170
|
64175
|
64232
|
64234
|
64294
|
64295
|
64296