fix test setup & improvements

collective · Mar 2, 2020 · 99f0f63 · 99f0f63
1 parent d5fb5ca
commit 99f0f63
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/src/collective/contact/core/testing.py b/src/collective/contact/core/testing.py
@@ -31,6 +31,7 @@ def setUpPloneSite(self, portal):
         # # insert some test data
         setRoles(portal, TEST_USER_ID, ['Manager'])
         create_test_contact_data(portal)
+        create_test_held_positions(portal)
 
 
 COLLECTIVE_CONTACT_CORE_FIXTURE = CollectiveContactCoreLayer()

diff --git a/src/collective/contact/core/tests/test_subscribers.py b/src/collective/contact/core/tests/test_subscribers.py
@@ -1,6 +1,6 @@
 # -*- coding: utf8 -*-
 
-from collective.contact.core.testing import INTEGRATION
+from collective.contact.core.testing import COLLECTIVE_CONTACT_CORE_ACCEPTANCE_TESTING
 from ecreall.helpers.testing.search import BaseSearchTest
 from plone import api
 from plone.app.testing import setRoles
@@ -12,7 +12,7 @@
 
 class TestUtils(unittest.TestCase, BaseSearchTest):
 
-    layer = INTEGRATION
+    layer = COLLECTIVE_CONTACT_CORE_ACCEPTANCE_TESTING
 
     def setUp(self):
         super(TestUtils, self).setUp()

diff --git a/src/collective/contact/core/utils.py b/src/collective/contact/core/utils.py
@@ -4,7 +4,7 @@
 from collective.contact.core.interfaces import IHeldPosition
 
 
-def get_gender_and_number(contacts, use_by=False, use_to=False):
+def get_gender_and_number(contacts, use_by=False, use_to=False):  # noqa for now 'is too complex'
     """Return gender and number of given contacts.
        Returns None if not genderable.
        Returns a 2 letters code if genderable: