From 329ef794d43bfca3ecb49768f7cbcdd7d3994b2d Mon Sep 17 00:00:00 2001 From: Konstantin Schaper Date: Mon, 5 Oct 2020 18:40:15 +0200 Subject: [PATCH] fix broken unit test --- .../buttons/DeletePermissionButton.test.tsx | 25 +++++++++++++++---- 1 file changed, 20 insertions(+), 5 deletions(-) diff --git a/scm-ui/ui-webapp/src/repos/permissions/components/buttons/DeletePermissionButton.test.tsx b/scm-ui/ui-webapp/src/repos/permissions/components/buttons/DeletePermissionButton.test.tsx index 354de5b871..2e34e07160 100644 --- a/scm-ui/ui-webapp/src/repos/permissions/components/buttons/DeletePermissionButton.test.tsx +++ b/scm-ui/ui-webapp/src/repos/permissions/components/buttons/DeletePermissionButton.test.tsx @@ -21,16 +21,19 @@ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE * SOFTWARE. */ -import React from "react"; +// eslint-disable-next-line @typescript-eslint/ban-ts-ignore +// @ts-ignore +import React, { FC } from "react"; +// eslint-disable-next-line no-restricted-imports import { mount, shallow } from "@scm-manager/ui-tests/enzyme-router"; +// eslint-disable-next-line no-restricted-imports import "@scm-manager/ui-tests/enzyme"; +// eslint-disable-next-line no-restricted-imports import "@scm-manager/ui-tests/i18n"; import DeletePermissionButton from "./DeletePermissionButton"; -import { confirmAlert } from "@scm-manager/ui-components"; - jest.mock("@scm-manager/ui-components", () => ({ - confirmAlert: jest.fn(), + ConfirmAlert: (({ children }) =>
{children}
) as FC, DeleteButton: require.requireActual("@scm-manager/ui-components").DeleteButton })); @@ -40,6 +43,9 @@ describe("DeletePermissionButton", () => { _links: {} }; + // eslint-disable-next-line @typescript-eslint/ban-ts-ignore + // @ts-ignore + // eslint-disable-next-line @typescript-eslint/no-empty-function const navLink = shallow( {}} />); expect(navLink.text()).toBe(""); }); @@ -53,6 +59,9 @@ describe("DeletePermissionButton", () => { } }; + // eslint-disable-next-line @typescript-eslint/ban-ts-ignore + // @ts-ignore + // eslint-disable-next-line @typescript-eslint/no-empty-function const deleteIcon = mount( {}} />); expect(deleteIcon.html()).not.toBe(""); }); @@ -66,10 +75,13 @@ describe("DeletePermissionButton", () => { } }; + // eslint-disable-next-line @typescript-eslint/ban-ts-ignore + // @ts-ignore + // eslint-disable-next-line @typescript-eslint/no-empty-function const button = mount( {}} />); button.find(".fa-trash").simulate("click"); - expect(confirmAlert.mock.calls.length).toBe(1); + expect(button.find(".modal")).toBeTruthy(); }); it("should call the delete permission function with delete url", () => { @@ -82,11 +94,14 @@ describe("DeletePermissionButton", () => { }; let calledUrl = null; + function capture(permission) { calledUrl = permission._links.delete.href; } const button = mount( + // eslint-disable-next-line @typescript-eslint/ban-ts-ignore + // @ts-ignore ); button.find(".fa-trash").simulate("click");