change execute contract message

Author: James0126

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@terra-money/msg-reader",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Convert common terra.js msg to a sentence",
   "main": "dist/index.js",
   "author": "Terra <[email protected]>",

package.json

@@ -8,6 +8,7 @@ const {
   MsgDeposit,
   MsgVote,
   MsgMigrateCode,
+  MsgExecuteContract,
   Coin,
   Coins,
 } = require("@terra-money/terra.js")
@@ -30,7 +31,7 @@ describe("Bank", () => {
   test("Send multiple coins", () => {
     const msg = new MsgSend(address, recipient, coins)
     expect(readMsg(msg)).toBe(
-      "Send 1000000ukrw, 1000000uluna to terra17lmam6zguazs5q5u6z5mmx76uj63gldnse2pdp"
+      "Send 1000000ukrw,1000000uluna to terra17lmam6zguazs5q5u6z5mmx76uj63gldnse2pdp"
     )
   })
 })
@@ -87,6 +88,23 @@ describe("Gov", () => {
   })
 })
 
+describe("Wasm", () => {
+  const contract = "terra10llyp6v3j3her8u3ce66ragytu45kcmd9asj3u"
+  test("Execute contract with coins", () => {
+    const msg = new MsgExecuteContract(address, contract, { send: "" }, [coin])
+    expect(readMsg(msg)).toBe(
+      "Execute send on terra10llyp6v3j3her8u3ce66ragytu45kcmd9asj3u with 1000000uluna"
+    )
+  })
+
+  test("Execute contract without coins", () => {
+    const msg = new MsgExecuteContract(address, contract, { send: "" })
+    expect(readMsg(msg)).toBe(
+      "Execute send on terra10llyp6v3j3her8u3ce66ragytu45kcmd9asj3u"
+    )
+  })
+})
+
 test("Default", () => {
   const msg = new MsgMigrateCode(address, 1, "codes")
   expect(readMsg(msg)).toBe("")

src/index.test.js

@@ -52,10 +52,11 @@ export const readMsg = (msg: Msg) => {
 
       case "wasm/MsgExecuteContract": {
         const { contract, execute_msg, coins } = data.value
-        const key = Object.keys(execute_msg)[0]
-        return `Execute ${key || "default"} on ${contract} ${
-          coins && `(-${formatCoins(coins)})`
-        }`
+        const [key] = Object.keys(execute_msg)
+        const payload = key ? ` ${key}` : ""
+        const suffix = coins.length ? ` with ${formatCoins(coins)}` : ""
+
+        return `Execute${payload} on ${contract}${suffix}`
       }
 
       default: