fix: should throw TypeError if drawImage param mismatched (#1190)

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: Brooooooklyn

__test__/draw.spec.ts

@@ -433,6 +433,70 @@ test('drawImage-another-Canvas', async (t) => {
   await snapshotImage(t)
 })
 
+test('drawImage-throws-TypeError-for-invalid-image-type', (t) => {
+  const { ctx } = t.context
+
+  // Test with plain object
+  t.throws(
+    () => {
+      // @ts-expect-error - Testing invalid type
+      ctx.drawImage({}, 0, 0)
+    },
+    {
+      instanceOf: TypeError,
+      message: /Value is not one of these types/,
+    },
+  )
+
+  // Test with number
+  t.throws(
+    () => {
+      // @ts-expect-error - Testing invalid type
+      ctx.drawImage(42, 0, 0)
+    },
+    {
+      instanceOf: TypeError,
+      message: /Value is not one of these types/,
+    },
+  )
+
+  // Test with string
+  t.throws(
+    () => {
+      // @ts-expect-error - Testing invalid type
+      ctx.drawImage('not an image', 0, 0)
+    },
+    {
+      instanceOf: TypeError,
+      message: /Value is not one of these types/,
+    },
+  )
+
+  // Test with null
+  t.throws(
+    () => {
+      // @ts-expect-error - Testing invalid type
+      ctx.drawImage(null, 0, 0)
+    },
+    {
+      instanceOf: TypeError,
+      message: /Value is not one of these types/,
+    },
+  )
+
+  // Test with undefined
+  t.throws(
+    () => {
+      // @ts-expect-error - Testing invalid type
+      ctx.drawImage(undefined, 0, 0)
+    },
+    {
+      instanceOf: TypeError,
+      message: /Value is not one of these types/,
+    },
+  )
+})
+
 test('ellipse', async (t) => {
   const { ctx } = t.context
   // Draw the ellipse

src/ctx.rs

@@ -1910,7 +1910,7 @@ impl CanvasRenderingContext2D {
   pub fn draw_image(
     &mut self,
     env: &Env,
-    image: Either3<&mut CanvasElement, &mut SVGCanvas, &mut Image>,
+    image: Unknown,
     sx: Option<f64>,
     sy: Option<f64>,
     s_width: Option<f64>,
@@ -1920,6 +1920,17 @@ impl CanvasRenderingContext2D {
     d_width: Option<f64>,
     d_height: Option<f64>,
   ) -> Result<()> {
+    let Ok(image) = (unsafe {
+      <Either3<&mut CanvasElement, &mut SVGCanvas, &mut Image> as FromNapiValue>::from_napi_value(
+        env.raw(),
+        image.raw(),
+      )
+    }) else {
+      return env.throw_type_error(
+        "Value is not one of these types: `CanvasElement`, `SVGCanvas`, `Image`",
+        Some("InvalidArg"),
+      );
+    };
     let bitmap = match image {
       Either3::A(canvas) => BitmapRef::Owned(canvas.ctx.as_ref().context.surface.get_bitmap()),
       Either3::B(svg) => BitmapRef::Owned(svg.ctx.as_ref().context.surface.get_bitmap()),