[chip-tool] Ensure that the event subscription success is dispatched …

…to SubscribeEvent (#17842)

examples/chip-tool/commands/clusters/ReportCommand.h

@@ -32,8 +32,7 @@ class ReportCommand : public ModelCommand, public chip::app::ReadClient::Callbac
  ModelCommand(commandName, credsIssuerConfig), mBufferedReadAdapter(*this)
  {}
 
- virtual void OnAttributeSubscription(){};
- virtual void OnEventSubscription(){};
+ virtual void OnSubscription(){};
 
  /////////// ReadClient Callback Interface /////////
  void OnAttributeData(const chip::app::ConcreteDataAttributePath & path, chip::TLV::TLVReader * data,
@@ -105,7 +104,7 @@ class ReportCommand : public ModelCommand, public chip::app::ReadClient::Callbac
  SetCommandExitStatus(mError);
  }
 
- void OnSubscriptionEstablished(uint64_t subscriptionId) override { OnAttributeSubscription(); }
+ void OnSubscriptionEstablished(uint64_t subscriptionId) override { OnSubscription(); }
 
 protected:
  CHIP_ERROR ReportAttribute(ChipDevice * device, std::vector<chip::EndpointId> endpointIds,
@@ -431,11 +430,9 @@ class SubscribeAttribute : public ReportCommand
  mDataVersion);
  }
 
- chip::System::Clock::Timeout GetWaitDuration() const override { return ReportCommand::GetWaitDuration(); }
-
- void OnAttributeSubscription() override
+ void OnSubscription() override
  {
- // The ReadClient instance can not be released directly into the OnAttributeSubscription
+ // The ReadClient instance can not be released directly into the OnSubscription
  // callback since it happens to be called by ReadClient itself which is doing additional
  // work after that.
  chip::DeviceLayer::PlatformMgr().ScheduleWork(
@@ -547,9 +544,7 @@ class SubscribeEvent : public ReportCommand
  chip::app::ReadClient::InteractionType::Subscribe, mMinInterval, mMaxInterval);
  }
 
- chip::System::Clock::Timeout GetWaitDuration() const override { return ReportCommand::GetWaitDuration(); }
-
- void OnEventSubscription() override
+ void OnSubscription() override
  {
  // The ReadClient instance can not be released directly into the OnEventSubscription
  // callback since it happens to be called by ReadClient itself which is doing additional