Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Commit

Permalink
remove annoying test
Browse files Browse the repository at this point in the history
  • Loading branch information
Kerry Archibald committed Apr 30, 2023
1 parent c12abc4 commit c5df9ba
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1,070 deletions.
22 changes: 1 addition & 21 deletions test/components/views/settings/Notifications-test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ import {
TweakName,
ConditionKind,
IPushRuleCondition,
PushRuleKind,
} from "matrix-js-sdk/src/matrix";
import { randomString } from "matrix-js-sdk/src/randomstring";
import { IThreepid, ThreepidMedium } from "matrix-js-sdk/src/@types/threepids";
Expand Down Expand Up @@ -390,7 +389,7 @@ describe("<Notifications />", () => {
);
});

xit("displays error when pusher update fails", async () => {
it("displays error when pusher update fails", async () => {
mockClient.setPusher.mockRejectedValue({});
await getComponentAndWait();

Expand Down Expand Up @@ -818,25 +817,6 @@ describe("<Notifications />", () => {
),
).toBeInTheDocument();
});

fit("adds a new keyword", async () => {
const { container } = await getComponentAndWait();

await userEvent.click(screen.getByLabelText("Keyword"));
await userEvent.type(screen.getByLabelText("Keyword"), "jest");

expect(container).toMatchSnapshot();

await flushPromises();
expect(screen.getByLabelText("Keyword")).toHaveValue("jest");

fireEvent.click(screen.getByText("Add"));

expect(mockClient.addPushRule).toHaveBeenCalledWith("global", PushRuleKind.ContentSpecific, "jest", {
actions: [PushRuleActionName.Notify, { set_tweak: "highlight", value: false }],
pattern: "jest",
});
});
});

describe("clear all notifications", () => {
Expand Down
Loading

0 comments on commit c5df9ba

Please sign in to comment.