[koha-commits] main Koha release repository branch master updated. v3.14.00-296-g1487949

Tue Jan 14 22:42:08 CET 2014

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  1487949011452ca0852f6a068c73670ef6dd9430 (commit)
       via  19b5ea868bf18883d188b6fad5faae80c5fa9c46 (commit)
       via  89b3b35e6255474ec9e6ae8860960ff250d622ba (commit)
       via  457ae7e20567e9a46ee5ceae9e8cb130064b84ba (commit)
       via  a75be3ab0d3e3837d85b69e4392c13210127ec14 (commit)
       via  0db732980aec0e7f7257b559419781adfab538ed (commit)
       via  6a6df3f5d755aa462f727fa21464883b31d51462 (commit)
      from  397d95821cca73927a6bb51c57b075af6a95bd10 (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 1487949011452ca0852f6a068c73670ef6dd9430
Author: alex_h <dhorsman at xtra.co.nz>
Date:   Tue Jan 14 14:18:46 2014 +1300

    bug 11544: increasing test coverage for Installer_PerlModules.pm

    To test
    1. run prove t/Installer_PerlModules.t
    2. apply patch
    3. run prove t/Installer_PerlModules.t
       notices more tests are run

    Signed-off-by: Magnus Enger <magnus at enger.priv.no>
    Adds another passing test.

    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Tests pass :)

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 19b5ea868bf18883d188b6fad5faae80c5fa9c46
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Tue Jan 14 21:43:56 2014 +0000

    Bug 11541: (follow-up) move Installer.t to t/db_dependent

    This test requires a valid Koha context, so this patch
    moves it to t/db_dependent.

    To test:

    [1] Verify that prove -v t/db_dependent/Installer.t passes.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 89b3b35e6255474ec9e6ae8860960ff250d622ba
Author: Galen Charlton <gmc at esilibrary.com>
Date:   Tue Jan 14 21:42:45 2014 +0000

    Bug 11541: (follow-up) update tests

    This patch updates some of the tests to more clearly
    check that fields in the C4::Installer object are getting
    set correctly from the context.

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 457ae7e20567e9a46ee5ceae9e8cb130064b84ba
Author: Kyle M Hall <kyle at bywatersolutions.com>
Date:   Tue Jan 14 15:04:27 2014 -0500

    Bug 11541: (follow-up) repair some warnings

    Tests succeed, but cause many warnings:

    prove t/Installer.t
    t/Installer.t .. 1/9 "my" variable $self masks earlier declaration in
    same scope at t/Installer.t line 19.
    "my" variable $self masks earlier declaration in same scope at
    t/Installer.t line 20.
    "my" variable $self masks earlier declaration in same scope at
    t/Installer.t line 21.
    "my" variable $self masks earlier declaration in same scope at
    t/Installer.t line 22.
    "my" variable $self masks earlier declaration in same scope at
    t/Installer.t line 23.
    Odd number of elements in anonymous hash at t/Installer.t line 17.
    Use of uninitialized value within @ARGV in anonymous hash ({}) at
    t/Installer.t line 17.

    The patch remedies that.

    Test Plan:
    1) Apply first patch on this bug
    2) Run "prove t/Installer.t"
    3) Note the warnings
    4) Apply this patch
    5) Repeat step 2
    6) Note the warnings are gone

    Signed-off-by: Aleisha Amohia <aleishaamohia at hotmail.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit a75be3ab0d3e3837d85b69e4392c13210127ec14
Author: Aleisha Amohia <aleishaamohia at hotmail.com>
Date:   Tue Jan 14 14:18:36 2014 +1300

    Bug 11541: increase test coverage of Installer.pm

    To test
    1. Run prove t/Installer.t
    2. Apply patch
    3. Run prove t/Installer.t
       Notice more tests are run

    Signed-off-by: Francesca Moore <francescalamoore at gmail.com>
    Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    All tests pass :)

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 0db732980aec0e7f7257b559419781adfab538ed
Author: Tomas Cohen Arazi <tomascohen at gmail.com>
Date:   Tue Jan 14 11:35:31 2014 -0300

    Bug 11540: (follow-up) add a couple of tests more

    This patch makes sure all scenarios are covered for SetUTF8Flag.

    Signed-off-by: Emma Heath <emmaheath.student at wegc.school.nz>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

commit 6a6df3f5d755aa462f727fa21464883b31d51462
Author: Emma Heath <emmaheath.student at wegc.school.nz>
Date:   Tue Jan 14 14:18:47 2014 +1300

    Bug 11540: increase test coverage of Charset.pm

    To test
    1. run prove t/Charset.t
    2. apply patch
    3. run prove t/Charset.t
     notice more tests are run

    Signed-off-by: Tom Houlker <thomas.houlker at hibs.school.nz>
    Signed-off-by: Tomas Cohen Arazi <tomascohen at gmail.com>
    Signed-off-by: Katrin Fischer <Katrin.Fischer.83 at web.de>
    Tests pass!

    Signed-off-by: Galen Charlton <gmc at esilibrary.com>

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

Summary of changes:
 t/Charset.t                |   44 +++++++++++++++++++++++++++++++++++++++++---
 t/Installer_PerlModules.t  |    3 ++-
 t/db_dependent/Installer.t |   23 +++++++++++++++++++++++
 3 files changed, 66 insertions(+), 4 deletions(-)
 create mode 100644 t/db_dependent/Installer.t

hooks/post-receive
-- 
main Koha release repository