feat(storage): remove whitespace from bucket name

contracts/cw-storage/src/contract.rs

@@ -142,4 +142,27 @@ mod tests {
 
         assert_eq!(err, ContractError::Bucket(BucketError::EmptyName));
     }
+
+    #[test]
+    fn whitespace_initialization() {
+        let mut deps = mock_dependencies();
+
+        let msg = InstantiateMsg {
+            bucket: "foo bar".to_string(),
+            limits: BucketLimits {
+                max_total_size: None,
+                max_objects: None,
+                max_object_size: None,
+                max_object_pins: None,
+            },
+        };
+        let info = mock_info("creator", &[]);
+
+        let res = instantiate(deps.as_mut(), mock_env(), info, msg).unwrap();
+        assert_eq!(0, res.messages.len());
+
+        let res = query(deps.as_ref(), mock_env(), QueryMsg::Bucket {}).unwrap();
+        let value: BucketResponse = from_binary(&res).unwrap();
+        assert_eq!("foobar", value.name);
+    }
 }

contracts/cw-storage/src/msg.rs

@@ -13,6 +13,8 @@ type Cursor = String;
 #[cw_serde]
 pub struct InstantiateMsg {
     /// The name of the bucket.
+    /// The name could not be empty or contains whitespaces.
+    /// If name contains whitespace, they will be removed.
     pub bucket: String,
     /// The limits of the bucket.
     pub limits: BucketLimits,

contracts/cw-storage/src/state.rs

@@ -16,11 +16,12 @@ pub struct Bucket {
 
 impl Bucket {
     pub fn new(name: String, limits: Limits) -> Result<Self, BucketError> {
-        if name.is_empty() {
+        let n: String = name.split_whitespace().collect();
+        if n.is_empty() {
             return Err(EmptyName);
         }
 
-        Ok(Self { name, limits })
+        Ok(Self { name: n, limits })
     }
 }