From ef33c878dec12625dfeb9c8282777e99d47f1f4e Mon Sep 17 00:00:00 2001 From: Maurits van Rees Date: Thu, 17 Sep 2015 19:05:12 +0200 Subject: [PATCH] Avoid pyflakes warnings for long package names. Several lines in test_setup.py are too long when we generate a package with name test.nested.plone_addon, like we do in the tests. --- CHANGES.rst | 3 ++- .../+package.name+/tests/test_setup.py.bob | 11 +++++++---- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/CHANGES.rst b/CHANGES.rst index 47310cbb..3af86529 100644 --- a/CHANGES.rst +++ b/CHANGES.rst @@ -4,7 +4,8 @@ Changelog 1.0 (unreleased) ---------------- -- Nothing changed yet. +- Avoid pyflakes warnings for long package names. + [maurits] 1.0b1 (2015-09-17) diff --git a/bobtemplates/plone_addon/src/+package.namespace+/+package.name+/tests/test_setup.py.bob b/bobtemplates/plone_addon/src/+package.namespace+/+package.name+/tests/test_setup.py.bob index c08155aa..bfb58f2e 100644 --- a/bobtemplates/plone_addon/src/+package.namespace+/+package.name+/tests/test_setup.py.bob +++ b/bobtemplates/plone_addon/src/+package.namespace+/+package.name+/tests/test_setup.py.bob @@ -17,12 +17,14 @@ class TestSetup(unittest.TestCase): self.installer = api.portal.get_tool('portal_quickinstaller') def test_product_installed(self): - """Test if {{{ package.dottedname }}} is installed with portal_quickinstaller.""" - self.assertTrue(self.installer.isProductInstalled('{{{ package.dottedname }}}')) + """Test if {{{ package.dottedname }}} is installed.""" + self.assertTrue(self.installer.isProductInstalled( + '{{{ package.dottedname }}}')) def test_browserlayer(self): """Test that I{{{ package.browserlayer }}} is registered.""" - from {{{ package.dottedname }}}.interfaces import I{{{ package.browserlayer }}} + from {{{ package.dottedname }}}.interfaces import ( + I{{{ package.browserlayer }}}) from plone.browserlayer import utils self.assertIn(I{{{ package.browserlayer }}}, utils.registered_layers()) @@ -38,7 +40,8 @@ class TestUninstall(unittest.TestCase): def test_product_uninstalled(self): """Test if {{{ package.dottedname }}} is cleanly uninstalled.""" - self.assertFalse(self.installer.isProductInstalled('{{{ package.dottedname }}}')) + self.assertFalse(self.installer.isProductInstalled( + '{{{ package.dottedname }}}')) {{% if plone.is_plone5 %}} def test_browserlayer_removed(self):