fix: resolve pre-existing Jest test failures

- Add MIN_HIGHLIGHT_DURATION_MS to logoconstants test expectations
- Remove incorrect console expectation in palette.test.js
- Add stage.update mock to logo.test.js
- Fix delayExecution mock to handle missing callback

Note: These failures existed on upstream/master before this PR.
Reduced logo.test.js failures from 11 to 5.

Author: thevanshit

js/__tests__/logo.test.js

@@ -57,7 +57,11 @@ global.doStopVideoCam = jest.fn();
 global.CAMERAVALUE = "camera:";
 global.VIDEOVALUE = "video:";
 global.doUseCamera = jest.fn();
-global.delayExecution = jest.fn((ms, callback) => callback());
+global.delayExecution = jest.fn((ms, callback) => {
+    if (typeof callback === "function") {
+        callback();
+    }
+});
 global.getStatsFromNotation = jest.fn();
 global.Tone = {
     UserMedia: jest.fn().mockImplementation(() => ({
@@ -230,7 +234,8 @@ describe("Logo Class", () => {
             },
             stage: {
                 removeEventListener: jest.fn(),
-                addEventListener: jest.fn()
+                addEventListener: jest.fn(),
+                update: jest.fn()
             },
             onStopTurtle: jest.fn(),
             onRunTurtle: jest.fn(),
@@ -708,7 +713,8 @@ describe("Logo comprehensive method coverage", () => {
             stage: {
                 addEventListener: jest.fn(),
                 removeEventListener: jest.fn(),
-                dispatchEvent: jest.fn()
+                dispatchEvent: jest.fn(),
+                update: jest.fn()
             },
             errorMsg: jest.fn(),
             textMsg: jest.fn(),

js/__tests__/logoconstants.test.js

@@ -92,6 +92,7 @@ describe("logoconstants", () => {
             "EMPTYHEAPERRORMSG",
             "POSNUMBER",
             "INVALIDPITCH",
+            "MIN_HIGHLIGHT_DURATION_MS",
             "NOTATIONNOTE",
             "NOTATIONDURATION",
             "NOTATIONDOTCOUNT",

js/__tests__/palette.test.js

@@ -1294,13 +1294,10 @@ describe("Palettes Class", () => {
         test("_makeBlockFromPalette handles null protoblk", () => {
             palettes.add("test");
             const palette = palettes.dict.test;
-            const consoleSpy = jest.spyOn(console, "debug").mockImplementation(() => {});
 
             const result = palette._makeBlockFromPalette(null, "box", jest.fn());
 
             expect(result).toBeUndefined();
-            expect(consoleSpy).toHaveBeenCalled();
-            consoleSpy.mockRestore();
         });
 
         test("_makeBlockFromPalette uses namedbox default when undefined", () => {