Bug 18222

Summary: Fix tests broken by Bug 18026
Product: Koha Reporter: Olli-Antti Kivilahti <olli-antti.kivilahti>
Component: Test SuiteAssignee: Olli-Antti Kivilahti <olli-antti.kivilahti>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: jonathan.druart, katrin.fischer, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 18026    
Bug Blocks:    
Attachments: Bug 18222 - Fix tests broken by Buug 18026
Bug 18222 - Fix tests broken by Buug 18026
Bug 18222 - Fix tests broken by Buug 18026

Description Olli-Antti Kivilahti 2017-03-07 11:25:21 UTC
root@koha_ci_1:/home/koha/Koha# KOHA_CONF=$KOHA_CONF perl t/Koha_Template_Plugin_Koha.t
1..3
ok 1 - use Koha::Template::Plugin::Koha;
ok 2
# Subtest: Koha::Template::Plugin::Koha::Version tests
    1..2
    not ok 1 - Correct development version
    #   Failed test 'Correct development version'
    #   at t/Koha_Template_Plugin_Koha.t line 54.
    #     Structures begin differing at:
    #          $got->{minor} = '24'
    #     $expected->{minor} = Does not exist
    not ok 2 - Correct maintenance version
    #   Failed test 'Correct maintenance version'
    #   at t/Koha_Template_Plugin_Koha.t line 69.
    #     Structures begin differing at:
    #          $got->{minor} = '07'
    #     $expected->{minor} = Does not exist
    # Looks like you failed 2 tests of 2.
not ok 3 - Koha::Template::Plugin::Koha::Version tests
#   Failed test 'Koha::Template::Plugin::Koha::Version tests'
#   at t/Koha_Template_Plugin_Koha.t line 76.
# Looks like you failed 1 test of 3.
Comment 1 Olli-Antti Kivilahti 2017-03-07 11:28:32 UTC
Created attachment 60865 [details] [review]
Bug 18222 - Fix tests broken by Buug 18026

root@koha_ci_1:/home/koha/Koha# KOHA_CONF=$KOHA_CONF perl t/Koha_Template_Plugin_Koha.t
1..3
ok 1 - use Koha::Template::Plugin::Koha;
ok 2
    1..2
    ok 1 - Correct development version
    ok 2 - Correct maintenance version
ok 3 - Koha::Template::Plugin::Koha::Version tests
Comment 2 Marcel de Rooy 2017-03-07 12:40:22 UTC
Created attachment 60868 [details] [review]
Bug 18222 - Fix tests broken by Buug 18026

root@koha_ci_1:/home/koha/Koha# KOHA_CONF=$KOHA_CONF perl t/Koha_Template_Plugin_Koha.t
1..3
ok 1 - use Koha::Template::Plugin::Koha;
ok 2
    1..2
    ok 1 - Correct development version
    ok 2 - Correct maintenance version
ok 3 - Koha::Template::Plugin::Koha::Version tests

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 3 Marcel de Rooy 2017-03-07 12:41:40 UTC
Combining SO and QA here
Passed QA
Comment 4 Jonathan Druart 2017-03-07 13:27:17 UTC
Created attachment 60869 [details] [review]
Bug 18222 - Fix tests broken by Buug 18026

root@koha_ci_1:/home/koha/Koha# KOHA_CONF=$KOHA_CONF perl t/Koha_Template_Plugin_Koha.t
1..3
ok 1 - use Koha::Template::Plugin::Koha;
ok 2
    1..2
    ok 1 - Correct development version
    ok 2 - Correct maintenance version
ok 3 - Koha::Template::Plugin::Koha::Version tests

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Jonathan Druart 2017-03-07 13:27:45 UTC
(In reply to Marcel de Rooy from comment #3)
> Combining SO and QA here
> Passed QA

Did not see it, I was going to do the same :)
Comment 6 Katrin Fischer 2017-03-26 19:46:26 UTC
This appears to have been pushed to master already?
Comment 7 Katrin Fischer 2017-03-26 19:50:18 UTC
perl t/Koha_Template_Plugin_Koha.t passes on 16.11.x - so I think this is not needed there. Also pushed to master - switching status.
Comment 8 Jonathan Druart 2017-03-27 14:38:09 UTC
No, it's not pushed yet.
Comment 9 Jonathan Druart 2017-03-27 14:39:05 UTC
Yes it is!!