test: fix failing logo, palette, and logoconstants tests due to recent master refactors

Author: vanshika2720

js/__tests__/logo.test.js

@@ -57,7 +57,7 @@ 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 => Promise.resolve());
 global.getStatsFromNotation = jest.fn();
 global.Tone = {
     UserMedia: jest.fn().mockImplementation(() => ({
@@ -230,7 +230,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 +709,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(),
@@ -916,6 +918,7 @@ describe("Logo comprehensive method coverage", () => {
         const originalTone = global.Tone;
         const open = jest.fn();
         global.Tone = { UserMedia: jest.fn(() => ({ open })) };
+        if (logo.deps) logo.deps.Tone = global.Tone;
 
         logo.initMediaDevices();
         expect(open).toHaveBeenCalled();
@@ -929,11 +932,13 @@ describe("Logo comprehensive method coverage", () => {
                 })
             }))
         };
+        if (logo.deps) logo.deps.Tone = global.Tone;
         logo.initMediaDevices();
         expect(mockActivity.errorMsg).toHaveBeenCalledWith("The microphone is not available.");
         expect(logo.mic).toBeNull();
 
         global.Tone = originalTone;
+        if (logo.deps) logo.deps.Tone = global.Tone;
     });
 
     test("processShow handles image/url/loadFile/default branches", () => {
@@ -957,6 +962,7 @@ describe("Logo comprehensive method coverage", () => {
             return 2;
         });
         global.delayExecution = jest.fn(() => Promise.resolve());
+        if (logo.deps && logo.deps.utils) logo.deps.utils.delayExecution = global.delayExecution;
 
         turtle0.singer.embeddedGraphics = {};
         await logo.dispatchTurtleSignals(0, 0.5, 3, 0);
@@ -1062,6 +1068,7 @@ describe("Logo comprehensive method coverage", () => {
     test("doStopTurtles covers companion/camera/recorder/showBlocks branches", () => {
         const clearIntervalSpy = jest.spyOn(global, "clearInterval").mockImplementation(() => {});
         global.instruments = { 0: { flute: {} }, 1: { piano: {} } };
+        if (logo.deps && logo.deps.instruments) logo.deps.instruments = global.instruments;
         turtle0.singer.killAllVoices = jest.fn();
         turtle0.companionTurtle = 1;
         turtle1.interval = 888;
@@ -1142,6 +1149,7 @@ describe("Logo comprehensive method coverage", () => {
         logo.processShow = jest.fn();
         logo.processSpeak = jest.fn();
         global.delayExecution = jest.fn(() => Promise.resolve());
+        if (logo.deps && logo.deps.utils) logo.deps.utils.delayExecution = global.delayExecution;
 
         turtle0.singer.suppressOutput = false;
         turtle0.embeddedGraphicsFinished = false;
@@ -1217,7 +1225,8 @@ describe("Logo comprehensive method coverage", () => {
             storage: { saveLocally: jest.fn() },
             config: { showBlocksAfterRun: false },
             callbacks: { onStopTurtle: jest.fn(), onRunTurtle: jest.fn() },
-            meSpeak: { speak: jest.fn() }
+            meSpeak: { speak: jest.fn() },
+            classes: { Notation: global.Notation, Synth: global.Synth }
         };
 
         const depLogo = new Logo(deps);

js/__tests__/logoconstants.test.js

@@ -98,7 +98,8 @@ describe("logoconstants", () => {
             "NOTATIONTUPLETVALUE",
             "NOTATIONROUNDDOWN",
             "NOTATIONINSIDECHORD",
-            "NOTATIONSTACCATO"
+            "NOTATIONSTACCATO",
+            "MIN_HIGHLIGHT_DURATION_MS"
         ];
         expect(Object.keys(constants).sort()).toEqual(expectedKeys.sort());
     });

js/__tests__/palette.test.js

@@ -1294,13 +1294,9 @@ 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", () => {