[Erp5-report] r32259 yusuke - /erp5/trunk/products/ERP5/tests/testAdvancedInvoicing.py

[nobody at svn.erp5.org]

Author: yusuke
Date: Thu Feb  4 11:38:16 2010
New Revision: 32259

URL: http://svn.erp5.org?rev=32259&view=rev
Log:
clean up a bit

Modified:
    erp5/trunk/products/ERP5/tests/testAdvancedInvoicing.py

Modified: erp5/trunk/products/ERP5/tests/testAdvancedInvoicing.py
URL: http://svn.erp5.org/erp5/trunk/products/ERP5/tests/testAdvancedInvoicing.py?rev=32259&r1=32258&r2=32259&view=diff
==============================================================================
--- erp5/trunk/products/ERP5/tests/testAdvancedInvoicing.py [utf8] (original)
+++ erp5/trunk/products/ERP5/tests/testAdvancedInvoicing.py [utf8] Thu Feb  4 11:38:16 2010
@@ -1148,53 +1148,60 @@
     self.assertEquals(sale_invoice.getSimulationState(), 'auto_planned')
 
     # other as anonymous
-    self.failIfUserCanAccessDocument(self.other.getId(), sale_invoice)
-    self.failIfUserCanAddDocument(self.other.getId(), sale_invoice)
-    self.failIfUserCanDeleteDocument(self.other.getId(), sale_invoice)
-    self.failIfUserCanModifyDocument(self.other.getId(), sale_invoice)
-    self.failIfUserCanViewDocument(self.other.getId(), sale_invoice)
+    username = self.other.getReference()
+    self.failIfUserCanAccessDocument(username, sale_invoice)
+    self.failIfUserCanAddDocument(username, sale_invoice)
+    self.failIfUserCanDeleteDocument(username, sale_invoice)
+    self.failIfUserCanModifyDocument(username, sale_invoice)
+    self.failIfUserCanViewDocument(username, sale_invoice)
 
     # assignee
-    self.assertUserCanAccessDocument(self.assignee.getId(), sale_invoice)
-    self.assertUserCanAddDocument(self.assignee.getId(), sale_invoice)
-    self.assertUserCanDeleteDocument(self.assignee.getId(), sale_invoice)
-    self.assertUserCanModifyDocument(self.assignee.getId(), sale_invoice)
-    self.assertUserCanViewDocument(self.assignee.getId(), sale_invoice)
+    username = self.assignee.getReference()
+    self.assertUserCanAccessDocument(username, sale_invoice)
+    self.assertUserCanAddDocument(username, sale_invoice)
+    self.assertUserCanDeleteDocument(username, sale_invoice)
+    self.assertUserCanModifyDocument(username, sale_invoice)
+    self.assertUserCanViewDocument(username, sale_invoice)
 
     # assignor
-    self.assertUserCanAccessDocument(self.assignor.getId(), sale_invoice)
-    self.assertUserCanAddDocument(self.assignor.getId(), sale_invoice)
-    self.assertUserCanDeleteDocument(self.assignor.getId(), sale_invoice)
-    self.assertUserCanModifyDocument(self.assignor.getId(), sale_invoice)
-    self.assertUserCanViewDocument(self.assignor.getId(), sale_invoice)
+    username = self.assignor.getReference()
+    self.assertUserCanAccessDocument(username, sale_invoice)
+    self.assertUserCanAddDocument(username, sale_invoice)
+    self.assertUserCanDeleteDocument(username, sale_invoice)
+    self.assertUserCanModifyDocument(username, sale_invoice)
+    self.assertUserCanViewDocument(username, sale_invoice)
 
     # associate
-    self.assertUserCanAccessDocument(self.associate.getId(), sale_invoice)
-    self.assertUserCanAddDocument(self.associate.getId(), sale_invoice)
-    self.assertUserCanDeleteDocument(self.associate.getId(), sale_invoice)
-    self.assertUserCanModifyDocument(self.associate.getId(), sale_invoice)
-    self.assertUserCanViewDocument(self.associate.getId(), sale_invoice)
+    username = self.associate.getReference()
+    self.assertUserCanAccessDocument(username, sale_invoice)
+    self.assertUserCanAddDocument(username, sale_invoice)
+    self.assertUserCanDeleteDocument(username, sale_invoice)
+    self.assertUserCanModifyDocument(username, sale_invoice)
+    self.assertUserCanViewDocument(username, sale_invoice)
 
     # auditor
-    self.assertUserCanAccessDocument(self.auditor.getId(), sale_invoice)
-    self.failIfUserCanAddDocument(self.auditor.getId(), sale_invoice)
-    self.failIfUserCanDeleteDocument(self.auditor.getId(), sale_invoice)
-    self.failIfUserCanModifyDocument(self.auditor.getId(), sale_invoice)
-    self.assertUserCanViewDocument(self.auditor.getId(), sale_invoice)
+    username = self.auditor.getReference()
+    self.assertUserCanAccessDocument(username, sale_invoice)
+    self.failIfUserCanAddDocument(username, sale_invoice)
+    self.failIfUserCanDeleteDocument(username, sale_invoice)
+    self.failIfUserCanModifyDocument(username, sale_invoice)
+    self.assertUserCanViewDocument(username, sale_invoice)
 
     # author
-    self.failIfUserCanAccessDocument(self.author.getId(), sale_invoice)
-    self.failIfUserCanAddDocument(self.author.getId(), sale_invoice)
-    self.failIfUserCanDeleteDocument(self.author.getId(), sale_invoice)
-    self.failIfUserCanModifyDocument(self.author.getId(), sale_invoice)
-    self.failIfUserCanViewDocument(self.author.getId(), sale_invoice)
+    username = self.author.getReference()
+    self.failIfUserCanAccessDocument(username, sale_invoice)
+    self.failIfUserCanAddDocument(username, sale_invoice)
+    self.failIfUserCanDeleteDocument(username, sale_invoice)
+    self.failIfUserCanModifyDocument(username, sale_invoice)
+    self.failIfUserCanViewDocument(username, sale_invoice)
 
     # manager
-    self.assertUserCanAccessDocument(self.manager.getId(), sale_invoice)
-    self.assertUserCanAddDocument(self.manager.getId(), sale_invoice)
-    self.assertUserCanDeleteDocument(self.manager.getId(), sale_invoice)
-    self.assertUserCanModifyDocument(self.manager.getId(), sale_invoice)
-    self.assertUserCanViewDocument(self.manager.getId(), sale_invoice)
+    username = self.manager.getReference()
+    self.assertUserCanAccessDocument(username, sale_invoice)
+    self.assertUserCanAddDocument(username, sale_invoice)
+    self.assertUserCanDeleteDocument(username, sale_invoice)
+    self.assertUserCanModifyDocument(username, sale_invoice)
+    self.assertUserCanViewDocument(username, sale_invoice)
 
     self.assertSameSet(('confirm_action', 'plan_action',),
         [action['id'] for action in self.portal.portal_workflow.getActionsFor(sale_invoice)