First JNI test & changes necessary to make it work (#24355)

* * Change various references of "SessionHandle &" to instead use "const SessionHandle &" to match how SessionHandle is often used as a parameter & to support rvalue
* Add Java/JNI hooks into for-test classes (like MessagingContext) to allow for unit tests of Java & JNI functionality
* Add first JNI unit test, GetConnectedDeviceCallbackJniTest, testing success/failure cases for device callback

* Restyled by whitespace

* Restyled by clang-format

* Restyled by gn

* * Fix a user-reported bug where we were seeing the following callstack:

JNIEnv::NewObject(_jclass*, _jmethodID*, ...)
chip::Controller::GetConnectedDeviceCallback::OnDeviceConnectionFailureFn(void*, chip::ScopedNodeId const&, chip::ChipError)
chip::OperationalSessionSetup::DequeueConnectionCallbacks(chip::ChipError)
chip::OperationalSessionSetup::OnNodeAddressResolutionFailed(chip::PeerId const&, chip::ChipError)
chip::AddressResolve::Impl::Resolver::HandleAction(chip::IntrusiveList<chip::AddressResolve::Impl::NodeLookupHandle, (chip::IntrusiveMode)0, chip::IntrusiveListBaseHook<chip::AddressResolve::Impl::NodeLookupHandle, (chip::IntrusiveMode)0> >::Iterator&)
chip::AddressResolve::Impl::Resolver::HandleTimer()
chip::System::LayerImplSelect::HandleEvents()

Though JNI tests were added to try to catch this at presubmit, those tests did not fail, which leads me to believe there's a difference in the JRE (wherein the version with the user-reported bug), a cast to an int is not being done, whereas in android emulator, the cast to int is done

* Add a README pointing to building android guide & specifying that these tests must be run externally due to android emulator dependency

* Fix path to android_building.md

* Restyled by clang-format

* Restyled by prettier-markdown

* Missed const correctness changes for SessionHandle&

* Restyled by clang-format

* More missed const correctness changes for SessionHandle&

* Restyled by clang-format

* More missed const correctness changes for SessionHandle&

* Restyled by clang-format

* Only build JNI test libs if chip_link_tests

* Restyled by gn

* Add missing tests.gni import

Co-authored-by: Restyled.io <commits@restyled.io>

examples/all-clusters-app/esp32/main/include/ShellCommands.h

@@ -134,7 +134,7 @@ class CASECommands
 
 private:
  CASECommands() {}
- static void OnConnected(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+ static void OnConnected(void * context, Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle)
  {
  streamer_printf(streamer_get(), "Establish CASESession Success!\r\n");
  GetInstance().SetOnConnecting(false);

examples/all-clusters-minimal-app/esp32/main/include/ShellCommands.h

@@ -134,7 +134,7 @@ class CASECommands
 
 private:
  CASECommands() {}
- static void OnConnected(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+ static void OnConnected(void * context, Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle)
  {
  streamer_printf(streamer_get(), "Establish CASESession Success!\r\n");
  GetInstance().SetOnConnecting(false);

examples/chip-tool/commands/clusters/ModelCommand.cpp

@@ -47,7 +47,7 @@ CHIP_ERROR ModelCommand::RunCommand()
 }
 
 void ModelCommand::OnDeviceConnectedFn(void * context, chip::Messaging::ExchangeManager & exchangeMgr,
- chip::SessionHandle & sessionHandle)
+ const chip::SessionHandle & sessionHandle)
 {
  ModelCommand * command = reinterpret_cast<ModelCommand *>(context);
  VerifyOrReturn(command != nullptr, ChipLogError(chipTool, "OnDeviceConnectedFn: context is null"));

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

@@ -70,7 +70,7 @@ class ModelCommand : public CHIPCommand
  std::vector<chip::EndpointId> mEndPointId;
 
  static void OnDeviceConnectedFn(void * context, chip::Messaging::ExchangeManager & exchangeMgr,
- chip::SessionHandle & sessionHandle);
+ const chip::SessionHandle & sessionHandle);
  static void OnDeviceConnectionFailureFn(void * context, const chip::ScopedNodeId & peerId, CHIP_ERROR error);
 
  chip::Callback::Callback<chip::OnDeviceConnected> mOnDeviceConnectedCallback;

examples/chip-tool/commands/delay/WaitForCommissioneeCommand.cpp

@@ -34,7 +34,7 @@ CHIP_ERROR WaitForCommissioneeCommand::RunCommand()
 }
 
 void WaitForCommissioneeCommand::OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr,
- SessionHandle & sessionHandle)
+ const SessionHandle & sessionHandle)
 {
  auto * command = reinterpret_cast<WaitForCommissioneeCommand *>(context);
  VerifyOrReturn(command != nullptr, ChipLogError(chipTool, "OnDeviceConnectedFn: context is null"));

examples/chip-tool/commands/delay/WaitForCommissioneeCommand.h

@@ -48,7 +48,7 @@ class WaitForCommissioneeCommand : public CHIPCommand
  chip::Optional<bool> mExpireExistingSession;
 
  static void OnDeviceConnectedFn(void * context, chip::Messaging::ExchangeManager & exchangeMgr,
- chip::SessionHandle & sessionHandle);
+ const chip::SessionHandle & sessionHandle);
  static void OnDeviceConnectionFailureFn(void * context, const chip::ScopedNodeId & peerId, CHIP_ERROR error);
 
  chip::Callback::Callback<chip::OnDeviceConnected> mOnDeviceConnectedCallback;

