diff --git a/homeassistant/components/hddtemp/sensor.py b/homeassistant/components/hddtemp/sensor.py index 4d9bbeb9516..9e00f0ee3fc 100644 --- a/homeassistant/components/hddtemp/sensor.py +++ b/homeassistant/components/hddtemp/sensor.py @@ -32,7 +32,6 @@ _LOGGER = logging.getLogger(__name__) ATTR_DEVICE = "device" ATTR_MODEL = "model" -DEFAULT_HOST = "localhost" DEFAULT_PORT = 7634 DEFAULT_NAME = "HD Temperature" DEFAULT_TIMEOUT = 5 @@ -42,7 +41,7 @@ SCAN_INTERVAL = timedelta(minutes=1) PLATFORM_SCHEMA = SENSOR_PLATFORM_SCHEMA.extend( { 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_NAME, default=DEFAULT_NAME): cv.string, } @@ -57,7 +56,7 @@ def setup_platform( ) -> None: """Set up the HDDTemp sensor.""" name = config.get(CONF_NAME) - host = config.get(CONF_HOST) + host = config[CONF_HOST] port = config.get(CONF_PORT) disks = config.get(CONF_DISKS) diff --git a/tests/components/hddtemp/test_sensor.py b/tests/components/hddtemp/test_sensor.py index 56ad9fdcb0e..1b05f6e6c3e 100644 --- a/tests/components/hddtemp/test_sensor.py +++ b/tests/components/hddtemp/test_sensor.py @@ -9,13 +9,19 @@ from homeassistant.const import UnitOfTemperature from homeassistant.core import HomeAssistant 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 = { "sensor": {