Attachment 66568 Details for Bug 18897 – Bug 18897: Koha_Elasticsearch_Indexer.t doesn't skip when ES module(s) not installed

[patch] Bug 18897: Koha_Elasticsearch_Indexer.t doesn't skip when ES module(s) not installed

Bug-18897-KohaElasticsearchIndexert-doesnt-skip-wh.patch (text/plain), 1.60 KB, created by Lee Jamison on 2017-08-28 20:46:12 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Lee Jamison

Created: 2017-08-28 20:46:12 UTC

Size: 1.60 KB

patch

obsolete

>From e47aa16ebbc2784553d1f9f2042d59653970aea7 Mon Sep 17 00:00:00 2001
>From: Lee Jamison <ldjamison@marywood.edu>
>Date: Fri, 25 Aug 2017 07:18:15 -0400
>Subject: [PATCH] Bug 18897: Koha_Elasticsearch_Indexer.t doesn't skip when ES
> module(s) not installed
>
>If Catmandu::Importer::MARC is not installed,
>Koha_Elasticsearch_Indexer.t fails instead of skipping.
>It should skip as ES is not currently mandatory.
>
>Test Plan:
>1. Run "prove t/db_dependent/Koha_Elasticsearch_Indexer.t"
>   as koha-shell
>2. Note failure on test 1/5 indicating module not installed.
>3. Apply patch.
>4. Re-run "prove t/db_dependent/Koha_Elasticsearch_Indexer.t"
>5. Note test now skips.
>
>Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
>
>Signed-off-by: Lee Jamison <ldjamison@marywood.edu>
>---
> t/db_dependent/Koha_Elasticsearch_Indexer.t | 11 ++++++++++-
> 1 file changed, 10 insertions(+), 1 deletion(-)
>
>diff --git a/t/db_dependent/Koha_Elasticsearch_Indexer.t b/t/db_dependent/Koha_Elasticsearch_Indexer.t
>index 9a525bf..1cb3a7d 100644
>--- a/t/db_dependent/Koha_Elasticsearch_Indexer.t
>+++ b/t/db_dependent/Koha_Elasticsearch_Indexer.t
>@@ -17,7 +17,7 @@
> 
> use Modern::Perl;
> 
>-use Test::More tests => 6;
>+use Test::More;
> use Test::MockModule;
> use t::lib::Mocks;
> 
>@@ -27,6 +27,15 @@ use Koha::Database;
> 
> my $schema = Koha::Database->schema();
> 
>+eval { require Koha::SearchEngine::Elasticsearch::Indexer; };
>+if ( $@ ) {
>+    my $es_dep_msg = "Required module Catmandu::Importer::MARC is not installed";
>+    plan( skip_all => $es_dep_msg );
>+}
>+else {
>+    plan tests => 6;
>+}
>+
> use_ok('Koha::SearchEngine::Elasticsearch::Indexer');
> 
> my $indexer;
>-- 
>2.1.4

Actions: View | Diff | Splinter Review

Attachments on bug 18897: 64818 | 66478 | 66479 | 66568 | 66569 | 66777 | 66779 | 67521 | 67522 | 67523 | 67539 | 67540 | 67553 | 67554 | 67555 | 67556 | 67557