Skip to content

Commit

Permalink
[FIX] tests: missing closing brackets in querySelectors
Browse files Browse the repository at this point in the history
Some brackets were missing in the querySelectors of the tests. It
didn't seem to affect the tests, but we should fix it anyway.

Task: 4276932
Part-of: #5115
Signed-off-by: Vincent Schippefilt (vsc) <[email protected]>
  • Loading branch information
hokolomopo committed Oct 22, 2024
1 parent f392c9b commit 449d752
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions tests/bottom_bar/bottom_bar_component.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -229,8 +229,8 @@ describe("BottomBar component", () => {

triggerMouseEvent(".o-sheet", "contextmenu");
await nextTick();
expect(fixture.querySelector(".o-menu-item[data-name='move_left'")).toBeNull();
expect(fixture.querySelector(".o-menu-item[data-name='move_right'")).toBeNull();
expect(fixture.querySelector(".o-menu-item[data-name='move_left']")).toBeNull();
expect(fixture.querySelector(".o-menu-item[data-name='move_right']")).toBeNull();
});

describe("Rename a sheet", () => {
Expand Down Expand Up @@ -411,7 +411,7 @@ describe("BottomBar component", () => {

triggerMouseEvent(".o-sheet", "contextmenu");
await nextTick();
expect(fixture.querySelector(".o-menu-item[data-name='delete'")).toBeNull();
expect(fixture.querySelector(".o-menu-item[data-name='delete']")).toBeNull();
});

test("Can open the list of sheets", async () => {
Expand Down Expand Up @@ -454,7 +454,7 @@ describe("BottomBar component", () => {
const { model } = await mountBottomBar();
createSheet(model, { sheetId: "42", hidden: true });
await click(fixture, ".o-list-sheets");
const menuItem = fixture.querySelector<HTMLElement>(".o-menu-item[data-name='42'")!;
const menuItem = fixture.querySelector<HTMLElement>(".o-menu-item[data-name='42']")!;
expect(toHex(menuItem.style.color)).toEqual("#808080");
await click(menuItem);
expect(model.getters.getActiveSheetId()).toBe("42");
Expand All @@ -466,7 +466,7 @@ describe("BottomBar component", () => {
createSheet(model, { sheetId: "42", hidden: true });
model.updateMode("readonly");
await click(fixture, ".o-list-sheets");
const menuItem = fixture.querySelector<HTMLElement>(".o-menu-item[data-name='42'");
const menuItem = fixture.querySelector<HTMLElement>(".o-menu-item[data-name='42']");
expect(menuItem!.classList).toContain("disabled");
});

Expand Down

0 comments on commit 449d752

Please sign in to comment.