Take out the dependency on getNewId() in the settings dialog test

The returned id from this method is increasing on every call, making snapshot testing with code using it flaky.

Signed-off-by: Mike Lischke <mike@lischke-online.de>

Author: mike-lischke

tests/ui/SettingsDialog.spec.tsx

@@ -8,6 +8,7 @@ import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
 
 import type { IUISettings } from "../../src/core/AppStorage.js";
 import { AppStorage } from "../../src/core/AppStorage.js";
+import * as utils from "../../src/core/utils.js";
 import { requisitions } from "../../src/supplement/Requisitions.js";
 import { SettingsDialog } from "../../src/ui/SettingsDialog.js";
 
@@ -51,6 +52,8 @@ describe.sequential("SettingsDialog (class)", () => {
 
     beforeEach(() => {
         vi.restoreAllMocks();
+        let nextId = 3;
+        vi.spyOn(utils, "getNewId").mockImplementation(() => nextId++);
         renderResult = null;
     });