Return invalid comamnd on MoveColor command with RateX and RateY both at 0. Add a test step to TC_CC_5.2 to validate the expected failure

Author: jmartinez-silabs

src/app/clusters/color-control-server/color-control-server.cpp

@@ -2251,7 +2251,7 @@ bool ColorControlServer::moveColorCommand(app::CommandHandler * commandObj, cons
 
     if (rateX == 0 && rateY == 0)
     {
-        commandObj->AddStatus(commandPath, Status::Success);
+        commandObj->AddStatus(commandPath, Status::InvalidCommand);
         return true;
     }
 

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

@@ -217,6 +217,22 @@ tests:
               minValue: 0
               maxValue: 3
 
+    - label: "Step 5a: TH sends MoveColor command to DUT with invalid parameters RateX=0 and RateY=0"
+      command: "MoveColor"
+      PICS: CC.S.F03 && CC.S.C08.Rsp
+      arguments:
+          values:
+              - name: "RateX"
+                value: 0
+              - name: "RateY"
+                value: 0
+              - name: "OptionsMask"
+                value: 0
+              - name: "OptionsOverride"
+                value: 0
+      response:
+          error: INVALID_COMMAND
+
     - label: "Turn off light that we turned on"
       PICS: OO.S.C00.Rsp
       cluster: "On/Off"