4
0
Эх сурвалжийг харах

Update and remove energy meter tests, fix typo in config

Individual devices no longer need unit tests, so lighten the load
slightly by removing a test that doesn't test anything new.

Update other energy meter tests for the unit changes.

Add missing whitespace from last change.
Jason Rumney 2 жил өмнө
parent
commit
95faa3975a

+ 1 - 1
custom_components/tuya_local/devices/wdyk_3phase_energymonitor.yaml

@@ -235,7 +235,7 @@ secondary_entities:
         unit: kWh
         class: total_increasing
         mapping:
-          -scale: 1000
+          - scale: 1000
   - entity: binary_sensor
     name: Locking
     class: lock

+ 0 - 105
tests/devices/test_compteur_energy_meter.py

@@ -1,105 +0,0 @@
-"""Tests for the Compteur Energy meter"""
-from homeassistant.components.sensor import (
-    SensorDeviceClass,
-    STATE_CLASS_MEASUREMENT,
-    STATE_CLASS_TOTAL_INCREASING,
-)
-from homeassistant.const import (
-    UnitOfElectricCurrent,
-    UnitOfElectricPotential,
-    UnitOfEnergy,
-    UnitOfPower,
-)
-
-from ..const import COMPTEUR_SMARTMETER_PAYLOAD
-from ..mixins.sensor import MultiSensorTests
-from .base_device_tests import TuyaDeviceTestCase
-
-
-ENERGY_DP = "17"
-CURRENT_DP = "18"
-POWER_DP = "19"
-VOLTAGE_DP = "20"
-UNKNOWN21_DP = "21"
-UNKNOWN22_DP = "22"
-UNKNOWN23_DP = "23"
-UNKNOWN24_DP = "24"
-UNKNOWN25_DP = "25"
-UNKNOWN26_DP = "26"
-
-
-class TestCompteurEnergyMeter(MultiSensorTests, TuyaDeviceTestCase):
-    __test__ = True
-
-    def setUp(self):
-        self.setUpForConfig(
-            "compteur_energy_meter.yaml",
-            COMPTEUR_SMARTMETER_PAYLOAD,
-        )
-        self.setUpMultiSensors(
-            [
-                {
-                    "dps": ENERGY_DP,
-                    "name": "sensor",
-                    "unit": UnitOfEnergy.KILO_WATT_HOUR,
-                    "device_class": SensorDeviceClass.ENERGY,
-                    "state_class": STATE_CLASS_TOTAL_INCREASING,
-                    "testdata": (12345, 12.345),
-                },
-                {
-                    "dps": VOLTAGE_DP,
-                    "name": "sensor_voltage",
-                    "unit": UnitOfElectricPotential.VOLT,
-                    "device_class": SensorDeviceClass.VOLTAGE,
-                    "state_class": STATE_CLASS_MEASUREMENT,
-                    "testdata": (2348, 234.8),
-                },
-                {
-                    "dps": CURRENT_DP,
-                    "name": "sensor_current",
-                    "unit": UnitOfElectricCurrent.AMPERE,
-                    "device_class": SensorDeviceClass.CURRENT,
-                    "state_class": STATE_CLASS_MEASUREMENT,
-                    "testdata": (4567, 4.567),
-                },
-                {
-                    "dps": POWER_DP,
-                    "name": "sensor_power",
-                    "unit": UnitOfPower.WATT,
-                    "state_class": STATE_CLASS_MEASUREMENT,
-                    "device_class": SensorDeviceClass.POWER,
-                    "testdata": (890, 89.0),
-                },
-            ]
-        )
-        self.mark_secondary(
-            [
-                "sensor_voltage",
-                "sensor_current",
-                "sensor_power",
-            ]
-        )
-
-    def test_multi_sensor_extra_state_attributes(self):
-        self.dps[UNKNOWN21_DP] = 21
-        self.dps[UNKNOWN22_DP] = 22
-        self.dps[UNKNOWN23_DP] = 23
-        self.dps[UNKNOWN24_DP] = 24
-        self.dps[UNKNOWN25_DP] = 25
-        self.dps[UNKNOWN26_DP] = 26
-
-        for k, v in self.multiSensor.items():
-            if k == "sensor":
-                self.assertDictEqual(
-                    v.extra_state_attributes,
-                    {
-                        "unknown_21": 21,
-                        "unknown_22": 22,
-                        "unknown_23": 23,
-                        "unknown_24": 24,
-                        "unknown_25": 25,
-                        "unknown_26": 26,
-                    },
-                )
-            else:
-                self.assertEqual(v.extra_state_attributes, {})

+ 6 - 4
tests/devices/test_parkside_plgs2012a1_smart_charger.py

@@ -68,9 +68,10 @@ class TestParksidePLGS2012A1Charger(
                     "name": "number_charge_current",
                     "dps": CURRENT_DPS,
                     "device_class": NumberDeviceClass.CURRENT,
-                    "max": 30000,
-                    "step": 100,
-                    "unit": UnitOfElectricCurrent.MILLIAMPERE,
+                    "max": 30.000,
+                    "step": 0.1,
+                    "scale": 1000,
+                    "unit": UnitOfElectricCurrent.AMPERE,
                 },
                 {
                     "name": "number_charge_voltage",
@@ -117,8 +118,9 @@ class TestParksidePLGS2012A1Charger(
                 {
                     "name": "sensor_max_current",
                     "dps": MAXCURRENT_DPS,
-                    "unit": UnitOfElectricCurrent.MILLIAMPERE,
+                    "unit": UnitOfElectricCurrent.AMPERE,
                     "device_class": SensorDeviceClass.CURRENT,
+                    "testdata": (1234, 1.234),
                 },
                 {
                     "name": "sensor_max_temperature_count",

+ 6 - 6
tests/devices/test_tompd63lw_breaker.py

@@ -44,16 +44,16 @@ class TestTOMPD63lw(MultiSensorTests, TuyaDeviceTestCase):
                 {
                     "name": "sensor_current_a",
                     "dps": PHASEA_DP,
-                    "unit": UnitOfElectricCurrent.MILLIAMPERE,
+                    "unit": UnitOfElectricCurrent.AMPERE,
                     "device_class": SensorDeviceClass.CURRENT,
-                    "testdata": ("CHoAQgQADlwAAA==", 16900),
+                    "testdata": ("CHoAQgQADlwAAA==", 16.9),
                 },
                 {
                     "name": "sensor_power_a",
                     "dps": PHASEA_DP,
-                    "unit": UnitOfPower.WATT,
+                    "unit": UnitOfPower.KILO_WATT,
                     "device_class": SensorDeviceClass.POWER,
-                    "testdata": ("CHoAQgQADlwAAA==", 3676),
+                    "testdata": ("CHoAQgQADlwAAA==", 3.676),
                 },
             ]
         )
@@ -74,8 +74,8 @@ class TestTOMPD63lw(MultiSensorTests, TuyaDeviceTestCase):
     def test_phasea_encoding(self):
         self.dps[PHASEA_DP] = "CMAAD6AAAzUAAA=="
         self.assertEqual(self.multiSensor["sensor_voltage_a"].native_value, 224.0)
-        self.assertEqual(self.multiSensor["sensor_current_a"].native_value, 4000)
-        self.assertEqual(self.multiSensor["sensor_power_a"].native_value, 821.0)
+        self.assertEqual(self.multiSensor["sensor_current_a"].native_value, 4.0)
+        self.assertEqual(self.multiSensor["sensor_power_a"].native_value, 0.821)
 
     def test_phasea_missing(self):
         self.dps[PHASEA_DP] = None