[feature] Admin added UUID readonly field

openwisp_network_topology/admin.py

@@ -13,7 +13,7 @@
     MultitenantOrgFilter,
     MultitenantRelatedOrgFilter,
 )
-from openwisp_utils.admin import ReceiveUrlAdmin
+from openwisp_utils.admin import ReceiveUrlAdmin, UUIDAdmin
 
 from . import settings as app_settings
 from .contextmanagers import log_failure
@@ -61,7 +61,7 @@ class Media:
 
 @admin.register(Topology)
 class TopologyAdmin(
-    MultitenantAdminMixin, BaseAdmin, ReceiveUrlAdmin, GraphVisualizerUrls
+    MultitenantAdminMixin, GraphVisualizerUrls, UUIDAdmin, ReceiveUrlAdmin, BaseAdmin
 ):
     model = Topology
     list_display = [
@@ -73,7 +73,14 @@ class TopologyAdmin(
         'modified',
         'organization',
     ]
-    readonly_fields = ['protocol', 'version', 'revision', 'metric', 'receive_url']
+    readonly_fields = [
+        'uuid',
+        'protocol',
+        'version',
+        'revision',
+        'metric',
+        'receive_url',
+    ]
     list_filter = [
         'parser',
         'strategy',
@@ -82,6 +89,7 @@ class TopologyAdmin(
     search_fields = ['label', 'id']
     actions = ['update_selected', 'unpublish_selected', 'publish_selected']
     fields = [
+        'uuid',
         'label',
         'organization',
         'parser',

openwisp_network_topology/tests/test_admin.py

@@ -189,11 +189,14 @@ def test_node_add(self):
         response = self.client.get(path)
         self.assertNotContains(response, 'Links to other nodes')
 
-    def test_topology_visualize_button(self):
+    def test_topology_change_form(self):
         topology = self.topology_model.objects.first()
         path = reverse('{0}_topology_change'.format(self.prefix), args=[topology.pk])
         response = self.client.get(path)
         self.assertContains(response, 'View topology graph')
+        self.assertContains(
+            response, '<div class="readonly">{0}</div>'.format(topology.pk)
+        )
 
     def test_topology_visualize_view(self):
         t = self.topology_model.objects.first()