From 58665bfd014d2b6c6cd2ab125120f34eca83cce9 Mon Sep 17 00:00:00 2001 From: Vivien Nicolas Date: Tue, 21 Feb 2023 18:40:35 +0100 Subject: [PATCH] [TC-SC-5.2] Add a UserPrompt with to manually '!PICS_SDK_CI_ONLY' validates that the DUT has not sent a response (#25198) * [TC-SC-5.2] Add a UserPrompt with to manually '!PICS_SDK_CI_ONLY' validates that the DUT has not sent a response * Update generated tests code --- .../suites/certification/Test_TC_SC_5_2.yaml | 11 +++++ .../chip-tool/zap-generated/test/Commands.h | 43 +++++++++++++------ 2 files changed, 40 insertions(+), 14 deletions(-) diff --git a/src/app/tests/suites/certification/Test_TC_SC_5_2.yaml b/src/app/tests/suites/certification/Test_TC_SC_5_2.yaml index f6748e1877530d..d5abcb70c8a20f 100644 --- a/src/app/tests/suites/certification/Test_TC_SC_5_2.yaml +++ b/src/app/tests/suites/certification/Test_TC_SC_5_2.yaml @@ -135,6 +135,17 @@ tests: - name: "ms" value: 1000 + - label: "Verify there is no response from DUT" + cluster: "LogCommands" + command: "UserPrompt" + PICS: "!PICS_SDK_CI_ONLY" + arguments: + values: + - name: "message" + value: "Verify that there is no response from DUT" + - name: "expectedValue" + value: "y" + - label: "TH sends ViewGroup command" PICS: G.S.F00 command: "ViewGroup" diff --git a/zzz_generated/chip-tool/zap-generated/test/Commands.h b/zzz_generated/chip-tool/zap-generated/test/Commands.h index ab6a7832a1a809..6af25550e4d4ec 100644 --- a/zzz_generated/chip-tool/zap-generated/test/Commands.h +++ b/zzz_generated/chip-tool/zap-generated/test/Commands.h @@ -41956,7 +41956,7 @@ class Test_TC_SC_5_1Suite : public TestCommand class Test_TC_SC_5_2Suite : public TestCommand { public: - Test_TC_SC_5_2Suite(CredentialIssuerCommands * credsIssuerConfig) : TestCommand("Test_TC_SC_5_2", 14, credsIssuerConfig) + Test_TC_SC_5_2Suite(CredentialIssuerCommands * credsIssuerConfig) : TestCommand("Test_TC_SC_5_2", 15, credsIssuerConfig) { AddArgument("nodeId", 0, UINT64_MAX, &mNodeId); AddArgument("cluster", &mCluster); @@ -42037,6 +42037,10 @@ class Test_TC_SC_5_2Suite : public TestCommand shouldContinue = true; break; case 9: + VerifyOrReturn(CheckValue("status", chip::to_underlying(status.mStatus), 0)); + shouldContinue = true; + break; + case 10: VerifyOrReturn(CheckValue("status", chip::to_underlying(status.mStatus), 0)); { chip::app::Clusters::Groups::Commands::ViewGroupResponse::DecodableType value; @@ -42046,7 +42050,7 @@ class Test_TC_SC_5_2Suite : public TestCommand VerifyOrReturn(CheckValueAsString("groupName", value.groupName, chip::CharSpan("Test Group 0101", 15))); } break; - case 10: + case 11: VerifyOrReturn(CheckValue("status", chip::to_underlying(status.mStatus), 0)); { chip::app::Clusters::Groups::Commands::ViewGroupResponse::DecodableType value; @@ -42056,15 +42060,15 @@ class Test_TC_SC_5_2Suite : public TestCommand VerifyOrReturn(CheckValueAsString("groupName", value.groupName, chip::CharSpan("", 0))); } break; - case 11: - VerifyOrReturn(CheckValue("status", chip::to_underlying(status.mStatus), 0)); - break; case 12: VerifyOrReturn(CheckValue("status", chip::to_underlying(status.mStatus), 0)); break; case 13: VerifyOrReturn(CheckValue("status", chip::to_underlying(status.mStatus), 0)); break; + case 14: + VerifyOrReturn(CheckValue("status", chip::to_underlying(status.mStatus), 0)); + break; default: LogErrorOnFailure(ContinueOnChipMainThread(CHIP_ERROR_INVALID_ARGUMENT)); } @@ -42232,7 +42236,18 @@ class Test_TC_SC_5_2Suite : public TestCommand return WaitForMs(kIdentityAlpha, value); } case 9: { - LogStep(9, "TH sends ViewGroup command"); + LogStep(9, "Verify there is no response from DUT"); + VerifyOrDo(!ShouldSkip("!PICS_SDK_CI_ONLY"), return ContinueOnChipMainThread(CHIP_NO_ERROR)); + ListFreer listFreer; + chip::app::Clusters::LogCommands::Commands::UserPrompt::Type value; + value.message = + chip::Span("Verify that there is no response from DUTgarbage: not in length on purpose", 41); + value.expectedValue.Emplace(); + value.expectedValue.Value() = chip::Span("ygarbage: not in length on purpose", 1); + return UserPrompt(kIdentityAlpha, value); + } + case 10: { + LogStep(10, "TH sends ViewGroup command"); VerifyOrDo(!ShouldSkip("G.S.F00"), return ContinueOnChipMainThread(CHIP_NO_ERROR)); ListFreer listFreer; chip::app::Clusters::Groups::Commands::ViewGroup::Type value; @@ -42242,8 +42257,8 @@ class Test_TC_SC_5_2Suite : public TestCommand ); } - case 10: { - LogStep(10, "TH sends ViewGroup command"); + case 11: { + LogStep(11, "TH sends ViewGroup command"); VerifyOrDo(!ShouldSkip("!(G.S.F00)"), return ContinueOnChipMainThread(CHIP_NO_ERROR)); ListFreer listFreer; chip::app::Clusters::Groups::Commands::ViewGroup::Type value; @@ -42253,8 +42268,8 @@ class Test_TC_SC_5_2Suite : public TestCommand ); } - case 11: { - LogStep(11, "TH removes the GroupKeySet"); + case 12: { + LogStep(12, "TH removes the GroupKeySet"); ListFreer listFreer; chip::app::Clusters::GroupKeyManagement::Commands::KeySetRemove::Type value; value.groupKeySetID = 419U; @@ -42263,8 +42278,8 @@ class Test_TC_SC_5_2Suite : public TestCommand ); } - case 12: { - LogStep(12, "TH cleans up groups using RemoveAllGroups command"); + case 13: { + LogStep(13, "TH cleans up groups using RemoveAllGroups command"); ListFreer listFreer; chip::app::Clusters::Groups::Commands::RemoveAllGroups::Type value; return SendCommand(kIdentityAlpha, GetEndpoint(1), Groups::Id, Groups::Commands::RemoveAllGroups::Id, value, @@ -42272,8 +42287,8 @@ class Test_TC_SC_5_2Suite : public TestCommand ); } - case 13: { - LogStep(13, "TH removes ACL Operate privileges for Group 0x0103"); + case 14: { + LogStep(14, "TH removes ACL Operate privileges for Group 0x0103"); ListFreer listFreer; chip::app::DataModel::List value;