examples/chip-tool/commands/pairing/CloseSessionCommand.cpp

@@ -71,7 +71,7 @@ CHIP_ERROR CloseSessionCommand::CloseSession(Messaging::ExchangeManager & exchan
 }
 
 void CloseSessionCommand::OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr,
- SessionHandle & sessionHandle)
+ const SessionHandle & sessionHandle)
 {
  auto * command = reinterpret_cast<CloseSessionCommand *>(context);
  VerifyOrReturn(command != nullptr, ChipLogError(chipTool, "OnDeviceConnectedFn: context is null"));

examples/chip-tool/commands/pairing/CloseSessionCommand.h

@@ -47,7 +47,7 @@ class CloseSessionCommand : public CHIPCommand
  chip::Optional<uint16_t> mTimeoutSecs;
 
  static void OnDeviceConnectedFn(void * context, chip::Messaging::ExchangeManager & exchangeMgr,
- chip::SessionHandle & sessionHandle);
+ const chip::SessionHandle & sessionHandle);
  static void OnDeviceConnectionFailureFn(void * context, const chip::ScopedNodeId & peerId, CHIP_ERROR error);
 
  // Try to send the action CloseSession status report.

examples/chip-tool/commands/tests/TestCommand.cpp

@@ -52,7 +52,7 @@ CHIP_ERROR TestCommand::WaitForCommissionee(const char * identity,
 }
 
 void TestCommand::OnDeviceConnectedFn(void * context, chip::Messaging::ExchangeManager & exchangeMgr,
- chip::SessionHandle & sessionHandle)
+ const chip::SessionHandle & sessionHandle)
 {
  ChipLogProgress(chipTool, " **** Test Setup: Device Connected\n");
  auto * command = static_cast<TestCommand *>(context);

examples/chip-tool/commands/tests/TestCommand.h

@@ -71,7 +71,7 @@ class TestCommand : public TestRunner,
  void OnResponse(const chip::app::StatusIB & status, chip::TLV::TLVReader * data) override{};
 
  static void OnDeviceConnectedFn(void * context, chip::Messaging::ExchangeManager & exchangeMgr,
- chip::SessionHandle & sessionHandle);
+ const chip::SessionHandle & sessionHandle);
  static void OnDeviceConnectionFailureFn(void * context, const chip::ScopedNodeId & peerId, CHIP_ERROR error);
 
  CHIP_ERROR ContinueOnChipMainThread(CHIP_ERROR err) override;

examples/platform/linux/CommissionerMain.cpp

@@ -252,7 +252,7 @@ class PairingCommand : public Controller::DevicePairingDelegate
 private:
 #if CHIP_DEVICE_CONFIG_APP_PLATFORM_ENABLED
  static void OnDeviceConnectedFn(void * context, chip::Messaging::ExchangeManager & exchangeMgr,
- chip::SessionHandle & sessionHandle);
+ const chip::SessionHandle & sessionHandle);
  static void OnDeviceConnectionFailureFn(void * context, const ScopedNodeId & peerId, CHIP_ERROR error);
 
  chip::Callback::Callback<chip::OnDeviceConnected> mOnDeviceConnectedCallback;
@@ -357,7 +357,8 @@ void PairingCommand::OnReadCommissioningInfo(const ReadCommissioningInfo & info)
 
 #if CHIP_DEVICE_CONFIG_APP_PLATFORM_ENABLED
 
