Summary: | 00-load.t prematurely stops all testing | ||
---|---|---|---|
Product: | Koha | Reporter: | Mark Tompsett <mtompset> |
Component: | Test Suite | Assignee: | Mark Tompsett <mtompset> |
Status: | CLOSED FIXED | QA Contact: | Galen Charlton <gmcharlt> |
Severity: | normal | ||
Priority: | P5 - low | CC: | f.demians, kyle, mirko |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16582 | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Bug 16618: 00-load.t prematurely stops all testing
[SIGNED-OFF] Bug 16618: 00-load.t prematurely stops all testing Bug 16618: 00-load.t prematurely stops all testing Bug 16618: 00-load.t prematurely stops all testing |
Description
Mark Tompsett
2016-05-30 15:46:06 UTC
Created attachment 51918 [details] [review] Bug 16618: 00-load.t prematurely stops all testing Rather than add/remove regular expressions to skip modules (like bug 9054), encapsulate the decision logic into a separate function. Currently there are three libraries which trigger halts: Koha::NorwegianDB (which was already there) Koha::ElasticSearch::Indexer Koha::SearchEngine::Elasticsearch::Search TEST PLAN --------- 1) prove t/00-load.t -- should barf horribly on Catmandu stuff if not: sudo apt-get remove libcatmandu-marc-perl then repeat step. 2) apply patch 3) prove t/00-load.t -- should not barf horribly 4) run koha qa test tools NOTE: The four optional modules for Koha::NorwegianDB are listed in the PerlDependencies.pm, while there is no mention of Catmandu libraries at all there. This may be another bug which needs fixing. TECH NOTES (for ideas of how to tinker around): These three things should trigger the three module cases: sudo apt-get remove libcatmandu-marc-perl sudo apt-get remove libcatmandu-store-elasticsearch-perl sudo apt-get remove libconvert-basen-perl You probably had koha-perldeps installed before, so the following wil mostly fix: sudo apt-get install koha-perldeps libcatmandu-marc-perl And in case you didn't have elastic search stuff installed: echo deb http://packages.elastic.co/elasticsearch/1.7/debian stable main | sudo tee /etc/apt/sources.list.d/elasticsearch.list wget -O- https://packages.elastic.co/GPG-KEY-elasticsearch | sudo apt-key add - sudo apt-get update cd ~ wget http://debian.koha-community.org/koha/otherthings/elasticsearch_deps.tar.gz tar xvf elasticsearch_deps.tar.gz cd es_deps sudo dpkg i lib* sudo apt-get install -f Created attachment 52066 [details] [review] [SIGNED-OFF] Bug 16618: 00-load.t prematurely stops all testing Rather than add/remove regular expressions to skip modules (like bug 9054), encapsulate the decision logic into a separate function. Currently there are three libraries which trigger halts: Koha::NorwegianDB (which was already there) Koha::ElasticSearch::Indexer Koha::SearchEngine::Elasticsearch::Search TEST PLAN --------- 1) prove t/00-load.t -- should barf horribly on Catmandu stuff if not: sudo apt-get remove libcatmandu-marc-perl then repeat step. 2) apply patch 3) prove t/00-load.t -- should not barf horribly 4) run koha qa test tools NOTE: The four optional modules for Koha::NorwegianDB are listed in the PerlDependencies.pm, while there is no mention of Catmandu libraries at all there. This may be another bug which needs fixing. TECH NOTES (for ideas of how to tinker around): These three things should trigger the three module cases: sudo apt-get remove libcatmandu-marc-perl sudo apt-get remove libcatmandu-store-elasticsearch-perl sudo apt-get remove libconvert-basen-perl You probably had koha-perldeps installed before, so the following wil mostly fix: sudo apt-get install koha-perldeps libcatmandu-marc-perl And in case you didn't have elastic search stuff installed: echo deb http://packages.elastic.co/elasticsearch/1.7/debian stable main | sudo tee /etc/apt/sources.list.d/elasticsearch.list wget -O- https://packages.elastic.co/GPG-KEY-elasticsearch | sudo apt-key add - sudo apt-get update cd ~ wget http://debian.koha-community.org/koha/otherthings/elasticsearch_deps.tar.gz tar xvf elasticsearch_deps.tar.gz cd es_deps sudo dpkg i lib* sudo apt-get install -f Signed-off-by: Srdjan <srdjan@catalyst.net.nz> *** Bug 16657 has been marked as a duplicate of this bug. *** Created attachment 52101 [details] [review] Bug 16618: 00-load.t prematurely stops all testing Rather than add/remove regular expressions to skip modules (like bug 9054), encapsulate the decision logic into a separate function. Currently there are three libraries which trigger halts: Koha::NorwegianDB (which was already there) Koha::ElasticSearch::Indexer Koha::SearchEngine::Elasticsearch::Search TEST PLAN --------- 1) prove t/00-load.t -- should barf horribly on Catmandu stuff if not: sudo apt-get remove libcatmandu-marc-perl then repeat step. 2) apply patch 3) prove t/00-load.t -- should not barf horribly 4) run koha qa test tools NOTE: The four optional modules for Koha::NorwegianDB are listed in the PerlDependencies.pm, while there is no mention of Catmandu libraries at all there. This may be another bug which needs fixing. TECH NOTES (for ideas of how to tinker around): These three things should trigger the three module cases: sudo apt-get remove libcatmandu-marc-perl sudo apt-get remove libcatmandu-store-elasticsearch-perl sudo apt-get remove libconvert-basen-perl You probably had koha-perldeps installed before, so the following wil mostly fix: sudo apt-get install koha-perldeps libcatmandu-marc-perl And in case you didn't have elastic search stuff installed: echo deb http://packages.elastic.co/elasticsearch/1.7/debian stable main | sudo tee /etc/apt/sources.list.d/elasticsearch.list wget -O- https://packages.elastic.co/GPG-KEY-elasticsearch | sudo apt-key add - sudo apt-get update cd ~ wget http://debian.koha-community.org/koha/otherthings/elasticsearch_deps.tar.gz tar xvf elasticsearch_deps.tar.gz cd es_deps sudo dpkg i lib* sudo apt-get install -f Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 52106 [details] [review] Bug 16618: 00-load.t prematurely stops all testing Rather than add/remove regular expressions to skip modules (like bug 9054), encapsulate the decision logic into a separate function. Currently there are three libraries which trigger halts: Koha::NorwegianDB (which was already there) Koha::ElasticSearch::Indexer Koha::SearchEngine::Elasticsearch::Search TEST PLAN --------- 1) prove t/00-load.t -- should barf horribly on Catmandu stuff if not: sudo apt-get remove libcatmandu-marc-perl then repeat step. 2) apply patch 3) prove t/00-load.t -- should not barf horribly 4) run koha qa test tools NOTE: The four optional modules for Koha::NorwegianDB are listed in the PerlDependencies.pm, while there is no mention of Catmandu libraries at all there. This may be another bug which needs fixing. TECH NOTES (for ideas of how to tinker around): These three things should trigger the three module cases: sudo apt-get remove libcatmandu-marc-perl sudo apt-get remove libcatmandu-store-elasticsearch-perl sudo apt-get remove libconvert-basen-perl You probably had koha-perldeps installed before, so the following wil mostly fix: sudo apt-get install koha-perldeps libcatmandu-marc-perl And in case you didn't have elastic search stuff installed: echo deb http://packages.elastic.co/elasticsearch/1.7/debian stable main | sudo tee /etc/apt/sources.list.d/elasticsearch.list wget -O- https://packages.elastic.co/GPG-KEY-elasticsearch | sudo apt-key add - sudo apt-get update cd ~ wget http://debian.koha-community.org/koha/otherthings/elasticsearch_deps.tar.gz tar xvf elasticsearch_deps.tar.gz cd es_deps sudo dpkg i lib* sudo apt-get install -f Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Galen Charlton <gmcharlt@gmail.com> Pushed to master for 16.11, thanks Mark! Pushed in 16.05. Will be in 16.05.01. |