Removing id_graph_stats getter and related tests (#2600)

Author: silva-fj

tee-worker/app-libs/stf/src/getter.rs

@@ -124,8 +124,6 @@ pub enum TrustedGetter {
 	// litentry
 	#[codec(index = 5)]
 	id_graph(Identity),
-	#[codec(index = 6)]
-	id_graph_stats(Identity),
 }
 
 impl TrustedGetter {
@@ -141,7 +139,6 @@ impl TrustedGetter {
 			TrustedGetter::evm_account_storages(sender_identity, ..) => sender_identity,
 			// litentry
 			TrustedGetter::id_graph(sender_identity) => sender_identity,
-			TrustedGetter::id_graph_stats(sender_identity) => sender_identity,
 		}
 	}
 
@@ -253,14 +250,6 @@ impl ExecuteGetter for TrustedGetterSigned {
 				},
 			// litentry
 			TrustedGetter::id_graph(who) => Some(IdentityManagement::id_graph(&who).encode()),
-
-			// TODO: we need to re-think it
-			//       currently, _who is ignored meaning it's actually not a "trusted" getter.
-			//       In fact, in the production no one should have access to the concrete identities
-			//       but maybe it makes sense to get some statistic information
-			// Disabled until it's resolved
-			// Disabled the test `lit-id-graph-stats` too
-			TrustedGetter::id_graph_stats(_who) => None,
 		}
 	}
 

tee-worker/cli/lit_id_graph_stats.sh

@@ -16,7 +16,6 @@
 
 pub mod get_storage;
 pub mod id_graph;
-pub mod id_graph_stats;
 pub mod link_identity;
 pub mod remove_identity;
 pub mod request_batch_vc;

tee-worker/cli/src/trusted_base_cli/commands/litentry/id_graph_stats.rs

@@ -21,9 +21,9 @@ use crate::{
 		get_shard::GetShardCommand,
 		get_shard_vault::GetShardVaultCommand,
 		litentry::{
-			get_storage::GetStorageCommand, id_graph_stats::IDGraphStats,
-			link_identity::LinkIdentityCommand, remove_identity::RemoveIdentityCommand,
-			request_batch_vc::RequestBatchVcCommand, request_vc::RequestVcCommand,
+			get_storage::GetStorageCommand, link_identity::LinkIdentityCommand,
+			remove_identity::RemoveIdentityCommand, request_batch_vc::RequestBatchVcCommand,
+			request_vc::RequestVcCommand,
 			send_erroneous_parentchain_call::SendErroneousParentchainCallCommand,
 		},
 		nonce::NonceCommand,
@@ -81,9 +81,6 @@ pub enum TrustedBaseCommand {
 	/// send an erroneous parentchain call intentionally, only used in tests
 	SendErroneousParentchainCall(SendErroneousParentchainCallCommand),
 
-	/// Disabled for now: get count of all keys account + identity in the IDGraphs
-	IDGraphStats(IDGraphStats),
-
 	/// Link the given identity to the prime identity, with specified networks
 	LinkIdentity(LinkIdentityCommand),
 
@@ -115,7 +112,6 @@ impl TrustedBaseCommand {
 			// Litentry's commands below
 			TrustedBaseCommand::GetStorage(cmd) => cmd.run(cli, trusted_cli),
 			TrustedBaseCommand::SendErroneousParentchainCall(cmd) => cmd.run(cli, trusted_cli),
-			TrustedBaseCommand::IDGraphStats(cmd) => cmd.run(cli, trusted_cli),
 			TrustedBaseCommand::LinkIdentity(cmd) => cmd.run(cli, trusted_cli),
 			TrustedBaseCommand::IDGraph(cmd) => cmd.run(cli, trusted_cli),
 			TrustedBaseCommand::RequestVc(cmd) => cmd.run(cli, trusted_cli),

tee-worker/cli/src/trusted_base_cli/commands/litentry/mod.rs

@@ -26,7 +26,6 @@ export default {
                 __Unused_evm_account_codes: "Null",
                 __Unused_evm_account_storages: "Null",
                 id_graph: "(LitentryIdentity)",
-                id_graph_stats: "(LitentryIdentity)",
             },
         },
         //important