Implement clearbanned method and test

Author: GideonBature

client/src/client_sync/v17/mod.rs

@@ -72,6 +72,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v17/network.rs

@@ -25,6 +25,22 @@ macro_rules! impl_client_v17__addnode {
     };
 }
 
+/// Implements Bitcoin Core JSON-RPC API method `clearbanned`
+#[macro_export]
+macro_rules! impl_client_v17__clearbanned {
+    () => {
+        impl Client {
+            pub fn clear_banned(&self) -> Result<()> {
+                match self.call("clearbanned", &[]) {
+                    Ok(serde_json::Value::Null) => Ok(()),
+                    Ok(res) => Err(Error::Returned(res.to_string())),
+                    Err(err) => Err(err.into()),
+                }
+            }
+        }
+    };
+}
+
 /// Implements Bitcoin Core JSON-RPC API method `getaddednodeinfo`
 #[macro_export]
 macro_rules! impl_client_v17__getaddednodeinfo {

client/src/client_sync/v18/mod.rs

@@ -76,6 +76,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v19/mod.rs

@@ -74,6 +74,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v20.rs

@@ -71,6 +71,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v21/mod.rs

@@ -73,6 +73,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v22/mod.rs

@@ -73,6 +73,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v23/mod.rs

@@ -75,6 +75,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v24.rs

@@ -72,6 +72,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v25.rs

@@ -72,6 +72,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v26/mod.rs

@@ -77,6 +77,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v27.rs

@@ -73,6 +73,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v28/mod.rs

@@ -75,6 +75,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

client/src/client_sync/v29/mod.rs

@@ -75,6 +75,7 @@ crate::impl_client_v17__submitblock!();
 
 // == Network ==
 crate::impl_client_v17__addnode!();
+crate::impl_client_v17__clearbanned!();
 crate::impl_client_v17__getaddednodeinfo!();
 crate::impl_client_v17__getnettotals!();
 crate::impl_client_v17__getnetworkinfo!();

integration_test/tests/network.rs

@@ -29,6 +29,15 @@ fn network__add_node() {
     }
 }
 
+#[test]
+fn network__clear_banned() {
+    let node = Node::with_wallet(Wallet::None, &[]);
+    let dummy_subnet = "192.0.2.2";
+
+    node.client.set_ban(dummy_subnet, SetBanCommand::Add).expect("setban add");
+    node.client.clear_banned().expect("clearbanned");
+}
+
 #[test]
 fn network__get_added_node_info() {
     let node = Node::with_wallet(Wallet::None, &[]);