Bug 16649

Summary: OpenLibrarySearch.t fails when building packages
Product: Koha Reporter: Mirko Tietgen <mirko>
Component: Test SuiteAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Galen Charlton <gmcharlt>
Severity: minor    
Priority: P5 - low CC: f.demians, jonathan.druart, julian.maurice, kyle
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 6624    
Bug Blocks:    
Attachments: Bug 16649: Make OpenLibrarySearch test pass even if launches offline
[SIGNED-OFF] Bug 16649: Make OpenLibrarySearch test pass even if launches offline
Bug 16649: Make OpenLibrarySearch test pass even if launches offline
Bug 16649: Make OpenLibrarySearch test pass even if launches offline

Description Mirko Tietgen 2016-06-03 14:15:49 UTC
Use of uninitialized value $_[0] in string eq at /usr/share/perl5/JSON.pm line 159.
malformed JSON string, neither array, object, number, string or atom, at character offset 0 (before "(end of string)") at t/OpenLibrarySearch.t line 25
# Looks like your test exited with 22 before it could output anything.
t/OpenLibrarySearch.t ............... 
Dubious, test returned 22 (wstat 5632, 0x1600)
Failed 1/1 subtests
Comment 1 Jonathan Druart 2016-06-04 11:28:56 UTC Comment hidden (obsolete)
Comment 2 Srdjan Jankovic 2016-06-06 05:21:34 UTC Comment hidden (obsolete)
Comment 3 Mirko Tietgen 2016-06-06 06:18:38 UTC Comment hidden (obsolete)
Comment 4 Galen Charlton 2016-06-07 01:54:26 UTC
Created attachment 52107 [details] [review]
Bug 16649: Make OpenLibrarySearch test pass even if launches offline

Test plan:
  prove t/OpenLibrarySearch.t
should return green even if you are offline

Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Signed-off-by: Mirko Tietgen <mirko@abunchofthings.net>
Signed-off-by: Galen Charlton <gmcharlt@gmail.com>
Comment 5 Jonathan Druart 2016-06-07 08:14:21 UTC
3 signed-offs and trivial patch, skipping QA.
Comment 6 Kyle M Hall 2016-06-10 17:05:59 UTC
Pushed to master for 16.11, thanks Jonathan!
Comment 7 Frédéric Demians 2016-06-15 07:44:25 UTC
Pushed in 16.05. Will be in 16.05.01.
Comment 8 Julian Maurice 2016-06-16 09:06:41 UTC
Not needed in 3.22.x