jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-07-24 21:57:51 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove configurable name in config flow from SABnzbd (#131073)

Browse Source
This commit is contained in:
Jan-Philipp Benecke 2024-11-22 07:58:43 +01:00 committed by GitHub
parent 8a292184a5
commit fa3d2a3031
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
5 changed files with 4 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
homeassistant/components/sabnzbd/config_flow.py
View File

@ -8,9 +8,9 @@ from typing import Any
import voluptuous as vol import voluptuous as vol
from homeassistant.config_entries import ConfigFlow, ConfigFlowResult 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 from .sab import get_client
_LOGGER = logging.getLogger(__name__) _LOGGER = logging.getLogger(__name__)
@ -18,7 +18,6 @@ _LOGGER = logging.getLogger(__name__)
USER_SCHEMA = vol.Schema( USER_SCHEMA = vol.Schema(
{ {
vol.Required(CONF_API_KEY): str, vol.Required(CONF_API_KEY): str,
vol.Optional(CONF_NAME, default=DEFAULT_NAME): str,
vol.Required(CONF_URL): str, vol.Required(CONF_URL): str,
} }
) )

1
homeassistant/components/sabnzbd/const.py
View File

@ -7,7 +7,6 @@ ATTR_SPEED = "speed"
ATTR_API_KEY = "api_key" ATTR_API_KEY = "api_key"
DEFAULT_HOST = "localhost" DEFAULT_HOST = "localhost"
DEFAULT_NAME = "SABnzbd"
DEFAULT_PORT = 8080 DEFAULT_PORT = 8080
DEFAULT_SPEED_LIMIT = "100" DEFAULT_SPEED_LIMIT = "100"
DEFAULT_SSL = False DEFAULT_SSL = False

1
homeassistant/components/sabnzbd/strings.json
View File

@ -4,7 +4,6 @@
"user": { "user": {
"data": { "data": {
"api_key": "[%key:common::config_flow::data::api_key%]", "api_key": "[%key:common::config_flow::data::api_key%]",
"name": "[%key:common::config_flow::data::name%]",
"url": "[%key:common::config_flow::data::url%]" "url": "[%key:common::config_flow::data::url%]"
}, },
"data_description": { "data_description": {

3
tests/components/sabnzbd/conftest.py
View File

@ -6,7 +6,7 @@ from unittest.mock import AsyncMock, patch
import pytest import pytest
from homeassistant.components.sabnzbd import DOMAIN 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.core import HomeAssistant
from homeassistant.setup import async_setup_component from homeassistant.setup import async_setup_component
@ -42,7 +42,6 @@ async def mock_config_entry(hass: HomeAssistant, sabnzbd: AsyncMock) -> MockConf
title="Sabnzbd", title="Sabnzbd",
entry_id="01JD2YVVPBC62D620DGYNG2R8H", entry_id="01JD2YVVPBC62D620DGYNG2R8H",
data={ data={
CONF_NAME: "Sabnzbd",
CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0", CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0",
CONF_URL: "http://localhost:8080", CONF_URL: "http://localhost:8080",
}, },

4
tests/components/sabnzbd/test_config_flow.py
View File

@ -8,12 +8,11 @@ import pytest
from homeassistant import config_entries from homeassistant import config_entries
from homeassistant.components.sabnzbd import DOMAIN from homeassistant.components.sabnzbd import DOMAIN
from homeassistant.config_entries import SOURCE_USER 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.core import HomeAssistant
from homeassistant.data_entry_flow import FlowResultType from homeassistant.data_entry_flow import FlowResultType
VALID_CONFIG = { VALID_CONFIG = {
CONF_NAME: "Sabnzbd",
CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0", CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0",
CONF_URL: "http://localhost:8080", 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["title"] == "edc3eee7330e"
assert result2["data"] == { assert result2["data"] == {
CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0", CONF_API_KEY: "edc3eee7330e4fdda04489e3fbc283d0",
CONF_NAME: "Sabnzbd",
CONF_URL: "http://localhost:8080", CONF_URL: "http://localhost:8080",
} }
assert len(mock_setup_entry.mock_calls) == 1 assert len(mock_setup_entry.mock_calls) == 1