diff --git a/homeassistant/components/binary_sensor/knx.py b/homeassistant/components/binary_sensor/knx.py index d63a5a5b400..2b33d6850d6 100644 --- a/homeassistant/components/binary_sensor/knx.py +++ b/homeassistant/components/binary_sensor/knx.py @@ -35,7 +35,7 @@ DEPENDENCIES = ['knx'] AUTOMATION_SCHEMA = vol.Schema({ vol.Optional(CONF_HOOK, default=CONF_DEFAULT_HOOK): cv.string, vol.Optional(CONF_COUNTER, default=CONF_DEFAULT_COUNTER): cv.port, - vol.Required(CONF_ACTION, default=None): cv.SCRIPT_SCHEMA + vol.Required(CONF_ACTION): cv.SCRIPT_SCHEMA }) AUTOMATIONS_SCHEMA = vol.All( diff --git a/homeassistant/components/media_player/braviatv_psk.py b/homeassistant/components/media_player/braviatv_psk.py index c78951e91b7..122eb3b9739 100755 --- a/homeassistant/components/media_player/braviatv_psk.py +++ b/homeassistant/components/media_player/braviatv_psk.py @@ -42,7 +42,7 @@ TV_NO_INFO = 'No info: TV resumed after pause' PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend({ vol.Required(CONF_HOST): cv.string, vol.Required(CONF_PSK): cv.string, - vol.Optional(CONF_MAC, default=None): cv.string, + vol.Optional(CONF_MAC): cv.string, vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string, vol.Optional(CONF_AMP, default=False): cv.boolean, vol.Optional(CONF_ANDROID, default=True): cv.boolean, diff --git a/tests/helpers/test_config_validation.py b/tests/helpers/test_config_validation.py index 26262f50ac4..66f0597fc93 100644 --- a/tests/helpers/test_config_validation.py +++ b/tests/helpers/test_config_validation.py @@ -524,18 +524,14 @@ def test_enum(): def test_socket_timeout(): # pylint: disable=invalid-name """Test socket timeout validator.""" - TEST_CONF_TIMEOUT = 'timeout' # pylint: disable=invalid-name - - schema = vol.Schema( - {vol.Required(TEST_CONF_TIMEOUT, default=None): cv.socket_timeout}) + schema = vol.Schema(cv.socket_timeout) with pytest.raises(vol.Invalid): - schema({TEST_CONF_TIMEOUT: 0.0}) + schema(0.0) with pytest.raises(vol.Invalid): - schema({TEST_CONF_TIMEOUT: -1}) + schema(-1) - assert _GLOBAL_DEFAULT_TIMEOUT == schema({TEST_CONF_TIMEOUT: - None})[TEST_CONF_TIMEOUT] + assert _GLOBAL_DEFAULT_TIMEOUT == schema(None) - assert schema({TEST_CONF_TIMEOUT: 1})[TEST_CONF_TIMEOUT] == 1.0 + assert schema(1) == 1.0