stop using attribute storage for TimeSource to make the code more

suitable for unit testing

Author: fessehaeve

examples/all-clusters-app/all-clusters-common/all-clusters-app.matter

@@ -7590,7 +7590,7 @@ endpoint 0 {
     emits event MissingTrustedTimeSource;
     callback attribute UTCTime;
     callback attribute granularity;
-    ram      attribute timeSource default = 0x00;
+    callback attribute timeSource;
     callback attribute trustedTimeSource;
     callback attribute defaultNTP;
     callback attribute timeZone;

examples/all-clusters-app/all-clusters-common/all-clusters-app.zap

@@ -4915,10 +4915,10 @@
               "side": "server",
               "type": "TimeSourceEnum",
               "included": 1,
-              "storageOption": "RAM",
+              "storageOption": "External",
               "singleton": 0,
               "bounded": 0,
-              "defaultValue": "0x00",
+              "defaultValue": null,
               "reportable": 1,
               "minInterval": 1,
               "maxInterval": 65534,

examples/light-switch-app/light-switch-common/light-switch-app.matter

@@ -2778,7 +2778,7 @@ endpoint 0 {
     emits event MissingTrustedTimeSource;
     callback attribute UTCTime;
     callback attribute granularity;
-    ram      attribute timeSource default = 0x00;
+    callback attribute timeSource;
     callback attribute trustedTimeSource;
     callback attribute defaultNTP;
     callback attribute timeZone;

examples/light-switch-app/light-switch-common/light-switch-app.zap

@@ -3599,10 +3599,10 @@
               "side": "server",
               "type": "TimeSourceEnum",
               "included": 1,
-              "storageOption": "RAM",
+              "storageOption": "External",
               "singleton": 0,
               "bounded": 0,
-              "defaultValue": "0x00",
+              "defaultValue": null,
               "reportable": 1,
               "minInterval": 1,
               "maxInterval": 65534,

src/app/clusters/time-synchronization-server/time-synchronization-server.cpp

@@ -381,12 +381,8 @@ void TimeSynchronizationServer::OnTimeSyncCompletionFn(TimeSourceEnum timeSource
         }
         return;
     }
-    mGranularity  = granularity;
-    Status status = TimeSource::Set(kRootEndpointId, timeSource);
-    if (!(status == Status::Success || status == Status::UnsupportedAttribute))
-    {
-        ChipLogError(Zcl, "Writing TimeSource failed.");
-    }
+    mGranularity = granularity;
+    mTimeSource  = timeSource;
 }
 
 void TimeSynchronizationServer::OnFallbackNTPCompletionFn(bool timeSyncSuccessful)
@@ -395,11 +391,7 @@ void TimeSynchronizationServer::OnFallbackNTPCompletionFn(bool timeSyncSuccessfu
     {
         mGranularity = GranularityEnum::kMillisecondsGranularity;
         // Non-matter SNTP because we know it's external and there's only one source
-        Status status = TimeSource::Set(kRootEndpointId, TimeSourceEnum::kNonMatterSNTP);
-        if (!(status == Status::Success || status == Status::UnsupportedAttribute))
-        {
-            ChipLogError(Zcl, "Writing TimeSource failed.");
-        }
+        mTimeSource = TimeSourceEnum::kNonMatterSNTP;
     }
     else
     {
@@ -795,13 +787,8 @@ CHIP_ERROR TimeSynchronizationServer::SetUTCTime(EndpointId ep, uint64_t utcTime
         return err;
     }
     GetDelegate()->UTCTimeAvailabilityChanged(utcTime);
-    mGranularity  = static_cast<GranularityEnum>(to_underlying(granularity) - 1);
-    Status status = TimeSource::Set(ep, source);
-    if (!(status == Status::Success || status == Status::UnsupportedAttribute))
-    {
-        ChipLogError(Zcl, "Writing TimeSource failed.");
-        return StatusIB(status).ToChipError();
-    }
+    mGranularity = static_cast<GranularityEnum>(to_underlying(granularity) - 1);
+    mTimeSource  = source;
     return CHIP_NO_ERROR;
 }
 

src/app/clusters/time-synchronization-server/time-synchronization-server.h

@@ -27,7 +27,6 @@
 #include <app/AppConfig.h>
 #include <app/server/Server.h>
 #include <app/util/af-types.h>
-#include <app/util/config.h>
 #include <credentials/FabricTable.h>
 #include <lib/core/TLV.h>
 
@@ -139,6 +138,7 @@ class TimeSynchronizationServer : public FabricTable::Delegate
     TimeSyncDataProvider::TimeZoneObj mTimeZoneObj{ Span<TimeSyncDataProvider::TimeZoneStore>(mTz), 0 };
     TimeSyncDataProvider::DSTOffsetObj mDstOffsetObj{ DataModel::List<Structs::DSTOffsetStruct::Type>(mDst), 0 };
     GranularityEnum mGranularity = GranularityEnum::kNoTimeGranularity;
+    TimeSourceEnum mTimeSource   = TimeSourceEnum::kNone;
 
     TimeSyncDataProvider::TimeZoneStore mTz[CHIP_CONFIG_TIME_ZONE_LIST_MAX_SIZE];
     Structs::DSTOffsetStruct::Type mDst[CHIP_CONFIG_DST_OFFSET_LIST_MAX_SIZE];

src/app/zap-templates/zcl/zcl-with-test-extensions.json

@@ -383,7 +383,8 @@
             "LocalTime",
             "Granularity",
             "TimeZoneListMaxSize",
-            "DSTOffsetListMaxSize"
+            "DSTOffsetListMaxSize",
+            "TimeSource"
         ],
         "Temperature Control": ["SupportedTemperatureLevels"],
         "Dishwasher Mode": [

src/app/zap-templates/zcl/zcl.json

@@ -381,7 +381,8 @@
             "LocalTime",
             "Granularity",
             "TimeZoneListMaxSize",
-            "DSTOffsetListMaxSize"
+            "DSTOffsetListMaxSize",
+            "TimeSource"
         ],
         "Temperature Control": ["SupportedTemperatureLevels"],
         "Dishwasher Mode": [