From f34ebf733d18a8768e0da2dddec8b375d8077387 Mon Sep 17 00:00:00 2001 From: Pascal Vizeli Date: Wed, 4 Oct 2017 18:31:50 +0200 Subject: [PATCH] HassIO replace config changes (#9695) * Update flow * fix tests * Update hassio.py --- homeassistant/components/hassio.py | 11 ++++------- tests/components/test_hassio.py | 4 ++-- 2 files changed, 6 insertions(+), 9 deletions(-) diff --git a/homeassistant/components/hassio.py b/homeassistant/components/hassio.py index 4bcb762cbd3..1be8ebcf5dd 100644 --- a/homeassistant/components/hassio.py +++ b/homeassistant/components/hassio.py @@ -17,7 +17,7 @@ import async_timeout import voluptuous as vol import homeassistant.helpers.config_validation as cv -from homeassistant.const import CONTENT_TYPE_TEXT_PLAIN +from homeassistant.const import CONTENT_TYPE_TEXT_PLAIN, SERVER_PORT from homeassistant.components.http import ( HomeAssistantView, KEY_AUTHENTICATED, CONF_API_PASSWORD, CONF_SERVER_PORT, CONF_SSL_CERTIFICATE) @@ -129,16 +129,13 @@ class HassIO(object): This method return a coroutine. """ + port = http_config.get(CONF_SERVER_PORT) or SERVER_PORT options = { 'ssl': CONF_SSL_CERTIFICATE in http_config, + 'port': port, + 'password': http_config.get(CONF_API_PASSWORD), } - if http_config.get(CONF_SERVER_PORT): - options['port'] = http_config[CONF_SERVER_PORT] - - if http_config.get(CONF_API_PASSWORD): - options['password'] = http_config[CONF_API_PASSWORD] - return self.send_command("/homeassistant/options", payload=options) @asyncio.coroutine diff --git a/tests/components/test_hassio.py b/tests/components/test_hassio.py index 26a8372352f..f7c967da862 100644 --- a/tests/components/test_hassio.py +++ b/tests/components/test_hassio.py @@ -105,8 +105,8 @@ def test_setup_api_push_api_data_default(hass, aioclient_mock): assert aioclient_mock.call_count == 2 assert not aioclient_mock.mock_calls[-1][2]['ssl'] - assert 'password' not in aioclient_mock.mock_calls[-1][2] - assert 'port' not in aioclient_mock.mock_calls[-1][2] + assert aioclient_mock.mock_calls[-1][2]['password'] is None + assert aioclient_mock.mock_calls[-1][2]['port'] == 8123 @asyncio.coroutine