Skip to content

Commit 4da4ad4

Browse files
committed
Reset state of:
self.db.security.set_props_only_default(False) at end of testGetPermission. I thought each test_X module had a fresh environment an load of all modules. I guess that is not the case as not resetting the props_only default to false seemed to bleed into the testAuthFilter in text_xmlrpc.py. However the funny part is it only caused problem in travis ci. Not in my manual running of the full test suite on two platforms. However I am pulling errors because the framework is not skipping the postgres tests for text_xmlrpc. Maybe that failure is hiding something?? If I run just the test_xmlrpc module I would not expect an issue since the security test suite won't be invoked. I am using different versions of the test harness and python so maybe.....
1 parent 3498801 commit 4da4ad4

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

test/test_security.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,11 @@ def testGetPermission(self):
132132
self.assertEquals(get('View', 'issue', properties=['title'],
133133
check=dummy), apci)
134134

135+
# Reset to default. Somehow this setting looks like it
136+
# was bleeding through to other tests in test_xmlrpc.
137+
# Is the security module being loaded only once for all tests??
138+
self.db.security.set_props_only_default(False)
139+
135140
def testDBinit(self):
136141
self.db.user.create(username="demo", roles='User')
137142
self.db.user.create(username="anonymous", roles='Anonymous')

0 commit comments

Comments
 (0)