fix: report divide-by-zero on mod block instead of Not a Number

js/blocks/NumberBlocks.js

@@ -56,6 +56,10 @@ function setupNumberBlocks(activity) {
      */
     const handleMathError = (logo, error, blk, onError = NANERRORMSG) => {
         logo.stopTurtle = true;
+        if (error && error.message === "DivByZeroError") {
+            activity.errorMsg(ZERODIVIDEERRORMSG, blk);
+            return;
+        }
         activity.errorMsg(onError, blk);
     };
 

js/utils/__tests__/mathutils.test.js

@@ -130,7 +130,7 @@ describe("MathUtility", () => {
         });
 
         test("throws error for invalid inputs", () => {
-            expect(() => MathUtility.doMod("a", 3)).toThrow("Invalid number input");
+            expect(() => MathUtility.doMod("a", 3)).toThrow("NanError");
         });
 
         // Edge case tests
@@ -159,7 +159,7 @@ describe("MathUtility", () => {
         });
 
         test("throws error when second arg is string", () => {
-            expect(() => MathUtility.doMod(10, "a")).toThrow("Invalid number input");
+            expect(() => MathUtility.doMod(10, "a")).toThrow("NanError");
         });
     });
 
@@ -556,7 +556,7 @@ describe("MathUtility", () => {
 
     describe("edge cases - Infinity, NaN, and boundary values", () => {
         test("doMod throws an error when divisor is zero", () => {
-            expect(() => MathUtility.doMod(5, 0)).toThrow();
+            expect(() => MathUtility.doMod(5, 0)).toThrow("DivByZeroError");
         });
 
         test("doSqrt handles Infinity", () => {

js/utils/mathutils.js

@@ -130,11 +130,11 @@ class MathUtility {
     static doMod(a, b) {
         if (typeof a === "number" && typeof b === "number") {
             if (Number(b) === 0) {
-                throw new Error("Division by zero");
+                throw new Error("DivByZeroError");
             }
             return Number(a) % Number(b);
         } else {
-            throw new Error("Invalid number input");
+            throw new Error("NanError");
         }
     }