From 8a5af2ae7bfeecdb7e81ff040ae0908cbba46a8a Mon Sep 17 00:00:00 2001
From: alexstroke <busyfifer@gmail.com>
Date: Wed, 14 Feb 2024 12:07:31 +0100
Subject: [PATCH] [fix] delete config test and restore transfer test

Signed-off-by: alexstroke <111361420+astrokov7@users.noreply.github.com>
---
 .../extra_functional/deprecated_config.rs     | 46 -------------------
 .../tests/integration/extra_functional/mod.rs |  1 -
 client/tests/integration/transfer_asset.rs    |  1 -
 3 files changed, 48 deletions(-)
 delete mode 100644 client/tests/integration/extra_functional/deprecated_config.rs

diff --git a/client/tests/integration/extra_functional/deprecated_config.rs b/client/tests/integration/extra_functional/deprecated_config.rs
deleted file mode 100644
index 0fef86fcf33..00000000000
--- a/client/tests/integration/extra_functional/deprecated_config.rs
+++ /dev/null
@@ -1,46 +0,0 @@
-use iroha_client::data_model::Level;
-use test_network::*;
-
-#[deprecated(
-    since = "2.0.0-pre-rc.20",
-    note = "This test is meaningless since it doesn't test anything useful"
-)]
-#[ignore = "ignore, more in #4094"]
-#[test]
-fn config_endpoints() {
-    const NEW_LOG_LEVEL: Level = Level::ERROR;
-
-    let (rt, peer, test_client) = <PeerBuilder>::new().with_port(10_685).start_with_runtime();
-    wait_for_genesis_committed(&vec![test_client.clone()], 0);
-
-    let init_log_level = rt.block_on(async move {
-        peer.iroha
-            .as_ref()
-            .unwrap()
-            .kiso
-            .get_dto()
-            .await
-            .unwrap()
-            .logger
-            .level
-    });
-
-    // Just to be sure this test suite is not useless
-    assert_ne!(init_log_level, NEW_LOG_LEVEL);
-
-    // Retrieving through API
-    let mut dto = test_client.get_config().expect("Client can always get it");
-    assert_eq!(dto.logger.level, init_log_level);
-
-    // Updating the log level
-    dto.logger.level = NEW_LOG_LEVEL;
-    test_client.set_config(dto).expect("New config is valid");
-
-    // Checking the updated value
-    dto = test_client.get_config().unwrap();
-    assert_eq!(dto.logger.level, NEW_LOG_LEVEL);
-
-    // Restoring value
-    dto.logger.level = init_log_level;
-    test_client.set_config(dto).expect("Also valid DTO");
-}
diff --git a/client/tests/integration/extra_functional/mod.rs b/client/tests/integration/extra_functional/mod.rs
index 773ccaca0d4..ad4db9c11d0 100644
--- a/client/tests/integration/extra_functional/mod.rs
+++ b/client/tests/integration/extra_functional/mod.rs
@@ -1,5 +1,4 @@
 mod connected_peers;
-mod deprecated_config;
 mod multiple_blocks_created;
 mod offline_peers;
 mod restart_peer;
diff --git a/client/tests/integration/transfer_asset.rs b/client/tests/integration/transfer_asset.rs
index 20ddacf8de1..062dfeeca7d 100644
--- a/client/tests/integration/transfer_asset.rs
+++ b/client/tests/integration/transfer_asset.rs
@@ -58,7 +58,6 @@ fn simulate_transfer_fixed() {
     since = "2.0.0-pre-rc.20",
     note = "This test suite is deprecated and refers to test_tranfer_assets.py"
 )]
-#[ignore = "migrated to client cli python tests"]
 #[test]
 #[should_panic(expected = "insufficient funds")]
 fn simulate_insufficient_funds() {