Skip to content

Commit

Permalink
- Fix security test cases
Browse files Browse the repository at this point in the history
  • Loading branch information
afabiani committed Nov 6, 2018
1 parent 7374f0c commit 16b4d0b
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions geonode/security/tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -127,11 +127,11 @@ def test_set_bulk_permissions(self):
# Check GeoFence Rules have been correctly created
geofence_rules_count = get_geofence_rules_count()
_log("1. geofence_rules_count: %s " % geofence_rules_count)
self.assertEquals(geofence_rules_count, 6)
self.assertEquals(geofence_rules_count, 4)
set_geofence_all(test_perm_layer)
geofence_rules_count = get_geofence_rules_count()
_log("2. geofence_rules_count: %s " % geofence_rules_count)
self.assertEquals(geofence_rules_count, 7)
self.assertEquals(geofence_rules_count, 5)

self.client.logout()

Expand All @@ -147,7 +147,7 @@ def test_set_bulk_permissions(self):
# Check GeoFence Rules have been correctly created
geofence_rules_count = get_geofence_rules_count()
_log("4. geofence_rules_count: %s " % geofence_rules_count)
self.assertEquals(geofence_rules_count, 8)
self.assertEquals(geofence_rules_count, 5)

# Validate maximum priority
geofence_rules_highest_priority = get_highest_priority()
Expand Down Expand Up @@ -213,14 +213,14 @@ def test_perm_specs_synchronization(self):
layer.set_permissions(perm_spec)
geofence_rules_count = get_geofence_rules_count()
_log("1. geofence_rules_count: %s " % geofence_rules_count)
self.assertEquals(geofence_rules_count, 1)
self.assertEquals(geofence_rules_count, 0)

perm_spec = {
"users": {"admin": ["view_resourcebase"]}, "groups": {}}
layer.set_permissions(perm_spec)
geofence_rules_count = get_geofence_rules_count()
_log("2. geofence_rules_count: %s " % geofence_rules_count)
self.assertEquals(geofence_rules_count, 3)
self.assertEquals(geofence_rules_count, 2)

perm_spec = {'users': {"admin": ['change_layer_data']}}
layer.set_permissions(perm_spec)
Expand All @@ -232,13 +232,13 @@ def test_perm_specs_synchronization(self):
layer.set_permissions(perm_spec)
geofence_rules_count = get_geofence_rules_count()
_log("4. geofence_rules_count: %s " % geofence_rules_count)
self.assertEquals(geofence_rules_count, 3)
self.assertEquals(geofence_rules_count, 2)

perm_spec = {'groups': {'bar': ['change_resourcebase']}}
layer.set_permissions(perm_spec)
geofence_rules_count = get_geofence_rules_count()
_log("5. geofence_rules_count: %s " % geofence_rules_count)
self.assertEquals(geofence_rules_count, 1)
self.assertEquals(geofence_rules_count, 0)

# Reset GeoFence Rules
purge_geofence_all()
Expand Down Expand Up @@ -944,7 +944,7 @@ def test_not_superuser_permissions(self):
# Check GeoFence Rules have been correctly created
geofence_rules_count = get_geofence_rules_count()
_log("3. geofence_rules_count: %s " % geofence_rules_count)
self.assertEquals(geofence_rules_count, 3)
self.assertEquals(geofence_rules_count, 1)

# 5. change_resourcebase_permissions
# should be impossible for the user without change_resourcebase_permissions
Expand Down

0 comments on commit 16b4d0b

Please sign in to comment.