Skip to content

Commit 53c1002

Browse files
Merge branch 'master' into idm-4.2-troubleshoot
2 parents 5750461 + f7494fa commit 53c1002

File tree

2 files changed

+74
-9
lines changed

2 files changed

+74
-9
lines changed

src/app/tests/suites/certification/Test_TC_S_2_6.yaml

+47-9
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ tests:
101101
CHIP:TOO: }
102102
disabled: true
103103

104-
- label: "Step 1b: Repeat Step 1a with TH2 and TH3."
104+
- label: "Step 1b: Repeat Step 1a with TH2."
105105
PICS: S.S.C03.Rsp
106106
verification: |
107107
./chip-tool scenesmanagement remove-all-scenes 0x0000 2 1 --commissioner-name beta
@@ -116,7 +116,11 @@ tests:
116116
[1700826575.191974][15971:15973] CHIP:TOO: status: 0
117117
[1700826575.191985][15971:15973] CHIP:TOO: groupID: 0
118118
[1700826575.191995][15971:15973] CHIP:TOO: }
119+
disabled: true
119120

121+
- label: "Step 1C: Repeat Step 1a with TH3."
122+
PICS: S.S.C03.Rsp
123+
verification: |
120124
./chip-tool scenesmanagement remove-all-scenes 0x0000 3 1 --commissioner-name gamma
121125
122126
Verify the RemoveAllScenesResponse with following fields:
@@ -282,10 +286,10 @@ tests:
282286
[1701242801.625919][7636:7638] CHIP:TOO: RemainingCapacity: 7
283287
[1701242801.625930][7636:7638] CHIP:TOO: FabricIndex: 2
284288
[1701242801.625941][7636:7638] CHIP:TOO: }
289+
disabled: true
285290

286-
287-
TH3:
288-
291+
- label: "Step 2e: Repeat Step 2b and 2c with TH3."
292+
verification: |
289293
Please use Interactive mode to Verify the subscription of an event
290294
Here the command to enter interactive mode:--
291295
./chip-tool interactive start
@@ -966,11 +970,11 @@ tests:
966970
disabled: true
967971

968972
- label:
969-
Verify that the DUT sends a report data to TH2 for FabricSceneInfo
970-
after the MinIntervalFloor time; store the RemainingCapacity field
971-
from this fabric’s entry reported in FabricSceneInfo into
972-
Remaining2ndCapacity; verify Remaining2ndCapacity equals
973-
(MaxRemainingCapacity).
973+
"Step 10b: Verify that the DUT sends a report data to TH2 for
974+
FabricSceneInfo after the MinIntervalFloor time; store the
975+
RemainingCapacity field from this fabric’s entry reported in
976+
FabricSceneInfo into Remaining2ndCapacity; verify Remaining2ndCapacity
977+
equals (MaxRemainingCapacity)."
974978
verification: |
975979
./chip-tool scenesmanagement subscribe fabric-scene-info 100 200 2 1 --commissioner-name beta
976980
@@ -989,3 +993,37 @@ tests:
989993
[1705915866.717640][21433:21435] CHIP:TOO: }
990994
991995
disabled: true
996+
997+
- label:
998+
"Step 11a: TH1 removes the TH2 fabric by sending the RemoveFabric
999+
command to the DUT with the FabricIndex set to th2FabricIndex"
1000+
verification: |
1001+
./chip-tool operationalcredentials remove-fabric th2FabricIndex 1 0
1002+
1003+
On TH1(chip-tool) verify the success with the nocresponse with statuscode is success(0)
1004+
1005+
[1784416866.004187][21433:21435] CHIP:DMG: Received Command Response Data, Endpoint=0 Cluster=0x0000_003E Command=0x0000_0008
1006+
[1784416866.004214][21433:21435] CHIP:TOO: Endpoint: 0 Cluster: 0x0000_003E Command 0x0000_0008
1007+
[1784416866.004236][21433:21435] CHIP:TOO: NOCResponse: {
1008+
[1784416866.004250][21433:21435] CHIP:TOO: statusCode: 0
1009+
[1784416866.004255][21433:21435] CHIP:TOO: fabricIndex: th2FabricIndex
1010+
[1784416866.004259][21433:21435] CHIP:TOO: }
1011+
[1784416866.004270][21433:21435] CHIP:DMG: ICR moving to [AwaitingDe]
1012+
disabled: true
1013+
1014+
- label:
1015+
"Step 11b: TH1 removes the TH3 fabric by sending the RemoveFabric
1016+
command to the DUT with the FabricIndex set to th3FabricIndex"
1017+
verification: |
1018+
./chip-tool operationalcredentials remove-fabric th3FabricIndex 1 0
1019+
1020+
On TH1(chip-tool) verify the success with the nocresponse with statuscode is success(0)
1021+
1022+
[1784416866.004187][21433:21435] CHIP:DMG: Received Command Response Data, Endpoint=0 Cluster=0x0000_003E Command=0x0000_0008
1023+
[1784416866.004214][21433:21435] CHIP:TOO: Endpoint: 0 Cluster: 0x0000_003E Command 0x0000_0008
1024+
[1784416866.004236][21433:21435] CHIP:TOO: NOCResponse: {
1025+
[1784416866.004250][21433:21435] CHIP:TOO: statusCode: 0
1026+
[1784416866.004255][21433:21435] CHIP:TOO: fabricIndex: th3FabricIndex
1027+
[1784416866.004259][21433:21435] CHIP:TOO: }
1028+
[1784416866.004270][21433:21435] CHIP:DMG: ICR moving to [AwaitingDe]
1029+
disabled: true

src/python_testing/spec_parsing_support.py

+27
Original file line numberDiff line numberDiff line change
@@ -496,6 +496,33 @@ def remove_problem(location: typing.Union[CommandPathLocation, FeaturePathLocati
496496
clusters[id] = new
497497

498498
# TODO: All these fixups should be removed BEFORE SVE if at all possible
499+
# Workaround for Color Control cluster - the spec uses a non-standard conformance. Set all to optional now, will need
500+
# to implement either arithmetic conformance handling (once spec changes land here) or specific test
501+
# https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/7808 for spec changes.
502+
# see 3.2.8. Defined Primaries Information Attribute Set, affects Primary<#>X/Y/Intensity attributes.
503+
cc_id = Clusters.ColorControl.id
504+
cc_attr = Clusters.ColorControl.Attributes
505+
affected_attributes = [cc_attr.Primary1X,
506+
cc_attr.Primary1Y,
507+
cc_attr.Primary1Intensity,
508+
cc_attr.Primary2X,
509+
cc_attr.Primary2Y,
510+
cc_attr.Primary2Intensity,
511+
cc_attr.Primary3X,
512+
cc_attr.Primary3Y,
513+
cc_attr.Primary3Intensity,
514+
cc_attr.Primary4X,
515+
cc_attr.Primary4Y,
516+
cc_attr.Primary4Intensity,
517+
cc_attr.Primary5X,
518+
cc_attr.Primary5Y,
519+
cc_attr.Primary5Intensity,
520+
cc_attr.Primary6X,
521+
cc_attr.Primary6Y,
522+
cc_attr.Primary6Intensity,
523+
]
524+
for a in affected_attributes:
525+
clusters[cc_id].attributes[a.attribute_id].conformance = optional()
499526

500527
# Workaround for temp control cluster - this is parsed incorrectly in the DM XML and is missing all its attributes
501528
# Remove this workaround when https://github.com/csa-data-model/projects/issues/330 is fixed

0 commit comments

Comments
 (0)