fix(Hash.update): add check for string type when Value type argument is passed.

- Expand test coverage on digest method
- Validate string check with Value type argument is working as expected in both host and guest

Signed-off-by: Elijah Kotyluk <[email protected]>

Author: ElijahKotyluk

packages/graalvm/src/main/kotlin/elide/runtime/node/crypto/NodeCryptoHash.kt

@@ -76,7 +76,8 @@ public class NodeHash(
             }
             arr
           }
-          else -> data.asString().toByteArray(Charsets.UTF_8)
+          data.isString -> data.asString().toByteArray(Charsets.UTF_8)
+          else -> throw IllegalArgumentException("Unsupported item type, must be of type string or an instance of Buffer, TypedArray, or Dataview. Received: ${data.javaClass.name}")
         }
       }
       is Iterable<*> -> { // @TODO(elijahkotyluk) Handle Polyglot lists as an iterable, may need to revisit

packages/graalvm/src/test/kotlin/elide/runtime/node/NodeCryptoTest.kt

@@ -12,6 +12,7 @@
  */
 package elide.runtime.node
 
+import org.graalvm.polyglot.Value
 import org.junit.jupiter.api.assertThrows
 import kotlin.test.Test
 import kotlin.test.assertEquals
@@ -388,7 +389,7 @@ import elide.testing.annotations.TestCase
 
   @Test fun `createHash should allow chainable updates`() = conforms {
     val hash = crypto.provide().createHash("sha256")
-    hash.update("hello").update(" ").update("world")
+    hash.update("hello").update(" ").update(Value.asValue("world"))
     val digest = hash.digest("hex")
 
     assertEquals(
@@ -576,6 +577,9 @@ import elide.testing.annotations.TestCase
     assertThrows<IllegalArgumentException> {
       hash.update({ str: String -> str })
     }
+    assertThrows<IllegalArgumentException> {
+      hash.update(Value.asValue(12345))
+    }
   }.guest {
     //language=javascript
     """