[koha-commits] main Koha release repository branch master updated. v16.05.00-101-g1fda410

[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, master has been updated
       via  1fda4105a8a0f1e2165e27cea8314b578196dba2 (commit)
       via  7f4682df0dfa460c0e2ef9117e032165a45d6e79 (commit)
       via  5361440f6538127c8c3ec61d963d5f0692c64a71 (commit)
       via  b4479624de2010fff8851732e94d086b2cca6921 (commit)
      from  4fc8b3d6cf6453f7ff550b9fbde1a7f6315dd066 (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 1fda4105a8a0f1e2165e27cea8314b578196dba2
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Fri Jun 10 18:02:14 2016 +0000

    Bug 16636 [QA Followup] - Move tests to t/External
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 7f4682df0dfa460c0e2ef9117e032165a45d6e79
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Jun 10 10:31:34 2016 +0200

    Bug 16636: [QA Follow-up] Move External_BakerTaylor.t to t
    
    When we mock the preferences completely now, we do not need
    db access anymore.
    
    Test plan:
    Run the test without exporting KOHA_CONF.
    The test passes with some warns about koha-conf.xml.
    If you point KOHA_CONF to etc/koha-conf.xml, you have no warns.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit 5361440f6538127c8c3ec61d963d5f0692c64a71
Author: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
Date:   Fri Jun 10 09:30:51 2016 +0200

    Bug 16636: [QA Follow-up] Make BakerTaylor plack safe
    
    Initialize file level lexicals each call. Do not call _initialize
    outside the module.
    Adjust test by mocking preferences.
    
    Test plan:
    Run t/db_dependent/External_BakerTaylor.t.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    Tested module with trivial script under Plack/memcached by toggling
    the associated preferences.
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

commit b4479624de2010fff8851732e94d086b2cca6921
Author: Mark Tompsett <mtompset at hotmail.com>
Date:   Wed Jun 1 20:43:30 2016 -0400

    Bug 16636: t/00-load.t warning from C4/External/BakerTaylor.pm
    
    Since the tests are expecting an initialize function, the
    initialize call was just moved outside of the INIT block.
    
    TEST PLAN
    ---------
    1) prove t/00-load.t
       -- warnings about INIT for BakerTaylor
    2) prove `git grep -l BakerTaylor | grep [.]t$`
       -- should all run okay
    3) apply patch
    4) repeat steps 1 and 2
       -- warning should be gone, and everything else run okay
    5) run koha qa test tools
    
    Signed-off-by: Srdjan <srdjan at catalyst.net.nz>
    
    Signed-off-by: Florent Mara <florent.mara at gmail.com>
    
    NOTE: Tweaked test plan based on comment #4,
          Added sign off based on comment #6.
    
    Signed-off-by: Marcel de Rooy <m.de.rooy at rijksmuseum.nl>
    
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>

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

Summary of changes:
 C4/External/BakerTaylor.pm                       |   23 ++++++++++-----
 t/External/BakerTaylor.t                         |   33 +++++++++++++++++++++
 t/{External_Syndetics.t => External/Syndetics.t} |    0
 t/db_dependent/External_BakerTaylor.t            |   34 ----------------------
 4 files changed, 48 insertions(+), 42 deletions(-)
 create mode 100755 t/External/BakerTaylor.t
 rename t/{External_Syndetics.t => External/Syndetics.t} (100%)
 delete mode 100755 t/db_dependent/External_BakerTaylor.t


hooks/post-receive
-- 
main Koha release repository