Commit 4d23aeea authored by Jérome Perrin's avatar Jérome Perrin

CodingStyleTest: install full text business template in a more clever way

We don't want to install more than one full text business template.
Also, we prefer erp5_full_text_mroonga_catalog, since we recommand
mroonga over myisam.

This also reverts commit ab44309f (
full_text_myisam_catalog: skip coding style test) that is no longer
needed with this new approach.
parent ab44309f
...@@ -44,13 +44,22 @@ class CodingStyleTest(CodingStyleTestCase, testXHTML.TestXHTMLMixin): ...@@ -44,13 +44,22 @@ class CodingStyleTest(CodingStyleTestCase, testXHTML.TestXHTMLMixin):
def getBusinessTemplateList(self): def getBusinessTemplateList(self):
# install erp5_upgrader for CodingStyleTestCase.test_run_upgrader # install erp5_upgrader for CodingStyleTestCase.test_run_upgrader
# XXX also install erp5_full_text_myisam_catalog to workaround missing test
# dependencies and the fact that test dependencies are not checked # XXX also install a full test business template, to workaround the fact
# recursively. # that test dependencies are not checked recursively, but unless the tested
# business template itself is a full text business template, as we are not
# supposed to have more than one full text business template installed.
if self.tested_business_template in (
'erp5_full_text_mroonga_catalog',
'erp5_full_text_myisam_catalog',
'erp5_full_text_sphinxse_catalog',
):
return ('erp5_upgrader', self.tested_business_template)
return ( return (
'erp5_upgrader', 'erp5_upgrader',
'erp5_full_text_myisam_catalog', 'erp5_full_text_mroonga_catalog',
self.tested_business_template) self.tested_business_template,
)
def _installBusinessTemplateList(self, def _installBusinessTemplateList(self,
bt_list, bt_list,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment