Fix permissions

src/ralph/lib/mixins/tests/test_models.py

@@ -8,5 +8,5 @@ class AdminUrlTestCase(TestCase):
     def test_returned_url(self):
         obj = Foo.objects.create(bar='test')
         self.assertEqual(
-            '/tests/foo/{}/'.format(obj.pk), obj.get_absolute_url()
+            '/tests/foo/{}/change/'.format(obj.pk), obj.get_absolute_url()
         )

src/ralph/lib/permissions/__init__.py

@@ -0,0 +1 @@
+default_app_config = 'ralph.lib.permissions.apps.PermissionAppConfig'

src/ralph/lib/permissions/apps.py

@@ -2,16 +2,14 @@
 from django.apps import AppConfig
 from django.db.models.signals import post_migrate
 
-from ralph.lib.permissions.models import create_permissions
-from ralph.lib.permissions.views import update_extra_view_permissions
-
 
 class PermissionAppConfig(AppConfig):
     name = 'ralph.lib.permissions'
     verbose_name = 'Permissions'
 
     def ready(self):
-        default_app_config = 'ralph.lib.permissions.apps.PermissionAppConfig'
+        from ralph.lib.permissions.models import create_permissions
+        from ralph.lib.permissions.views import update_extra_view_permissions
         post_migrate.disconnect(
             dispatch_uid='django.contrib.auth.management.create_permissions'
         )

src/ralph/lib/permissions/views.py

@@ -70,8 +70,9 @@ def update_extra_view_permissions(sender, **kwargs):
     logger.info('Updating extra views permissions...')
     admin_classes = {}
     for model, admin_class in ralph_site._registry.items():
-        for change_view in admin_class.change_views:
-            admin_classes[change_view] = model
+        if admin_class.change_views:
+            for change_view in admin_class.change_views:
+                admin_classes[change_view] = model
 
     old_permission = Permission.objects.filter(
         codename__startswith='can_view_extra_'

src/ralph/virtual/management/commands/openstack_sync.py

@@ -5,7 +5,7 @@
 from enum import auto, Enum
 from functools import lru_cache
 
-import reversion as revisions
+from reversion import revisions
 from django.conf import settings
 from django.core.exceptions import MultipleObjectsReturned, ObjectDoesNotExist
 from django.core.management.base import BaseCommand