[change] Removed deprecated API url #101

Closes #101

Author: prajwal296

openwisp_network_topology/api/views.py

@@ -141,16 +141,6 @@ def post(self, request, pk, format=None):
             ) % (topology.get_parser_display(), e.__class__.__name__, e)
             return Response({'detail': error}, status=400)
         success_message = _('data received successfully')
-        deprecated_url = reverse('receive_topology_deprecated', args=[pk])
-        if request.path == deprecated_url:
-            expected_path = reverse('receive_topology', args=[pk])
-            expected_path = f'{expected_path}?key={key}'
-            warning = _(
-                'This URL is depercated and will be removed in '
-                f'future versions, use {expected_path}'
-            )
-            logger.warning(warning)
-            success_message = f'{success_message}. {warning}'
         return Response({'detail': success_message})
 
 

openwisp_network_topology/tests/test_api.py

@@ -105,26 +105,6 @@ def test_receive(self):
         self.assertEqual(self.node_model.objects.count(), 2)
         self.assertEqual(self.link_model.objects.count(), 1)
 
-    def test_receive_with_deprecated_url(self):
-        self._set_receive()
-        self.node_model.objects.all().delete()
-        data = self._load('static/netjson-1-link.json')
-        t = self.topology_model.objects.first()
-        path = reverse('receive_topology_deprecated', args=[t.pk])
-        path = f'{path}?key=test'
-        response = self.client.post(path, data=data, content_type='text/plain')
-        self.assertEqual(response.status_code, 200)
-        expected_path = reverse('receive_topology', args=[t.pk])
-        expected_path = f'{expected_path}?key=test'
-        message = (
-            'data received successfully. '
-            'This URL is depercated and will be removed in '
-            f'future versions, use {expected_path}'
-        )
-        self.assertEqual(response.data['detail'], message)
-        self.assertEqual(self.node_model.objects.count(), 2)
-        self.assertEqual(self.link_model.objects.count(), 1)
-
     def test_receive_404(self):
         # topology is set to FETCH strategy
         response = self.client.post(self.receive_url, content_type='text/plain')

openwisp_network_topology/utils.py

@@ -44,11 +44,6 @@ def get_api_urls(views_module):
             views_module.network_graph_history,
             name='network_graph_history',
         ),
-        url(
-            r'^receive/(?P<pk>[^/\?]+)/$',
-            views_module.receive_topology,
-            name='receive_topology_deprecated',
-        ),
         url(
             r'^topology/(?P<pk>[^/\?]+)/receive/$',
             views_module.receive_topology,