From 73ed2ab16407f7817c3c76b099b8b7614d528b7a Mon Sep 17 00:00:00 2001 From: Paulus Schoutsen Date: Fri, 2 Nov 2018 20:09:12 +0100 Subject: [PATCH 1/3] bump frontend to 20181026.4 --- homeassistant/components/frontend/__init__.py | 2 +- requirements_all.txt | 2 +- requirements_test_all.txt | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/homeassistant/components/frontend/__init__.py b/homeassistant/components/frontend/__init__.py index c669e804d9d..17fa7a249ee 100644 --- a/homeassistant/components/frontend/__init__.py +++ b/homeassistant/components/frontend/__init__.py @@ -24,7 +24,7 @@ from homeassistant.core import callback from homeassistant.helpers.translation import async_get_translations from homeassistant.loader import bind_hass -REQUIREMENTS = ['home-assistant-frontend==20181026.3'] +REQUIREMENTS = ['home-assistant-frontend==20181026.4'] DOMAIN = 'frontend' DEPENDENCIES = ['api', 'websocket_api', 'http', 'system_log', diff --git a/requirements_all.txt b/requirements_all.txt index 018072210a4..2ffd2ad6b21 100644 --- a/requirements_all.txt +++ b/requirements_all.txt @@ -466,7 +466,7 @@ hole==0.3.0 holidays==0.9.8 # homeassistant.components.frontend -home-assistant-frontend==20181026.3 +home-assistant-frontend==20181026.4 # homeassistant.components.homekit_controller # homekit==0.10 diff --git a/requirements_test_all.txt b/requirements_test_all.txt index 096884db304..e7d00f75799 100644 --- a/requirements_test_all.txt +++ b/requirements_test_all.txt @@ -97,7 +97,7 @@ hdate==0.6.5 holidays==0.9.8 # homeassistant.components.frontend -home-assistant-frontend==20181026.3 +home-assistant-frontend==20181026.4 # homeassistant.components.homematicip_cloud homematicip==0.9.8 From 255607f3a5325b93da09e099815aeeab6a1cf718 Mon Sep 17 00:00:00 2001 From: jjlawren Date: Sat, 3 Nov 2018 06:36:22 -0500 Subject: [PATCH 2/3] Remove FFmpeg input tests (#18131) * Remove FFmpeg input tests * Not needed here * Removing tests for removed functionality * Minor lint * Fix tests to reflect removed config option * Remove async service registration by request * More lint * Unused imports * Make it a non-breaking change * Update ffmpeg.py --- .../components/binary_sensor/ffmpeg_motion.py | 4 - .../components/binary_sensor/ffmpeg_noise.py | 4 - homeassistant/components/camera/ffmpeg.py | 2 - homeassistant/components/camera/onvif.py | 7 +- homeassistant/components/ffmpeg.py | 31 +------ tests/components/binary_sensor/test_ffmpeg.py | 6 -- tests/components/test_ffmpeg.py | 88 +++---------------- 7 files changed, 15 insertions(+), 127 deletions(-) diff --git a/homeassistant/components/binary_sensor/ffmpeg_motion.py b/homeassistant/components/binary_sensor/ffmpeg_motion.py index df811d47e56..d0e597e13c0 100644 --- a/homeassistant/components/binary_sensor/ffmpeg_motion.py +++ b/homeassistant/components/binary_sensor/ffmpeg_motion.py @@ -49,10 +49,6 @@ async def async_setup_platform(hass, config, async_add_entities, discovery_info=None): """Set up the FFmpeg binary motion sensor.""" manager = hass.data[DATA_FFMPEG] - - if not await manager.async_run_test(config.get(CONF_INPUT)): - return - entity = FFmpegMotion(hass, manager, config) async_add_entities([entity]) diff --git a/homeassistant/components/binary_sensor/ffmpeg_noise.py b/homeassistant/components/binary_sensor/ffmpeg_noise.py index a2625c3de8d..3c2397d692b 100644 --- a/homeassistant/components/binary_sensor/ffmpeg_noise.py +++ b/homeassistant/components/binary_sensor/ffmpeg_noise.py @@ -46,10 +46,6 @@ async def async_setup_platform(hass, config, async_add_entities, discovery_info=None): """Set up the FFmpeg noise binary sensor.""" manager = hass.data[DATA_FFMPEG] - - if not await manager.async_run_test(config.get(CONF_INPUT)): - return - entity = FFmpegNoise(hass, manager, config) async_add_entities([entity]) diff --git a/homeassistant/components/camera/ffmpeg.py b/homeassistant/components/camera/ffmpeg.py index dfbcc4d70bc..6bd68b05bb5 100644 --- a/homeassistant/components/camera/ffmpeg.py +++ b/homeassistant/components/camera/ffmpeg.py @@ -32,8 +32,6 @@ PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend({ async def async_setup_platform(hass, config, async_add_entities, discovery_info=None): """Set up a FFmpeg camera.""" - if not await hass.data[DATA_FFMPEG].async_run_test(config.get(CONF_INPUT)): - return async_add_entities([FFmpegCamera(hass, config)]) diff --git a/homeassistant/components/camera/onvif.py b/homeassistant/components/camera/onvif.py index 2576dfa7f92..722577fed39 100644 --- a/homeassistant/components/camera/onvif.py +++ b/homeassistant/components/camera/onvif.py @@ -74,9 +74,6 @@ SERVICE_PTZ_SCHEMA = vol.Schema({ def setup_platform(hass, config, add_entities, discovery_info=None): """Set up a ONVIF camera.""" - if not hass.data[DATA_FFMPEG].async_run_test(config.get(CONF_HOST)): - return - def handle_ptz(service): """Handle PTZ service call.""" pan = service.data.get(ATTR_PAN, None) @@ -93,8 +90,8 @@ def setup_platform(hass, config, add_entities, discovery_info=None): for camera in target_cameras: camera.perform_ptz(pan, tilt, zoom) - hass.services.async_register(DOMAIN, SERVICE_PTZ, handle_ptz, - schema=SERVICE_PTZ_SCHEMA) + hass.services.register(DOMAIN, SERVICE_PTZ, handle_ptz, + schema=SERVICE_PTZ_SCHEMA) add_entities([ONVIFHassCamera(hass, config)]) diff --git a/homeassistant/components/ffmpeg.py b/homeassistant/components/ffmpeg.py index f28dbd52336..791f6d29175 100644 --- a/homeassistant/components/ffmpeg.py +++ b/homeassistant/components/ffmpeg.py @@ -40,12 +40,11 @@ CONF_OUTPUT = 'output' CONF_RUN_TEST = 'run_test' DEFAULT_BINARY = 'ffmpeg' -DEFAULT_RUN_TEST = True CONFIG_SCHEMA = vol.Schema({ DOMAIN: vol.Schema({ vol.Optional(CONF_FFMPEG_BIN, default=DEFAULT_BINARY): cv.string, - vol.Optional(CONF_RUN_TEST, default=DEFAULT_RUN_TEST): cv.boolean, + vol.Optional(CONF_RUN_TEST): cv.boolean, }), }, extra=vol.ALLOW_EXTRA) @@ -60,8 +59,7 @@ async def async_setup(hass, config): manager = FFmpegManager( hass, - conf.get(CONF_FFMPEG_BIN, DEFAULT_BINARY), - conf.get(CONF_RUN_TEST, DEFAULT_RUN_TEST) + conf.get(CONF_FFMPEG_BIN, DEFAULT_BINARY) ) # Register service @@ -95,40 +93,17 @@ async def async_setup(hass, config): class FFmpegManager: """Helper for ha-ffmpeg.""" - def __init__(self, hass, ffmpeg_bin, run_test): + def __init__(self, hass, ffmpeg_bin): """Initialize helper.""" self.hass = hass self._cache = {} self._bin = ffmpeg_bin - self._run_test = run_test @property def binary(self): """Return ffmpeg binary from config.""" return self._bin - async def async_run_test(self, input_source): - """Run test on this input. TRUE is deactivate or run correct. - - This method must be run in the event loop. - """ - from haffmpeg import Test - - if self._run_test: - # if in cache - if input_source in self._cache: - return self._cache[input_source] - - # run test - ffmpeg_test = Test(self.binary, loop=self.hass.loop) - success = await ffmpeg_test.run_test(input_source) - if not success: - _LOGGER.error("FFmpeg '%s' test fails!", input_source) - self._cache[input_source] = False - return False - self._cache[input_source] = True - return True - class FFmpegBase(Entity): """Interface object for FFmpeg.""" diff --git a/tests/components/binary_sensor/test_ffmpeg.py b/tests/components/binary_sensor/test_ffmpeg.py index 4e6629c0afd..2c17207af32 100644 --- a/tests/components/binary_sensor/test_ffmpeg.py +++ b/tests/components/binary_sensor/test_ffmpeg.py @@ -15,9 +15,6 @@ class TestFFmpegNoiseSetup: self.hass = get_test_home_assistant() self.config = { - 'ffmpeg': { - 'run_test': False, - }, 'binary_sensor': { 'platform': 'ffmpeg_noise', 'input': 'testinputvideo', @@ -80,9 +77,6 @@ class TestFFmpegMotionSetup: self.hass = get_test_home_assistant() self.config = { - 'ffmpeg': { - 'run_test': False, - }, 'binary_sensor': { 'platform': 'ffmpeg_motion', 'input': 'testinputvideo', diff --git a/tests/components/test_ffmpeg.py b/tests/components/test_ffmpeg.py index 774bb471f57..f11611ece75 100644 --- a/tests/components/test_ffmpeg.py +++ b/tests/components/test_ffmpeg.py @@ -1,6 +1,6 @@ """The tests for Home Assistant ffmpeg.""" import asyncio -from unittest.mock import patch, MagicMock +from unittest.mock import MagicMock import homeassistant.components.ffmpeg as ffmpeg from homeassistant.components.ffmpeg import ( @@ -10,7 +10,7 @@ from homeassistant.core import callback from homeassistant.setup import setup_component, async_setup_component from tests.common import ( - get_test_home_assistant, assert_setup_component, mock_coro) + get_test_home_assistant, assert_setup_component) @callback @@ -85,14 +85,14 @@ class TestFFmpegSetup: def test_setup_component(self): """Set up ffmpeg component.""" - with assert_setup_component(2): + with assert_setup_component(1): setup_component(self.hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) assert self.hass.data[ffmpeg.DATA_FFMPEG].binary == 'ffmpeg' def test_setup_component_test_service(self): """Set up ffmpeg component test services.""" - with assert_setup_component(2): + with assert_setup_component(1): setup_component(self.hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) assert self.hass.services.has_service(ffmpeg.DOMAIN, 'start') @@ -103,7 +103,7 @@ class TestFFmpegSetup: @asyncio.coroutine def test_setup_component_test_register(hass): """Set up ffmpeg component test register.""" - with assert_setup_component(2): + with assert_setup_component(1): yield from async_setup_component( hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) @@ -118,7 +118,7 @@ def test_setup_component_test_register(hass): @asyncio.coroutine def test_setup_component_test_register_no_startup(hass): """Set up ffmpeg component test register without startup.""" - with assert_setup_component(2): + with assert_setup_component(1): yield from async_setup_component( hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) @@ -133,7 +133,7 @@ def test_setup_component_test_register_no_startup(hass): @asyncio.coroutine def test_setup_component_test_service_start(hass): """Set up ffmpeg component test service start.""" - with assert_setup_component(2): + with assert_setup_component(1): yield from async_setup_component( hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) @@ -149,7 +149,7 @@ def test_setup_component_test_service_start(hass): @asyncio.coroutine def test_setup_component_test_service_stop(hass): """Set up ffmpeg component test service stop.""" - with assert_setup_component(2): + with assert_setup_component(1): yield from async_setup_component( hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) @@ -165,7 +165,7 @@ def test_setup_component_test_service_stop(hass): @asyncio.coroutine def test_setup_component_test_service_restart(hass): """Set up ffmpeg component test service restart.""" - with assert_setup_component(2): + with assert_setup_component(1): yield from async_setup_component( hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) @@ -182,7 +182,7 @@ def test_setup_component_test_service_restart(hass): @asyncio.coroutine def test_setup_component_test_service_start_with_entity(hass): """Set up ffmpeg component test service start.""" - with assert_setup_component(2): + with assert_setup_component(1): yield from async_setup_component( hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) @@ -194,71 +194,3 @@ def test_setup_component_test_service_start_with_entity(hass): assert ffmpeg_dev.called_start assert ffmpeg_dev.called_entities == ['test.ffmpeg_device'] - - -@asyncio.coroutine -def test_setup_component_test_run_test_false(hass): - """Set up ffmpeg component test run_test false.""" - with assert_setup_component(2): - yield from async_setup_component( - hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: { - 'run_test': False, - }}) - - manager = hass.data[ffmpeg.DATA_FFMPEG] - with patch('haffmpeg.Test.run_test', return_value=mock_coro(False)): - yield from manager.async_run_test("blabalblabla") - - assert len(manager._cache) == 0 - - -@asyncio.coroutine -def test_setup_component_test_run_test(hass): - """Set up ffmpeg component test run_test.""" - with assert_setup_component(2): - yield from async_setup_component( - hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) - - manager = hass.data[ffmpeg.DATA_FFMPEG] - - with patch('haffmpeg.Test.run_test', return_value=mock_coro(True)) \ - as mock_test: - yield from manager.async_run_test("blabalblabla") - - assert mock_test.called - assert mock_test.call_count == 1 - assert len(manager._cache) == 1 - assert manager._cache['blabalblabla'] - - yield from manager.async_run_test("blabalblabla") - - assert mock_test.called - assert mock_test.call_count == 1 - assert len(manager._cache) == 1 - assert manager._cache['blabalblabla'] - - -@asyncio.coroutine -def test_setup_component_test_run_test_test_fail(hass): - """Set up ffmpeg component test run_test.""" - with assert_setup_component(2): - yield from async_setup_component( - hass, ffmpeg.DOMAIN, {ffmpeg.DOMAIN: {}}) - - manager = hass.data[ffmpeg.DATA_FFMPEG] - - with patch('haffmpeg.Test.run_test', return_value=mock_coro(False)) \ - as mock_test: - yield from manager.async_run_test("blabalblabla") - - assert mock_test.called - assert mock_test.call_count == 1 - assert len(manager._cache) == 1 - assert not manager._cache['blabalblabla'] - - yield from manager.async_run_test("blabalblabla") - - assert mock_test.called - assert mock_test.call_count == 1 - assert len(manager._cache) == 1 - assert not manager._cache['blabalblabla'] From 6d3167fcd461d5857ac0ccc87d99f1f845cc987a Mon Sep 17 00:00:00 2001 From: Paulus Schoutsen Date: Sat, 3 Nov 2018 12:36:51 +0100 Subject: [PATCH 3/3] Bumped version to 0.81.5 --- homeassistant/const.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/homeassistant/const.py b/homeassistant/const.py index 8133916a660..66153dcaf46 100644 --- a/homeassistant/const.py +++ b/homeassistant/const.py @@ -2,7 +2,7 @@ """Constants used by Home Assistant components.""" MAJOR_VERSION = 0 MINOR_VERSION = 81 -PATCH_VERSION = '4' +PATCH_VERSION = '5' __short_version__ = '{}.{}'.format(MAJOR_VERSION, MINOR_VERSION) __version__ = '{}.{}'.format(__short_version__, PATCH_VERSION) REQUIRED_PYTHON_VER = (3, 5, 3)