Custom error message instead "can't convert to BigInt"

aeternity · Nov 22, 2021 · 3348eb1 · 3348eb1
1 parent 37c72be
commit 3348eb1
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/src/types/FateBytes.js b/src/types/FateBytes.js
@@ -12,7 +12,13 @@ const toByteArray = (value) => {
         return HexStringToByteArray(value)
     }
 
-    return Int2ByteArray(BigInt(value))
+    try {
+        return Int2ByteArray(BigInt(value))
+    } catch (error) {
+        if (!/^Cannot convert .* to a BigInt$/.test(error.message)) throw error
+    }
+
+    throw new Error(`Should be one of: Array, ArrayBuffer, hex string, Number, BigInt; got ${value} instead`)
 }
 
 class FateBytes extends FateData {

diff --git a/tests/Encoder.js b/tests/Encoder.js
@@ -64,6 +64,14 @@ test('Encode hash arguments', t => {
     )
 })
 
+test('Ensures hash in a proper format', t => {
+    t.plan(1)
+    t.throws(
+        () => encoder.encode(CONTRACT, 'test_hash', [{}]),
+        { message: 'Should be one of: Array, ArrayBuffer, hex string, Number, BigInt; got [object Object] instead' }
+    )
+})
+
 test('Encode signature arguments', t => {
     t.plan(1)
     const encoded = encoder.encode(