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

Make host required for hddtemp

Browse Source
This commit is contained in:
Robert Resch 2025-07-09 16:06:40 +02:00
parent 59fe6da47c
commit 943d873782
No known key found for this signature in database
GPG Key ID: 9D9D9DCB43120143
2 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
homeassistant/components/hddtemp/sensor.py
View File

@ -32,7 +32,6 @@ _LOGGER = logging.getLogger(__name__)
ATTR_DEVICE = "device" ATTR_DEVICE = "device"
ATTR_MODEL = "model" ATTR_MODEL = "model"
DEFAULT_HOST = "localhost"
DEFAULT_PORT = 7634 DEFAULT_PORT = 7634
DEFAULT_NAME = "HD Temperature" DEFAULT_NAME = "HD Temperature"
DEFAULT_TIMEOUT = 5 DEFAULT_TIMEOUT = 5
@ -42,7 +41,7 @@ SCAN_INTERVAL = timedelta(minutes=1)
PLATFORM_SCHEMA = SENSOR_PLATFORM_SCHEMA.extend( PLATFORM_SCHEMA = SENSOR_PLATFORM_SCHEMA.extend(
{ {
vol.Optional(CONF_DISKS, default=[]): vol.All(cv.ensure_list, [cv.string]), vol.Optional(CONF_DISKS, default=[]): vol.All(cv.ensure_list, [cv.string]),
vol.Optional(CONF_HOST, default=DEFAULT_HOST): cv.string, vol.Required(CONF_HOST): cv.string,
vol.Optional(CONF_PORT, default=DEFAULT_PORT): cv.port, vol.Optional(CONF_PORT, default=DEFAULT_PORT): cv.port,
vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string, vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string,
} }
@ -57,7 +56,7 @@ def setup_platform(
) -> None: ) -> None:
"""Set up the HDDTemp sensor.""" """Set up the HDDTemp sensor."""
name = config.get(CONF_NAME) name = config.get(CONF_NAME)
host = config.get(CONF_HOST) host = config[CONF_HOST]
port = config.get(CONF_PORT) port = config.get(CONF_PORT)
disks = config.get(CONF_DISKS) disks = config.get(CONF_DISKS)

14
tests/components/hddtemp/test_sensor.py
View File

@ -9,13 +9,19 @@ from homeassistant.const import UnitOfTemperature
from homeassistant.core import HomeAssistant from homeassistant.core import HomeAssistant
from homeassistant.setup import async_setup_component from homeassistant.setup import async_setup_component
VALID_CONFIG_MINIMAL = {"sensor": {"platform": "hddtemp"}} VALID_CONFIG_MINIMAL = {"sensor": {"platform": "hddtemp", "host": "foobar.local"}}
VALID_CONFIG_NAME = {"sensor": {"platform": "hddtemp", "name": "FooBar"}} VALID_CONFIG_NAME = {
"sensor": {"platform": "hddtemp", "host": "foobar.local", "name": "FooBar"}
}
VALID_CONFIG_ONE_DISK = {"sensor": {"platform": "hddtemp", "disks": ["/dev/sdd1"]}} VALID_CONFIG_ONE_DISK = {
"sensor": {"platform": "hddtemp", "host": "foobar.local", "disks": ["/dev/sdd1"]}
}
VALID_CONFIG_WRONG_DISK = {"sensor": {"platform": "hddtemp", "disks": ["/dev/sdx1"]}} VALID_CONFIG_WRONG_DISK = {
"sensor": {"platform": "hddtemp", "host": "foobar.local", "disks": ["/dev/sdx1"]}
}
VALID_CONFIG_MULTIPLE_DISKS = { VALID_CONFIG_MULTIPLE_DISKS = {
"sensor": { "sensor": {