Add logging (project-chip#42921)

Author: andy31415

src/app/clusters/groups-server/GroupsCluster.cpp

@@ -341,7 +341,7 @@ bool emberAfGroupsClusterRemoveAllGroupsCallback(app::CommandHandler * commandOb
     }
 #endif
 
-    TEMPORARY_RETURN_IGNORED provider->RemoveEndpoint(fabricIndex, commandPath.mEndpointId);
+    LogErrorOnFailure(provider->RemoveEndpoint(fabricIndex, commandPath.mEndpointId));
     status = Status::Success;
     MatterReportingAttributeChangeCallback(kRootEndpointId, GroupKeyManagement::Id, GroupKeyManagement::Attributes::GroupTable::Id);
 exit:

src/app/clusters/scenes-server/CodegenIntegration.cpp

@@ -167,43 +167,43 @@ void ScenesServer::GroupWillBeRemoved(FabricIndex aFabricIx, EndpointId aEndpoin
     ScenesManagementCluster * cluster = FindClusterOnEndpoint(aEndpointId);
     VerifyOrReturn(cluster != nullptr);
 
-    TEMPORARY_RETURN_IGNORED cluster->GroupWillBeRemoved(aFabricIx, aGroupId);
+    LogErrorOnFailure(cluster->GroupWillBeRemoved(aFabricIx, aGroupId));
 }
 
 void ScenesServer::MakeSceneInvalid(EndpointId aEndpointId, FabricIndex aFabricIx)
 {
     ScenesManagementCluster * cluster = FindClusterOnEndpoint(aEndpointId);
     VerifyOrReturn(cluster != nullptr);
 
-    TEMPORARY_RETURN_IGNORED cluster->MakeSceneInvalid(aFabricIx);
+    LogErrorOnFailure(cluster->MakeSceneInvalid(aFabricIx));
 }
 
 void ScenesServer::MakeSceneInvalidForAllFabrics(EndpointId aEndpointId)
 {
     ScenesManagementCluster * cluster = FindClusterOnEndpoint(aEndpointId);
     VerifyOrReturn(cluster != nullptr);
-    TEMPORARY_RETURN_IGNORED cluster->MakeSceneInvalidForAllFabrics();
+    LogErrorOnFailure(cluster->MakeSceneInvalidForAllFabrics());
 }
 
 void ScenesServer::StoreCurrentScene(FabricIndex aFabricIx, EndpointId aEndpointId, GroupId aGroupId, SceneId aSceneId)
 {
     ScenesManagementCluster * cluster = FindClusterOnEndpoint(aEndpointId);
     VerifyOrReturn(cluster != nullptr);
-    TEMPORARY_RETURN_IGNORED cluster->StoreCurrentScene(aFabricIx, aGroupId, aSceneId);
+    LogErrorOnFailure(cluster->StoreCurrentScene(aFabricIx, aGroupId, aSceneId));
 }
 
 void ScenesServer::RecallScene(FabricIndex aFabricIx, EndpointId aEndpointId, GroupId aGroupId, SceneId aSceneId)
 {
     ScenesManagementCluster * cluster = FindClusterOnEndpoint(aEndpointId);
     VerifyOrReturn(cluster != nullptr);
-    TEMPORARY_RETURN_IGNORED cluster->RecallScene(aFabricIx, aGroupId, aSceneId);
+    LogErrorOnFailure(cluster->RecallScene(aFabricIx, aGroupId, aSceneId));
 }
 
 void ScenesServer::RemoveFabric(EndpointId aEndpointId, FabricIndex aFabricIndex)
 {
     ScenesManagementCluster * cluster = FindClusterOnEndpoint(aEndpointId);
     VerifyOrReturn(cluster != nullptr);
-    TEMPORARY_RETURN_IGNORED cluster->RemoveFabric(aFabricIndex);
+    LogErrorOnFailure(cluster->RemoveFabric(aFabricIndex));
 }
 
 } // namespace chip::app::Clusters::ScenesManagement