[Koha-bugs] [Bug 18897] Koha_Elasticsearch_Indexer.t doesn' t skip when ES module not installed

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Fri Sep 1 20:52:11 CEST 2017


https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18897

Lee Jamison <ldjamison at marywood.edu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #66777|0                           |1
        is obsolete|                            |

--- Comment #10 from Lee Jamison <ldjamison at marywood.edu> ---
Created attachment 66779
  -->
https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=66779&action=edit
Bug 18897: Use Module::Load::Conditional::can_load

Rather than rely on eval(), Jonathan Druart recommended in
comment #6 to use Use Module::Load::Conditional::can_load.
This addresses that concern.

TEST PLAN
---------
$ git fetch
$ git checkout -b bug_18897 origin/master
$ git bz apply 18897
$ sudo apt-get remove libcatmandu-store-elasticsearch-perl
...
$ prove t/db_dependent/Koha_Elasticsearch_Indexer.t
-- Note that the message says the correct library is missing.
$ sudo apt-get install libcatmandu-store-elasticsearch-perl
-- Note it runs as expected.
run the koha qa test tools

Signed-off-by: Lee Jamison <ldjamison at marywood.edu>

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list