jeans/core
jeans/core
1
0
Fork 0
mirror of https://github.com/home-assistant/core.git synced 2025-04-23 16:57:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use device class naming in Renault (#96038)

Browse Source
This commit is contained in:
Joost Lekkerkerker 2023-07-07 10:34:11 +02:00 committed by GitHub
parent 8c5df60cc3
commit 84979f8e92
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
6 changed files with 51 additions and 67 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
homeassistant/components/renault/binary_sensor.py
View File

@ -87,7 +87,6 @@ BINARY_SENSOR_TYPES: tuple[RenaultBinarySensorEntityDescription, ...] = tuple(
device_class=BinarySensorDeviceClass.PLUG,
on_key="plugStatus",
on_value=PlugState.PLUGGED.value,
translation_key="plugged_in",
),
RenaultBinarySensorEntityDescription(
key="charging",
@ -95,7 +94,6 @@ BINARY_SENSOR_TYPES: tuple[RenaultBinarySensorEntityDescription, ...] = tuple(
device_class=BinarySensorDeviceClass.BATTERY_CHARGING,
on_key="chargingStatus",
on_value=ChargeState.CHARGE_IN_PROGRESS.value,
translation_key="charging",
),
RenaultBinarySensorEntityDescription(
key="hvac_status",
@ -112,7 +110,6 @@ BINARY_SENSOR_TYPES: tuple[RenaultBinarySensorEntityDescription, ...] = tuple(
device_class=BinarySensorDeviceClass.LOCK,
on_key="lockStatus",
on_value="unlocked",
translation_key="lock_status",
),
RenaultBinarySensorEntityDescription(
key="hatch_status",

1
homeassistant/components/renault/sensor.py
View File

@ -165,7 +165,6 @@ SENSOR_TYPES: tuple[RenaultSensorEntityDescription[Any], ...] = (
entity_class=RenaultSensor[KamereonVehicleBatteryStatusData],
native_unit_of_measurement=PERCENTAGE,
state_class=SensorStateClass.MEASUREMENT,
translation_key="battery_level",
),
RenaultSensorEntityDescription(
key="charge_state",

12
homeassistant/components/renault/strings.json
View File

@ -34,9 +34,6 @@
},
"entity": {
"binary_sensor": {
"charging": {
"name": "[%key:component::binary_sensor::entity_component::battery_charging::name%]"
},
"hatch_status": {
"name": "Hatch"
},
@ -46,15 +43,9 @@
"hvac_status": {
"name": "HVAC"
},
"lock_status": {
"name": "[%key:component::binary_sensor::entity_component::lock::name%]"
},
"passenger_door_status": {
"name": "Passenger door"
},
"plugged_in": {
"name": "[%key:component::binary_sensor::entity_component::plug::name%]"
},
"rear_left_door_status": {
"name": "Rear left door"
},
@ -101,9 +92,6 @@
"battery_last_activity": {
"name": "Last battery activity"
},
"battery_level": {
"name": "Battery level"
},
"battery_temperature": {
"name": "Battery temperature"
},

6
tests/components/renault/const.py
View File

@ -151,7 +151,7 @@ MOCK_VEHICLES = {
},
{
ATTR_DEVICE_CLASS: SensorDeviceClass.BATTERY,
ATTR_ENTITY_ID: "sensor.reg_number_battery_level",
ATTR_ENTITY_ID: "sensor.reg_number_battery",
ATTR_STATE: "60",
ATTR_STATE_CLASS: SensorStateClass.MEASUREMENT,
ATTR_UNIQUE_ID: "vf1aaaaa555777999_battery_level",
@ -386,7 +386,7 @@ MOCK_VEHICLES = {
},
{
ATTR_DEVICE_CLASS: SensorDeviceClass.BATTERY,
ATTR_ENTITY_ID: "sensor.reg_number_battery_level",
ATTR_ENTITY_ID: "sensor.reg_number_battery",
ATTR_STATE: "50",
ATTR_STATE_CLASS: SensorStateClass.MEASUREMENT,
ATTR_UNIQUE_ID: "vf1aaaaa555777999_battery_level",
@ -621,7 +621,7 @@ MOCK_VEHICLES = {
},
{
ATTR_DEVICE_CLASS: SensorDeviceClass.BATTERY,
ATTR_ENTITY_ID: "sensor.reg_number_battery_level",
ATTR_ENTITY_ID: "sensor.reg_number_battery",
ATTR_STATE: "60",
ATTR_STATE_CLASS: SensorStateClass.MEASUREMENT,
ATTR_UNIQUE_ID: "vf1aaaaa555777123_battery_level",

36
tests/components/renault/snapshots/test_binary_sensor.ambr
View File

@ -54,7 +54,7 @@
'original_name': 'Lock',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'lock_status',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_lock_status',
'unit_of_measurement': None,
}),
@ -325,7 +325,7 @@
'original_name': 'Plug',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'plugged_in',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_plugged_in',
'unit_of_measurement': None,
}),
@ -353,7 +353,7 @@
'original_name': 'Charging',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'charging',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_charging',
'unit_of_measurement': None,
}),
@ -381,7 +381,7 @@
'original_name': 'Lock',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'lock_status',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_lock_status',
'unit_of_measurement': None,
}),
@ -674,7 +674,7 @@
'original_name': 'Plug',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'plugged_in',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_plugged_in',
'unit_of_measurement': None,
}),
@ -702,7 +702,7 @@
'original_name': 'Charging',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'charging',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_charging',
'unit_of_measurement': None,
}),
@ -828,7 +828,7 @@
'original_name': 'Plug',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'plugged_in',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_plugged_in',
'unit_of_measurement': None,
}),
@ -856,7 +856,7 @@
'original_name': 'Charging',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'charging',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_charging',
'unit_of_measurement': None,
}),
@ -912,7 +912,7 @@
'original_name': 'Lock',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'lock_status',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_lock_status',
'unit_of_measurement': None,
}),
@ -1216,7 +1216,7 @@
'original_name': 'Lock',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'lock_status',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_lock_status',
'unit_of_measurement': None,
}),
@ -1487,7 +1487,7 @@
'original_name': 'Plug',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'plugged_in',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_plugged_in',
'unit_of_measurement': None,
}),
@ -1515,7 +1515,7 @@
'original_name': 'Charging',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'charging',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_charging',
'unit_of_measurement': None,
}),
@ -1543,7 +1543,7 @@
'original_name': 'Lock',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'lock_status',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_lock_status',
'unit_of_measurement': None,
}),
@ -1836,7 +1836,7 @@
'original_name': 'Plug',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'plugged_in',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_plugged_in',
'unit_of_measurement': None,
}),
@ -1864,7 +1864,7 @@
'original_name': 'Charging',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'charging',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_charging',
'unit_of_measurement': None,
}),
@ -1990,7 +1990,7 @@
'original_name': 'Plug',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'plugged_in',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_plugged_in',
'unit_of_measurement': None,
}),
@ -2018,7 +2018,7 @@
'original_name': 'Charging',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'charging',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_charging',
'unit_of_measurement': None,
}),
@ -2074,7 +2074,7 @@
'original_name': 'Lock',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'lock_status',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_lock_status',
'unit_of_measurement': None,
}),

