Mock arguments to ClientResponseError() in miele tests (home-assistant#156676)

Author: astrandb

tests/components/miele/test_button.py

@@ -1,6 +1,6 @@
 """Tests for Miele button module."""
 
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, Mock
 
 from aiohttp import ClientResponseError
 import pytest
@@ -70,7 +70,7 @@ async def test_api_failure(
     setup_platform: MockConfigEntry,
 ) -> None:
     """Test handling of exception from API."""
-    mock_miele_client.send_action.side_effect = ClientResponseError("test", "Test")
+    mock_miele_client.send_action.side_effect = ClientResponseError(Mock(), Mock())
 
     with pytest.raises(
         HomeAssistantError, match=f"Failed to set state for {ENTITY_ID}"

tests/components/miele/test_climate.py

@@ -1,6 +1,6 @@
 """Tests for miele climate module."""
 
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, Mock
 
 from aiohttp import ClientResponseError
 import pytest
@@ -108,7 +108,7 @@ async def test_api_failure(
 ) -> None:
     """Test handling of exception from API."""
     mock_miele_client.set_target_temperature.side_effect = ClientResponseError(
-        "test", "Test"
+        Mock(), Mock()
     )
 
     with pytest.raises(

tests/components/miele/test_fan.py

@@ -1,7 +1,7 @@
 """Tests for miele fan module."""
 
 from typing import Any
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, Mock
 
 from aiohttp import ClientResponseError
 import pytest
@@ -141,7 +141,7 @@ async def test_api_failure(
     service: str,
 ) -> None:
     """Test handling of exception from API."""
-    mock_miele_client.send_action.side_effect = ClientResponseError("test", "Test")
+    mock_miele_client.send_action.side_effect = ClientResponseError(Mock(), Mock())
 
     with pytest.raises(HomeAssistantError):
         await hass.services.async_call(
@@ -156,7 +156,7 @@ async def test_set_percentage(
     setup_platform: None,
 ) -> None:
     """Test handling of exception at set_percentage."""
-    mock_miele_client.send_action.side_effect = ClientResponseError("test", "Test")
+    mock_miele_client.send_action.side_effect = ClientResponseError(Mock(), Mock())
 
     with pytest.raises(
         HomeAssistantError, match=f"Failed to set state for {ENTITY_ID}"

tests/components/miele/test_init.py

@@ -3,7 +3,7 @@
 from datetime import timedelta
 import http
 import time
-from unittest.mock import MagicMock, patch
+from unittest.mock import MagicMock, Mock, patch
 
 from aiohttp import ClientConnectionError, ClientResponseError
 from freezegun.api import FrozenDateTimeFactory
@@ -215,7 +215,7 @@ async def test_setup_all_platforms(
 @pytest.mark.parametrize(
     "side_effect",
     [
-        ClientResponseError("test", "Test"),
+        ClientResponseError(Mock(), Mock()),
         TimeoutError,
     ],
     ids=[

tests/components/miele/test_light.py

@@ -1,6 +1,6 @@
 """Tests for miele light module."""
 
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, Mock
 
 from aiohttp import ClientResponseError
 import pytest
@@ -84,7 +84,7 @@ async def test_api_failure(
     service: str,
 ) -> None:
     """Test handling of exception from API."""
-    mock_miele_client.send_action.side_effect = ClientResponseError("test", "Test")
+    mock_miele_client.send_action.side_effect = ClientResponseError(Mock(), Mock())
 
     with pytest.raises(
         HomeAssistantError, match=f"Failed to set state for {ENTITY_ID}"

tests/components/miele/test_services.py

@@ -1,7 +1,7 @@
 """Tests the services provided by the miele integration."""
 
 from datetime import timedelta
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, Mock
 
 from aiohttp import ClientResponseError
 import pytest
@@ -138,7 +138,7 @@ async def test_service_api_errors(
     device = device_registry.async_get_device(identifiers={(DOMAIN, TEST_APPLIANCE)})
 
     # Test http error
-    mock_miele_client.set_program.side_effect = ClientResponseError("TestInfo", "test")
+    mock_miele_client.set_program.side_effect = ClientResponseError(Mock(), Mock())
     with pytest.raises(HomeAssistantError, match=error):
         await hass.services.async_call(
             DOMAIN,
@@ -162,7 +162,7 @@ async def test_get_service_api_errors(
     device = device_registry.async_get_device(identifiers={(DOMAIN, TEST_APPLIANCE)})
 
     # Test http error
-    mock_miele_client.get_programs.side_effect = ClientResponseError("TestInfo", "test")
+    mock_miele_client.get_programs.side_effect = ClientResponseError(Mock(), Mock())
     with pytest.raises(HomeAssistantError, match="'Get programs' action failed"):
         await hass.services.async_call(
             DOMAIN,

tests/components/miele/test_switch.py

@@ -1,6 +1,6 @@
 """Tests for miele switch module."""
 
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, Mock
 
 from aiohttp import ClientResponseError
 import pytest
@@ -99,7 +99,7 @@ async def test_api_failure(
     entity: str,
 ) -> None:
     """Test handling of exception from API."""
-    mock_miele_client.send_action.side_effect = ClientResponseError("test", "Test")
+    mock_miele_client.send_action.side_effect = ClientResponseError(Mock(), Mock())
 
     with pytest.raises(HomeAssistantError, match=f"Failed to set state for {entity}"):
         await hass.services.async_call(

tests/components/miele/test_vacuum.py

@@ -1,6 +1,6 @@
 """Tests for miele vacuum module."""
 
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, Mock
 
 from aiohttp import ClientResponseError
 from pymiele import MieleDevices
@@ -142,7 +142,7 @@ async def test_api_failure(
     service: str,
 ) -> None:
     """Test handling of exception from API."""
-    mock_miele_client.send_action.side_effect = ClientResponseError("test", "Test")
+    mock_miele_client.send_action.side_effect = ClientResponseError(Mock(), Mock())
 
     with pytest.raises(
         HomeAssistantError, match=f"Failed to set state for {ENTITY_ID}"