@scm-manager/ui-components 4.0.0-REACT19-20250917-125211 → 4.0.0-REACT19-20250920-130317

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scm-manager/ui-components",
-  "version": "4.0.0-REACT19-20250917-125211",
+  "version": "4.0.0-REACT19-20250920-130317",
   "description": "UI Components for SCM-Manager and its plugins",
   "main": "src/index.ts",
   "files": [
@@ -11,7 +11,7 @@
   "author": "Sebastian Sdorra <sebastian.sdorra@cloudogu.com>",
   "license": "AGPL-3.0-only",
   "scripts": {
-    "test": "jest",
+    "test": "vitest",
     "typecheck": "tsc",
     "storybook": "storybook dev -p 6006",
     "build-storybook": "storybook build",
@@ -34,13 +34,12 @@
   "devDependencies": {
     "@scm-manager/babel-preset": "^2.13.1",
     "@scm-manager/eslint-config": "^2.18.2",
-    "@scm-manager/jest-preset": "^2.15.0-alpha1",
     "@scm-manager/prettier-config": "^2.12.0",
     "@scm-manager/tsconfig": "^2.13.0",
-    "@scm-manager/ui-shortcuts": "4.0.0-REACT19-20250917-125211",
-    "@scm-manager/ui-syntaxhighlighting": "4.0.0-REACT19-20250917-125211",
-    "@scm-manager/ui-text": "4.0.0-REACT19-20250917-125211",
-    "@scm-manager/ui-types": "4.0.0-REACT19-20250917-125211",
+    "@scm-manager/ui-shortcuts": "4.0.0-REACT19-20250920-130317",
+    "@scm-manager/ui-syntaxhighlighting": "4.0.0-REACT19-20250920-130317",
+    "@scm-manager/ui-text": "4.0.0-REACT19-20250920-130317",
+    "@scm-manager/ui-types": "4.0.0-REACT19-20250920-130317",
     "@storybook/addon-actions": "^9.0.8",
     "@storybook/addon-docs": "^9.1.5",
     "@storybook/addon-essentials": "^9.0.0-alpha.12",
@@ -56,23 +55,27 @@
     "@types/react": "^19.1.1",
     "@types/react-select": "^2.0.19",
     "@types/unist": "^2.0.3",
+    "@vitest/coverage-v8": "^3.2.4",
+    "@vitest/web-worker": "^3.2.4",
+    "ci-info": "^4.3.1",
     "decode-named-character-reference": "^1.0.1",
     "fetch-mock": "^7.5.1",
     "gitdiff-parser": "^0.2.2",
     "minimatch": "^10.0.3",
-    "react-test-renderer": "^19.1.1",
     "react-router-dom": "^7.9.1",
+    "react-test-renderer": "^19.1.1",
     "sass-loader": "^12.3.0",
     "storybook-addon-i18next": "^1.3.0",
-    "storybook-addon-themes": "^6.1.0"
+    "storybook-addon-themes": "^6.1.0",
+    "vitest": "^3.2.4"
   },
   "dependencies": {
-    "@scm-manager/ui-api": "4.0.0-REACT19-20250917-125211",
-    "@scm-manager/ui-buttons": "4.0.0-REACT19-20250917-125211",
-    "@scm-manager/ui-core": "4.0.0-REACT19-20250917-125211",
-    "@scm-manager/ui-extensions": "4.0.0-REACT19-20250917-125211",
-    "@scm-manager/ui-layout": "4.0.0-REACT19-20250917-125211",
-    "@scm-manager/ui-overlays": "4.0.0-REACT19-20250917-125211",
+    "@scm-manager/ui-api": "4.0.0-REACT19-20250920-130317",
+    "@scm-manager/ui-buttons": "4.0.0-REACT19-20250920-130317",
+    "@scm-manager/ui-core": "4.0.0-REACT19-20250920-130317",
+    "@scm-manager/ui-extensions": "4.0.0-REACT19-20250920-130317",
+    "@scm-manager/ui-layout": "4.0.0-REACT19-20250920-130317",
+    "@scm-manager/ui-overlays": "4.0.0-REACT19-20250920-130317",
     "deepmerge": "^4.2.2",
     "hast-util-sanitize": "^3.0.2",
     "react-diff-view": "2.6.0",
@@ -94,9 +97,6 @@
       "@scm-manager/babel-preset"
     ]
   },
-  "jest": {
-    "preset": "@scm-manager/jest-preset"
-  },
   "prettier": "@scm-manager/prettier-config",
   "eslintConfig": {
     "extends": "@scm-manager/eslint-config"

package/src/Paginator.test.tsx

@@ -18,14 +18,6 @@ import React from "react";
 import { render, screen } from "@testing-library/react";
 import Paginator from "./Paginator";
 
-describe("Temporary test to be removed", () => {
-  it("true", () => {
-    expect(true).toBe(true);
-  });
-});
-
-//TODO Fix jest preset to include TextEncoder in its environment
-/*
 describe("paginator rendering tests", () => {
   const dummyLink = {
     href: "https://dummy",
@@ -41,12 +33,11 @@ describe("paginator rendering tests", () => {
 
     render(<Paginator collection={collection} />);
 
-    const buttons = await screen.findAllByRole("button");
+    const buttons: HTMLButtonElement[] = await screen.findAllByRole("button");
 
-    expect(buttons.length).toBe(7);
+    assert.lengthOf(buttons, 7);
     buttons.forEach((button) => {
-      // @ts-ignore
-      expect(button).toBeDisabled();
+      assert.strictEqual(button.disabled, true, "all buttons should be disabled");
     });
   });
 
@@ -64,28 +55,24 @@ describe("paginator rendering tests", () => {
 
     render(<Paginator collection={collection} />);
 
-    const buttons = await screen.findAllByRole("button");
+    const buttons: HTMLButtonElement[] = await screen.findAllByRole("button");
 
-    expect(buttons.length).toBe(5);
+    assert.lengthOf(buttons, 5);
+
+    assert.strictEqual(buttons[0].disabled, true, "previous button should be disabled");
+    assert.strictEqual(buttons[1].disabled, false, "next button should be enabled");
 
-    // previous button
-    expect(buttons[0]).toBeDisabled();
-    // last button
-    expect(buttons[1]).toBeEnabled();
-    // first button
     const firstButton = buttons[2];
-    expect(firstButton).toBeDisabled();
-    expect(firstButton).toHaveTextContent("1");
+    assert.strictEqual(firstButton.disabled, true, "first button should be disabled");
+    assert.strictEqual(firstButton.textContent?.trim(), "1", "first button be 1");
 
-    // next button
     const nextButton = buttons[3];
-    expect(nextButton).toBeEnabled();
-    expect(nextButton).toHaveTextContent("2");
+    assert.strictEqual(nextButton.disabled, false, "next button should be enabled");
+    assert.strictEqual(nextButton.textContent?.trim(), "2", "next button be 2");
 
-    // last button
     const lastButton = buttons[4];
-    expect(lastButton).toBeEnabled();
-    expect(lastButton).toHaveTextContent("148");
+    assert.strictEqual(lastButton.disabled, false, "last button should be enabled");
+    assert.strictEqual(lastButton.textContent?.trim(), "148", "last button be 148");
   });
 
   it("should render buttons for second page", async () => {
@@ -103,33 +90,28 @@ describe("paginator rendering tests", () => {
 
     render(<Paginator collection={collection} />);
 
-    const buttons = await screen.findAllByRole("button");
+    const buttons: HTMLButtonElement[] = await screen.findAllByRole("button");
+
+    assert.lengthOf(buttons, 6);
 
-    expect(buttons.length).toBe(6);
+    assert.strictEqual(buttons[0].disabled, false, "previous button should be enabled");
+    assert.strictEqual(buttons[1].disabled, false, "next button should be enabled");
 
-    // previous button
-    expect(buttons[0]).toBeEnabled();
-    // last button
-    expect(buttons[1]).toBeEnabled();
-    // first button
     const firstButton = buttons[2];
-    expect(firstButton).toBeEnabled();
-    expect(firstButton).toHaveTextContent("1");
+    assert.strictEqual(firstButton.disabled, false, "first button should be enabled");
+    assert.strictEqual(firstButton.textContent?.trim(), "1", "first button be 1");
 
-    // current button
     const currentButton = buttons[3];
-    expect(currentButton).toBeDisabled();
-    expect(currentButton).toHaveTextContent("2");
+    assert.strictEqual(currentButton.disabled, true, "current button should be disabled");
+    assert.strictEqual(currentButton.textContent?.trim(), "2", "current button be 2");
 
-    // next button
     const nextButton = buttons[4];
-    expect(nextButton).toBeEnabled();
-    expect(nextButton).toHaveTextContent("3");
+    assert.strictEqual(nextButton.disabled, false, "next button should be enabled");
+    assert.strictEqual(nextButton.textContent?.trim(), "3", "next button be 3");
 
-    // last button
     const lastButton = buttons[5];
-    expect(lastButton).toBeEnabled();
-    expect(lastButton).toHaveTextContent("148");
+    assert.strictEqual(lastButton.disabled, false, "last button should be enabled");
+    assert.strictEqual(lastButton.textContent?.trim(), "148", "last button be 148");
   });
 
   it("should render buttons for last page", async () => {
@@ -145,28 +127,24 @@ describe("paginator rendering tests", () => {
 
     render(<Paginator collection={collection} />);
 
-    const buttons = await screen.findAllByRole("button");
+    const buttons: HTMLButtonElement[] = await screen.findAllByRole("button");
+
+    assert.lengthOf(buttons, 5);
 
-    expect(buttons.length).toBe(5);
+    assert.strictEqual(buttons[0].disabled, false, "previous button should be enabled");
+    assert.strictEqual(buttons[1].disabled, true, "next button should be disabled");
 
-    // previous button
-    expect(buttons[0]).toBeEnabled();
-    // last button
-    expect(buttons[1]).toBeDisabled();
-    // first button
     const firstButton = buttons[2];
-    expect(firstButton).toBeEnabled();
-    expect(firstButton).toHaveTextContent("1");
+    assert.strictEqual(firstButton.disabled, false, "first button should be enabled");
+    assert.strictEqual(firstButton.textContent?.trim(), "1", "first button be 1");
 
-    // next button
     const nextButton = buttons[3];
-    expect(nextButton).toBeEnabled();
-    expect(nextButton).toHaveTextContent("147");
+    assert.strictEqual(nextButton.disabled, false, "next button should be enabled");
+    assert.strictEqual(nextButton.textContent?.trim(), "147", "next button be 147");
 
-    // last button
     const lastButton = buttons[4];
-    expect(lastButton).toBeDisabled();
-    expect(lastButton).toHaveTextContent("148");
+    assert.strictEqual(lastButton.disabled, true, "last button should be disabled");
+    assert.strictEqual(lastButton.textContent?.trim(), "148", "last button be 148");
   });
 
   it("should render buttons for penultimate page", async () => {
@@ -184,33 +162,28 @@ describe("paginator rendering tests", () => {
 
     render(<Paginator collection={collection} />);
 
-    const buttons = await screen.findAllByRole("button");
+    const buttons: HTMLButtonElement[] = await screen.findAllByRole("button");
 
-    expect(buttons.length).toBe(6);
+    assert.lengthOf(buttons, 6);
 
-    // previous button
-    expect(buttons[0]).toBeEnabled();
-    // last button
-    expect(buttons[1]).toBeEnabled();
+    assert.strictEqual(buttons[0].disabled, false, "previous button should be enabled");
+    assert.strictEqual(buttons[1].disabled, false, "next button should be enabled");
 
-    // first button
     const firstButton = buttons[2];
-    expect(firstButton).toBeEnabled();
-    expect(firstButton).toHaveTextContent("1");
+    assert.strictEqual(firstButton.disabled, false, "first button should be enabled");
+    assert.strictEqual(firstButton.textContent?.trim(), "1", "first button be 1");
 
-    const currentButton = buttons[3];
-    expect(currentButton).toBeEnabled();
-    expect(currentButton).toHaveTextContent("146");
+    const previousButton = buttons[3];
+    assert.strictEqual(previousButton.disabled, false, "previous button should be enabled");
+    assert.strictEqual(previousButton.textContent?.trim(), "146", "previous button be 146");
 
-    // current button
-    const nextButton = buttons[4];
-    expect(nextButton).toBeDisabled();
-    expect(nextButton).toHaveTextContent("147");
+    const currentButton = buttons[4];
+    assert.strictEqual(currentButton.disabled, true, "current button should be disabled");
+    assert.strictEqual(currentButton.textContent?.trim(), "147", "current button be 147");
 
-    // last button
     const lastButton = buttons[5];
-    expect(lastButton).toBeEnabled();
-    expect(lastButton).toHaveTextContent("148");
+    assert.strictEqual(lastButton.disabled, false, "last button should be enabled");
+    assert.strictEqual(lastButton.textContent?.trim(), "148", "last button be 148");
   });
 
   it("should render buttons for a page in the middle", async () => {
@@ -228,39 +201,32 @@ describe("paginator rendering tests", () => {
 
     render(<Paginator collection={collection} />);
 
-    const buttons = await screen.findAllByRole("button");
+    const buttons: HTMLButtonElement[] = await screen.findAllByRole("button");
 
-    expect(buttons.length).toBe(7);
+    assert.lengthOf(buttons, 7);
 
-    // previous button
-    expect(buttons[0]).toBeEnabled();
-    // next button
-    expect(buttons[1]).toBeEnabled();
+    assert.strictEqual(buttons[0].disabled, false, "previous button should be enabled");
+    assert.strictEqual(buttons[1].disabled, false, "next button should be enabled");
 
-    // first button
-    const firstButton = buttons[2];
-    expect(firstButton).toBeEnabled();
-    expect(firstButton).toHaveTextContent("1");
+    const firstButton = buttons[2] as HTMLButtonElement;
+    assert.strictEqual(firstButton.disabled, false, "first button should be enabled");
+    assert.strictEqual(firstButton.textContent?.trim(), "1", "first button be 1");
 
-    // previous Button
     const previousButton = buttons[3];
-    expect(previousButton).toBeEnabled();
-    expect(previousButton).toHaveTextContent("41");
+    assert.strictEqual(previousButton.disabled, false, "previous button should be enabled");
+    assert.strictEqual(previousButton.textContent?.trim(), "41", "previous button be 41");
 
-    // current button
     const currentButton = buttons[4];
-    expect(currentButton).toBeDisabled();
-    expect(currentButton).toHaveTextContent("42");
+    assert.strictEqual(currentButton.disabled, true, "current button should be disabled");
+    assert.strictEqual(currentButton.textContent?.trim(), "42", "current button be 42");
 
-    // next button
     const nextButton = buttons[5];
-    expect(nextButton).toBeEnabled();
-    expect(nextButton).toHaveTextContent("43");
+    assert.strictEqual(nextButton.disabled, false, "next button should be enabled");
+    assert.strictEqual(nextButton.textContent?.trim(), "43", "next button be 43");
 
-    // last button
     const lastButton = buttons[6];
-    expect(lastButton).toBeEnabled();
-    expect(lastButton).toHaveTextContent("148");
+    assert.strictEqual(lastButton.disabled, false, "last button should be enabled");
+    assert.strictEqual(lastButton.textContent?.trim(), "148", "last button be 148");
   });
 
   it("should call the function with the last previous url", async () => {
@@ -285,9 +251,9 @@ describe("paginator rendering tests", () => {
 
     render(<Paginator collection={collection} onPageChange={callMe} />);
 
-    const previousButton = await screen.getByRole("button", { name: /paginator.previous/i });
+    const previousButton = screen.getByRole("button", { name: /paginator.previous/i });
     previousButton.click();
 
-    expect(urlToOpen).toBe("https://www.scm-manager.org");
+    assert.strictEqual(urlToOpen, "https://www.scm-manager.org", "the url passed to the function is not correct");
   });
-});*/
+});

