You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Instead, the test should verifies that every user in a group with the proper permission (attribute permission_required in every view) does have access, and doesn't if not. Ie: permission are grouped based and not user based in Borgia even if possible with Django.
Permissions are set in the configuration thus it doesn't seems right to test group rather than permission directly. For instance the external group may have every permissions the president group have if Borgia is configured in this way.
The text was updated successfully, but these errors were encountered:
Currently, view based test (for instance
BaseGeneralUsersViewsTestCase
) don't test the permission.It only test if a member in a certain group (3 are currently tested) have access or not :
In
tests/tests_views.py
, in the classBaseBorgiaViewsTestCase
:Instead, the test should verifies that every user in a group with the proper permission (attribute
permission_required
in every view) does have access, and doesn't if not. Ie: permission are grouped based and not user based in Borgia even if possible with Django.Permissions are set in the configuration thus it doesn't seems right to test group rather than permission directly. For instance the external group may have every permissions the president group have if Borgia is configured in this way.
The text was updated successfully, but these errors were encountered: