Use hass_client_no_auth test fixture in integrations s-x (home-assist…

…ant#55585)

tests/components/smappee/test_config_flow.py

@@ -395,7 +395,7 @@ async def test_abort_cloud_flow_if_local_device_exists(hass):
 
 
 async def test_full_user_flow(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Check full flow."""
     assert await setup.async_setup_component(
@@ -422,7 +422,7 @@ async def test_full_user_flow(
         },
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     resp = await client.get(f"/auth/external/callback?code=abcd&state={state}")
     assert resp.status == 200
     assert resp.headers["content-type"] == "text/html; charset=utf-8"

tests/components/somfy/test_config_flow.py

@@ -34,7 +34,7 @@ async def test_abort_if_existing_entry(hass):
 
 
 async def test_full_flow(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Check full flow."""
     assert await setup.async_setup_component(
@@ -67,7 +67,7 @@ async def test_full_flow(
         f"&state={state}"
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     resp = await client.get(f"/auth/external/callback?code=abcd&state={state}")
     assert resp.status == 200
     assert resp.headers["content-type"] == "text/html; charset=utf-8"

tests/components/spotify/test_config_flow.py

@@ -42,7 +42,7 @@ async def test_zeroconf_abort_if_existing_entry(hass):
 
 
 async def test_full_flow(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Check a full flow."""
     assert await setup.async_setup_component(
@@ -78,7 +78,7 @@ async def test_full_flow(
         "user-top-read,user-read-playback-position,user-read-recently-played,user-follow-read"
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     resp = await client.get(f"/auth/external/callback?code=abcd&state={state}")
     assert resp.status == 200
     assert resp.headers["content-type"] == "text/html; charset=utf-8"
@@ -114,7 +114,7 @@ async def test_full_flow(
 
 
 async def test_abort_if_spotify_error(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Check Spotify errors causes flow to abort."""
     await setup.async_setup_component(
@@ -138,7 +138,7 @@ async def test_abort_if_spotify_error(
             "redirect_uri": "https://example.com/auth/external/callback",
         },
     )
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.get(f"/auth/external/callback?code=abcd&state={state}")
 
     aioclient_mock.post(
@@ -162,7 +162,7 @@ async def test_abort_if_spotify_error(
 
 
 async def test_reauthentication(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Test Spotify reauthentication."""
     await setup.async_setup_component(
@@ -199,7 +199,7 @@ async def test_reauthentication(
             "redirect_uri": "https://example.com/auth/external/callback",
         },
     )
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.get(f"/auth/external/callback?code=abcd&state={state}")
 
     aioclient_mock.post(
@@ -229,7 +229,7 @@ async def test_reauthentication(
 
 
 async def test_reauth_account_mismatch(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Test Spotify reauthentication with different account."""
     await setup.async_setup_component(
@@ -264,7 +264,7 @@ async def test_reauth_account_mismatch(
             "redirect_uri": "https://example.com/auth/external/callback",
         },
     )
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.get(f"/auth/external/callback?code=abcd&state={state}")
 
     aioclient_mock.post(

tests/components/toon/test_config_flow.py

@@ -41,7 +41,7 @@ async def test_abort_if_no_configuration(hass):
 
 
 async def test_full_flow_implementation(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Test registering an integration and finishing flow works."""
     await setup_component(hass)
@@ -75,7 +75,7 @@ async def test_full_flow_implementation(
         "&tenant_id=eneco&issuer=identity.toon.eu"
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     resp = await client.get(f"/auth/external/callback?code=abcd&state={state}")
     assert resp.status == 200
     assert resp.headers["content-type"] == "text/html; charset=utf-8"
@@ -105,7 +105,7 @@ async def test_full_flow_implementation(
 
 
 async def test_no_agreements(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Test abort when there are no displays."""
     await setup_component(hass)
@@ -125,7 +125,7 @@ async def test_no_agreements(
         result["flow_id"], {"implementation": "eneco"}
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.get(f"/auth/external/callback?code=abcd&state={state}")
     aioclient_mock.post(
         "https://api.toon.eu/token",
@@ -145,7 +145,7 @@ async def test_no_agreements(
 
 
 async def test_multiple_agreements(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Test abort when there are no displays."""
     await setup_component(hass)
@@ -165,7 +165,7 @@ async def test_multiple_agreements(
         result["flow_id"], {"implementation": "eneco"}
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.get(f"/auth/external/callback?code=abcd&state={state}")
 
     aioclient_mock.post(
@@ -195,7 +195,7 @@ async def test_multiple_agreements(
 
 
 async def test_agreement_already_set_up(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Test showing display form again if display already exists."""
     await setup_component(hass)
@@ -216,7 +216,7 @@ async def test_agreement_already_set_up(
         result["flow_id"], {"implementation": "eneco"}
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.get(f"/auth/external/callback?code=abcd&state={state}")
     aioclient_mock.post(
         "https://api.toon.eu/token",
@@ -236,7 +236,7 @@ async def test_agreement_already_set_up(
 
 
 async def test_toon_abort(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Test we abort on Toon error."""
     await setup_component(hass)
@@ -255,7 +255,7 @@ async def test_toon_abort(
         result["flow_id"], {"implementation": "eneco"}
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.get(f"/auth/external/callback?code=abcd&state={state}")
     aioclient_mock.post(
         "https://api.toon.eu/token",
@@ -289,7 +289,7 @@ async def test_import(hass, current_request_with_host):
 
 
 async def test_import_migration(
-    hass, aiohttp_client, aioclient_mock, current_request_with_host
+    hass, hass_client_no_auth, aioclient_mock, current_request_with_host
 ):
     """Test if importing step with migration works."""
     old_entry = MockConfigEntry(domain=DOMAIN, unique_id=123, version=1)
@@ -317,7 +317,7 @@ async def test_import_migration(
         flows[0]["flow_id"], {"implementation": "eneco"}
     )
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.get(f"/auth/external/callback?code=abcd&state={state}")
     aioclient_mock.post(
         "https://api.toon.eu/token",

tests/components/traccar/test_init.py

@@ -28,7 +28,7 @@ def mock_dev_track(mock_device_tracker_conf):
 
 
 @pytest.fixture(name="client")
-async def traccar_client(loop, hass, aiohttp_client):
+async def traccar_client(loop, hass, hass_client_no_auth):
     """Mock client for Traccar (unauthenticated)."""
     assert await async_setup_component(hass, "persistent_notification", {})
 
@@ -37,7 +37,7 @@ async def traccar_client(loop, hass, aiohttp_client):
     await hass.async_block_till_done()
 
     with patch("homeassistant.components.device_tracker.legacy.update_config"):
-        return await aiohttp_client(hass.http.app)
+        return await hass_client_no_auth()
 
 
 @pytest.fixture(autouse=True)

tests/components/twilio/test_init.py

@@ -5,7 +5,7 @@
 from homeassistant.core import callback
 
 
-async def test_config_flow_registers_webhook(hass, aiohttp_client):
+async def test_config_flow_registers_webhook(hass, hass_client_no_auth):
     """Test setting up Twilio and sending webhook."""
     await async_process_ha_core_config(
         hass,
@@ -29,7 +29,7 @@ def handle_event(event):
 
     hass.bus.async_listen(twilio.RECEIVED_DATA, handle_event)
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     await client.post(f"/api/webhook/{webhook_id}", data={"hello": "twilio"})
 
     assert len(twilio_events) == 1

tests/components/webhook/test_trigger.py

@@ -17,7 +17,7 @@ async def setup_http(hass):
     await hass.async_block_till_done()
 
 
-async def test_webhook_json(hass, aiohttp_client):
+async def test_webhook_json(hass, hass_client_no_auth):
     """Test triggering with a JSON webhook."""
     events = []
 
@@ -46,7 +46,7 @@ def store_event(event):
     )
     await hass.async_block_till_done()
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
 
     await client.post("/api/webhook/json_webhook", json={"hello": "world"})
     await hass.async_block_till_done()
@@ -56,7 +56,7 @@ def store_event(event):
     assert events[0].data["id"] == 0
 
 
-async def test_webhook_post(hass, aiohttp_client):
+async def test_webhook_post(hass, hass_client_no_auth):
     """Test triggering with a POST webhook."""
     events = []
 
@@ -82,7 +82,7 @@ def store_event(event):
     )
     await hass.async_block_till_done()
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
 
     await client.post("/api/webhook/post_webhook", data={"hello": "world"})
     await hass.async_block_till_done()
@@ -91,7 +91,7 @@ def store_event(event):
     assert events[0].data["hello"] == "yo world"
 
 
-async def test_webhook_query(hass, aiohttp_client):
+async def test_webhook_query(hass, hass_client_no_auth):
     """Test triggering with a query POST webhook."""
     events = []
 
@@ -117,7 +117,7 @@ def store_event(event):
     )
     await hass.async_block_till_done()
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
 
     await client.post("/api/webhook/query_webhook?hello=world")
     await hass.async_block_till_done()
@@ -126,7 +126,7 @@ def store_event(event):
     assert events[0].data["hello"] == "yo world"
 
 
-async def test_webhook_reload(hass, aiohttp_client):
+async def test_webhook_reload(hass, hass_client_no_auth):
     """Test reloading a webhook."""
     events = []
 
@@ -152,7 +152,7 @@ def store_event(event):
     )
     await hass.async_block_till_done()
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
 
     await client.post("/api/webhook/post_webhook", data={"hello": "world"})
     await hass.async_block_till_done()

tests/components/websocket_api/conftest.py

@@ -13,12 +13,12 @@ async def websocket_client(hass, hass_ws_client):
 
 
 @pytest.fixture
-async def no_auth_websocket_client(hass, aiohttp_client):
+async def no_auth_websocket_client(hass, hass_client_no_auth):
     """Websocket connection that requires authentication."""
     assert await async_setup_component(hass, "websocket_api", {})
     await hass.async_block_till_done()
 
-    client = await aiohttp_client(hass.http.app)
+    client = await hass_client_no_auth()
     ws = await client.ws_connect(URL)
 
     auth_ok = await ws.receive_json()