clippy

Author: jstuczyn

contracts/node-families/src/contract.rs

@@ -3,9 +3,7 @@
 
 //! CosmWasm entry points for the node families contract.
 
-use cosmwasm_std::{
-    entry_point, to_json_binary, Binary, Deps, DepsMut, Env, MessageInfo, Response,
-};
+use cosmwasm_std::{entry_point, Binary, Deps, DepsMut, Env, MessageInfo, Response};
 use node_families_contract_common::{
     ExecuteMsg, InstantiateMsg, MigrateMsg, NodeFamiliesContractError, QueryMsg,
 };
@@ -28,7 +26,11 @@ pub fn instantiate(
     cw2::set_contract_version(deps.storage, CONTRACT_NAME, CONTRACT_VERSION)?;
     set_build_information!(deps.storage)?;
 
-    // NYM_POOL_STORAGE.initialise(deps, env, info.sender, &msg.pool_denomination, msg.grants)?;
+    let _ = env;
+    let _ = info;
+    let _ = msg;
+
+    // NodeFamiliesStorage::new().initialise(deps, env, info.sender, &msg)?;
 
     Ok(Response::default())
 }
@@ -42,14 +44,21 @@ pub fn execute(
     info: MessageInfo,
     msg: ExecuteMsg,
 ) -> Result<Response, NodeFamiliesContractError> {
-    todo!()
+    let _ = deps;
+    let _ = env;
+    let _ = info;
+    let _ = msg;
+    Ok(Response::default())
 }
 
 /// Read-only dispatcher. Concrete handlers live in [`crate::queries`] and are
 /// wired up here as variants are added to [`QueryMsg`].
 #[entry_point]
 pub fn query(deps: Deps, env: Env, msg: QueryMsg) -> Result<Binary, NodeFamiliesContractError> {
-    todo!()
+    let _ = deps;
+    let _ = env;
+    let _ = msg;
+    Ok(Binary::default())
 }
 
 /// Migration entry point.

contracts/node-families/src/helpers.rs

@@ -8,6 +8,7 @@
 /// the storage layer's unique-name index. Callers should pass the normalised
 /// value to [`node_families_contract_common::NodeFamily::name`] when creating a family and when looking one
 /// up by name.
+#[allow(dead_code)]
 pub fn normalise_family_name(name: &str) -> String {
     name.chars()
         .filter(|c| c.is_ascii_alphanumeric())

contracts/node-families/src/storage/mod.rs

@@ -665,10 +665,6 @@ mod tests {
     #[test]
     fn register_new_family_assigns_sequential_ids() {
         let mut tester = init_contract_tester();
-        let s = NodeFamiliesStorage::new();
-        let env = tester.env();
-        let alice = tester.addr_make("alice");
-        let bob = tester.addr_make("bob");
 
         let f1 = tester.add_dummy_family();
         let f2 = tester.add_dummy_family();
@@ -695,13 +691,8 @@ mod tests {
         .unwrap();
 
         // unique-index defence-in-depth check
-        let res = s.register_new_family(
-            tester.storage_mut(),
-            &env,
-            bob,
-            "shared".into(),
-            "".into(),
-        );
+        let res =
+            s.register_new_family(tester.storage_mut(), &env, bob, "shared".into(), "".into());
         assert!(res.is_err());
     }
 
@@ -731,7 +722,6 @@ mod tests {
     fn add_pending_invitation_persists() {
         let mut tester = init_contract_tester();
         let s = NodeFamiliesStorage::new();
-        let env = tester.env();
         let alice = tester.addr_make("alice");
         let f = tester.make_family(&alice);
         let expires_at = tester.env().block.time.seconds() + 100;