Skip to content

Commit 7ade08a

Browse files
Ryan Hanszclaude
authored andcommitted
test: Remove weak tests that only check presence
Remove tests that only verify presence of elements without testing behavior: - PrintOptionsDialog: removed 4 tests (renders dialog, open=false, descriptions) - CompactResourceList: removed 2 tests (heading, numbered list) Remaining tests verify actual user interactions and semantic behavior. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 158d97a commit 7ade08a

2 files changed

Lines changed: 0 additions & 52 deletions

File tree

frontend/tests/components/CompactResourceList.test.tsx

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -24,21 +24,6 @@ describe("CompactResourceList", () => {
2424
},
2525
];
2626

27-
it("renders the section heading", () => {
28-
render(<CompactResourceList resources={mockResources} />);
29-
30-
expect(
31-
screen.getByText("Selected Resources - Contact Information"),
32-
).toBeInTheDocument();
33-
});
34-
35-
it("renders numbered list of resources", () => {
36-
render(<CompactResourceList resources={mockResources} />);
37-
38-
expect(screen.getByText("1.")).toBeInTheDocument();
39-
expect(screen.getByText("2.")).toBeInTheDocument();
40-
});
41-
4227
it("displays resource names", () => {
4328
render(<CompactResourceList resources={mockResources} />);
4429

frontend/tests/components/PrintOptionsDialog.test.tsx

Lines changed: 0 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -14,17 +14,6 @@ describe("PrintOptionsDialog", () => {
1414
jest.clearAllMocks();
1515
});
1616

17-
it("renders dialog with title and description", () => {
18-
render(<PrintOptionsDialog {...defaultProps} />);
19-
20-
expect(
21-
screen.getByText("What would you like to print?"),
22-
).toBeInTheDocument();
23-
expect(
24-
screen.getByText("Choose what to include in your printed report."),
25-
).toBeInTheDocument();
26-
});
27-
2817
it("shows both options when hasActionPlan is true", () => {
2918
render(<PrintOptionsDialog {...defaultProps} />);
3019

@@ -63,32 +52,6 @@ describe("PrintOptionsDialog", () => {
6352
expect(defaultProps.onOpenChange).toHaveBeenCalledWith(false);
6453
});
6554

66-
it("does not render content when open is false", () => {
67-
render(<PrintOptionsDialog {...defaultProps} open={false} />);
68-
69-
expect(
70-
screen.queryByText("What would you like to print?"),
71-
).not.toBeInTheDocument();
72-
});
73-
74-
it("displays description for Action Plan Only option", () => {
75-
render(<PrintOptionsDialog {...defaultProps} />);
76-
77-
expect(
78-
screen.getByText(
79-
"Includes contact info for selected resources (no descriptions)",
80-
),
81-
).toBeInTheDocument();
82-
});
83-
84-
it("displays description for Full Referrals option", () => {
85-
render(<PrintOptionsDialog {...defaultProps} />);
86-
87-
expect(
88-
screen.getByText("Includes complete resource details with descriptions"),
89-
).toBeInTheDocument();
90-
});
91-
9255
it("has proper accessibility attributes", () => {
9356
render(<PrintOptionsDialog {...defaultProps} />);
9457

0 commit comments

Comments
 (0)