diff --git a/tests/components/knx/test_telegrams.py b/tests/components/knx/test_telegrams.py index 4d72a9583a1..2eda718f5ac 100644 --- a/tests/components/knx/test_telegrams.py +++ b/tests/components/knx/test_telegrams.py @@ -51,8 +51,8 @@ MOCK_TELEGRAMS = [ def assert_telegram_history(telegrams: list[TelegramDict]) -> bool: """Assert that the mock telegrams are equal to the given telegrams. Omitting timestamp.""" assert len(telegrams) == len(MOCK_TELEGRAMS) - for index in range(len(telegrams)): - test_telegram = copy(telegrams[index]) # don't modify the original + for index, value in enumerate(telegrams): + test_telegram = copy(value) # don't modify the original comp_telegram = MOCK_TELEGRAMS[index] assert datetime.fromisoformat(test_telegram["timestamp"]) if isinstance(test_telegram["payload"], tuple): diff --git a/tests/components/modbus/test_sensor.py b/tests/components/modbus/test_sensor.py index 71cb64cc1b6..20ff558fce6 100644 --- a/tests/components/modbus/test_sensor.py +++ b/tests/components/modbus/test_sensor.py @@ -901,7 +901,7 @@ async def test_virtual_sensor( hass: HomeAssistant, entity_registry: er.EntityRegistry, mock_do_cycle, expected ) -> None: """Run test for sensor.""" - for i in range(len(expected)): + for i, expected_value in enumerate(expected): entity_id = f"{SENSOR_DOMAIN}.{TEST_ENTITY_NAME}".replace(" ", "_") unique_id = f"{SLAVE_UNIQUE_ID}" if i: @@ -909,7 +909,7 @@ async def test_virtual_sensor( unique_id = f"{unique_id}_{i}" entry = entity_registry.async_get(entity_id) state = hass.states.get(entity_id).state - assert state == expected[i] + assert state == expected_value assert entry.unique_id == unique_id @@ -1071,12 +1071,12 @@ async def test_virtual_swap_sensor( hass: HomeAssistant, mock_do_cycle, expected ) -> None: """Run test for sensor.""" - for i in range(len(expected)): + for i, expected_value in enumerate(expected): entity_id = f"{SENSOR_DOMAIN}.{TEST_ENTITY_NAME}".replace(" ", "_") if i: entity_id = f"{entity_id}_{i}" state = hass.states.get(entity_id).state - assert state == expected[i] + assert state == expected_value @pytest.mark.parametrize( diff --git a/tests/components/plant/test_init.py b/tests/components/plant/test_init.py index 8a728665ce2..97286a28cde 100644 --- a/tests/components/plant/test_init.py +++ b/tests/components/plant/test_init.py @@ -204,8 +204,8 @@ def test_daily_history_one_day(hass: HomeAssistant) -> None: """Test storing data for the same day.""" dh = plant.DailyHistory(3) values = [-2, 10, 0, 5, 20] - for i in range(len(values)): - dh.add_measurement(values[i]) + for i, value in enumerate(values): + dh.add_measurement(value) max_value = max(values[0 : i + 1]) assert len(dh._days) == 1 assert dh.max == max_value @@ -222,6 +222,6 @@ def test_daily_history_multiple_days(hass: HomeAssistant) -> None: values = [10, 1, 7, 3] max_values = [10, 10, 10, 7] - for i in range(len(days)): - dh.add_measurement(values[i], days[i]) + for i, value in enumerate(days): + dh.add_measurement(values[i], value) assert max_values[i] == dh.max diff --git a/tests/components/sensor/test_recorder.py b/tests/components/sensor/test_recorder.py index 0abe5e56e44..896742d87c3 100644 --- a/tests/components/sensor/test_recorder.py +++ b/tests/components/sensor/test_recorder.py @@ -4786,10 +4786,10 @@ async def test_validate_statistics_unit_change_no_conversion( with session_scope(hass=hass, read_only=True) as session: db_states = list(session.query(StatisticsMeta)) assert len(db_states) == len(expected_result) - for i in range(len(db_states)): - assert db_states[i].statistic_id == expected_result[i]["statistic_id"] + for i, db_state in enumerate(db_states): + assert db_state.statistic_id == expected_result[i]["statistic_id"] assert ( - db_states[i].unit_of_measurement + db_state.unit_of_measurement == expected_result[i]["unit_of_measurement"] ) @@ -4920,10 +4920,10 @@ async def test_validate_statistics_unit_change_equivalent_units( with session_scope(hass=hass, read_only=True) as session: db_states = list(session.query(StatisticsMeta)) assert len(db_states) == len(expected_result) - for i in range(len(db_states)): - assert db_states[i].statistic_id == expected_result[i]["statistic_id"] + for i, db_state in enumerate(db_states): + assert db_state.statistic_id == expected_result[i]["statistic_id"] assert ( - db_states[i].unit_of_measurement + db_state.unit_of_measurement == expected_result[i]["unit_of_measurement"] ) @@ -5005,10 +5005,10 @@ async def test_validate_statistics_unit_change_equivalent_units_2( with session_scope(hass=hass, read_only=True) as session: db_states = list(session.query(StatisticsMeta)) assert len(db_states) == len(expected_result) - for i in range(len(db_states)): - assert db_states[i].statistic_id == expected_result[i]["statistic_id"] + for i, db_state in enumerate(db_states): + assert db_state.statistic_id == expected_result[i]["statistic_id"] assert ( - db_states[i].unit_of_measurement + db_state.unit_of_measurement == expected_result[i]["unit_of_measurement"] ) diff --git a/tests/components/statistics/test_sensor.py b/tests/components/statistics/test_sensor.py index 6508ccd608e..5a716fd8ce8 100644 --- a/tests/components/statistics/test_sensor.py +++ b/tests/components/statistics/test_sensor.py @@ -1314,13 +1314,13 @@ async def test_state_characteristics(hass: HomeAssistant) -> None: # With all values in buffer - for i in range(len(VALUES_NUMERIC)): + for i, value in enumerate(VALUES_NUMERIC): current_time += timedelta(minutes=1) freezer.move_to(current_time) async_fire_time_changed(hass, current_time) hass.states.async_set( "sensor.test_monitored", - str(VALUES_NUMERIC[i]), + str(value), {ATTR_UNIT_OF_MEASUREMENT: UnitOfTemperature.CELSIUS}, ) hass.states.async_set(