Renamed TestOperationlStateDelegate to TestOperationalStateClusterObj…

…ects.
project-chip · Aug 3, 2023 · 1338948 · 1338948
1 parent 16608a9
commit 1338948
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 8 deletions.
diff --git a/src/app/clusters/operational-state-server/operational-state-cluster-objects.h b/src/app/clusters/operational-state-server/operational-state-cluster-objects.h
@@ -22,6 +22,8 @@
 #include <app/util/af-enums.h>
 #include <lib/support/CommonIterator.h>
 
+#include <utility>
+
 namespace chip {
 namespace app {
 namespace Clusters {
@@ -39,7 +41,7 @@ struct GenericOperationalState : public app::Clusters::detail::Structs::Operatio
 {
     GenericOperationalState(uint8_t state = to_underlying(OperationalStateEnum::kStopped), Optional<CharSpan> label = NullOptional)
     {
-        Set(state, label);
+        Set(state, std::move(label));
     }
 
     GenericOperationalState(const GenericOperationalState & op) { *this = op; }
@@ -85,7 +87,7 @@ struct GenericOperationalError : public app::Clusters::detail::Structs::ErrorSta
     GenericOperationalError(uint8_t state, Optional<chip::CharSpan> label = NullOptional,
                             Optional<chip::CharSpan> details = NullOptional)
     {
-        Set(state, label, details);
+        Set(state, std::move(label), std::move(details));
     }
 
     GenericOperationalError(const GenericOperationalError & error) { *this = error; }
@@ -148,7 +150,7 @@ struct GenericOperationalError : public app::Clusters::detail::Structs::ErrorSta
  */
 struct GenericOperationalPhase
 {
-    GenericOperationalPhase(app::DataModel::Nullable<CharSpan> name) { Set(name); }
+    GenericOperationalPhase(app::DataModel::Nullable<CharSpan> name) { Set(std::move(name)); }
 
     GenericOperationalPhase(const GenericOperationalPhase & ph) { *this = ph; }
 

diff --git a/src/app/tests/BUILD.gn b/src/app/tests/BUILD.gn
@@ -98,7 +98,7 @@ source_set("scenes-table-test-srcs") {
 
 source_set("operational-state-test-srcs") {
   sources = [
-    "${chip_root}/src/app/clusters/operational-state-server/operational-state-server.h",
+    "${chip_root}/src/app/clusters/operational-state-server/operational-state-cluster-objects.h",
   ]
 
   public_deps = [
@@ -133,7 +133,7 @@ chip_test_suite("tests") {
     "TestInteractionModelEngine.cpp",
     "TestMessageDef.cpp",
     "TestNumericAttributeTraits.cpp",
-    "TestOperationalStateDelegate.cpp",
+    "TestOperationalStateClusterObjects.cpp",
     "TestPendingNotificationMap.cpp",
     "TestReadInteraction.cpp",
     "TestReportingEngine.cpp",

diff --git a/...pp/tests/TestOperationalStateDelegate.cpp → ...ts/TestOperationalStateClusterObjects.cpp b/...pp/tests/TestOperationalStateDelegate.cpp → ...ts/TestOperationalStateClusterObjects.cpp
@@ -579,13 +579,13 @@ int TestTearDown(void * inContext)
 
 } // namespace
 
-int TestOperationalStateDelegate()
+int TestOperationalStateClusterObjects()
 {
-    nlTestSuite theSuite = { "Test Operational State delegate tests", &sTests[0], TestSetup, TestTearDown };
+    nlTestSuite theSuite = { "Test Operational State Cluster Objects tests", &sTests[0], TestSetup, TestTearDown };
 
     // Run test suit againt one context.
     nlTestRunner(&theSuite, nullptr);
     return nlTestRunnerStats(&theSuite);
 }
 
-CHIP_REGISTER_TEST_SUITE(TestOperationalStateDelegate)
+CHIP_REGISTER_TEST_SUITE(TestOperationalStateClusterObjects)