From 96edaebdd30c43be4d64939e4a4577b5c0ba7e6b Mon Sep 17 00:00:00 2001
From: Erik Montnemery <erik@montnemery.com>
Date: Mon, 19 Aug 2024 13:15:50 +0200
Subject: [PATCH] Remove unused function otbr.async_get_active_dataset_tlvs
 (#124210)

---
 homeassistant/components/otbr/__init__.py | 13 -----
 tests/components/otbr/test_init.py        | 71 -----------------------
 2 files changed, 84 deletions(-)

diff --git a/homeassistant/components/otbr/__init__.py b/homeassistant/components/otbr/__init__.py
index 97c2f40eb99..1d79b1c52e9 100644
--- a/homeassistant/components/otbr/__init__.py
+++ b/homeassistant/components/otbr/__init__.py
@@ -81,16 +81,3 @@ async def async_unload_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
 async def async_reload_entry(hass: HomeAssistant, entry: ConfigEntry) -> None:
     """Handle an options update."""
     await hass.config_entries.async_reload(entry.entry_id)
-
-
-async def async_get_active_dataset_tlvs(hass: HomeAssistant) -> bytes | None:
-    """Get current active operational dataset in TLVS format, or None.
-
-    Returns None if there is no active operational dataset.
-    Raises if the http status is 400 or higher or if the response is invalid.
-    """
-    if DOMAIN not in hass.data:
-        raise HomeAssistantError("OTBR API not available")
-
-    data: OTBRData = hass.data[DOMAIN]
-    return await data.get_active_dataset_tlvs()
diff --git a/tests/components/otbr/test_init.py b/tests/components/otbr/test_init.py
index 0c56e9ac8da..5cf29662be3 100644
--- a/tests/components/otbr/test_init.py
+++ b/tests/components/otbr/test_init.py
@@ -1,7 +1,6 @@
 """Test the Open Thread Border Router integration."""
 
 import asyncio
-from http import HTTPStatus
 from typing import Any
 from unittest.mock import ANY, AsyncMock, MagicMock, patch
 
@@ -13,7 +12,6 @@ from zeroconf.asyncio import AsyncServiceInfo
 from homeassistant.components import otbr, thread
 from homeassistant.components.thread import discovery
 from homeassistant.core import HomeAssistant
-from homeassistant.exceptions import HomeAssistantError
 from homeassistant.helpers import issue_registry as ir
 from homeassistant.setup import async_setup_component
 
@@ -346,78 +344,9 @@ async def test_remove_entry(
 
     aioclient_mock.get(f"{BASE_URL}/node/dataset/active", text="0E")
 
-    assert await otbr.async_get_active_dataset_tlvs(hass) == bytes.fromhex("0E")
-
     config_entry = hass.config_entries.async_entries(otbr.DOMAIN)[0]
     await hass.config_entries.async_remove(config_entry.entry_id)
 
-    with pytest.raises(HomeAssistantError):
-        assert await otbr.async_get_active_dataset_tlvs(hass)
-
-
-async def test_get_active_dataset_tlvs(
-    hass: HomeAssistant, aioclient_mock: AiohttpClientMocker, otbr_config_entry_multipan
-) -> None:
-    """Test async_get_active_dataset_tlvs."""
-
-    mock_response = (
-        "0E080000000000010000000300001035060004001FFFE00208F642646DA209B1C00708FDF57B5A"
-        "0FE2AAF60510DE98B5BA1A528FEE049D4B4B01835375030D4F70656E5468726561642048410102"
-        "25A40410F5DD18371BFD29E1A601EF6FFAD94C030C0402A0F7F8"
-    )
-
-    aioclient_mock.get(f"{BASE_URL}/node/dataset/active", text=mock_response)
-
-    assert await otbr.async_get_active_dataset_tlvs(hass) == bytes.fromhex(
-        mock_response
-    )
-
-
-async def test_get_active_dataset_tlvs_empty(
-    hass: HomeAssistant, aioclient_mock: AiohttpClientMocker, otbr_config_entry_multipan
-) -> None:
-    """Test async_get_active_dataset_tlvs."""
-
-    aioclient_mock.get(f"{BASE_URL}/node/dataset/active", status=HTTPStatus.NO_CONTENT)
-    assert await otbr.async_get_active_dataset_tlvs(hass) is None
-
-
-async def test_get_active_dataset_tlvs_addon_not_installed(hass: HomeAssistant) -> None:
-    """Test async_get_active_dataset_tlvs when the multi-PAN addon is not installed."""
-
-    with pytest.raises(HomeAssistantError):
-        await otbr.async_get_active_dataset_tlvs(hass)
-
-
-async def test_get_active_dataset_tlvs_404(
-    hass: HomeAssistant, aioclient_mock: AiohttpClientMocker, otbr_config_entry_multipan
-) -> None:
-    """Test async_get_active_dataset_tlvs with error."""
-
-    aioclient_mock.get(f"{BASE_URL}/node/dataset/active", status=HTTPStatus.NOT_FOUND)
-    with pytest.raises(HomeAssistantError):
-        await otbr.async_get_active_dataset_tlvs(hass)
-
-
-async def test_get_active_dataset_tlvs_201(
-    hass: HomeAssistant, aioclient_mock: AiohttpClientMocker, otbr_config_entry_multipan
-) -> None:
-    """Test async_get_active_dataset_tlvs with error."""
-
-    aioclient_mock.get(f"{BASE_URL}/node/dataset/active", status=HTTPStatus.CREATED)
-    with pytest.raises(HomeAssistantError):
-        assert await otbr.async_get_active_dataset_tlvs(hass)
-
-
-async def test_get_active_dataset_tlvs_invalid(
-    hass: HomeAssistant, aioclient_mock: AiohttpClientMocker, otbr_config_entry_multipan
-) -> None:
-    """Test async_get_active_dataset_tlvs with error."""
-
-    aioclient_mock.get(f"{BASE_URL}/node/dataset/active", text="unexpected")
-    with pytest.raises(HomeAssistantError):
-        assert await otbr.async_get_active_dataset_tlvs(hass)
-
 
 async def test_remove_extra_entries(
     hass: HomeAssistant, aioclient_mock: AiohttpClientMocker