Do not run the C extension tests when we do not have the C extensions.

zopefoundation · Oct 19, 2017 · 79938a5 · 79938a5
1 parent c2a27b4
commit 79938a5
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 11 deletions.
diff --git a/src/AccessControl/tests/testSecurityManager.py b/src/AccessControl/tests/testSecurityManager.py
@@ -255,13 +255,14 @@ def _getTargetClass(self):
         from AccessControl.ImplPython import SecurityManager
         return SecurityManager
 
+from AccessControl.Implementation import C_EXTENSION
+if C_EXTENSION:
+    class C_SecurityManagerTests(SecurityManagerTestBase,
+                                 ISecurityManagerConformance,
+                                 unittest.TestCase):
+        # The C version mixes in the Python version, which is why we
+        # can test for conformance to ISecurityManager.
 
-class C_SecurityManagerTests(SecurityManagerTestBase,
-                             ISecurityManagerConformance,
-                             unittest.TestCase):
-    # The C version mixes in the Python version, which is why we
-    # can test for conformance to ISecurityManager.
-
-    def _getTargetClass(self):
-        from AccessControl.ImplC import SecurityManager
-        return SecurityManager
+        def _getTargetClass(self):
+            from AccessControl.ImplC import SecurityManager
+            return SecurityManager
diff --git a/src/AccessControl/tests/testZopeSecurityPolicy.py b/src/AccessControl/tests/testZopeSecurityPolicy.py
@@ -749,11 +749,15 @@ def loop():
 
 
 def test_suite():
+    from AccessControl.Implementation import C_EXTENSION
+
     suite = unittest.TestSuite()
     suite.addTest(unittest.makeSuite(Python_ZSPTests))
-    suite.addTest(unittest.makeSuite(C_ZSPTests))
+    if C_EXTENSION:
+        suite.addTest(unittest.makeSuite(C_ZSPTests))
     suite.addTest(unittest.makeSuite(Python_SMTests))
-    suite.addTest(unittest.makeSuite(C_SMTests))
+    if C_EXTENSION:
+        suite.addTest(unittest.makeSuite(C_SMTests))
     suite.addTest(DocTestSuite())
     suite.addTest(unittest.makeSuite(GetRolesWithMultiThreadTest))
     return suite