From 1fe816f76162269eeac67365982f39844f3b7231 Mon Sep 17 00:00:00 2001
From: Gagan Deep <pandafy.dev@gmail.com>
Date: Thu, 23 Nov 2023 17:56:30 +0530
Subject: [PATCH] [req-changes] Renamed methods

---
 openwisp_monitoring/device/apps.py              | 6 +++---
 openwisp_monitoring/device/base/models.py       | 2 +-
 openwisp_monitoring/device/tasks.py             | 4 ++--
 openwisp_monitoring/device/tests/test_models.py | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/openwisp_monitoring/device/apps.py b/openwisp_monitoring/device/apps.py
index c13b6b10..73970987 100644
--- a/openwisp_monitoring/device/apps.py
+++ b/openwisp_monitoring/device/apps.py
@@ -124,7 +124,7 @@ def connect_device_signals(self):
         post_save.connect(
             self.organization_post_save_receiver,
             sender=Organization,
-            dispatch_uid='post_save_organization_clear_management_ip_monitoring_status',
+            dispatch_uid='post_save_organization_disabled_monitoring',
         )
 
     @classmethod
@@ -143,9 +143,9 @@ def device_post_delete_receiver(cls, instance, **kwargs):
     @classmethod
     def organization_post_save_receiver(cls, instance, *args, **kwargs):
         if instance.is_active is False:
-            from .tasks import organization_disabled_handler
+            from .tasks import handle_disabled_organization
 
-            organization_disabled_handler.delay(str(instance.id))
+            handle_disabled_organization.delay(str(instance.id))
 
     def device_recovery_detection(self):
         if not app_settings.DEVICE_RECOVERY_DETECTION:
diff --git a/openwisp_monitoring/device/base/models.py b/openwisp_monitoring/device/base/models.py
index ab0f8aa0..b3e3f5d4 100644
--- a/openwisp_monitoring/device/base/models.py
+++ b/openwisp_monitoring/device/base/models.py
@@ -436,7 +436,7 @@ def is_metric_critical(metric):
         return False
 
     @classmethod
-    def organization_disabled_handler(cls, organization_id):
+    def handle_disabled_organization(cls, organization_id):
         """
         Clears the management IP of all devices belonging to a
         disabled organization and set their monitoring status to 'unknown'.
diff --git a/openwisp_monitoring/device/tasks.py b/openwisp_monitoring/device/tasks.py
index afab46e8..03ab8577 100644
--- a/openwisp_monitoring/device/tasks.py
+++ b/openwisp_monitoring/device/tasks.py
@@ -66,6 +66,6 @@ def write_device_metrics(pk, data, time=None, current=False):
 
 
 @shared_task(base=OpenwispCeleryTask)
-def organization_disabled_handler(organization_id):
+def handle_disabled_organization(organization_id):
     DeviceMonitoring = load_model('device_monitoring', 'DeviceMonitoring')
-    DeviceMonitoring.organization_disabled_handler(organization_id)
+    DeviceMonitoring.handle_disabled_organization(organization_id)
diff --git a/openwisp_monitoring/device/tests/test_models.py b/openwisp_monitoring/device/tests/test_models.py
index d190715e..e539c070 100644
--- a/openwisp_monitoring/device/tests/test_models.py
+++ b/openwisp_monitoring/device/tests/test_models.py
@@ -720,7 +720,7 @@ def test_deleting_device_deletes_tsdb(self):
         self.assertEqual(self._read_metric(ping1), [])
         self.assertNotEqual(self._read_metric(ping2), [])
 
-    def test_organization_disabled_handler(self):
+    def test_handle_disabled_organization(self):
         device_monitoring, _, _, _ = self._create_env()
         device = device_monitoring.device
         device.management_ip = '10.10.0.5'