-void PairingCommand::OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+void PairingCommand::OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr,
+ const SessionHandle & sessionHandle)
 {
  ChipLogProgress(Controller, "OnDeviceConnectedFn");
  CommissionerDiscoveryController * cdc = GetCommissionerDiscoveryController();

examples/tv-app/linux/AppImpl.cpp

@@ -89,7 +89,7 @@ MyPincodeService gMyPincodeService;
 class MyPostCommissioningListener : public PostCommissioningListener
 {
  void CommissioningCompleted(uint16_t vendorId, uint16_t productId, NodeId nodeId, Messaging::ExchangeManager & exchangeMgr,
- SessionHandle & sessionHandle) override
+ const SessionHandle & sessionHandle) override
  {
  // read current binding list
  chip::Controller::BindingCluster cluster(exchangeMgr, sessionHandle, kTargetBindingClusterEndpointId);
@@ -184,7 +184,7 @@ class MyPostCommissioningListener : public PostCommissioningListener
  }
 
  void cacheContext(uint16_t vendorId, uint16_t productId, NodeId nodeId, Messaging::ExchangeManager & exchangeMgr,
- SessionHandle & sessionHandle)
+ const SessionHandle & sessionHandle)
  {
  mVendorId = vendorId;
  mProductId = productId;

examples/tv-casting-app/tv-casting-common/commands/clusters/ModelCommand.cpp

@@ -53,7 +53,8 @@ CHIP_ERROR ModelCommand::RunCommand()
  return CHIP_NO_ERROR;
 }
 
