[fix] Remove duplicate self-match assertion from test_device_update_config_in_progress

Consolidated the self-ID exclusion assertion into
test_update_in_progress_ignores_current_task to avoid redundancy.

Author: mn-ram

openwisp_controller/connection/tests/test_models.py

@@ -1047,24 +1047,6 @@ def test_device_update_config_in_progress(
             mocked_get_working_connection.assert_not_called()
             update_config.assert_not_called()
 
-        # Regression: when the only active task matches the current task's own ID,
-        # _is_update_in_progress must return False (update should proceed).
-        with mock.patch("celery.app.control.Inspect.active") as mocked_active:
-            current_task_id = "current-task-id"
-            mocked_active.return_value = {
-                "worker1": [
-                    {
-                        "name": _TASK_NAME,
-                        "args": [str(conf.device.pk)],
-                        "id": current_task_id,
-                    }
-                ]
-            }
-            result = _is_update_in_progress(
-                conf.device.pk, current_task_id=current_task_id
-            )
-            self.assertFalse(result)
-
     def test_update_in_progress_ignores_current_task(self):
         """Regression test: _is_update_in_progress must not match the
         currently running task, which would cause it to skip itself."""