[fix] Skip WHOIS and estimated location for deactivated devices #1325

- add guard in WHOISService to skip processing for deactivated devices
- add guard in EstimatedLocationService to prevent task scheduling
- fix N+1 query by including _is_deactivated in clear_last_ip_command
  queryset
- add regression tests ensuring tasks are skipped when device is
  deactivated
- ensure deterministic tests by enabling whois on organization
  settings

Fixes #1325

Author: tomsebastian10

openwisp_controller/config/whois/commands.py

@@ -39,7 +39,7 @@ def clear_last_ip_command(stdout, stderr, interactive=True):
         )
         # include the FK field 'organization' in .only() so the related
         # `organization__config_settings` traversal is not deferred
-        .only("last_ip", "organization", "key")
+        .only("last_ip", "organization", "key", "_is_deactivated")
     )
     # Filter out devices that have WHOIS information for their last IP
     devices = devices.exclude(last_ip=None).exclude(

openwisp_controller/config/whois/tests/tests.py

@@ -349,7 +349,7 @@ def test_last_ip_management_command_queries(self):
         )
         args = ["--noinput"]
 
-        with self.assertNumQueries(7):
+        with self.assertNumQueries(4):
             call_command("clear_last_ip", *args, stdout=out, stderr=StringIO())
 
     @mock.patch.object(app_settings, "WHOIS_CONFIGURED", True)
@@ -1219,4 +1219,7 @@ def test_process_ip_runs_when_active(self, mock_task):
         service = WHOISService(self.device)
         service.process_ip_data_and_location()
 
-        self.assertEqual(mock_task.call_count, 1)
+        mock_task.assert_called_once_with(
+            device_pk=self.device.pk,
+            initial_ip_address=self.device._initial_last_ip,
+        )

openwisp_controller/geo/estimated_location/service.py

@@ -63,6 +63,8 @@ def is_estimated_location_enabled(self):
         return self.check_estimated_location_enabled(self.device.organization_id)
 
     def trigger_estimated_location_task(self, ip_address):
+        if self.device.is_deactivated():
+            return
         try:
             current_app.send_task(
                 "whois_estimated_location_task",