Trim docs whitespaces for metadata (#613)

crates/metadata/src/specs.rs

@@ -352,7 +352,7 @@ impl<S> ConstructorSpecBuilder<S> {
     {
         let mut this = self;
         debug_assert!(this.spec.docs.is_empty());
-        this.spec.docs = docs.into_iter().collect::<Vec<_>>();
+        this.spec.docs = docs.into_iter().map(str::trim).collect::<Vec<_>>();
         this
     }
 }

crates/metadata/src/tests.rs

@@ -169,3 +169,32 @@ fn spec_contract_json() {
         })
     )
 }
+
+#[test]
+fn trim_docs() {
+    // given
+    let name = "foo";
+    let cs = ConstructorSpec::from_name(name)
+        .selector(123_456_789u32.to_be_bytes())
+        .docs(vec![" foobar      "])
+        .done();
+    let mut registry = Registry::new();
+    let compact_spec = cs.into_compact(&mut registry);
+
+    // when
+    let json = serde_json::to_value(&compact_spec).unwrap();
+    let deserialized: ConstructorSpec<CompactForm> =
+        serde_json::from_value(json.clone()).unwrap();
+
+    // then
+    assert_eq!(
+        json,
+        json!({
+            "name": ["foo"],
+            "selector": "0x075bcd15",
+            "args": [],
+            "docs": ["foobar"]
+        })
+    );
+    assert_eq!(deserialized, compact_spec);
+}