diff --git a/homeassistant/components/abode/camera.py b/homeassistant/components/abode/camera.py index afe017bfcc7..326e845b16b 100644 --- a/homeassistant/components/abode/camera.py +++ b/homeassistant/components/abode/camera.py @@ -30,7 +30,7 @@ async def async_setup_entry( data: AbodeSystem = hass.data[DOMAIN] async_add_entities( - AbodeCamera(data, device, TIMELINE.CAPTURE_IMAGE) # pylint: disable=no-member + AbodeCamera(data, device, TIMELINE.CAPTURE_IMAGE) for device in data.abode.get_devices(generic_type=CONST.TYPE_CAMERA) ) diff --git a/homeassistant/components/aten_pe/switch.py b/homeassistant/components/aten_pe/switch.py index 13214b04628..3293a3e7a09 100644 --- a/homeassistant/components/aten_pe/switch.py +++ b/homeassistant/components/aten_pe/switch.py @@ -4,7 +4,7 @@ from __future__ import annotations import logging from typing import Any -from atenpdu import AtenPE, AtenPEError # pylint: disable=import-error +from atenpdu import AtenPE, AtenPEError import voluptuous as vol from homeassistant.components.switch import ( diff --git a/homeassistant/components/avea/light.py b/homeassistant/components/avea/light.py index 5b306b058d3..a33fbfeab79 100644 --- a/homeassistant/components/avea/light.py +++ b/homeassistant/components/avea/light.py @@ -3,7 +3,7 @@ from __future__ import annotations from typing import Any -import avea # pylint: disable=import-error +import avea from homeassistant.components.light import ( ATTR_BRIGHTNESS, diff --git a/homeassistant/components/azure_service_bus/notify.py b/homeassistant/components/azure_service_bus/notify.py index b318c5224df..23235a23dff 100644 --- a/homeassistant/components/azure_service_bus/notify.py +++ b/homeassistant/components/azure_service_bus/notify.py @@ -4,13 +4,13 @@ from __future__ import annotations import json import logging -# pylint: disable-next=import-error, no-name-in-module +# pylint: disable-next=no-name-in-module from azure.servicebus import ServiceBusMessage -# pylint: disable-next=import-error, no-name-in-module +# pylint: disable-next=no-name-in-module from azure.servicebus.aio import ServiceBusClient, ServiceBusSender -# pylint: disable-next=import-error, no-name-in-module +# pylint: disable-next=no-name-in-module from azure.servicebus.exceptions import ( MessagingEntityNotFoundError, ServiceBusConnectionError, diff --git a/homeassistant/components/beewi_smartclim/sensor.py b/homeassistant/components/beewi_smartclim/sensor.py index 0bb3a5bbb69..08f2410ee06 100644 --- a/homeassistant/components/beewi_smartclim/sensor.py +++ b/homeassistant/components/beewi_smartclim/sensor.py @@ -1,7 +1,7 @@ """Platform for beewi_smartclim integration.""" from __future__ import annotations -from beewi_smartclim import BeewiSmartClimPoller # pylint: disable=import-error +from beewi_smartclim import BeewiSmartClimPoller import voluptuous as vol from homeassistant.components.sensor import ( diff --git a/homeassistant/components/bloomsky/binary_sensor.py b/homeassistant/components/bloomsky/binary_sensor.py index 7b59039a89e..b99fdfe0c78 100644 --- a/homeassistant/components/bloomsky/binary_sensor.py +++ b/homeassistant/components/bloomsky/binary_sensor.py @@ -49,7 +49,7 @@ def setup_platform( class BloomSkySensor(BinarySensorEntity): """Representation of a single binary sensor in a BloomSky device.""" - def __init__(self, bs, device, sensor_name): # pylint: disable=invalid-name + def __init__(self, bs, device, sensor_name): """Initialize a BloomSky binary sensor.""" self._bloomsky = bs self._device_id = device["DeviceID"] diff --git a/homeassistant/components/bloomsky/sensor.py b/homeassistant/components/bloomsky/sensor.py index 6cefcdb3346..35c9a40a46a 100644 --- a/homeassistant/components/bloomsky/sensor.py +++ b/homeassistant/components/bloomsky/sensor.py @@ -93,7 +93,7 @@ def setup_platform( class BloomSkySensor(SensorEntity): """Representation of a single sensor in a BloomSky device.""" - def __init__(self, bs, device, sensor_name): # pylint: disable=invalid-name + def __init__(self, bs, device, sensor_name): """Initialize a BloomSky sensor.""" self._bloomsky = bs self._device_id = device["DeviceID"] diff --git a/homeassistant/components/bluetooth/wrappers.py b/homeassistant/components/bluetooth/wrappers.py index 2ae036080f8..3a0abc855b5 100644 --- a/homeassistant/components/bluetooth/wrappers.py +++ b/homeassistant/components/bluetooth/wrappers.py @@ -199,7 +199,7 @@ class HaBleakClientWrapper(BleakClient): when an integration does this. """ - def __init__( # pylint: disable=super-init-not-called, keyword-arg-before-vararg + def __init__( # pylint: disable=super-init-not-called self, address_or_ble_device: str | BLEDevice, disconnected_callback: Callable[[BleakClient], None] | None = None, diff --git a/homeassistant/components/bluetooth_tracker/device_tracker.py b/homeassistant/components/bluetooth_tracker/device_tracker.py index f4fc6a8df08..4bfbe72d8b5 100644 --- a/homeassistant/components/bluetooth_tracker/device_tracker.py +++ b/homeassistant/components/bluetooth_tracker/device_tracker.py @@ -7,7 +7,7 @@ from datetime import datetime, timedelta import logging from typing import Final -import bluetooth # pylint: disable=import-error +import bluetooth from bt_proximity import BluetoothRSSI import voluptuous as vol diff --git a/homeassistant/components/browser/__init__.py b/homeassistant/components/browser/__init__.py index b01f04fa140..9dc3e1fe66a 100644 --- a/homeassistant/components/browser/__init__.py +++ b/homeassistant/components/browser/__init__.py @@ -16,8 +16,7 @@ SERVICE_BROWSE_URL = "browse_url" SERVICE_BROWSE_URL_SCHEMA = vol.Schema( { - # pylint: disable-next=no-value-for-parameter - vol.Required(ATTR_URL, default=ATTR_URL_DEFAULT): vol.Url() + vol.Required(ATTR_URL, default=ATTR_URL_DEFAULT): vol.Url(), } ) diff --git a/homeassistant/components/caldav/calendar.py b/homeassistant/components/caldav/calendar.py index 57bf8e81e03..f30f79f7275 100644 --- a/homeassistant/components/caldav/calendar.py +++ b/homeassistant/components/caldav/calendar.py @@ -43,7 +43,6 @@ OFFSET = "!!" PLATFORM_SCHEMA = PLATFORM_SCHEMA.extend( { - # pylint: disable=no-value-for-parameter vol.Required(CONF_URL): vol.Url(), vol.Optional(CONF_CALENDARS, default=[]): vol.All(cv.ensure_list, [cv.string]), vol.Inclusive(CONF_USERNAME, "authentication"): cv.string, diff --git a/homeassistant/components/color_extractor/__init__.py b/homeassistant/components/color_extractor/__init__.py index 0e27f396c6d..fb04ebb76a4 100644 --- a/homeassistant/components/color_extractor/__init__.py +++ b/homeassistant/components/color_extractor/__init__.py @@ -81,7 +81,7 @@ async def async_setup(hass: HomeAssistant, hass_config: ConfigType) -> bool: except UnidentifiedImageError as ex: _LOGGER.error( "Bad image from %s '%s' provided, are you sure it's an image? %s", - image_type, # pylint: disable=used-before-assignment + image_type, image_reference, ex, ) diff --git a/homeassistant/components/config/config_entries.py b/homeassistant/components/config/config_entries.py index 9691994512c..77e2548d424 100644 --- a/homeassistant/components/config/config_entries.py +++ b/homeassistant/components/config/config_entries.py @@ -143,7 +143,6 @@ class ConfigManagerFlowIndexView(FlowManagerIndexView): ) async def post(self, request): """Handle a POST request.""" - # pylint: disable=no-value-for-parameter try: return await super().post(request) except DependencyError as exc: @@ -175,7 +174,6 @@ class ConfigManagerFlowResourceView(FlowManagerResourceView): ) async def post(self, request, flow_id): """Handle a POST request.""" - # pylint: disable=no-value-for-parameter return await super().post(request, flow_id) def _prepare_result_json(self, result): @@ -212,7 +210,6 @@ class OptionManagerFlowIndexView(FlowManagerIndexView): handler in request is entry_id. """ - # pylint: disable=no-value-for-parameter return await super().post(request) @@ -234,7 +231,6 @@ class OptionManagerFlowResourceView(FlowManagerResourceView): ) async def post(self, request, flow_id): """Handle a POST request.""" - # pylint: disable=no-value-for-parameter return await super().post(request, flow_id) diff --git a/homeassistant/components/conversation/default_agent.py b/homeassistant/components/conversation/default_agent.py index 04aafc8a99d..09245fde8dc 100644 --- a/homeassistant/components/conversation/default_agent.py +++ b/homeassistant/components/conversation/default_agent.py @@ -54,9 +54,7 @@ _DEFAULT_ERROR_TEXT = "Sorry, I couldn't understand that" _ENTITY_REGISTRY_UPDATE_FIELDS = ["aliases", "name", "original_name"] REGEX_TYPE = type(re.compile("")) -TRIGGER_CALLBACK_TYPE = Callable[ # pylint: disable=invalid-name - [str, RecognizeResult], Awaitable[str | None] -] +TRIGGER_CALLBACK_TYPE = Callable[[str, RecognizeResult], Awaitable[str | None]] def json_load(fp: IO[str]) -> JsonObjectType: diff --git a/homeassistant/components/decora/light.py b/homeassistant/components/decora/light.py index b46732178b8..d060b69c3f6 100644 --- a/homeassistant/components/decora/light.py +++ b/homeassistant/components/decora/light.py @@ -8,8 +8,8 @@ import logging import time from typing import TYPE_CHECKING, Any, Concatenate, ParamSpec, TypeVar -from bluepy.btle import BTLEException # pylint: disable=import-error -import decora # pylint: disable=import-error +from bluepy.btle import BTLEException +import decora import voluptuous as vol from homeassistant import util diff --git a/homeassistant/components/decora_wifi/light.py b/homeassistant/components/decora_wifi/light.py index c103636563c..a9d43736743 100644 --- a/homeassistant/components/decora_wifi/light.py +++ b/homeassistant/components/decora_wifi/light.py @@ -4,7 +4,6 @@ from __future__ import annotations import logging from typing import Any -# pylint: disable=import-error from decora_wifi import DecoraWiFiSession from decora_wifi.models.person import Person from decora_wifi.models.residence import Residence diff --git a/homeassistant/components/demo/light.py b/homeassistant/components/demo/light.py index 7009df75caa..d8451bdd683 100644 --- a/homeassistant/components/demo/light.py +++ b/homeassistant/components/demo/light.py @@ -106,7 +106,7 @@ class DemoLight(LightEntity): state: bool, available: bool = False, brightness: int = 180, - ct: int | None = None, # pylint: disable=invalid-name + ct: int | None = None, effect_list: list[str] | None = None, effect: str | None = None, hs_color: tuple[int, int] | None = None, diff --git a/homeassistant/components/dhcp/__init__.py b/homeassistant/components/dhcp/__init__.py index b3cfd1b65f2..29b25d0781b 100644 --- a/homeassistant/components/dhcp/__init__.py +++ b/homeassistant/components/dhcp/__init__.py @@ -415,9 +415,7 @@ class DHCPWatcher(WatcherBase): """Start watching for dhcp packets.""" # Local import because importing from scapy has side effects such as opening # sockets - from scapy import ( # pylint: disable=import-outside-toplevel,unused-import # noqa: F401 - arch, - ) + from scapy import arch # pylint: disable=import-outside-toplevel # noqa: F401 from scapy.layers.dhcp import DHCP # pylint: disable=import-outside-toplevel from scapy.layers.inet import IP # pylint: disable=import-outside-toplevel from scapy.layers.l2 import Ether # pylint: disable=import-outside-toplevel diff --git a/homeassistant/components/digital_ocean/binary_sensor.py b/homeassistant/components/digital_ocean/binary_sensor.py index 59c6f7961c2..e2bd09ba15e 100644 --- a/homeassistant/components/digital_ocean/binary_sensor.py +++ b/homeassistant/components/digital_ocean/binary_sensor.py @@ -65,7 +65,7 @@ class DigitalOceanBinarySensor(BinarySensorEntity): _attr_attribution = ATTRIBUTION - def __init__(self, do, droplet_id): # pylint: disable=invalid-name + def __init__(self, do, droplet_id): """Initialize a new Digital Ocean sensor.""" self._digital_ocean = do self._droplet_id = droplet_id diff --git a/homeassistant/components/digital_ocean/switch.py b/homeassistant/components/digital_ocean/switch.py index 2791d83d6bc..b226dbab0a9 100644 --- a/homeassistant/components/digital_ocean/switch.py +++ b/homeassistant/components/digital_ocean/switch.py @@ -63,7 +63,7 @@ class DigitalOceanSwitch(SwitchEntity): _attr_attribution = ATTRIBUTION - def __init__(self, do, droplet_id): # pylint: disable=invalid-name + def __init__(self, do, droplet_id): """Initialize a new Digital Ocean sensor.""" self._digital_ocean = do self._droplet_id = droplet_id diff --git a/homeassistant/components/dlib_face_detect/image_processing.py b/homeassistant/components/dlib_face_detect/image_processing.py index dcae1c1eb40..42031b28844 100644 --- a/homeassistant/components/dlib_face_detect/image_processing.py +++ b/homeassistant/components/dlib_face_detect/image_processing.py @@ -3,7 +3,7 @@ from __future__ import annotations import io -import face_recognition # pylint: disable=import-error +import face_recognition from homeassistant.components.image_processing import ImageProcessingFaceEntity from homeassistant.const import ATTR_LOCATION, CONF_ENTITY_ID, CONF_NAME, CONF_SOURCE @@ -11,7 +11,6 @@ from homeassistant.core import HomeAssistant, split_entity_id from homeassistant.helpers.entity_platform import AddEntitiesCallback from homeassistant.helpers.typing import ConfigType, DiscoveryInfoType -# pylint: disable=unused-import from homeassistant.components.image_processing import ( # noqa: F401, isort:skip PLATFORM_SCHEMA, ) diff --git a/homeassistant/components/dlib_face_identify/image_processing.py b/homeassistant/components/dlib_face_identify/image_processing.py index 373f2c2b928..e6aaa6848d0 100644 --- a/homeassistant/components/dlib_face_identify/image_processing.py +++ b/homeassistant/components/dlib_face_identify/image_processing.py @@ -4,7 +4,6 @@ from __future__ import annotations import io import logging -# pylint: disable=import-error import face_recognition import voluptuous as vol diff --git a/homeassistant/components/dsmr/sensor.py b/homeassistant/components/dsmr/sensor.py index 3d198e38f36..e4f9d0e9ab9 100644 --- a/homeassistant/components/dsmr/sensor.py +++ b/homeassistant/components/dsmr/sensor.py @@ -509,7 +509,7 @@ async def async_setup_entry( if stop_listener and ( hass.state == CoreState.not_running or hass.is_running ): - stop_listener() # pylint: disable=not-callable + stop_listener() if transport: transport.close() diff --git a/homeassistant/components/duckdns/__init__.py b/homeassistant/components/duckdns/__init__.py index 278c3c989db..d477bd41a26 100644 --- a/homeassistant/components/duckdns/__init__.py +++ b/homeassistant/components/duckdns/__init__.py @@ -138,6 +138,6 @@ def async_track_time_interval_backoff( def remove_listener() -> None: """Remove interval listener.""" if remove: - remove() # pylint: disable=not-callable + remove() return remove_listener diff --git a/tests/components/alexa/test_flash_briefings.py b/tests/components/alexa/test_flash_briefings.py index 0a4acda79f5..c6c2b3cc421 100644 --- a/tests/components/alexa/test_flash_briefings.py +++ b/tests/components/alexa/test_flash_briefings.py @@ -14,7 +14,6 @@ SESSION_ID = "amzn1.echo-api.session.0000000-0000-0000-0000-00000000000" APPLICATION_ID = "amzn1.echo-sdk-ams.app.000000-d0ed-0000-ad00-000000d00ebe" REQUEST_ID = "amzn1.echo-api.request.0000000-0000-0000-0000-00000000000" -# pylint: disable=invalid-name calls = [] NPR_NEWS_MP3_URL = "https://pd.npr.org/anon.npr-mp3/npr/news/newscast.mp3" diff --git a/tests/components/alexa/test_intent.py b/tests/components/alexa/test_intent.py index 03546c0ed22..c63825b3c12 100644 --- a/tests/components/alexa/test_intent.py +++ b/tests/components/alexa/test_intent.py @@ -19,7 +19,6 @@ REQUEST_ID = "amzn1.echo-api.request.0000000-0000-0000-0000-00000000000" AUTHORITY_ID = "amzn1.er-authority.000000-d0ed-0000-ad00-000000d00ebe.ZODIAC" BUILTIN_AUTH_ID = "amzn1.er-authority.000000-d0ed-0000-ad00-000000d00ebe.TEST" -# pylint: disable=invalid-name calls = [] NPR_NEWS_MP3_URL = "https://pd.npr.org/anon.npr-mp3/npr/news/newscast.mp3" diff --git a/tests/components/api/test_init.py b/tests/components/api/test_init.py index 5ba9d60996b..116529b02a4 100644 --- a/tests/components/api/test_init.py +++ b/tests/components/api/test_init.py @@ -81,7 +81,6 @@ async def test_api_state_change( assert hass.states.get("test.test").state == "debug_state_change2" -# pylint: disable=invalid-name async def test_api_state_change_of_non_existing_entity( hass: HomeAssistant, mock_api_client: TestClient ) -> None: @@ -97,7 +96,6 @@ async def test_api_state_change_of_non_existing_entity( assert hass.states.get("test_entity.that_does_not_exist").state == new_state -# pylint: disable=invalid-name async def test_api_state_change_with_bad_data( hass: HomeAssistant, mock_api_client: TestClient ) -> None: @@ -109,7 +107,6 @@ async def test_api_state_change_with_bad_data( assert resp.status == HTTPStatus.BAD_REQUEST -# pylint: disable=invalid-name async def test_api_state_change_to_zero_value( hass: HomeAssistant, mock_api_client: TestClient ) -> None: @@ -127,7 +124,6 @@ async def test_api_state_change_to_zero_value( assert resp.status == HTTPStatus.OK -# pylint: disable=invalid-name async def test_api_state_change_push( hass: HomeAssistant, mock_api_client: TestClient ) -> None: @@ -154,7 +150,6 @@ async def test_api_state_change_push( assert len(events) == 1 -# pylint: disable=invalid-name async def test_api_fire_event_with_no_data( hass: HomeAssistant, mock_api_client: TestClient ) -> None: @@ -174,7 +169,6 @@ async def test_api_fire_event_with_no_data( assert len(test_value) == 1 -# pylint: disable=invalid-name async def test_api_fire_event_with_data( hass: HomeAssistant, mock_api_client: TestClient ) -> None: @@ -199,7 +193,6 @@ async def test_api_fire_event_with_data( assert len(test_value) == 1 -# pylint: disable=invalid-name async def test_api_fire_event_with_invalid_json( hass: HomeAssistant, mock_api_client: TestClient ) -> None: diff --git a/tests/components/arcam_fmj/test_media_player.py b/tests/components/arcam_fmj/test_media_player.py index b9c86140cb9..9287e8dbc18 100644 --- a/tests/components/arcam_fmj/test_media_player.py +++ b/tests/components/arcam_fmj/test_media_player.py @@ -228,9 +228,9 @@ async def test_sound_mode_list(player, state, modes, modes_enum) -> None: async def test_is_volume_muted(player, state) -> None: """Test muted.""" state.get_mute.return_value = True - assert player.is_volume_muted is True # pylint: disable=singleton-comparison + assert player.is_volume_muted is True state.get_mute.return_value = False - assert player.is_volume_muted is False # pylint: disable=singleton-comparison + assert player.is_volume_muted is False state.get_mute.return_value = None assert player.is_volume_muted is None diff --git a/tests/components/assist_pipeline/test_select.py b/tests/components/assist_pipeline/test_select.py index 1868d9b005e..1419eb58750 100644 --- a/tests/components/assist_pipeline/test_select.py +++ b/tests/components/assist_pipeline/test_select.py @@ -26,7 +26,6 @@ from tests.common import MockConfigEntry, MockPlatform, mock_entity_platform class SelectPlatform(MockPlatform): """Fake select platform.""" - # pylint: disable=method-hidden async def async_setup_entry( self, hass: HomeAssistant, diff --git a/tests/components/balboa/conftest.py b/tests/components/balboa/conftest.py index 04447d0b3cc..e5da4582454 100644 --- a/tests/components/balboa/conftest.py +++ b/tests/components/balboa/conftest.py @@ -29,7 +29,7 @@ def client_fixture() -> Generator[MagicMock, None, None]: client = mock_balboa.return_value callback: list[Callable] = [] - def on(_, _callback: Callable): # pylint: disable=invalid-name + def on(_, _callback: Callable): callback.append(_callback) return lambda: None diff --git a/tests/components/cast/test_media_player.py b/tests/components/cast/test_media_player.py index 6f7a13b47af..3d9feb3e43c 100644 --- a/tests/components/cast/test_media_player.py +++ b/tests/components/cast/test_media_player.py @@ -42,7 +42,6 @@ from tests.components.media_player import common from tests.test_util.aiohttp import AiohttpClientMocker from tests.typing import WebSocketGenerator -# pylint: disable=invalid-name FakeUUID = UUID("57355bce-9364-4aa6-ac1e-eb849dccf9e2") FakeUUID2 = UUID("57355bce-9364-4aa6-ac1e-eb849dccf9e4") FakeGroupUUID = UUID("57355bce-9364-4aa6-ac1e-eb849dccf9e3") diff --git a/tests/components/dhcp/test_init.py b/tests/components/dhcp/test_init.py index 0754febfc76..076138080cc 100644 --- a/tests/components/dhcp/test_init.py +++ b/tests/components/dhcp/test_init.py @@ -4,7 +4,7 @@ import threading from unittest.mock import MagicMock, patch import pytest -from scapy import arch # pylint: disable=unused-import # noqa: F401 +from scapy import arch # noqa: F401 from scapy.error import Scapy_Exception from scapy.layers.dhcp import DHCP from scapy.layers.l2 import Ether