added old-school uninstall method to remove the registry

src/collective/nitf/Extensions/Install.py

@@ -0,0 +1,12 @@
+# -*- coding: utf-8 -*-
+
+from Products.CMFCore.utils import getToolByName
+
+from collective.nitf.config import PROJECTNAME
+
+
+def uninstall(portal):
+    profile = 'profile-%s:uninstall' % PROJECTNAME
+    setup_tool = getToolByName(portal, 'portal_setup')
+    setup_tool.runAllImportStepsFromProfile(profile)
+    return "Ran all uninstall steps."

src/collective/nitf/configure.zcml

@@ -1,13 +1,14 @@
 <configure
     xmlns="http://namespaces.zope.org/zope"
     xmlns:browser="http://namespaces.zope.org/browser"
+    xmlns:five="http://namespaces.zope.org/five"
     xmlns:genericsetup="http://namespaces.zope.org/genericsetup"
     xmlns:grok="http://namespaces.zope.org/grok"
     xmlns:i18n="http://namespaces.zope.org/i18n"
     i18n_domain="collective.nitf">
 
+    <five:registerPackage package="." />
     <i18n:registerTranslations directory="locales" />
-
     <includeDependencies package="." />
 
     <permission
@@ -39,10 +40,8 @@
             action="gallery"
             description="Displays an image gallery stored in the news article"
             />
-
     </browser:menuItems>
 
-    <!-- Register an extension profile to make the product installable -->
     <genericsetup:registerProfile
         name="default"
         title="collective.nitf"

src/collective/nitf/tests/test_setup.py

@@ -24,7 +24,7 @@ def test_installed(self):
         qi = getattr(self.portal, 'portal_quickinstaller')
         self.assertTrue(qi.isProductInstalled(PROJECTNAME))
 
-    def test_browserlayer_installed(self):
+    def test_browserlayer(self):
         layers = [l.getName() for l in registered_layers()]
         self.assertTrue('INITFBrowserLayer' in layers,
                         'browser layer not installed')
@@ -52,12 +52,13 @@ def setUp(self):
     def test_uninstalled(self):
         qi = getattr(self.portal, 'portal_quickinstaller')
         qi.uninstallProducts(products=[PROJECTNAME])
-        self.assertTrue(not qi.isProductInstalled(PROJECTNAME))
+        self.assertFalse(qi.isProductInstalled(PROJECTNAME))
 
-    def test_browserlayer_uninstalled(self):
+    def test_browserlayer_removed(self):
+        # XXX: removal is implemented until plone.browserlayer 2.1.1
         layers = [l.getName() for l in registered_layers()]
-        self.assertTrue('INITFBrowserLayer' not in layers,
-                        'browser layer not removed')
+        self.assertFalse('INITFBrowserLayer' in layers,
+                         'browser layer not removed')
 
 
 def test_suite():