From 6e903fd429fad3b51b5f610b3d16ab8ed3f3f34c Mon Sep 17 00:00:00 2001 From: Johann Kellerman Date: Sat, 22 Oct 2016 05:40:23 +0200 Subject: [PATCH] Updater component - rename opt_out to reporting (#3979) * Updater component - rename opt_out to reporting * Fix tests --- homeassistant/components/updater.py | 6 +++--- tests/components/test_updater.py | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/homeassistant/components/updater.py b/homeassistant/components/updater.py index 76dd9d8c09e..f1467f0e4cb 100644 --- a/homeassistant/components/updater.py +++ b/homeassistant/components/updater.py @@ -27,12 +27,12 @@ DOMAIN = 'updater' ENTITY_ID = 'updater.updater' ATTR_RELEASE_NOTES = 'release_notes' UPDATER_UUID_FILE = '.uuid' -CONF_OPT_OUT = 'opt_out' +CONF_REPORTING = 'reporting' REQUIREMENTS = ['distro==1.0.0'] CONFIG_SCHEMA = vol.Schema({DOMAIN: { - vol.Optional(CONF_OPT_OUT, default=False): cv.boolean + vol.Optional(CONF_REPORTING, default=True): cv.boolean }}, extra=vol.ALLOW_EXTRA) @@ -63,7 +63,7 @@ def setup(hass, config): _LOGGER.warning('Updater not supported in development version') return False - huuid = None if config.get(CONF_OPT_OUT) else _load_uuid(hass) + huuid = _load_uuid(hass) if config.get(CONF_REPORTING) else None # Update daily, start 1 hour after startup _dt = datetime.now() + timedelta(hours=1) diff --git a/tests/components/test_updater.py b/tests/components/test_updater.py index 74fc7fc8cd4..8333a721beb 100644 --- a/tests/components/test_updater.py +++ b/tests/components/test_updater.py @@ -41,7 +41,7 @@ class TestUpdater(unittest.TestCase): with assert_setup_component(1) as config: setup_component(self.hass, updater.DOMAIN, {updater.DOMAIN: {}}) _dt = datetime.now() + timedelta(hours=1) - assert config['updater'] == {'opt_out': False} + assert config['updater'] == {'reporting': True} for secs in [-1, 0, 1]: fire_time_changed(self.hass, _dt + timedelta(seconds=secs)) @@ -61,7 +61,7 @@ class TestUpdater(unittest.TestCase): assert setup_component( self.hass, updater.DOMAIN, {updater.DOMAIN: {}}) _dt = datetime.now() + timedelta(hours=1) - assert config['updater'] == {'opt_out': False} + assert config['updater'] == {'reporting': True} self.assertIsNone(self.hass.states.get(updater.ENTITY_ID)) @@ -95,7 +95,7 @@ class TestUpdater(unittest.TestCase): with assert_setup_component(1) as config: assert not setup_component( self.hass, updater.DOMAIN, {updater.DOMAIN: {}}) - assert config['updater'] == {'opt_out': False} + assert config['updater'] == {'reporting': True} def test_uuid_function(self): """Test if the uuid function works."""