package/src/__snapshots__/storyshots.test.ts.snap

@@ -1,4 +1,4 @@
-// Jest Snapshot v1, https://goo.gl/fbAQLP
+// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
 
 exports[`Storyshots BranchSelector Default 1`] = `
 <div

package/src/comparators.test.ts

@@ -48,7 +48,7 @@ describe("key comparator tests", () => {
   });
 
   it("should not fail if item is undefined", () => {
-    const array: any[] = createObjects("key", ["a"]);
+    const array: Array<Record<string, unknown> | undefined> = createObjects("key", ["a"]);
     array.push(undefined);
     array.push(createObject("key", "z"));
     array.sort(byKey("key"));
@@ -78,7 +78,7 @@ describe("length comparator tests", () => {
   });
 
   it("should not fail if item is undefined", () => {
-    const array: any[] = createObjects("key", ["."]);
+    const array: Array<Record<string, unknown> | undefined> = createObjects("key", ["."]);
     array.push(undefined);
     array.push(createObject("key", ".."));
     array.sort(byValueLength("key"));
@@ -133,7 +133,7 @@ describe("nested key comparator tests", () => {
   });
 
   it("should not fail if item is undefined", () => {
-    const array: any[] = createObjects("key", "nested", ["a"]);
+    const array: Array<Record<string, unknown> | undefined> = createObjects("key", "nested", ["a"]);
     array.push(undefined);
     array.push(createObject("key", "nested", "z"));
     array.sort(byNestedKeys("key", "nested"));

package/src/languages.test.ts

@@ -36,5 +36,4 @@ describe("syntax highlighter", () => {
     const lang = determineLanguage("");
     expect(lang).toBe("text");
   });
-
 });

package/src/markdown/MarkdownHeadingRenderer.test.ts

@@ -14,17 +14,8 @@
  * along with this program. If not, see https://www.gnu.org/licenses/.
  */
 
-import React from "react";
 import { headingToAnchorId } from "./MarkdownHeadingRenderer";
 
-describe("Temporary test to be removed", () => {
-  it("true", () => {
-    expect(true).toBe(true);
-  });
-});
-
-//TODO Fix jest preset to include TextEncoder in its environment
-/*
 describe("headingToAnchorId tests", () => {
   it("should lower case the text", () => {
     expect(headingToAnchorId("Hello")).toBe("hello");
@@ -36,4 +27,4 @@ describe("headingToAnchorId tests", () => {
     expect(headingToAnchorId("awesome stuff")).toBe("awesome-stuff");
     expect(headingToAnchorId("a b c d e f")).toBe("a-b-c-d-e-f");
   });
-});*/
+});

package/src/markdown/MarkdownImageRenderer.test.ts

@@ -14,16 +14,8 @@
  * along with this program. If not, see https://www.gnu.org/licenses/.
  */
 
-import React from "react";
 import { createLocalLink } from "./MarkdownImageRenderer";
 
-describe("Temporary test to be removed", () => {
-  it("true", () => {
-    expect(true).toBe(true);
-  });
-});
-
-/*
 describe("createLocalLink tests", () => {
   const revision = "revision";
   const basePath = `/repo/namespace/name/code/sources/${revision}/`;
@@ -38,7 +30,7 @@ describe("createLocalLink tests", () => {
 
   it("should return modified contentLink for absolute link", () => {
     expect(createLocalLink(basePath, contentLink, revision, currentPath, "/path/anotherImg.jpg")).toBe(
-      "http://localhost:8081/scm/api/v2/repositories/namespace/name/content/revision/path/anotherImg.jpg"
+      "http://localhost:8081/scm/api/v2/repositories/namespace/name/content/revision/path/anotherImg.jpg",
     );
   });
 
@@ -49,8 +41,8 @@ describe("createLocalLink tests", () => {
         contentLink,
         "feature/awesome",
         currentPath,
-        link
-      )
+        link,
+      ),
     ).toContain("feature%2Fawesome");
   });
-});*/
+});

package/src/markdown/MarkdownLinkRenderer.test.tsx

@@ -17,14 +17,6 @@
 import { isAnchorLink, isExternalLink, isLinkWithProtocol, isInternalScmRepoLink } from "./paths";
 import { createLocalLink } from "./MarkdownLinkRenderer";
 
-describe("Temporary test to be removed", () => {
-  it("true", () => {
-    expect(true).toBe(true);
-  });
-});
-
-//TODO Fix jest preset to include TextEncoder in its environment
-/*
 describe("test isAnchorLink", () => {
   it("should return true", () => {
     expect(isAnchorLink("#some-thing")).toBe(true);
@@ -135,4 +127,3 @@ describe("test createLocalLink", () => {
     expect(localLink).toBe(expected);
   };
 });
-*/

package/src/modals/useRegisterModal.test.tsx

@@ -19,16 +19,18 @@ import React, { FC, ReactNode } from "react";
 import ActiveModalCountContext, { ModalStateContextType } from "./activeModalCountContext";
 import useRegisterModal from "./useRegisterModal";
 
-const createWrapper =
-  (context: ModalStateContextType): FC<{ children: ReactNode }> =>
-  ({ children }: { children: React.ReactNode }) => {
-    return <ActiveModalCountContext.Provider value={context}>{children}</ActiveModalCountContext.Provider>;
-  };
+const createWrapper = (context: ModalStateContextType): FC<{ children: ReactNode }> => {
+  const Wrapper: FC<{ children: ReactNode }> = ({ children }) => (
+    <ActiveModalCountContext.Provider value={context}>{children}</ActiveModalCountContext.Provider>
+  );
+  Wrapper.displayName = "ActiveModalCountContextWrapper";
+  return Wrapper;
+};
 
 describe("useRegisterModal", () => {
   it("should increment and not decrement on active registration", () => {
-    const increment = jest.fn();
-    const decrement = jest.fn();
+    const increment = vi.fn();
+    const decrement = vi.fn();
 
     renderHook(() => useRegisterModal(true), {
       wrapper: createWrapper({ value: 0, increment, decrement }),
@@ -39,8 +41,8 @@ describe("useRegisterModal", () => {
   });
 
   it("should not decrement on inactive registration", () => {
-    const increment = jest.fn();
-    const decrement = jest.fn();
+    const increment = vi.fn();
+    const decrement = vi.fn();
 
     renderHook(() => useRegisterModal(false), {
       wrapper: createWrapper({ value: 0, increment, decrement }),
@@ -51,8 +53,8 @@ describe("useRegisterModal", () => {
   });
 
   it("should not decrement on inactive de-registration", () => {
-    const increment = jest.fn();
-    const decrement = jest.fn();
+    const increment = vi.fn();
+    const decrement = vi.fn();
 
     const result = renderHook(() => useRegisterModal(false), {
       wrapper: createWrapper({ value: 0, increment, decrement }),
@@ -65,8 +67,8 @@ describe("useRegisterModal", () => {
   });
 
   it("should decrement once on active de-registration", () => {
-    const increment = jest.fn();
-    const decrement = jest.fn();
+    const increment = vi.fn();
+    const decrement = vi.fn();
 
     const result = renderHook(() => useRegisterModal(false, true), {
       wrapper: createWrapper({ value: 0, increment, decrement }),
@@ -79,11 +81,10 @@ describe("useRegisterModal", () => {
   });
 
   it("should decrement once when switching from active to inactive", () => {
-    const increment = jest.fn();
-    const decrement = jest.fn();
+    const increment = vi.fn();
+    const decrement = vi.fn();
 
     const result = renderHook(({ active }: { active: boolean }) => useRegisterModal(active), {
-      // @ts-ignore
       wrapper: createWrapper({ value: 0, increment, decrement }),
       initialProps: { active: true },
     });

package/src/repos/DiffExpander.test.ts

@@ -295,7 +295,7 @@ describe("with hunks the diff expander", () => {
     const preceedingHunk = diffExpander.getHunk(0).hunk;
     fetchMock.get(
       "http://localhost:8081/scm/api/v2/content/abc/CommitMessage.js?start=8&end=13",
-      "new line 9\nnew line 10\nnew line 11\nnew line 12\nnew line 13"
+      "new line 9\nnew line 10\nnew line 11\nnew line 12\nnew line 13",
     );
     let newFile: FileDiff;
     await diffExpander
@@ -324,7 +324,7 @@ describe("with hunks the diff expander", () => {
     const oldHunkCount = diffExpander.hunkCount();
     fetchMock.get(
       "http://localhost:8081/scm/api/v2/content/abc/CommitMessage.js?start=40&end=50",
-      "new line 40\nnew line 41\nnew line 42"
+      "new line 40\nnew line 41\nnew line 42",
     );
     let newFile: FileDiff;
     await diffExpander
@@ -337,7 +337,7 @@ describe("with hunks the diff expander", () => {
   it("should set end to -1 if requested to expand to the end", async () => {
     fetchMock.get(
       "http://localhost:8081/scm/api/v2/content/abc/CommitMessage.js?start=40&end=-1",
-      "new line 40\nnew line 41\nnew line 42"
+      "new line 40\nnew line 41\nnew line 42",
     );
     let newFile: FileDiff;
     await diffExpander

package/src/repos/changesets/ChangesetButtonGroup.test.tsx

@@ -14,52 +14,57 @@
  * along with this program. If not, see https://www.gnu.org/licenses/.
  */
 
-import * as changesets from "./changesets";
+import React from "react";
+import { BrowserRouter } from "react-router-dom";
 import { render } from "@testing-library/react";
 import { Branch, Changeset, Repository } from "@scm-manager/ui-types";
+import * as changesets from "./changesets";
 import ChangesetButtonGroup from "./ChangesetButtonGroup";
-import React from "react";
-import { BrowserRouter } from "react-router-dom";
 
-const createChangesetLink = jest.spyOn(changesets, "createChangesetLink");
-const createChangesetLinkByBranch = jest.spyOn(changesets, "createChangesetLinkByBranch");
-
-afterEach(() => {
-  jest.resetAllMocks();
+vi.mock("react-router", async (importOriginal) => {
+  const actual = await importOriginal();
+  return {
+    ...(actual as Record<string, unknown>),
+    // Mock the 'Link' component to render an <a> tag directly, bypassing the broken router context (Fixes basename error)
+    Link: vi.fn(({ to, children, ...props }) => (
+      <a href={to} {...props}>
+        {children}
+      </a>
+    )),
+  };
 });
 
-describe("Temporary test to be removed", () => {
-  it("true", () => {
-    expect(true).toBe(true);
-  });
+const createChangesetLink = vi.spyOn(changesets, "createChangesetLink").mockReturnValue("/changeset");
+const createChangesetLinkByBranch = vi.spyOn(changesets, "createChangesetLinkByBranch").mockReturnValue("/branch");
+
+afterEach(() => {
+  vi.resetAllMocks();
 });
 
-//TODO Fix jest preset to include TextEncoder in its environment
-/*describe("ChangesetButtonGroup", () => {
-  test("shouldCallCreateChangesetLinkWithoutBranch", async () => {
+describe("ChangesetButtonGroup", () => {
+  it("should call createChangesetLink without branch", () => {
     const { repository, changeset } = createTestData();
     render(
       <BrowserRouter>
         <ChangesetButtonGroup repository={repository} changeset={changeset}></ChangesetButtonGroup>
-      </BrowserRouter>
+      </BrowserRouter>,
     );
     expect(createChangesetLink).toHaveBeenCalled();
     expect(createChangesetLinkByBranch).toHaveBeenCalledTimes(0);
   });
 
-  test("shouldCallCreateChangesetLinkByBranchWithBranch", async () => {
+  it("should call createChangesetLink by branch with branch", () => {
     const { repository, changeset, branch } = createTestData();
     render(
       <BrowserRouter>
         <ChangesetButtonGroup repository={repository} changeset={changeset} branch={branch}></ChangesetButtonGroup>
-      </BrowserRouter>
+      </BrowserRouter>,
     );
     expect(createChangesetLinkByBranch).toHaveBeenCalled();
     expect(createChangesetLink).toHaveBeenCalledTimes(0);
   });
 });
 
-// TODO centralized test data
 function createTestData() {
   const repository: Repository = { _links: {}, name: "", namespace: "", type: "" };
   const changeset: Changeset = {
@@ -73,4 +78,4 @@ function createTestData() {
   };
   const branch: Branch = { _links: {}, name: "", revision: "" };
   return { repository, changeset, branch };
-}*/
+}

package/src/repos/changesets/ChangesetDiff.test.ts

@@ -16,14 +16,6 @@
 
 import { createUrl, isDiffSupported } from "./ChangesetDiff";
 
-describe("Temporary test to be removed", () => {
-  it("true", () => {
-    expect(true).toBe(true);
-  });
-});
-
-//TODO Fix jest preset to include TextEncoder in its environment
-/*
 describe("isDiffSupported tests", () => {
   it("should return true if diff link is defined", () => {
     const supported = isDiffSupported({
@@ -105,4 +97,4 @@ describe("createUrl tests", () => {
       }),
     ).toThrow();
   });
-});*/
+});

package/src/repositories.test.ts

@@ -27,10 +27,10 @@ describe("getProtocolLinkByType tests", () => {
         protocol: [
           {
             name: "http",
-            href: "http://scm.scm-manager.org/repo/scm/core"
-          }
-        ]
-      }
+            href: "http://scm.scm-manager.org/repo/scm/core",
+          },
+        ],
+      },
     };
 
     const link = getProtocolLinkByType(repository, "http");
@@ -46,14 +46,14 @@ describe("getProtocolLinkByType tests", () => {
         protocol: [
           {
             name: "http",
-            href: "http://scm.scm-manager.org/repo/scm/core"
+            href: "http://scm.scm-manager.org/repo/scm/core",
           },
           {
             name: "ssh",
-            href: "git@scm.scm-manager.org:scm/core"
-          }
-        ]
-      }
+            href: "git@scm.scm-manager.org:scm/core",
+          },
+        ],
+      },
     };
 
     const link = getProtocolLinkByType(repository, "http");
@@ -68,9 +68,9 @@ describe("getProtocolLinkByType tests", () => {
       _links: {
         protocol: {
           name: "http",
-          href: "http://scm.scm-manager.org/repo/scm/core"
-        }
-      }
+          href: "http://scm.scm-manager.org/repo/scm/core",
+        },
+      },
     };
 
     const link = getProtocolLinkByType(repository, "http");
@@ -86,14 +86,14 @@ describe("getProtocolLinkByType tests", () => {
         protocol: [
           {
             name: "http",
-            href: "http://scm.scm-manager.org/repo/scm/core"
+            href: "http://scm.scm-manager.org/repo/scm/core",
           },
           {
             name: "ssh",
-            href: "git@scm.scm-manager.org:scm/core"
-          }
-        ]
-      }
+            href: "git@scm.scm-manager.org:scm/core",
+          },
+        ],
+      },
     };
 
     const link = getProtocolLinkByType(repository, "awesome");
@@ -105,7 +105,7 @@ describe("getProtocolLinkByType tests", () => {
       namespace: "scm",
       name: "core",
       type: "git",
-      _links: {}
+      _links: {},
     };
 
     const link = getProtocolLinkByType(repository, "http");

package/src/validation.test.ts

@@ -88,7 +88,7 @@ describe("test mail validation", () => {
     "s.sdorra @ostfalia.de",
     "s.sdorra@[ostfalia.de",
     "abc.example.com",
-    "a@b@c@example.com"
+    "a@b@c@example.com",
   ];
   for (const mail of invalid) {
     it(`should return false for '${mail}'`, () => {
@@ -113,7 +113,7 @@ describe("test mail validation", () => {
     "example@s.example",
     "user.name+tag+sorting@example.com",
     "name/surname@example.com",
-    "user%example.com@example.org"
+    "user%example.com@example.org",
   ];
   for (const mail of valid) {
     it(`should return true for '${mail}'`, () => {