Empty provider updated

Author: andreilitvin

src/app/server-cluster/testing/EmptyProvider.cpp

@@ -35,34 +35,36 @@ CHIP_ERROR EmptyProvider::Endpoints(ListBuilder<app::DataModel::EndpointEntry> &
 
 CHIP_ERROR EmptyProvider::SemanticTags(EndpointId endpointId, ListBuilder<SemanticTag> & builder)
 {
-    return CHIP_NO_ERROR;
+    return CHIP_IM_GLOBAL_STATUS(UnsupportedEndpoint);
 }
 CHIP_ERROR EmptyProvider::DeviceTypes(EndpointId endpointId, ListBuilder<app::DataModel::DeviceTypeEntry> & builder)
 {
-    return CHIP_NO_ERROR;
+    return CHIP_IM_GLOBAL_STATUS(UnsupportedEndpoint);
 }
 
 CHIP_ERROR EmptyProvider::ClientClusters(EndpointId endpointId, ListBuilder<ClusterId> & builder)
 {
-    return CHIP_NO_ERROR;
+    return CHIP_IM_GLOBAL_STATUS(UnsupportedEndpoint);
 }
 CHIP_ERROR EmptyProvider::ServerClusters(EndpointId endpointId, ListBuilder<app::DataModel::ServerClusterEntry> & builder)
 {
-    return CHIP_NO_ERROR;
+    return CHIP_IM_GLOBAL_STATUS(UnsupportedEndpoint);
 }
 
 CHIP_ERROR EmptyProvider::Attributes(const app::ConcreteClusterPath & path, ListBuilder<app::DataModel::AttributeEntry> & builder)
 {
-    return CHIP_NO_ERROR;
+    return CHIP_IM_GLOBAL_STATUS(UnsupportedEndpoint);
 }
+
 CHIP_ERROR EmptyProvider::GeneratedCommands(const app::ConcreteClusterPath & path, ListBuilder<CommandId> & builder)
 {
-    return CHIP_NO_ERROR;
+    return CHIP_IM_GLOBAL_STATUS(UnsupportedEndpoint);
 }
+
 CHIP_ERROR EmptyProvider::AcceptedCommands(const app::ConcreteClusterPath & path,
                                            ListBuilder<app::DataModel::AcceptedCommandEntry> & builder)
 {
-    return CHIP_NO_ERROR;
+    return CHIP_IM_GLOBAL_STATUS(UnsupportedEndpoint);
 }
 
 void EmptyProvider::Temporary_ReportAttributeChanged(const app::AttributePathParams & path) {}

src/app/server-cluster/testing/EmptyProvider.h

@@ -22,10 +22,12 @@
 namespace chip {
 namespace Test {
 
-/// A provider that is emtpy - everything returns null
+/// A provider that is emtpy - it contains no endpoints and most
+/// calls fail with `Unsupported Endpoint`
 ///
-/// Over time this should be replaced with some code-generated/controllable provider
-/// to allow for better testing.
+/// This is a bare minimum implentation to have somethign that claims to be a `Provider`
+/// however it has no real implementation that is useful. Over time this should be replaced
+/// with some code-generated/controllable provider to allow for better testing.
 class EmptyProvider : public app::DataModel::Provider
 {
 public: