Ignore unnecessary-lambda warnings in tests (#119564)

This commit is contained in:
epenet 2024-06-13 16:58:41 +02:00 committed by GitHub
parent bb2883a5a8
commit ca8d3e0c83
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
8 changed files with 71 additions and 13 deletions

View File

@ -818,6 +818,7 @@ class MockModule:
if setup: if setup:
# We run this in executor, wrap it in function # We run this in executor, wrap it in function
# pylint: disable-next=unnecessary-lambda
self.setup = lambda *args: setup(*args) self.setup = lambda *args: setup(*args)
if async_setup is not None: if async_setup is not None:
@ -875,6 +876,7 @@ class MockPlatform:
if setup_platform is not None: if setup_platform is not None:
# We run this in executor, wrap it in function # We run this in executor, wrap it in function
# pylint: disable-next=unnecessary-lambda
self.setup_platform = lambda *args: setup_platform(*args) self.setup_platform = lambda *args: setup_platform(*args)
if async_setup_platform is not None: if async_setup_platform is not None:

View File

@ -1012,7 +1012,10 @@ async def test_template_triggers(hass: HomeAssistant) -> None:
events = [] events = []
async_track_state_change_event( async_track_state_change_event(
hass, "binary_sensor.test_binary", callback(lambda event: events.append(event)) hass,
"binary_sensor.test_binary",
# pylint: disable-next=unnecessary-lambda
callback(lambda event: events.append(event)),
) )
context = Context() context = Context()
@ -1051,7 +1054,10 @@ async def test_state_triggers(hass: HomeAssistant) -> None:
events = [] events = []
async_track_state_change_event( async_track_state_change_event(
hass, "binary_sensor.test_binary", callback(lambda event: events.append(event)) hass,
"binary_sensor.test_binary",
# pylint: disable-next=unnecessary-lambda
callback(lambda event: events.append(event)),
) )
context = Context() context = Context()

View File

@ -134,6 +134,7 @@ async def test_update_with_progress(hass: HomeAssistant) -> None:
async_track_state_change_event( async_track_state_change_event(
hass, hass,
"update.demo_update_with_progress", "update.demo_update_with_progress",
# pylint: disable-next=unnecessary-lambda
callback(lambda event: events.append(event)), callback(lambda event: events.append(event)),
) )
@ -170,6 +171,7 @@ async def test_update_with_progress_raising(hass: HomeAssistant) -> None:
events = [] events = []
async_track_state_change_event( async_track_state_change_event(
hass, hass,
# pylint: disable-next=unnecessary-lambda
"update.demo_update_with_progress", "update.demo_update_with_progress",
callback(lambda event: events.append(event)), callback(lambda event: events.append(event)),
) )

View File

@ -362,6 +362,7 @@ async def test_call_rate_delay_throttle_enabled(hass: HomeAssistant) -> None:
delay = 5.0 delay = 5.0
limit = pilight.CallRateDelayThrottle(hass, delay) limit = pilight.CallRateDelayThrottle(hass, delay)
# pylint: disable-next=unnecessary-lambda
action = limit.limited(lambda x: runs.append(x)) action = limit.limited(lambda x: runs.append(x))
for i in range(3): for i in range(3):
@ -385,6 +386,7 @@ def test_call_rate_delay_throttle_disabled(hass: HomeAssistant) -> None:
runs = [] runs = []
limit = pilight.CallRateDelayThrottle(hass, 0.0) limit = pilight.CallRateDelayThrottle(hass, 0.0)
# pylint: disable-next=unnecessary-lambda
action = limit.limited(lambda x: runs.append(x)) action = limit.limited(lambda x: runs.append(x))
for i in range(3): for i in range(3):

View File

@ -27,6 +27,7 @@ def create_mock_device() -> AsyncMock:
device.get_device_info.return_value = DEVICE_INFO device.get_device_info.return_value = DEVICE_INFO
device.get_instantaneous_demand.return_value = DEMAND device.get_instantaneous_demand.return_value = DEMAND
device.get_meter_list.return_value = METER_LIST device.get_meter_list.return_value = METER_LIST
# pylint: disable-next=unnecessary-lambda
device.get_meter_info.side_effect = lambda meter: METER_INFO.get(meter) device.get_meter_info.side_effect = lambda meter: METER_INFO.get(meter)
device.get_network_info.return_value = NETWORK_INFO device.get_network_info.return_value = NETWORK_INFO

View File

@ -1271,7 +1271,10 @@ async def test_master_state_with_template(hass: HomeAssistant) -> None:
events = [] events = []
async_track_state_change_event( async_track_state_change_event(
hass, "media_player.tv", callback(lambda event: events.append(event)) hass,
"media_player.tv",
# pylint: disable-next=unnecessary-lambda
callback(lambda event: events.append(event)),
) )
context = Context() context = Context()

View File

@ -586,7 +586,10 @@ async def test_entity_without_progress_support(
events = [] events = []
async_track_state_change_event( async_track_state_change_event(
hass, "update.update_available", callback(lambda event: events.append(event)) hass,
"update.update_available",
# pylint: disable-next=unnecessary-lambda
callback(lambda event: events.append(event)),
) )
await hass.services.async_call( await hass.services.async_call(
@ -624,7 +627,10 @@ async def test_entity_without_progress_support_raising(
events = [] events = []
async_track_state_change_event( async_track_state_change_event(
hass, "update.update_available", callback(lambda event: events.append(event)) hass,
"update.update_available",
# pylint: disable-next=unnecessary-lambda
callback(lambda event: events.append(event)),
) )
with ( with (

View File

@ -61,7 +61,10 @@ async def test_track_point_in_time(hass: HomeAssistant) -> None:
runs = [] runs = []
async_track_point_in_utc_time( async_track_point_in_utc_time(
hass, callback(lambda x: runs.append(x)), birthday_paulus hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: runs.append(x)),
birthday_paulus,
) )
async_fire_time_changed(hass, before_birthday) async_fire_time_changed(hass, before_birthday)
@ -78,7 +81,10 @@ async def test_track_point_in_time(hass: HomeAssistant) -> None:
assert len(runs) == 1 assert len(runs) == 1
async_track_point_in_utc_time( async_track_point_in_utc_time(
hass, callback(lambda x: runs.append(x)), birthday_paulus hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: runs.append(x)),
birthday_paulus,
) )
async_fire_time_changed(hass, after_birthday) async_fire_time_changed(hass, after_birthday)
@ -86,7 +92,10 @@ async def test_track_point_in_time(hass: HomeAssistant) -> None:
assert len(runs) == 2 assert len(runs) == 2
unsub = async_track_point_in_time( unsub = async_track_point_in_time(
hass, callback(lambda x: runs.append(x)), birthday_paulus hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: runs.append(x)),
birthday_paulus,
) )
unsub() unsub()
@ -107,6 +116,7 @@ async def test_track_point_in_time_drift_rearm(hass: HomeAssistant) -> None:
async_track_point_in_utc_time( async_track_point_in_utc_time(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
time_that_will_not_match_right_away, time_that_will_not_match_right_away,
) )
@ -3546,7 +3556,10 @@ async def test_track_time_interval(hass: HomeAssistant) -> None:
utc_now = dt_util.utcnow() utc_now = dt_util.utcnow()
unsub = async_track_time_interval( unsub = async_track_time_interval(
hass, callback(lambda x: specific_runs.append(x)), timedelta(seconds=10) hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)),
timedelta(seconds=10),
) )
async_fire_time_changed(hass, utc_now + timedelta(seconds=5)) async_fire_time_changed(hass, utc_now + timedelta(seconds=5))
@ -3578,6 +3591,7 @@ async def test_track_time_interval_name(hass: HomeAssistant) -> None:
unique_string = "xZ13" unique_string = "xZ13"
unsub = async_track_time_interval( unsub = async_track_time_interval(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
timedelta(seconds=10), timedelta(seconds=10),
name=unique_string, name=unique_string,
@ -3808,12 +3822,20 @@ async def test_async_track_time_change(
) )
freezer.move_to(time_that_will_not_match_right_away) freezer.move_to(time_that_will_not_match_right_away)
unsub = async_track_time_change(hass, callback(lambda x: none_runs.append(x))) unsub = async_track_time_change(
hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: none_runs.append(x)),
)
unsub_utc = async_track_utc_time_change( unsub_utc = async_track_utc_time_change(
hass, callback(lambda x: specific_runs.append(x)), second=[0, 30] hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)),
second=[0, 30],
) )
unsub_wildcard = async_track_time_change( unsub_wildcard = async_track_time_change(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: wildcard_runs.append(x)), callback(lambda x: wildcard_runs.append(x)),
second="*", second="*",
minute="*", minute="*",
@ -3872,7 +3894,11 @@ async def test_periodic_task_minute(
freezer.move_to(time_that_will_not_match_right_away) freezer.move_to(time_that_will_not_match_right_away)
unsub = async_track_utc_time_change( unsub = async_track_utc_time_change(
hass, callback(lambda x: specific_runs.append(x)), minute="/5", second=0 hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)),
minute="/5",
second=0,
) )
async_fire_time_changed( async_fire_time_changed(
@ -3918,6 +3944,7 @@ async def test_periodic_task_hour(
unsub = async_track_utc_time_change( unsub = async_track_utc_time_change(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
hour="/2", hour="/2",
minute=0, minute=0,
@ -3971,7 +3998,10 @@ async def test_periodic_task_wrong_input(hass: HomeAssistant) -> None:
with pytest.raises(ValueError): with pytest.raises(ValueError):
async_track_utc_time_change( async_track_utc_time_change(
hass, callback(lambda x: specific_runs.append(x)), hour="/two" hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)),
hour="/two",
) )
async_fire_time_changed( async_fire_time_changed(
@ -3995,6 +4025,7 @@ async def test_periodic_task_clock_rollback(
unsub = async_track_utc_time_change( unsub = async_track_utc_time_change(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
hour="/2", hour="/2",
minute=0, minute=0,
@ -4064,6 +4095,7 @@ async def test_periodic_task_duplicate_time(
unsub = async_track_utc_time_change( unsub = async_track_utc_time_change(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
hour="/2", hour="/2",
minute=0, minute=0,
@ -4109,6 +4141,7 @@ async def test_periodic_task_entering_dst(
unsub = async_track_time_change( unsub = async_track_time_change(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
hour=2, hour=2,
minute=30, minute=30,
@ -4160,6 +4193,7 @@ async def test_periodic_task_entering_dst_2(
unsub = async_track_time_change( unsub = async_track_time_change(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
second=list(range(59)), second=list(range(59)),
) )
@ -4210,6 +4244,7 @@ async def test_periodic_task_leaving_dst(
unsub = async_track_time_change( unsub = async_track_time_change(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
hour=2, hour=2,
minute=30, minute=30,
@ -4285,6 +4320,7 @@ async def test_periodic_task_leaving_dst_2(
unsub = async_track_time_change( unsub = async_track_time_change(
hass, hass,
# pylint: disable-next=unnecessary-lambda
callback(lambda x: specific_runs.append(x)), callback(lambda x: specific_runs.append(x)),
minute=30, minute=30,
second=0, second=0,