From fa3d2a30310ddd6ae7f835bcf88c0ac23bfe825e Mon Sep 17 00:00:00 2001
From: Jan-Philipp Benecke <jan-philipp@bnck.me>
Date: Fri, 22 Nov 2024 07:58:43 +0100
Subject: [PATCH] Remove configurable name in config flow from SABnzbd
 (#131073)

---
 homeassistant/components/sabnzbd/config_flow.py | 5 ++---
 homeassistant/components/sabnzbd/const.py       | 1 -
 homeassistant/components/sabnzbd/strings.json   | 1 -
 tests/components/sabnzbd/conftest.py            | 3 +--
 tests/components/sabnzbd/test_config_flow.py    | 4 +---
 5 files changed, 4 insertions(+), 10 deletions(-)

diff --git a/homeassistant/components/sabnzbd/config_flow.py b/homeassistant/components/sabnzbd/config_flow.py
index 0d0934fee11..b3bf48a252b 100644
--- a/homeassistant/components/sabnzbd/config_flow.py
+++ b/homeassistant/components/sabnzbd/config_flow.py
@@ -8,9 +8,9 @@ from typing import Any
 import voluptuous as vol
 
 from homeassistant.config_entries import ConfigFlow, ConfigFlowResult
-from homeassistant.const import CONF_API_KEY, CONF_NAME, CONF_URL
+from homeassistant.const import CONF_API_KEY, CONF_URL
 
-from .const import DEFAULT_NAME, DOMAIN
+from .const import DOMAIN
 from .sab import get_client
 
 _LOGGER = logging.getLogger(__name__)
@@ -18,7 +18,6 @@ _LOGGER = logging.getLogger(__name__)
 USER_SCHEMA = vol.Schema(
     {
         vol.Required(CONF_API_KEY): str,
-        vol.Optional(CONF_NAME, default=DEFAULT_NAME): str,
         vol.Required(CONF_URL): str,
     }
 )
diff --git a/homeassistant/components/sabnzbd/const.py b/homeassistant/components/sabnzbd/const.py
index 55346509133..991490f5716 100644
--- a/homeassistant/components/sabnzbd/const.py
+++ b/homeassistant/components/sabnzbd/const.py
@@ -7,7 +7,6 @@ ATTR_SPEED = "speed"
 ATTR_API_KEY = "api_key"
 
 DEFAULT_HOST = "localhost"
-DEFAULT_NAME = "SABnzbd"
 DEFAULT_PORT = 8080
 DEFAULT_SPEED_LIMIT = "100"
 DEFAULT_SSL = False
diff --git a/homeassistant/components/sabnzbd/strings.json b/homeassistant/components/sabnzbd/strings.json
index 5d573c6a8cb..d21ae1fd219 100644
--- a/homeassistant/components/sabnzbd/strings.json
+++ b/homeassistant/components/sabnzbd/strings.json
@@ -4,7 +4,6 @@
       "user": {
         "data": {
           "api_key": "[%key:common::config_flow::data::api_key%]",
-          "name": "[%key:common::config_flow::data::name%]",
           "url": "[%key:common::config_flow::data::url%]"
         },
         "data_description": {
diff --git a/tests/components/sabnzbd/conftest.py b/tests/components/sabnzbd/conftest.py
index fc01429378b..08cca82d3b6 100644
--- a/tests/components/sabnzbd/conftest.py
+++ b/tests/components/sabnzbd/conftest.py
@@ -6,7 +6,7 @@ from unittest.mock import AsyncMock, patch
 import pytest
 
 from homeassistant.components.sabnzbd import DOMAIN
-from homeassistant.const import CONF_API_KEY, CONF_NAME, CONF_URL
+from homeassistant.const import CONF_API_KEY, CONF_URL
 from homeassistant.core import HomeAssistant
 from homeassistant.setup import async_setup_component
 
@@ -42,7 +42,6 @@ async def mock_config_entry(hass: HomeAssistant, sabnzbd: AsyncMock) -> MockConf
         title="Sabnzbd",
         entry_id="01JD2YVVPBC62D620DGYNG2R8H",
         data={
-            CONF_NAME: "Sabnzbd",
             CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0",
             CONF_URL: "http://localhost:8080",
         },
diff --git a/tests/components/sabnzbd/test_config_flow.py b/tests/components/sabnzbd/test_config_flow.py
index fc0205915b8..aefbfedb4b5 100644
--- a/tests/components/sabnzbd/test_config_flow.py
+++ b/tests/components/sabnzbd/test_config_flow.py
@@ -8,12 +8,11 @@ import pytest
 from homeassistant import config_entries
 from homeassistant.components.sabnzbd import DOMAIN
 from homeassistant.config_entries import SOURCE_USER
-from homeassistant.const import CONF_API_KEY, CONF_NAME, CONF_URL
+from homeassistant.const import CONF_API_KEY, CONF_URL
 from homeassistant.core import HomeAssistant
 from homeassistant.data_entry_flow import FlowResultType
 
 VALID_CONFIG = {
-    CONF_NAME: "Sabnzbd",
     CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0",
     CONF_URL: "http://localhost:8080",
 }
@@ -43,7 +42,6 @@ async def test_create_entry(hass: HomeAssistant, mock_setup_entry: AsyncMock) ->
         assert result2["title"] == "edc3eee7330e"
         assert result2["data"] == {
             CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0",
-            CONF_NAME: "Sabnzbd",
             CONF_URL: "http://localhost:8080",
         }
         assert len(mock_setup_entry.mock_calls) == 1