Mercurial > p > roundup > code
diff test/test_mailgw.py @ 8119:c12377fb4144 permission-performance
Change permission representation
Now permissions are checked in different order. Permissions without a
check method (which are cheap to check) are checked first. Only if no
permission is found do we check permissions with check methods.
| author | Ralf Schlatterbeck <rsc@runtux.com> |
|---|---|
| date | Fri, 18 Oct 2024 16:52:42 +0200 |
| parents | 0713c286f71c |
| children | 28c5030757d3 |
line wrap: on
line diff
--- a/test/test_mailgw.py Thu Oct 17 19:13:26 2024 -0400 +++ b/test/test_mailgw.py Fri Oct 18 16:52:42 2024 +0200 @@ -241,7 +241,7 @@ self.db.security.getPermission('Create', 'issue'), self.db.security.getPermission('Create', 'msg'), ] - self.db.security.role['anonymous'].permissions = p + self.db.security.role['anonymous'].addPermission(*p) def _create_mailgw(self, message, args=()): class MailGW(self.instance.MailGW): @@ -2867,7 +2867,7 @@ This is a test submission of a new issue. ''' - self.db.security.role['anonymous'].permissions=[] + self.db.security.role['anonymous']._permissions={} anonid = self.db.user.lookup('anonymous') self.db.user.set(anonid, roles='Anonymous') try: @@ -2888,7 +2888,7 @@ self.db.security.getPermission('Register', 'user'), self.db.security.getPermission('Web Access', None), ] - self.db.security.role['anonymous'].permissions=p + self.db.security.role['anonymous'].addPermission(*p) try: self._handle_mail(message) except Unauthorized as value: @@ -2915,7 +2915,7 @@ self.db.security.getPermission('Register', 'user'), self.db.security.getPermission('Email Access', None), ] - self.db.security.role['anonymous'].permissions=p + self.db.security.role['anonymous'].addPermission(*p) self._handle_mail(message) m = self.db.user.list() m.sort()