[koha-commits] main Koha release repository branch 17.05.x updated. v17.05.04-90-g2b4d0cb

[Git repo owner]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".

The branch, 17.05.x has been updated
       via  2b4d0cb0b4e16b008ad9928a09259accfa1efc7a (commit)
       via  faf651e514a1d2f5fa84df0c64556cc9962afc99 (commit)
       via  088e332760b55c6f7a44097d8ab5d84003d8f0e0 (commit)
       via  a24ac206701bdde509baf47f005aa31bf42a1b6a (commit)
       via  b6b6acf285f98a7da665eac6741cdfc87b089b57 (commit)
      from  02d518b334596ed585596c614ab31c0656901634 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 2b4d0cb0b4e16b008ad9928a09259accfa1efc7a
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Mon Oct 2 22:00:57 2017 -0400

    Bug 18897: Add skipping to t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t
    
    Test Plan:
    $ git fetch
    $ git checkout -b bug_18897 origin/master
    $ sudo apt-get remove libcatmandu-store-elasticsearch-perl
    $ prove t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t
    -- nasty failure.
    $ git bz apply 18897
    -- apply all the patches
    $ prove t/db_dependent/Koha_SearchEngine_Elasticsearch_Search.t
    -- nicer skip message
    run the koha qa test tools
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>
    Signed-off-by: Fridolin Somers <fridolin.somers at biblibre.com>

commit faf651e514a1d2f5fa84df0c64556cc9962afc99
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Mon Oct 2 21:50:01 2017 -0400

    Bug 18897: Skipping t/db_dependend/00-strict elastic search stuffs as needed
    
    Test Plan:
    $ git fetch
    $ git checkout -b bug_18897 origin/master
    $ sudo apt-get remove libcatmandu-store-elasticsearch-perl
    $ prove t/db_dependent/00-strict.t
    -- nasty failure.
    $ git bz apply 18897
    -- interactive apply just the four patchs
    $ prove t/db_dependent/00-strict.t
    -- nicer skip message
    run the koha qa test tools
    
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit 088e332760b55c6f7a44097d8ab5d84003d8f0e0
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Fri Sep 1 14:15:44 2017 -0400

    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>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit a24ac206701bdde509baf47f005aa31bf42a1b6a
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Fri Aug 25 07:38:06 2017 -0400

    Bug 18897: Improve error message
    
    Test Plan:
    $ git fetch
    $ git checkout -b bug_18897 origin/master
    $ git bz apply 18897
    -- interactive apply just the first patch
    $ sudo apt-get remove libcatmandu-store-elasticsearch-perl
    ...
    $ prove t/db_dependent/Koha_Elasticsearch_Indexer.t
    -- Note that the message says the wrong library is missing.
    $ git bz apply 18897
    -- interactive apply this patch
    $ prove t/db_dependent/Koha_Elasticsearch_Indexer.t
    -- Note that the message says the correct library is missing.
    run the koha qa test tools
    
    Signed-off-by: Mark Tompsett <mtompset at hotmail.com>
    
    Signed-off-by: Lee Jamison <ldjamison at marywood.edu>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

commit b6b6acf285f98a7da665eac6741cdfc87b089b57
Author: Lee Jamison <ldjamison at marywood.edu>
Date:   Fri Aug 25 07:18:15 2017 -0400

    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 at hotmail.com>
    
    Signed-off-by: Lee Jamison <ldjamison at marywood.edu>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at theke.io>

-----------------------------------------------------------------------

Summary of changes:
 t/db_dependent/00-strict.t                         |   21 +++++++++++++++++++-
 t/db_dependent/Koha_Elasticsearch_Indexer.t        |   17 +++++++++++++++-
 .../Koha_SearchEngine_Elasticsearch_Search.t       |   17 +++++++++++++++-
 3 files changed, 52 insertions(+), 3 deletions(-)


hooks/post-receive
-- 
main Koha release repository