60
tests/components/renault/snapshots/test_sensor.ambr
View File

@ -327,7 +327,7 @@
'disabled_by': None,
'domain': 'sensor',
'entity_category': None,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'has_entity_name': True,
'hidden_by': None,
'icon': None,
@ -337,10 +337,10 @@
}),
'original_device_class': <SensorDeviceClass.BATTERY: 'battery'>,
'original_icon': None,
'original_name': 'Battery level',
'original_name': 'Battery',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'battery_level',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_battery_level',
'unit_of_measurement': '%',
}),
@ -777,12 +777,12 @@
StateSnapshot({
'attributes': ReadOnlyDict({
'device_class': 'battery',
'friendly_name': 'REG-NUMBER Battery level',
'friendly_name': 'REG-NUMBER Battery',
'state_class': <SensorStateClass.MEASUREMENT: 'measurement'>,
'unit_of_measurement': '%',
}),
'context': <ANY>,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'last_changed': <ANY>,
'last_updated': <ANY>,
'state': 'unknown',
@ -1023,7 +1023,7 @@
'disabled_by': None,
'domain': 'sensor',
'entity_category': None,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'has_entity_name': True,
'hidden_by': None,
'icon': None,
@ -1033,10 +1033,10 @@
}),
'original_device_class': <SensorDeviceClass.BATTERY: 'battery'>,
'original_icon': None,
'original_name': 'Battery level',
'original_name': 'Battery',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'battery_level',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_battery_level',
'unit_of_measurement': '%',
}),
@ -1471,12 +1471,12 @@
StateSnapshot({
'attributes': ReadOnlyDict({
'device_class': 'battery',
'friendly_name': 'REG-NUMBER Battery level',
'friendly_name': 'REG-NUMBER Battery',
'state_class': <SensorStateClass.MEASUREMENT: 'measurement'>,
'unit_of_measurement': '%',
}),
'context': <ANY>,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'last_changed': <ANY>,
'last_updated': <ANY>,
'state': 'unknown',
@ -1713,7 +1713,7 @@
'disabled_by': None,
'domain': 'sensor',
'entity_category': None,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'has_entity_name': True,
'hidden_by': None,
'icon': None,
@ -1723,10 +1723,10 @@
}),
'original_device_class': <SensorDeviceClass.BATTERY: 'battery'>,
'original_icon': None,
'original_name': 'Battery level',
'original_name': 'Battery',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'battery_level',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_battery_level',
'unit_of_measurement': '%',
}),
@ -2189,12 +2189,12 @@
StateSnapshot({
'attributes': ReadOnlyDict({
'device_class': 'battery',
'friendly_name': 'REG-NUMBER Battery level',
'friendly_name': 'REG-NUMBER Battery',
'state_class': <SensorStateClass.MEASUREMENT: 'measurement'>,
'unit_of_measurement': '%',
}),
'context': <ANY>,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'last_changed': <ANY>,
'last_updated': <ANY>,
'state': 'unknown',
@ -2726,7 +2726,7 @@
'disabled_by': None,
'domain': 'sensor',
'entity_category': None,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'has_entity_name': True,
'hidden_by': None,
'icon': None,
@ -2736,10 +2736,10 @@
}),
'original_device_class': <SensorDeviceClass.BATTERY: 'battery'>,
'original_icon': None,
'original_name': 'Battery level',
'original_name': 'Battery',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'battery_level',
'translation_key': None,
'unique_id': 'vf1aaaaa555777123_battery_level',
'unit_of_measurement': '%',
}),
@ -3176,12 +3176,12 @@
StateSnapshot({
'attributes': ReadOnlyDict({
'device_class': 'battery',
'friendly_name': 'REG-NUMBER Battery level',
'friendly_name': 'REG-NUMBER Battery',
'state_class': <SensorStateClass.MEASUREMENT: 'measurement'>,
'unit_of_measurement': '%',
}),
'context': <ANY>,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'last_changed': <ANY>,
'last_updated': <ANY>,
'state': '60',
@ -3422,7 +3422,7 @@
'disabled_by': None,
'domain': 'sensor',
'entity_category': None,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'has_entity_name': True,
'hidden_by': None,
'icon': None,
@ -3432,10 +3432,10 @@
}),
'original_device_class': <SensorDeviceClass.BATTERY: 'battery'>,
'original_icon': None,
'original_name': 'Battery level',
'original_name': 'Battery',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'battery_level',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_battery_level',
'unit_of_measurement': '%',
}),
@ -3870,12 +3870,12 @@
StateSnapshot({
'attributes': ReadOnlyDict({
'device_class': 'battery',
'friendly_name': 'REG-NUMBER Battery level',
'friendly_name': 'REG-NUMBER Battery',
'state_class': <SensorStateClass.MEASUREMENT: 'measurement'>,
'unit_of_measurement': '%',
}),
'context': <ANY>,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'last_changed': <ANY>,
'last_updated': <ANY>,
'state': '60',
@ -4112,7 +4112,7 @@
'disabled_by': None,
'domain': 'sensor',
'entity_category': None,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'has_entity_name': True,
'hidden_by': None,
'icon': None,
@ -4122,10 +4122,10 @@
}),
'original_device_class': <SensorDeviceClass.BATTERY: 'battery'>,
'original_icon': None,
'original_name': 'Battery level',
'original_name': 'Battery',
'platform': 'renault',
'supported_features': 0,
'translation_key': 'battery_level',
'translation_key': None,
'unique_id': 'vf1aaaaa555777999_battery_level',
'unit_of_measurement': '%',
}),
@ -4588,12 +4588,12 @@
StateSnapshot({
'attributes': ReadOnlyDict({
'device_class': 'battery',
'friendly_name': 'REG-NUMBER Battery level',
'friendly_name': 'REG-NUMBER Battery',
'state_class': <SensorStateClass.MEASUREMENT: 'measurement'>,
'unit_of_measurement': '%',
}),
'context': <ANY>,
'entity_id': 'sensor.reg_number_battery_level',
'entity_id': 'sensor.reg_number_battery',
'last_changed': <ANY>,
'last_updated': <ANY>,
'state': '50',