Skip to content

Commit fd370c2

Browse files
Fix test cases using EnergyMeasurementStruct (#2470)
Two fields were added to the type definition for EnergyMeasurementStruct and must be added to tests using this struct to fix test case failures.
1 parent 570354b commit fd370c2

File tree

6 files changed

+32
-18
lines changed

6 files changed

+32
-18
lines changed

drivers/SmartThings/matter-energy/src/test/test_battery_storage.lua

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -196,7 +196,7 @@ test.register_coroutine_test(
196196
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes
197197
.CumulativeEnergyImported:build_test_report_data(mock_device,
198198
BATTERY_STORAGE_EP,
199-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 100000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) --100Wh
199+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 100000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) --100Wh
200200

201201
test.socket.capability:__expect_send(
202202
mock_device:generate_test_message("importedEnergy",
@@ -219,7 +219,7 @@ test.register_coroutine_test(
219219
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes
220220
.CumulativeEnergyExported:build_test_report_data(mock_device,
221221
BATTERY_STORAGE_EP,
222-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 400000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) --400Wh
222+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 400000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) --400Wh
223223

224224
test.socket.capability:__expect_send(
225225
mock_device:generate_test_message("exportedEnergy",
@@ -245,7 +245,7 @@ test.register_coroutine_test(
245245
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes
246246
.CumulativeEnergyImported:build_test_report_data(mock_device,
247247
BATTERY_STORAGE_EP,
248-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 200000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) --200Wh
248+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 200000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) --200Wh
249249

250250
test.socket.capability:__expect_send(
251251
mock_device:generate_test_message("importedEnergy",
@@ -266,7 +266,7 @@ test.register_coroutine_test(
266266
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes
267267
.CumulativeEnergyExported:build_test_report_data(mock_device,
268268
BATTERY_STORAGE_EP,
269-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 400000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) --400Wh
269+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 400000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) --400Wh
270270

271271
test.socket.capability:__expect_send(
272272
mock_device:generate_test_message("exportedEnergy",

drivers/SmartThings/matter-energy/src/test/test_solar_power.lua

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ test.register_coroutine_test(
133133
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes
134134
.CumulativeEnergyExported:build_test_report_data(mock_device,
135135
SOLAR_POWER_EP_ONE,
136-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 100000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) --100Wh
136+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 100000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) --100Wh
137137

138138
test.socket.capability:__expect_send(
139139
mock_device:generate_test_message("main",
@@ -156,7 +156,7 @@ test.register_coroutine_test(
156156
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes
157157
.CumulativeEnergyExported:build_test_report_data(mock_device,
158158
SOLAR_POWER_EP_TWO,
159-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 150000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) --150Wh
159+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 150000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) --150Wh
160160

161161
test.socket.capability:__expect_send(
162162
mock_device:generate_test_message("main",
@@ -178,7 +178,7 @@ test.register_coroutine_test(
178178
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes
179179
.CumulativeEnergyExported:build_test_report_data(mock_device,
180180
SOLAR_POWER_EP_ONE,
181-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 100000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) --100Wh
181+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 100000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) --100Wh
182182

183183
test.socket.capability:__expect_send(
184184
mock_device:generate_test_message("main",
@@ -190,7 +190,7 @@ test.register_coroutine_test(
190190
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes
191191
.CumulativeEnergyImported:build_test_report_data(mock_device,
192192
SOLAR_POWER_EP_ONE,
193-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 100000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) --100Wh
193+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 100000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) --100Wh
194194
end
195195
)
196196

drivers/SmartThings/matter-switch/src/test/test_aqara_light_switch_h2.lua

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -119,6 +119,8 @@ local cumulative_report_val_19 = {
119119
end_timestamp = 0,
120120
start_systime = 0,
121121
end_systime = 0,
122+
apparent_energy = 0,
123+
reactive_energy = 0
122124
}
123125

124126
local cumulative_report_val_29 = {
@@ -127,6 +129,8 @@ local cumulative_report_val_29 = {
127129
end_timestamp = 0,
128130
start_systime = 0,
129131
end_systime = 0,
132+
apparent_energy = 0,
133+
reactive_energy = 0
130134
}
131135

132136
local cumulative_report_val_39 = {
@@ -135,6 +139,8 @@ local cumulative_report_val_39 = {
135139
end_timestamp = 0,
136140
start_systime = 0,
137141
end_systime = 0,
142+
apparent_energy = 0,
143+
reactive_energy = 0
138144
}
139145

140146
local function configure_buttons()

drivers/SmartThings/matter-switch/src/test/test_electrical_sensor.lua

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -110,6 +110,8 @@ local cumulative_report_val_19 = {
110110
end_timestamp = 0,
111111
start_systime = 0,
112112
end_systime = 0,
113+
apparent_energy = 0,
114+
reactive_energy = 0
113115
}
114116

115117
local cumulative_report_val_29 = {
@@ -118,6 +120,8 @@ local cumulative_report_val_29 = {
118120
end_timestamp = 0,
119121
start_systime = 0,
120122
end_systime = 0,
123+
apparent_energy = 0,
124+
reactive_energy = 0
121125
}
122126

123127
local cumulative_report_val_39 = {
@@ -126,6 +130,8 @@ local cumulative_report_val_39 = {
126130
end_timestamp = 0,
127131
start_systime = 0,
128132
end_systime = 0,
133+
apparent_energy = 0,
134+
reactive_energy = 0
129135
}
130136

131137
local periodic_report_val_23 = {
@@ -134,6 +140,8 @@ local periodic_report_val_23 = {
134140
end_timestamp = 0,
135141
start_systime = 0,
136142
end_systime = 0,
143+
apparent_energy = 0,
144+
reactive_energy = 0
137145
}
138146

139147
local function test_init()

drivers/SmartThings/matter-thermostat/src/test/test_matter_heat_pump.lua

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -633,7 +633,7 @@ test.register_message_test(
633633
clusters.ElectricalEnergyMeasurement.attributes
634634
.CumulativeEnergyImported:build_test_report_data(mock_device,
635635
HEAT_PUMP_EP,
636-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 15000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 }))
636+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 15000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 }))
637637
}
638638
},
639639
{
@@ -651,7 +651,7 @@ test.register_coroutine_test(
651651

652652
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes.CumulativeEnergyImported:build_test_report_data(mock_device,
653653
HEAT_PUMP_EP,
654-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 20000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) -- 20Wh
654+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 20000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) -- 20Wh
655655

656656
test.socket.capability:__expect_send(
657657
mock_device:generate_test_message("main",
@@ -675,7 +675,7 @@ test.register_coroutine_test(
675675

676676
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes.CumulativeEnergyImported:build_test_report_data(mock_device,
677677
HEAT_PUMP_EP,
678-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 30000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) -- 30Wh
678+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 30000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) -- 30Wh
679679

680680
test.socket.capability:__expect_send(
681681
mock_device:generate_test_message("main",
@@ -698,7 +698,7 @@ test.register_coroutine_test(
698698

699699
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes.CumulativeEnergyImported:build_test_report_data(mock_device,
700700
HEAT_PUMP_EP,
701-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 20000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) -- 20Wh
701+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 20000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) -- 20Wh
702702

703703
test.socket.capability:__expect_send(
704704
mock_device:generate_test_message("main",
@@ -722,12 +722,12 @@ test.register_coroutine_test(
722722
-- do not expect energyMeter event for this report.
723723
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes.PeriodicEnergyImported:build_test_report_data(mock_device,
724724
HEAT_PUMP_EP,
725-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 20000, start_timestamp = 0, end_timestamp = 800, start_systime = 0, end_systime = 0 })) }) -- 20Wh
725+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 20000, start_timestamp = 0, end_timestamp = 800, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) -- 20Wh
726726

727727
-- do not expect a powerConsumptionReport to be emitted
728728
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes.CumulativeEnergyImported:build_test_report_data(mock_device,
729729
HEAT_PUMP_EP,
730-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 50000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) -- 50Wh
730+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 50000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) -- 50Wh
731731

732732
test.socket.capability:__expect_send(
733733
mock_device:generate_test_message("main",

drivers/SmartThings/matter-thermostat/src/test/test_matter_water_heater.lua

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -249,7 +249,7 @@ test.register_message_test(
249249
clusters.ElectricalEnergyMeasurement.attributes
250250
.CumulativeEnergyImported:build_test_report_data(mock_device,
251251
ELECTRICAL_SENSOR_EP,
252-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 15000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 }))
252+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 15000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 }))
253253
}
254254
},
255255
{
@@ -267,7 +267,7 @@ test.register_coroutine_test(
267267

268268
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes.CumulativeEnergyImported:build_test_report_data(mock_device,
269269
ELECTRICAL_SENSOR_EP,
270-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 20000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) -- 20Wh
270+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 20000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) -- 20Wh
271271

272272
test.socket.capability:__expect_send(
273273
mock_device:generate_test_message("main",
@@ -288,7 +288,7 @@ test.register_coroutine_test(
288288

289289
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes.CumulativeEnergyImported:build_test_report_data(mock_device,
290290
ELECTRICAL_SENSOR_EP,
291-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 30000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) -- 30Wh
291+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 30000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) -- 30Wh
292292

293293
test.socket.capability:__expect_send(
294294
mock_device:generate_test_message("main",
@@ -303,7 +303,7 @@ test.register_coroutine_test(
303303

304304
test.socket.matter:__queue_receive({ mock_device.id, clusters.ElectricalEnergyMeasurement.attributes.CumulativeEnergyImported:build_test_report_data(mock_device,
305305
ELECTRICAL_SENSOR_EP,
306-
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 50000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0 })) }) -- 30Wh
306+
clusters.ElectricalEnergyMeasurement.types.EnergyMeasurementStruct({ energy = 50000, start_timestamp = 0, end_timestamp = 0, start_systime = 0, end_systime = 0, apparent_energy = 0, reactive_energy = 0 })) }) -- 30Wh
307307

308308
test.socket.capability:__expect_send(
309309
mock_device:generate_test_message("main",

0 commit comments

Comments
 (0)