[Zope-Checkins] CVS: Zope3/lib/python/Zope/App/Security/tests - testPrincipalRoleManager.py:1.1.2.10.2.1

[Jim Fulton]

Update of /cvs-repository/Zope3/lib/python/Zope/App/Security/tests
In directory cvs.zope.org:/tmp/cvs-serv29689

Modified Files:
      Tag: Zope-3x-security_defactor-branch
	testPrincipalRoleManager.py 
Log Message:
Cleaned up _make_principal a bit.


=== Zope3/lib/python/Zope/App/Security/tests/testPrincipalRoleManager.py 1.1.2.10 => 1.1.2.10.2.1 ===
         
     def _make_principal(self, id=None, title=None):
-        pregistry.definePrincipal(
+        p = pregistry.definePrincipal(
             id or 'APrincipal',
             title or 'A Principal',
             login = id or 'APrincipal')
-        return id or 'APrincipal'
+        return p.getId()
 
     def testUnboundPrincipalRole(self):
         role = rregistry.defineRole('ARole', 'A Role').getId()