Bug 19483

Summary: t/db_dependent/www/* crashes test harness due to misconfigured test plan
Product: Koha Reporter: Olli-Antti Kivilahti <olli-antti.kivilahti>
Component: Test SuiteAssignee: Olli-Antti Kivilahti <olli-antti.kivilahti>
Status: CLOSED FIXED QA Contact: Julian Maurice <julian.maurice>
Severity: enhancement    
Priority: P5 - low CC: fridolin.somers, julian.maurice, mtompset, nick
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 19483 - t/db_dependent/www/* crashes test harness due to misconfigured test plan
Bug 19483 - t/db_dependent/www/* crashes test harness due to misconfigured test plan
Bug 19483: Fix test plan in t/db_dependent/www/*

Description Olli-Antti Kivilahti 2017-10-18 11:32:35 UTC
prove t/db_dependent/www/

without defining KOHA_INTRANET_URL
bails out and marks the whole test suite as failed.
Test suite should not be failed if this optional WWW::Mechanize test suite is not activated.

After this patch, the tests are properly skipped without failing the whole tests.

This is important when running all tests under t, as this needlessly fails the test suite.

Also handling of 'skip_all' is inconsistent in t/db_dependent/www -tests, so this normalizes it to skip_all instead of bail_out
Comment 1 Olli-Antti Kivilahti 2017-10-18 11:36:28 UTC
Created attachment 68242 [details] [review]
Bug 19483 - t/db_dependent/www/* crashes test harness due to misconfigured test plan

prove t/db_dependent/www/

without defining KOHA_INTRANET_URL
bails out and marks the whole test suite as failed.
Test suite should not be failed if this optional WWW::Mechanize test suite is not activated.

After this patch, the tests are properly skipped without failing the whole tests.

This is important when running all tests under t, as this needlessly fails the test suite.

Also handling of 'skip_all' is inconsistent in t/db_dependent/www -tests, so this normalizes it to skip_all instead of bail_out
Comment 2 Mark Tompsett 2017-10-20 03:52:43 UTC
Created attachment 68307 [details] [review]
Bug 19483 - t/db_dependent/www/* crashes test harness due to misconfigured test plan

prove t/db_dependent/www/

without defining KOHA_INTRANET_URL
bails out and marks the whole test suite as failed.
Test suite should not be failed if this optional WWW::Mechanize test suite is not activated.

After this patch, the tests are properly skipped without failing the whole tests.

This is important when running all tests under t, as this needlessly fails the test suite.

Also handling of 'skip_all' is inconsistent in t/db_dependent/www -tests, so this normalizes it to skip_all instead of bail_out

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Comment 3 Julian Maurice 2018-01-12 13:53:35 UTC
Created attachment 70468 [details] [review]
Bug 19483: Fix test plan in t/db_dependent/www/*

t/db_dependent/www/* crashes test harness due to misconfigured test plan

prove t/db_dependent/www/

without defining KOHA_INTRANET_URL
bails out and marks the whole test suite as failed.
Test suite should not be failed if this optional WWW::Mechanize test
suite is not activated.

After this patch, the tests are properly skipped without failing the
whole tests.

This is important when running all tests under t, as this needlessly
fails the test suite.

Also handling of 'skip_all' is inconsistent in t/db_dependent/www
-tests, so this normalizes it to skip_all instead of bail_out

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Julian Maurice <julian.maurice@biblibre.com>
Comment 4 Jonathan Druart 2018-01-15 15:25:26 UTC
Pushed to master for 18.05, thanks to everybody involved!
Comment 5 Nick Clemens 2018-01-23 04:08:34 UTC
Awesome work all!

Pushed to Stable for 17.11.02
Comment 6 Fridolin Somers 2018-01-31 07:25:39 UTC
Pushed to 17.05.x for v17.05.09