From e4c73259f7ba1902854b97e58188e530df4983cc Mon Sep 17 00:00:00 2001 From: Aarni Koskela Date: Tue, 22 Nov 2022 09:10:36 +0200 Subject: [PATCH] Configure autoflake for pre-commit (#82414) --- .pre-commit-config.yaml | 7 +++++++ homeassistant/components/zha/core/__init__.py | 3 ++- requirements_test_pre_commit.txt | 1 + tests/components/bluetooth/__init__.py | 1 - .../bluetooth_le_tracker/test_device_tracker.py | 2 -- tests/components/config/test_config_entries.py | 1 - tests/components/fido/test_sensor.py | 2 -- tests/components/history/test_init.py | 1 - tests/components/kodi/util.py | 4 ---- tests/components/lastfm/test_sensor.py | 1 - tests/components/locative/test_init.py | 1 - tests/components/mqtt/test_subscription.py | 2 -- tests/components/ness_alarm/test_init.py | 10 ---------- tests/components/owntracks/test_init.py | 1 - tests/components/tellduslive/test_config_flow.py | 2 +- tests/components/xiaomi_ble/conftest.py | 4 ---- .../custom_components/test_embedded/switch.py | 1 - 17 files changed, 11 insertions(+), 33 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 1676b60a802..583776421e7 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -4,6 +4,13 @@ repos: hooks: - id: pyupgrade args: [--py39-plus] + - repo: https://github.com/PyCQA/autoflake + rev: v1.7.7 + hooks: + - id: autoflake + args: + - --in-place + - --remove-all-unused-imports - repo: https://github.com/psf/black rev: 22.10.0 hooks: diff --git a/homeassistant/components/zha/core/__init__.py b/homeassistant/components/zha/core/__init__.py index a416ff2eebe..755eac3c4ce 100644 --- a/homeassistant/components/zha/core/__init__.py +++ b/homeassistant/components/zha/core/__init__.py @@ -1,5 +1,6 @@ """Core module for Zigbee Home Automation.""" -# flake8: noqa from .device import ZHADevice from .gateway import ZHAGateway + +__all__ = ["ZHADevice", "ZHAGateway"] diff --git a/requirements_test_pre_commit.txt b/requirements_test_pre_commit.txt index ec6edeeea66..2258b8c055c 100644 --- a/requirements_test_pre_commit.txt +++ b/requirements_test_pre_commit.txt @@ -1,5 +1,6 @@ # Automatically generated from .pre-commit-config.yaml by gen_requirements_all.py, do not edit +autoflake==1.7.7 bandit==1.7.4 black==22.10.0 codespell==2.1.0 diff --git a/tests/components/bluetooth/__init__.py b/tests/components/bluetooth/__init__.py index fbe5be2e34d..e4dd2a8bb57 100644 --- a/tests/components/bluetooth/__init__.py +++ b/tests/components/bluetooth/__init__.py @@ -215,7 +215,6 @@ class MockBleakClient(BleakClient): async def disconnect(self, *args, **kwargs): """Mock disconnect.""" - pass async def get_services(self, *args, **kwargs): """Mock get_services.""" diff --git a/tests/components/bluetooth_le_tracker/test_device_tracker.py b/tests/components/bluetooth_le_tracker/test_device_tracker.py index 585c83f20a7..04f8c534cf2 100644 --- a/tests/components/bluetooth_le_tracker/test_device_tracker.py +++ b/tests/components/bluetooth_le_tracker/test_device_tracker.py @@ -31,7 +31,6 @@ class MockBleakClient: def __init__(self, *args, **kwargs): """Mock BleakClient.""" - pass async def __aenter__(self, *args, **kwargs): """Mock BleakClient.__aenter__.""" @@ -39,7 +38,6 @@ class MockBleakClient: async def __aexit__(self, *args, **kwargs): """Mock BleakClient.__aexit__.""" - pass class MockBleakClientTimesOut(MockBleakClient): diff --git a/tests/components/config/test_config_entries.py b/tests/components/config/test_config_entries.py index 29a2395a926..8ebf59323c8 100644 --- a/tests/components/config/test_config_entries.py +++ b/tests/components/config/test_config_entries.py @@ -69,7 +69,6 @@ async def test_get_entries(hass, client, clear_handlers): @callback def async_get_options_flow(config_entry): """Get options flow.""" - pass @classmethod @callback diff --git a/tests/components/fido/test_sensor.py b/tests/components/fido/test_sensor.py index bcece50f6e4..0ab1de94117 100644 --- a/tests/components/fido/test_sensor.py +++ b/tests/components/fido/test_sensor.py @@ -17,7 +17,6 @@ class FidoClientMock: def __init__(self, username, password, timeout=None, httpsession=None): """Fake Fido client init.""" - pass def get_phone_numbers(self): """Return Phone numbers.""" @@ -29,7 +28,6 @@ class FidoClientMock: async def fetch_data(self): """Return fake fetching data.""" - pass class FidoClientMockError(FidoClientMock): diff --git a/tests/components/history/test_init.py b/tests/components/history/test_init.py index 092c589035d..f0c4da26231 100644 --- a/tests/components/history/test_init.py +++ b/tests/components/history/test_init.py @@ -27,7 +27,6 @@ from tests.components.recorder.common import ( def test_setup(): """Test setup method of history.""" # Verification occurs in the fixture - pass def test_get_significant_states(hass_history): diff --git a/tests/components/kodi/util.py b/tests/components/kodi/util.py index 5b8b07583c5..9fb215e2d8a 100644 --- a/tests/components/kodi/util.py +++ b/tests/components/kodi/util.py @@ -68,7 +68,6 @@ class MockConnection: async def connect(self): """Mock connect.""" - pass @property def connected(self): @@ -82,7 +81,6 @@ class MockConnection: async def close(self): """Mock close.""" - pass @property def server(self): @@ -99,7 +97,6 @@ class MockWSConnection: async def connect(self): """Mock connect.""" - pass @property def connected(self): @@ -113,7 +110,6 @@ class MockWSConnection: async def close(self): """Mock close.""" - pass @property def server(self): diff --git a/tests/components/lastfm/test_sensor.py b/tests/components/lastfm/test_sensor.py index cbb37f94dc4..d20b54e738f 100644 --- a/tests/components/lastfm/test_sensor.py +++ b/tests/components/lastfm/test_sensor.py @@ -30,7 +30,6 @@ class MockUser: def get_image(self): """Get mock image.""" - pass def get_recent_tracks(self, limit): """Get mock recent tracks.""" diff --git a/tests/components/locative/test_init.py b/tests/components/locative/test_init.py index 94b94c2ec93..5f608803e4a 100644 --- a/tests/components/locative/test_init.py +++ b/tests/components/locative/test_init.py @@ -18,7 +18,6 @@ from homeassistant.setup import async_setup_component @pytest.fixture(autouse=True) def mock_dev_track(mock_device_tracker_conf): """Mock device tracker config loading.""" - pass @pytest.fixture diff --git a/tests/components/mqtt/test_subscription.py b/tests/components/mqtt/test_subscription.py index 3be66f0aa90..6487e5cb826 100644 --- a/tests/components/mqtt/test_subscription.py +++ b/tests/components/mqtt/test_subscription.py @@ -139,7 +139,6 @@ async def test_qos_encoding_default(hass, mqtt_mock_entry_no_yaml_config, caplog @callback def msg_callback(*args): """Do nothing.""" - pass sub_state = None sub_state = async_prepare_subscribe_topics( @@ -158,7 +157,6 @@ async def test_qos_encoding_custom(hass, mqtt_mock_entry_no_yaml_config, caplog) @callback def msg_callback(*args): """Do nothing.""" - pass sub_state = None sub_state = async_prepare_subscribe_topics( diff --git a/tests/components/ness_alarm/test_init.py b/tests/components/ness_alarm/test_init.py index 507a786c3d1..6a0158674e3 100644 --- a/tests/components/ness_alarm/test_init.py +++ b/tests/components/ness_alarm/test_init.py @@ -210,43 +210,33 @@ class MockClient: async def panic(self, code): """Handle panic.""" - pass async def disarm(self, code): """Handle disarm.""" - pass async def arm_away(self, code): """Handle arm_away.""" - pass async def arm_home(self, code): """Handle arm_home.""" - pass async def aux(self, output_id, state): """Handle auxiliary control.""" - pass async def keepalive(self): """Handle keepalive.""" - pass async def update(self): """Handle update.""" - pass def on_zone_change(self): """Handle on_zone_change.""" - pass def on_state_change(self): """Handle on_state_change.""" - pass async def close(self): """Handle close.""" - pass @pytest.fixture diff --git a/tests/components/owntracks/test_init.py b/tests/components/owntracks/test_init.py index 856d3ece298..f7f978f381e 100644 --- a/tests/components/owntracks/test_init.py +++ b/tests/components/owntracks/test_init.py @@ -35,7 +35,6 @@ LOCATION_MESSAGE = { @pytest.fixture(autouse=True) def mock_dev_track(mock_device_tracker_conf): """Mock device tracker config loading.""" - pass @pytest.fixture diff --git a/tests/components/tellduslive/test_config_flow.py b/tests/components/tellduslive/test_config_flow.py index ba233d04a78..0872060712e 100644 --- a/tests/components/tellduslive/test_config_flow.py +++ b/tests/components/tellduslive/test_config_flow.py @@ -16,7 +16,7 @@ from homeassistant.components.tellduslive import ( from homeassistant.config_entries import SOURCE_DISCOVERY from homeassistant.const import CONF_HOST -from tests.common import MockConfigEntry, mock_coro +from tests.common import MockConfigEntry def init_config_flow(hass, side_effect=None): diff --git a/tests/components/xiaomi_ble/conftest.py b/tests/components/xiaomi_ble/conftest.py index 2997943468f..3d68d78e27e 100644 --- a/tests/components/xiaomi_ble/conftest.py +++ b/tests/components/xiaomi_ble/conftest.py @@ -20,7 +20,6 @@ class MockBleakClient: def __init__(self, *args, **kwargs): """Mock BleakClient.""" - pass async def __aenter__(self, *args, **kwargs): """Mock BleakClient.__aenter__.""" @@ -28,15 +27,12 @@ class MockBleakClient: async def __aexit__(self, *args, **kwargs): """Mock BleakClient.__aexit__.""" - pass async def connect(self, *args, **kwargs): """Mock BleakClient.connect.""" - pass async def disconnect(self, *args, **kwargs): """Mock BleakClient.disconnect.""" - pass class MockBleakClientBattery5(MockBleakClient): diff --git a/tests/testing_config/custom_components/test_embedded/switch.py b/tests/testing_config/custom_components/test_embedded/switch.py index 3abd5b79085..46dac4419a6 100644 --- a/tests/testing_config/custom_components/test_embedded/switch.py +++ b/tests/testing_config/custom_components/test_embedded/switch.py @@ -5,4 +5,3 @@ async def async_setup_platform( hass, config, async_add_entities_callback, discovery_info=None ): """Find and return test switches.""" - pass