-void ModelCommand::OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+void ModelCommand::OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr,
+ const SessionHandle & sessionHandle)
 {
  ChipLogProgress(chipTool, "ModelCommand::OnDeviceConnectedFn");
  ModelCommand * command = reinterpret_cast<ModelCommand *>(context);

examples/tv-casting-app/tv-casting-common/include/TargetVideoPlayerInfo.h

@@ -69,7 +69,7 @@ class TargetVideoPlayerInfo
 
 private:
  static void HandleDeviceConnected(void * context, chip::Messaging::ExchangeManager & exchangeMgr,
- chip::SessionHandle & sessionHandle)
+ const chip::SessionHandle & sessionHandle)
  {
  TargetVideoPlayerInfo * _this = static_cast<TargetVideoPlayerInfo *>(context);
  _this->mDeviceProxy = chip::OperationalDeviceProxy(&exchangeMgr, sessionHandle);

src/app/OperationalSessionSetup.h

@@ -115,7 +115,7 @@ class OperationalDeviceProxy : public DeviceProxy
  * application code does incorrectly hold onto this information so do not follow those incorrect
  * implementations as an example.
  */
-typedef void (*OnDeviceConnected)(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+typedef void (*OnDeviceConnected)(void * context, Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle);
 typedef void (*OnDeviceConnectionFailure)(void * context, const ScopedNodeId & peerId, CHIP_ERROR error);
 
 /**

src/app/ReadClient.cpp

@@ -822,7 +822,7 @@ CHIP_ERROR ReadClient::RefreshLivenessCheckTimer()
  timeout = System::Clock::Seconds16(mMaxInterval) + publisherTransmissionTimeout;
  }
 
- // EFR32/MBED/INFINION/K32W's chrono count return long unsinged, but other platform returns unsigned
+ // EFR32/MBED/INFINION/K32W's chrono count return long unsigned, but other platform returns unsigned
  ChipLogProgress(
  DataManagement,
  "Refresh LivenessCheckTime for %lu milliseconds with SubscriptionId = 0x%08" PRIx32 " Peer = %02x:" ChipLogFormatX64,
@@ -1068,7 +1068,8 @@ CHIP_ERROR ReadClient::DefaultResubscribePolicy(CHIP_ERROR aTerminationCause)
  return CHIP_NO_ERROR;
 }
 
-void ReadClient::HandleDeviceConnected(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+void ReadClient::HandleDeviceConnected(void * context, Messaging::ExchangeManager & exchangeMgr,
+ const SessionHandle & sessionHandle)
 {
  ReadClient * const _this = static_cast<ReadClient *>(context);
  VerifyOrDie(_this != nullptr);

src/app/ReadClient.h

@@ -509,7 +509,8 @@ class ReadClient : public Messaging::ExchangeDelegate
  void StopResubscription();
  void ClearActiveSubscriptionState();
 
- static void HandleDeviceConnected(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+ static void HandleDeviceConnected(void * context, Messaging::ExchangeManager & exchangeMgr,
+ const SessionHandle & sessionHandle);
  static void HandleDeviceConnectionFailure(void * context, const ScopedNodeId & peerId, CHIP_ERROR error);
 
  CHIP_ERROR GetMinEventNumber(const ReadPrepareParams & aReadPrepareParams, Optional<EventNumber> & aEventMin);

src/app/clusters/bindings/BindingManager.cpp

@@ -123,7 +123,7 @@ CHIP_ERROR BindingManager::EstablishConnection(const ScopedNodeId & nodeId)
  return mLastSessionEstablishmentError;
 }
 
-void BindingManager::HandleDeviceConnected(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+void BindingManager::HandleDeviceConnected(Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle)
 {
  FabricIndex fabricToRemove = kUndefinedFabricIndex;
  NodeId nodeToRemove = kUndefinedNodeId;

src/app/clusters/bindings/BindingManager.h

@@ -143,7 +143,8 @@ class BindingManager
  Callback::Callback<OnDeviceConnectionFailure> * GetOnDeviceConnectionFailure() { return &mOnConnectionFailureCallback; }
 
  private:
- static void HandleDeviceConnected(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+ static void HandleDeviceConnected(void * context, Messaging::ExchangeManager & exchangeMgr,
+ const SessionHandle & sessionHandle)
  {
  ConnectionCallback * _this = static_cast<ConnectionCallback *>(context);
  _this->mBindingManager.HandleDeviceConnected(exchangeMgr, sessionHandle);
@@ -169,7 +170,7 @@ class BindingManager
  BoundDeviceChangedHandler mBoundDeviceChangedHandler;
  BindingManagerInitParams mInitParams;
 
- void HandleDeviceConnected(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+ void HandleDeviceConnected(Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle);
  void HandleDeviceConnectionFailure(const ScopedNodeId & peerId, CHIP_ERROR error);
 
  // Used to keep track of synchronous failures from FindOrEstablishSession.

src/app/clusters/ota-requestor/DefaultOTARequestor.cpp

@@ -422,7 +422,7 @@ CHIP_ERROR DefaultOTARequestor::GetUpdateStateAttribute(EndpointId endpointId, O
 }
 
 // Called whenever FindOrEstablishSession is successful
-void DefaultOTARequestor::OnConnected(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+void DefaultOTARequestor::OnConnected(void * context, Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle)
 {
  DefaultOTARequestor * requestorCore = static_cast<DefaultOTARequestor *>(context);
  VerifyOrDie(requestorCore != nullptr);
@@ -728,7 +728,7 @@ CHIP_ERROR DefaultOTARequestor::GenerateUpdateToken()
  return CHIP_NO_ERROR;
 }
 
-CHIP_ERROR DefaultOTARequestor::SendQueryImageRequest(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+CHIP_ERROR DefaultOTARequestor::SendQueryImageRequest(Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle)
 {
  VerifyOrReturnError(mProviderLocation.HasValue(), CHIP_ERROR_INCORRECT_STATE);
 
@@ -796,7 +796,7 @@ CHIP_ERROR DefaultOTARequestor::ExtractUpdateDescription(const QueryImageRespons
  return CHIP_NO_ERROR;
 }
 
-CHIP_ERROR DefaultOTARequestor::StartDownload(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+CHIP_ERROR DefaultOTARequestor::StartDownload(Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle)
 {
  VerifyOrReturnError(mBdxDownloader != nullptr, CHIP_ERROR_INCORRECT_STATE);
 
@@ -829,7 +829,8 @@ CHIP_ERROR DefaultOTARequestor::StartDownload(Messaging::ExchangeManager & excha
  return err;
 }
 
-CHIP_ERROR DefaultOTARequestor::SendApplyUpdateRequest(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle)
+CHIP_ERROR DefaultOTARequestor::SendApplyUpdateRequest(Messaging::ExchangeManager & exchangeMgr,
+ const SessionHandle & sessionHandle)
 {
  VerifyOrReturnError(mProviderLocation.HasValue(), CHIP_ERROR_INCORRECT_STATE);
  ReturnErrorOnFailure(GenerateUpdateToken());
@@ -844,7 +845,7 @@ CHIP_ERROR DefaultOTARequestor::SendApplyUpdateRequest(Messaging::ExchangeManage
 }
 
 CHIP_ERROR DefaultOTARequestor::SendNotifyUpdateAppliedRequest(Messaging::ExchangeManager & exchangeMgr,
- SessionHandle & sessionHandle)
+ const SessionHandle & sessionHandle)
 {
  VerifyOrReturnError(mProviderLocation.HasValue(), CHIP_ERROR_INCORRECT_STATE);
  ReturnErrorOnFailure(GenerateUpdateToken());

src/app/clusters/ota-requestor/DefaultOTARequestor.h

@@ -232,7 +232,7 @@ class DefaultOTARequestor : public OTARequestorInterface, public BDXDownloader::
  /**
  * Send QueryImage request using values matching Basic cluster
  */
- CHIP_ERROR SendQueryImageRequest(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+ CHIP_ERROR SendQueryImageRequest(Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle);
 
  /**
  * Validate and extract mandatory information from QueryImageResponse
@@ -263,17 +263,17 @@ class DefaultOTARequestor : public OTARequestorInterface, public BDXDownloader::
  /**
  * Start download of the software image returned in QueryImageResponse
  */
- CHIP_ERROR StartDownload(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+ CHIP_ERROR StartDownload(Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle);
 
  /**
  * Send ApplyUpdate request using values obtained from QueryImageResponse
  */
- CHIP_ERROR SendApplyUpdateRequest(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+ CHIP_ERROR SendApplyUpdateRequest(Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle);
 
  /**
  * Send NotifyUpdateApplied request
  */
- CHIP_ERROR SendNotifyUpdateAppliedRequest(Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+ CHIP_ERROR SendNotifyUpdateAppliedRequest(Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle);
 
  /**
  * Store current update information to KVS
@@ -288,7 +288,7 @@ class DefaultOTARequestor : public OTARequestorInterface, public BDXDownloader::
  /**
  * Session connection callbacks
  */
- static void OnConnected(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+ static void OnConnected(void * context, Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle);
  static void OnConnectionFailure(void * context, const ScopedNodeId & peerId, CHIP_ERROR error);
  Callback::Callback<OnDeviceConnected> mOnConnectedCallback;
  Callback::Callback<OnDeviceConnectionFailure> mOnConnectionFailureCallback;

src/controller/CHIPDeviceController.cpp

@@ -1679,7 +1679,7 @@ void DeviceCommissioner::CommissioningStageComplete(CHIP_ERROR err, Commissionin
 }
 
 void DeviceCommissioner::OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr,
- SessionHandle & sessionHandle)
+ const SessionHandle & sessionHandle)
 {
  // CASE session established.
  DeviceCommissioner * commissioner = static_cast<DeviceCommissioner *>(context);

src/controller/CHIPDeviceController.h

@@ -784,7 +784,7 @@ class DLL_EXPORT DeviceCommissioner : public DeviceController,
  /* Callback called when adding root cert to device results in failure */
  static void OnRootCertFailureResponse(void * context, CHIP_ERROR error);
 
- static void OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr, SessionHandle & sessionHandle);
+ static void OnDeviceConnectedFn(void * context, Messaging::ExchangeManager & exchangeMgr, const SessionHandle & sessionHandle);
  static void OnDeviceConnectionFailureFn(void * context, const ScopedNodeId & peerId, CHIP_ERROR error);
 
  static void OnDeviceAttestationInformationVerification(void * context,

src/controller/CommissionerDiscoveryController.cpp

@@ -177,7 +177,7 @@ void CommissionerDiscoveryController::Cancel()
 
 void CommissionerDiscoveryController::CommissioningSucceeded(uint16_t vendorId, uint16_t productId, NodeId nodeId,
  Messaging::ExchangeManager & exchangeMgr,
- SessionHandle & sessionHandle)
+ const SessionHandle & sessionHandle)
 {
  mVendorId = vendorId;
  mProductId = productId;

src/controller/CommissionerDiscoveryController.h

@@ -138,7 +138,8 @@ class DLL_EXPORT PostCommissioningListener
  *
  */
  virtual void CommissioningCompleted(uint16_t vendorId, uint16_t productId, NodeId nodeId,
- chip::Messaging::ExchangeManager & exchangeMgr, chip::SessionHandle & sessionHandle) = 0;
+ chip::Messaging::ExchangeManager & exchangeMgr,
+ const chip::SessionHandle & sessionHandle) = 0;
 
  virtual ~PostCommissioningListener() = default;
 };
@@ -217,7 +218,7 @@ class CommissionerDiscoveryController : public chip::Protocols::UserDirectedComm
  *
  */
  void CommissioningSucceeded(uint16_t vendorId, uint16_t productId, NodeId nodeId,
- chip::Messaging::ExchangeManager & exchangeMgr, chip::SessionHandle & sessionHandle);
+ chip::Messaging::ExchangeManager & exchangeMgr, const chip::SessionHandle & sessionHandle);
 
  /**
  * This method should be called by the commissioner to indicate that commissioning failed.