#667: Error rendering forbidden pages
------------------------+---------------------------------
Reporter: olemis | Owner: olemis
Type: defect | Status: assigned
Priority: blocker | Milestone: Release 8
Component: ui design | Version: 0.7.0
Resolution: | Keywords: permissions, chrome
------------------------+---------------------------------
Changes (by rjollos):
* status: review => assigned
* owner: rjollos => olemis
Comment:
I applied [attachment:t667_r1526822_error_mainnav.tests.diff​], executed
the functional tests and saw the following error:
{{{
======================================================================
FAIL: runTest (tests.functional.admin.RegressionTestBhTicket667)
No error after removing all permissions
----------------------------------------------------------------------
Traceback (most recent call last):
File
"/home/user/Workspace/bhdev/bloodhound.git/bloodhound_multiproduct/tests/functional/admin.py",
line 42, in runTest
prefix = tester.create_product()
File
"/home/user/Workspace/bhdev/bloodhound.git/bloodhound_multiproduct/tests/functional/__init__.py",
line 740, in create_product
tc.formvalue('new', 'action', 'new')
File
"/home/user/Workspace/bhdev/bloodhound.git/trac/trac/tests/functional/better_twill.py",
line 153, in better_formvalue
raise twill.errors.TwillAssertionError(*args)
TwillAssertionError: ('no matching forms!',
'/home/user/Workspace/bhdev/bloodhound.git/trac/testenv/trac/log/RegressionTestBhTicket667.html')
----------------------------------------------------------------------
}}}
I'm not sure what led to the error, but the bigger problem I see is that
revoking all permissions for anonymous and not restoring them after the
test is finished would affect other functional tests.
It reminds me though, I have been considering a change to the Trac
functional test environment whereby all permissions would be revoked from
`anonymous` and `authenticated`. I think that change would make it easier
to write tests by allowing a user to be created and their permissions to
be explicitly set - they wouldn't inherit any permissions. It would
probably require many changes to existing Trac functional tests, however.
--
Ticket URL: <https://issues.apache.org/bloodhound/ticket/667#comment:12>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker
