test: generalize shared DOM test helper

Rename the shared DOM helper to make it reusable outside palette tests.

Update palette.test.js to use the generic helper while preserving the
existing test behavior.

Author: mahesh-09-12

js/__tests__/palette.test.js

@@ -21,7 +21,7 @@
 const { Palettes, initPalettes } = require("../palette");
 const { mockDocById } = require("../../test/utils/domMocks");
 const { createMockActivity } = require("../../test/utils/activityFactory");
-const { createPaletteDOM } = require("../../test/utils/domFactory");
+const { createMockDOM } = require("../../test/utils/domFactory");
 const { setupImageMock } = require("../../test/utils/imageMock");
 const { setupSVGMock } = require("../../test/utils/svgMock");
 const { setupGlobalEnvironment } = require("../../test/setup/globalSetup");
@@ -101,7 +101,7 @@ describe("Palettes Class", () => {
         setupSVGMock();
 
         // Real DOM instead of fake one
-        const { paletteElement, paletteBody } = createPaletteDOM();
+        const { container: paletteElement, body: paletteBody } = createMockDOM();
         mockDocById({
             palette: paletteElement,
             PaletteBody: paletteBody

test/utils/domFactory.js

@@ -1,13 +1,13 @@
-function createPaletteDOM() {
-    const paletteElement = document.createElement("div");
-    paletteElement.style.visibility = "visible";
-    paletteElement.style.top = "100px";
+function createMockDOM() {
+    const container = document.createElement("div");
+    container.style.visibility = "visible";
+    container.style.top = "100px";
 
-    const paletteBody = document.createElement("div");
+    const body = document.createElement("div");
     const parent = document.createElement("div");
-    parent.appendChild(paletteBody);
+    parent.appendChild(body);
 
-    return { paletteElement, paletteBody };
+    return { container, body };
 }
 
-module.exports = { createPaletteDOM };
+module.exports = { createMockDOM };