Skip to content

Commit 69b00b1

Browse files
bjohansebasclaude
andcommitted
test: drop two overlay unit tests the e2e suite already covers
The accent-bar colors are asserted (both hues, including the error-to-warning flip) by the shared-state e2e, and the default pagination counter by the pagination e2e. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent 17a475c commit 69b00b1

1 file changed

Lines changed: 0 additions & 15 deletions

File tree

test/overlay.test.js

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -59,13 +59,6 @@ describe("overlay", () => {
5959
expect(badge.style.backgroundColor).toBe("rgb(255, 211, 14)");
6060
});
6161

62-
it("colors the top accent bar red for errors and yellow for warnings", () => {
63-
showProblems("errors", ["boom"]);
64-
expect(getCard().style.borderTopColor).toBe("rgb(255, 51, 72)");
65-
showProblems("warnings", ["careful"]);
66-
expect(getCard().style.borderTopColor).toBe("rgb(255, 211, 14)");
67-
});
68-
6962
it("highlights the file path and leaves the location uncolored", () => {
7063
showProblems("errors", ["./src/render.js 7:2\nModule parse failed"]);
7164
const pathSpan = [...getCard().querySelectorAll("span")].find(
@@ -215,14 +208,6 @@ describe("overlay", () => {
215208
configureOverlay({ paginate: true });
216209
});
217210

218-
it("shows one problem at a time with a counter by default", () => {
219-
showProblems("errors", ["first boom", "second boom", "third boom"]);
220-
221-
expect(getCard().textContent).toContain("first boom");
222-
expect(getCard().textContent).not.toContain("second boom");
223-
expect(getCard().textContent).toContain("1 / 3");
224-
});
225-
226211
it("navigates with the prev/next buttons and clamps at the ends", () => {
227212
showProblems("errors", ["first boom", "second boom"]);
228213

0 commit comments

Comments
 (0)