Create a separate 'constants' source set in src/app (project-chip#32228)

andy31415 · andreilitvin · web-flow · commit a6fd6ce388bc · 2024-02-23T14:55:29.000Z
* Create constants file

* Restyle

* Place everything into a `Revision` namespace to avoid name collision with a Tags enum

* Rename

---------

Co-authored-by: Andrei Litvin &lt;andreilitvin@google.com&gt;
diff --git a/src/app/BUILD.gn b/src/app/BUILD.gn
@@ -76,14 +76,6 @@ source_set("app_config") {
   deps = [ ":app_buildconfig" ]
 }
 
-source_set("revision_info") {
-  sources = [
-    "DataModelRevision.h",
-    "InteractionModelRevision.h",
-    "SpecificationVersion.h",
-  ]
-}
-
 source_set("paths") {
   sources = [
     "AttributePathParams.h",
@@ -130,6 +122,17 @@ config("config-controller-dynamic-server") {
   ]
 }
 
+source_set("constants") {
+  sources = [
+    "InteractionModelTimeout.h",
+    "SpecificationDefinedRevisions.h",
+  ]
+  public_deps = [
+    "${chip_root}/src/lib/core",
+    "${chip_root}/src/system",
+  ]
+}
+
 # interaction-model is a static-library because it currently requires global functions (app/util/...) that are stubbed in different test files that depend on the app static_library
 # which in tern depens on the interaction-model.
 # Using source_set prevents the unit test to build correctly.
@@ -149,7 +152,6 @@ static_library("interaction-model") {
     "InteractionModelEngine.cpp",
     "InteractionModelEngine.h",
     "InteractionModelHelper.h",
-    "InteractionModelTimeout.h",
     "OperationalSessionSetup.cpp",
     "OperationalSessionSetup.h",
     "OperationalSessionSetupPool.h",
@@ -181,6 +183,7 @@ static_library("interaction-model") {
 
   public_deps = [
     ":app_config",
+    ":constants",
     ":paths",
     ":subscription-manager",
     "${chip_root}/src/app/MessageDef",
@@ -276,9 +279,9 @@ static_library("app") {
 
   public_deps = [
     ":app_config",
+    ":constants",
     ":global-attributes",
     ":interaction-model",
-    ":revision_info",
     "${chip_root}/src/app/data-model",
     "${chip_root}/src/app/icd/server:icd-server-config",
     "${chip_root}/src/lib/address_resolve",
diff --git a/src/app/DataModelRevision.h b/src/app/DataModelRevision.h
diff --git a/src/app/InteractionModelTimeout.h b/src/app/InteractionModelTimeout.h
@@ -18,7 +18,6 @@
 #pragma once
 
 #include <system/SystemClock.h>
-#include <transport/Session.h>
 
 namespace chip {
 namespace app {
diff --git a/src/app/MessageDef/BUILD.gn b/src/app/MessageDef/BUILD.gn
@@ -113,8 +113,8 @@ source_set("MessageDef") {
 
   deps = [
     "${chip_root}/src/app:app_config",
+    "${chip_root}/src/app:constants",
     "${chip_root}/src/app:paths",
-    "${chip_root}/src/app:revision_info",
     "${chip_root}/src/lib/core",
     "${chip_root}/src/lib/support",
     "${chip_root}/src/protocols/interaction_model",
diff --git a/src/app/MessageDef/InvokeRequestMessage.cpp b/src/app/MessageDef/InvokeRequestMessage.cpp
@@ -74,7 +74,7 @@ CHIP_ERROR InvokeRequestMessage::Parser::PrettyPrint() const
             PRETTY_PRINT_DECDEPTH();
         }
         break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/InvokeResponseMessage.cpp b/src/app/MessageDef/InvokeResponseMessage.cpp
@@ -73,7 +73,7 @@ CHIP_ERROR InvokeResponseMessage::Parser::PrettyPrint() const
         }
 #endif // CHIP_DETAIL_LOGGING
         break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/MessageBuilder.cpp b/src/app/MessageDef/MessageBuilder.cpp
@@ -24,8 +24,7 @@ namespace chip {
 namespace app {
 CHIP_ERROR MessageBuilder::EncodeInteractionModelRevision()
 {
-    return mpWriter->Put(TLV::ContextTag(kInteractionModelRevisionTag),
-                         static_cast<InteractionModelRevision>(CHIP_DEVICE_INTERACTION_MODEL_REVISION));
+    return mpWriter->Put(TLV::ContextTag(Revision::kInteractionModelRevisionTag), Revision::kInteractionModelRevision);
 }
 } // namespace app
 } // namespace chip
diff --git a/src/app/MessageDef/MessageBuilder.h b/src/app/MessageDef/MessageBuilder.h
@@ -18,7 +18,7 @@
 #pragma once
 
 #include "StructBuilder.h"
-#include <app/InteractionModelRevision.h>
+#include <app/SpecificationDefinedRevisions.h>
 #include <app/util/basic-types.h>
 
 namespace chip {
diff --git a/src/app/MessageDef/MessageDefHelper.cpp b/src/app/MessageDef/MessageDefHelper.cpp
@@ -24,7 +24,7 @@
 #include "MessageDefHelper.h"
 #include <algorithm>
 #include <app/AppConfig.h>
-#include <app/InteractionModelRevision.h>
+#include <app/SpecificationDefinedRevisions.h>
 #include <app/util/basic-types.h>
 #include <inttypes.h>
 #include <lib/support/logging/CHIPLogging.h>
diff --git a/src/app/MessageDef/MessageParser.cpp b/src/app/MessageDef/MessageParser.cpp
@@ -16,7 +16,7 @@
 
 #include "MessageParser.h"
 #include "MessageDefHelper.h"
-#include <app/InteractionModelRevision.h>
+#include <app/SpecificationDefinedRevisions.h>
 
 namespace chip {
 namespace app {
@@ -52,7 +52,7 @@ CHIP_ERROR MessageParser::CheckInteractionModelRevision(TLV::TLVReader & aReader
 
 CHIP_ERROR MessageParser::GetInteractionModelRevision(InteractionModelRevision * const apInteractionModelRevision) const
 {
-    return GetUnsignedInteger(kInteractionModelRevisionTag, apInteractionModelRevision);
+    return GetUnsignedInteger(Revision::kInteractionModelRevisionTag, apInteractionModelRevision);
 }
 
 } // namespace app
diff --git a/src/app/MessageDef/MessageParser.h b/src/app/MessageDef/MessageParser.h
@@ -20,7 +20,7 @@
 
 #include "StructParser.h"
 #include <app/AppConfig.h>
-#include <app/InteractionModelRevision.h>
+#include <app/SpecificationDefinedRevisions.h>
 #include <app/util/basic-types.h>
 
 namespace chip {
diff --git a/src/app/MessageDef/ReadRequestMessage.cpp b/src/app/MessageDef/ReadRequestMessage.cpp
@@ -91,7 +91,7 @@ CHIP_ERROR ReadRequestMessage::Parser::PrettyPrint() const
         }
 #endif // CHIP_DETAIL_LOGGING
         break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/ReportDataMessage.cpp b/src/app/MessageDef/ReportDataMessage.cpp
@@ -112,7 +112,7 @@ CHIP_ERROR ReportDataMessage::Parser::PrettyPrint() const
             }
 #endif // CHIP_DETAIL_LOGGING
             break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/StatusResponseMessage.cpp b/src/app/MessageDef/StatusResponseMessage.cpp
@@ -52,7 +52,7 @@ CHIP_ERROR StatusResponseMessage::Parser::PrettyPrint() const
             }
 #endif // CHIP_DETAIL_LOGGING
             break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/SubscribeRequestMessage.cpp b/src/app/MessageDef/SubscribeRequestMessage.cpp
@@ -115,7 +115,7 @@ CHIP_ERROR SubscribeRequestMessage::Parser::PrettyPrint() const
         }
 #endif // CHIP_DETAIL_LOGGING
         break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/SubscribeResponseMessage.cpp b/src/app/MessageDef/SubscribeResponseMessage.cpp
@@ -59,7 +59,7 @@ CHIP_ERROR SubscribeResponseMessage::Parser::PrettyPrint() const
             }
 #endif // CHIP_DETAIL_LOGGING
             break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/TimedRequestMessage.cpp b/src/app/MessageDef/TimedRequestMessage.cpp
@@ -49,7 +49,7 @@ CHIP_ERROR TimedRequestMessage::Parser::PrettyPrint() const
             }
 #endif // CHIP_DETAIL_LOGGING
             break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/WriteRequestMessage.cpp b/src/app/MessageDef/WriteRequestMessage.cpp
@@ -88,7 +88,7 @@ CHIP_ERROR WriteRequestMessage::Parser::PrettyPrint() const
         }
 #endif // CHIP_DETAIL_LOGGING
         break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/MessageDef/WriteResponseMessage.cpp b/src/app/MessageDef/WriteResponseMessage.cpp
@@ -54,7 +54,7 @@ CHIP_ERROR WriteResponseMessage::Parser::PrettyPrint() const
             ReturnErrorOnFailure(writeResponses.PrettyPrint());
             PRETTY_PRINT_DECDEPTH();
             break;
-        case kInteractionModelRevisionTag:
+        case Revision::kInteractionModelRevisionTag:
             ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));
             break;
         default:
diff --git a/src/app/SpecificationDefinedRevisions.h b/src/app/SpecificationDefinedRevisions.h
@@ -20,16 +20,37 @@
 #include <inttypes.h>
 #include <stddef.h>
 
+#include <lib/core/DataModelTypes.h>
+
+namespace chip {
+namespace Revision {
+
 /**
- * CHIP_DEVICE_INTERACTION_MODEL_REVISION
- *
  * A monothonic number identifying the interaction model revision.
  *
  * See section 8.1.1. "Revision History" in the "Interaction Model
  * Specification" chapter of the core Matter specification.
  */
-#ifndef CHIP_DEVICE_INTERACTION_MODEL_REVISION
-#define CHIP_DEVICE_INTERACTION_MODEL_REVISION 11
-#endif
+inline constexpr InteractionModelRevision kInteractionModelRevision = 11;
+inline constexpr uint8_t kInteractionModelRevisionTag               = 0xFF;
+
+/**
+ * A monotonic number identifying the revision number of the Data Model against
+ * which the Node is certified.
+ *
+ * See section 7.1.1. "Revision History" in the "Data Model Specification"
+ * chapter of the core Matter specification.
+ */
+inline constexpr uint16_t kDataModelRevision = 17;
+
+/*
+ * A number identifying the specification version against which the
+ * Node is certified.
+ *
+ * See section 11.1.5.22. "SpecificationVersion Attribute" in "Service and
+ * Device Management" chapter of the core Matter specification.
+ */
+inline constexpr uint32_t kSpecificationVersion = 0x01030000;
 
-inline constexpr uint8_t kInteractionModelRevisionTag = 0xFF;
+} // namespace Revision
+} // namespace chip
diff --git a/src/app/SpecificationVersion.h b/src/app/SpecificationVersion.h
diff --git a/src/app/clusters/basic-information/basic-information.cpp b/src/app/clusters/basic-information/basic-information.cpp
@@ -20,10 +20,9 @@
 
 #include <app-common/zap-generated/attributes/Accessors.h>
 #include <app-common/zap-generated/cluster-objects.h>
-#include <app/DataModelRevision.h>
 #include <app/EventLogging.h>
 #include <app/InteractionModelEngine.h>
-#include <app/SpecificationVersion.h>
+#include <app/SpecificationDefinedRevisions.h>
 #include <app/util/attribute-storage.h>
 #include <lib/core/CHIPConfig.h>
 #include <platform/CHIPDeviceLayer.h>
@@ -314,7 +313,7 @@ CHIP_ERROR BasicAttrAccess::Read(const ConcreteReadAttributePath & aPath, Attrib
 
 CHIP_ERROR BasicAttrAccess::ReadDataModelRevision(AttributeValueEncoder & aEncoder)
 {
-    uint16_t revision = CHIP_DEVICE_DATA_MODEL_REVISION;
+    uint16_t revision = Revision::kDataModelRevision;
     return aEncoder.Encode(revision);
 }
 
@@ -399,7 +398,7 @@ CHIP_ERROR BasicAttrAccess::ReadProductAppearance(AttributeValueEncoder & aEncod
 
 CHIP_ERROR BasicAttrAccess::ReadSpecificationVersion(AttributeValueEncoder & aEncoder)
 {
-    uint32_t specification_version = CHIP_DEVICE_SPECIFICATION_VERSION;
+    uint32_t specification_version = Revision::kSpecificationVersion;
     return aEncoder.Encode(specification_version);
 }
 
diff --git a/src/protocols/secure_channel/BUILD.gn b/src/protocols/secure_channel/BUILD.gn
@@ -88,5 +88,5 @@ static_library("secure_channel") {
     "${chip_root}/src/transport",
   ]
 
-  deps = [ "${chip_root}/src/app:revision_info" ]
+  deps = [ "${chip_root}/src/app:constants" ]
 }
diff --git a/src/protocols/secure_channel/PairingSession.cpp b/src/protocols/secure_channel/PairingSession.cpp
@@ -18,9 +18,7 @@
 
 #include <protocols/secure_channel/PairingSession.h>
 
-#include <app/DataModelRevision.h>
-#include <app/InteractionModelRevision.h>
-#include <app/SpecificationVersion.h>
+#include <app/SpecificationDefinedRevisions.h>
 #include <lib/core/CHIPConfig.h>
 #include <lib/core/TLVTypes.h>
 #include <lib/support/SafeInt.h>
@@ -112,13 +110,13 @@ CHIP_ERROR PairingSession::EncodeSessionParameters(TLV::Tag tag, const Optional<
     ReturnErrorOnFailure(tlvWriter.Put(TLV::ContextTag(SessionParameters::Tag::kSessionActiveThreshold),
                                        mrpLocalConfig.mActiveThresholdTime.count()));
 
-    uint16_t dataModel = CHIP_DEVICE_DATA_MODEL_REVISION;
+    uint16_t dataModel = Revision::kDataModelRevision;
     ReturnErrorOnFailure(tlvWriter.Put(TLV::ContextTag(SessionParameters::Tag::kDataModelRevision), dataModel));
 
-    uint16_t interactionModel = CHIP_DEVICE_INTERACTION_MODEL_REVISION;
+    uint16_t interactionModel = Revision::kInteractionModelRevision;
     ReturnErrorOnFailure(tlvWriter.Put(TLV::ContextTag(SessionParameters::Tag::kInteractionModelRevision), interactionModel));
 
-    uint32_t specVersion = CHIP_DEVICE_SPECIFICATION_VERSION;
+    uint32_t specVersion = Revision::kSpecificationVersion;
     ReturnErrorOnFailure(tlvWriter.Put(TLV::ContextTag(SessionParameters::Tag::kSpecificationVersion), specVersion));
 
     uint16_t maxPathsPerInvoke = CHIP_CONFIG_MAX_PATHS_PER_INVOKE;

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ CHIP_ERROR InvokeRequestMessage::Parser::PrettyPrint() const`
`74`	`74`	`PRETTY_PRINT_DECDEPTH();`
`75`	`75`	`}`
`76`	`76`	`break;`
`77`		`- case kInteractionModelRevisionTag:`
	`77`	`+ case Revision::kInteractionModelRevisionTag:`
`78`	`78`	`ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));`
`79`	`79`	`break;`
`80`	`80`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ CHIP_ERROR InvokeResponseMessage::Parser::PrettyPrint() const`
`73`	`73`	`}`
`74`	`74`	`#endif // CHIP_DETAIL_LOGGING`
`75`	`75`	`break;`
`76`		`- case kInteractionModelRevisionTag:`
	`76`	`+ case Revision::kInteractionModelRevisionTag:`
`77`	`77`	`ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));`
`78`	`78`	`break;`
`79`	`79`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -24,8 +24,7 @@ namespace chip {`
`24`	`24`	`namespace app {`
`25`	`25`	`CHIP_ERROR MessageBuilder::EncodeInteractionModelRevision()`
`26`	`26`	`{`
`27`		`- return mpWriter->Put(TLV::ContextTag(kInteractionModelRevisionTag),`
`28`		`- static_cast<InteractionModelRevision>(CHIP_DEVICE_INTERACTION_MODEL_REVISION));`
	`27`	`+ return mpWriter->Put(TLV::ContextTag(Revision::kInteractionModelRevisionTag), Revision::kInteractionModelRevision);`
`29`	`28`	`}`
`30`	`29`	`} // namespace app`
`31`	`30`	`} // namespace chip`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@`
`16`	`16`
`17`	`17`	`#include "MessageParser.h"`
`18`	`18`	`#include "MessageDefHelper.h"`
`19`		`-#include <app/InteractionModelRevision.h>`
	`19`	`+#include <app/SpecificationDefinedRevisions.h>`
`20`	`20`
`21`	`21`	`namespace chip {`
`22`	`22`	`namespace app {`
`@@ -52,7 +52,7 @@ CHIP_ERROR MessageParser::CheckInteractionModelRevision(TLV::TLVReader & aReader`
`52`	`52`
`53`	`53`	`CHIP_ERROR MessageParser::GetInteractionModelRevision(InteractionModelRevision * const apInteractionModelRevision) const`
`54`	`54`	`{`
`55`		`- return GetUnsignedInteger(kInteractionModelRevisionTag, apInteractionModelRevision);`
	`55`	`+ return GetUnsignedInteger(Revision::kInteractionModelRevisionTag, apInteractionModelRevision);`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`} // namespace app`
Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ CHIP_ERROR ReadRequestMessage::Parser::PrettyPrint() const`
`91`	`91`	`}`
`92`	`92`	`#endif // CHIP_DETAIL_LOGGING`
`93`	`93`	`break;`
`94`		`- case kInteractionModelRevisionTag:`
	`94`	`+ case Revision::kInteractionModelRevisionTag:`
`95`	`95`	`ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));`
`96`	`96`	`break;`
`97`	`97`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -112,7 +112,7 @@ CHIP_ERROR ReportDataMessage::Parser::PrettyPrint() const`
`112`	`112`	`}`
`113`	`113`	`#endif // CHIP_DETAIL_LOGGING`
`114`	`114`	`break;`
`115`		`- case kInteractionModelRevisionTag:`
	`115`	`+ case Revision::kInteractionModelRevisionTag:`
`116`	`116`	`ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));`
`117`	`117`	`break;`
`118`	`118`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ CHIP_ERROR StatusResponseMessage::Parser::PrettyPrint() const`
`52`	`52`	`}`
`53`	`53`	`#endif // CHIP_DETAIL_LOGGING`
`54`	`54`	`break;`
`55`		`- case kInteractionModelRevisionTag:`
	`55`	`+ case Revision::kInteractionModelRevisionTag:`
`56`	`56`	`ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));`
`57`	`57`	`break;`
`58`	`58`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ CHIP_ERROR SubscribeRequestMessage::Parser::PrettyPrint() const`
`115`	`115`	`}`
`116`	`116`	`#endif // CHIP_DETAIL_LOGGING`
`117`	`117`	`break;`
`118`		`- case kInteractionModelRevisionTag:`
	`118`	`+ case Revision::kInteractionModelRevisionTag:`
`119`	`119`	`ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));`
`120`	`120`	`break;`
`121`	`121`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ CHIP_ERROR SubscribeResponseMessage::Parser::PrettyPrint() const`
`59`	`59`	`}`
`60`	`60`	`#endif // CHIP_DETAIL_LOGGING`
`61`	`61`	`break;`
`62`		`- case kInteractionModelRevisionTag:`
	`62`	`+ case Revision::kInteractionModelRevisionTag:`
`63`	`63`	`ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));`
`64`	`64`	`break;`
`65`	`65`	`default:`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ CHIP_ERROR TimedRequestMessage::Parser::PrettyPrint() const`
`49`	`49`	`}`
`50`	`50`	`#endif // CHIP_DETAIL_LOGGING`
`51`	`51`	`break;`
`52`		`- case kInteractionModelRevisionTag:`
	`52`	`+ case Revision::kInteractionModelRevisionTag:`
`53`	`53`	`ReturnErrorOnFailure(MessageParser::CheckInteractionModelRevision(reader));`
`54`	`54`	`break;`
`55`	`55`	`default:`