[icd] integrate ICD management command into CHIP tool (project-chip#3…

…0863) * [icd] integrate ICD management command into CHIP tool * Delete entry on failure * Fix build
thivya-amazon · Dec 12, 2023 · 704a2ec · 704a2ec
1 parent 0e8bed3
commit 704a2ec
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 5 deletions.
diff --git a/examples/chip-tool/BUILD.gn b/examples/chip-tool/BUILD.gn
@@ -102,9 +102,7 @@ static_library("chip-tool-utils") {
 
  public_deps = [
  "${chip_root}/examples/common/tracing:commandline",
- "${chip_root}/src/app/icd/client:handler",
  "${chip_root}/src/app/icd/client:manager",
- "${chip_root}/src/app/icd/client:handler",
  "${chip_root}/src/app/server",
  "${chip_root}/src/app/tests/suites/commands/interaction_model",
  "${chip_root}/src/controller/data_model",

diff --git a/examples/chip-tool/commands/common/CHIPCommand.cpp b/examples/chip-tool/commands/common/CHIPCommand.cpp
@@ -110,6 +110,12 @@ CHIP_ERROR CHIPCommand::MaybeSetUpStack()
  // TODO: Implement persistent ICD storage for the chip-tool.
  ReturnLogErrorOnFailure(sICDClientStorage.Init(&mDefaultStorage, &sSessionKeystore));
 
+ // chip-tool uses a non-persistent keystore.
+ // ICD storage lifetime is currently tied to the chip-tool's lifetime. Since chip-tool interactive mode is currently used for
+ // ICD commissioning and check-in validation, this temporary storage meets the test requirements.
+ // TODO: Implement persistent ICD storage for the chip-tool.
+ ReturnLogErrorOnFailure(sICDClientStorage.Init(&mDefaultStorage, &sSessionKeystore));
+
  chip::Controller::FactoryInitParams factoryInitParams;
 
  factoryInitParams.fabricIndependentStorage = &mDefaultStorage;

diff --git a/examples/chip-tool/commands/common/CHIPCommand.h b/examples/chip-tool/commands/common/CHIPCommand.h
@@ -25,8 +25,6 @@
 #include "Command.h"
 
 #include <TracingCommandLineArgument.h>
-#include <app/icd/client/CheckInHandler.h>
-#include <app/icd/client/DefaultCheckInDelegate.h>
 #include <app/icd/client/DefaultICDClientStorage.h>
 #include <commands/common/CredentialIssuerCommands.h>
 #include <commands/example/ExampleCredentialIssuerCommands.h>

diff --git a/examples/tv-casting-app/tv-casting-common/BUILD.gn b/examples/tv-casting-app/tv-casting-common/BUILD.gn
@@ -110,7 +110,6 @@ chip_data_model("tv-casting-common") {
 
  deps = [
  "${chip_root}/examples/common/tracing:commandline",
- "${chip_root}/src/app/icd/client:handler",
  "${chip_root}/src/app/icd/client:manager",
  "${chip_root}/src/app/tests/suites/commands/interaction_model",
  "${chip_root}/src/lib/support/jsontlv",