From: Michael D. <svn...@pl...> - 2008-08-05 19:48:44
|
Author: davismr Date: Tue Aug 5 19:48:38 2008 New Revision: 69521 Modified: PloneSurvey/trunk/content/Survey.py PloneSurvey/trunk/tests/testSetup.py Log: Move acl_users creation to seperate method, add test Modified: PloneSurvey/trunk/content/Survey.py ============================================================================== --- PloneSurvey/trunk/content/Survey.py (original) +++ PloneSurvey/trunk/content/Survey.py Tue Aug 5 19:48:38 2008 @@ -53,6 +53,10 @@ self.respondents = OOBTree() def at_post_create_script(self): + self.createLocalPas() + + security.declarePrivate('createLocalPas') + def createLocalPas(self): """Create PAS acl_users else login_form does not work""" # need Manager role to add an acl_users object remove_role = False @@ -546,7 +550,7 @@ def get_acl_users(self): """Fetch acl_users. Create if it does not yet exist.""" if not 'acl_users' in self.objectIds(): - self.at_post_create_script() + self.createLocalPas() return self.acl_users security.declareProtected(permissions.View, 'buildSpreadsheetUrl') Modified: PloneSurvey/trunk/tests/testSetup.py ============================================================================== --- PloneSurvey/trunk/tests/testSetup.py (original) +++ PloneSurvey/trunk/tests/testSetup.py Tue Aug 5 19:48:38 2008 @@ -175,10 +175,23 @@ sharing_tab_available = True assert sharing_tab_available, "Sharing tab not available" +class TestAclUsers(PloneSurveyTestCase): + """Test acl_users is created""" + + def afterSetUp(self): + self.folder.invokeFactory('Survey', 's1') + self.s1 = getattr(self.folder, 's1') + + def testAclUsersCreated(self): + s1 = getattr(self, 's1') + s1.at_post_create_script() + assert 'acl_users' in s1.objectIds(), "acl_users not created" + def test_suite(): from unittest import TestSuite, makeSuite suite = TestSuite() suite.addTest(makeSuite(TestInstallation)) suite.addTest(makeSuite(TestContentCreation)) suite.addTest(makeSuite(TestTypeActions)) + suite.addTest(makeSuite(TestAclUsers)) return suite |