From 372b573cb12f6c2ee7036facf38cc074f266f6a2 Mon Sep 17 00:00:00 2001 From: William Deegan Date: Sat, 15 Dec 2012 16:45:56 -0800 Subject: [PATCH] move creation of test user to test setup, rather than just in the new test_migrate_comment_with_creator() method --- plone/app/discussion/tests/test_migration.py | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/plone/app/discussion/tests/test_migration.py b/plone/app/discussion/tests/test_migration.py index d4ac2fd..98a12d5 100644 --- a/plone/app/discussion/tests/test_migration.py +++ b/plone/app/discussion/tests/test_migration.py @@ -50,6 +50,13 @@ class MigrationTest(unittest.TestCase): self.workflowTool.setChainForPortalTypes(('Discussion Item',), 'comment_review_workflow') + # Create a user Jimmy Jones so comments creator migration can work? + acl_users = getToolByName(self.portal, 'acl_users') + acl_users.userFolderAddUser('Jim', 'secret', ['Member'], []) + mt = getToolByName(self.portal, 'portal_membership') + member = mt.getMemberById('Jim') + member.fullname = 'Jimmy Jones' + self.doc = self.portal.doc def test_migrate_comment(self): @@ -98,12 +105,6 @@ class MigrationTest(unittest.TestCase): def test_migrate_comment_with_creator(self): - # Create a user Jimmy Jones so comments creator migration can work? - acl_users = getToolByName(self.portal, 'acl_users') - acl_users.userFolderAddUser('Jim', 'secret', ['Member'], []) - mt = getToolByName(self.portal, 'portal_membership') - member = mt.getMemberById('Jim') - member.fullname = 'Jimmy Jones' # Create a comment