Skip to content
This repository was archived by the owner on May 7, 2026. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions src/App.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,17 @@ describe("App layout", () => {
expect(await screen.findByTestId("overview")).toBeInTheDocument();
});

it("should expose a single level-one heading on the overview screen", async () => {
renderApp();

expect(await screen.findByTestId("overview")).toBeInTheDocument();
expect(screen.getAllByRole("heading", { level: 1 })).toHaveLength(1);
expect(screen.getByRole("heading", { level: 1, name: "PRism" })).toBeInTheDocument();
expect(
screen.getByRole("heading", { level: 2, name: "Review requests come first" }),
).toBeInTheDocument();
});

it("should render my-prs view", async () => {
useDashboardStore.setState({ currentView: "mine" });
renderApp();
Expand Down
14 changes: 14 additions & 0 deletions src/components/Overview/Overview.test.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -158,6 +158,20 @@ describe("Overview", () => {
expect(screen.getByText("PR #2")).toBeInTheDocument();
});

it("should use a level-two heading for the priority lane title", () => {
setupMock(
makeDashboard({
reviewRequests: [makePr(1)],
}),
);

renderWithProviders(<Overview />);

expect(
screen.getByRole("heading", { name: "Review requests come first", level: 2 }),
).toBeInTheDocument();
});

it("should render my PRs section", () => {
setupMock(
makeDashboard({
Expand Down
2 changes: 1 addition & 1 deletion src/components/Overview/Overview.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ export function Overview(): ReactElement {
<p className="text-[11px] font-semibold uppercase tracking-[0.24em] text-accent">
Priority lane
</p>
<h1 className="text-lg font-semibold text-white">Review requests come first</h1>
<h2 className="text-lg font-semibold text-white">Review requests come first</h2>
<p className="max-w-2xl text-sm text-dim">
Surface the PRs that need your attention now so approvals and requested changes do
not get buried under passive updates.
Expand Down
Loading