Skip to content

Commit f0f1d2a

Browse files
fix test and arbitrary props
1 parent 320ec0b commit f0f1d2a

2 files changed

Lines changed: 12 additions & 5 deletions

File tree

packages/ui-components/src/components/DescriptionList/DescriptionList.stories.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ const meta: Meta<typeof DL> = {
1616
docs: {
1717
description: {
1818
component: `
19-
A wrapper component that contains a list of terms and their corresponding descriptions.
19+
A component that contains a list of terms and their corresponding descriptions.
2020
2121
This component enforces structure by expecting child elements of \`DescriptionTerm\` and \`DescriptionDefinition\`.
2222

packages/ui-components/src/components/DescriptionList/DescriptionList.test.tsx

Lines changed: 11 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,19 @@ describe("DescriptionList", () => {
1414
it("renders child DescriptionTerm and DescriptionDefinition components correctly", () => {
1515
render(
1616
<DescriptionList>
17-
<DescriptionTerm>Term 1</DescriptionTerm>
18-
<DescriptionDefinition>Definition 1</DescriptionDefinition>
17+
<DescriptionTerm data-testid="term-1">Term 1</DescriptionTerm>
18+
<DescriptionDefinition data-testid="definition-1">Definition 1</DescriptionDefinition>
1919
</DescriptionList>
2020
)
21-
expect(screen.getByText("Term 1")).toBeInTheDocument()
22-
expect(screen.getByText("Definition 1")).toBeInTheDocument()
21+
22+
const termElement = screen.getByTestId("term-1")
23+
const definitionElement = screen.getByTestId("definition-1")
24+
25+
expect(termElement).toBeInTheDocument()
26+
expect(termElement).toHaveTextContent("Term 1")
27+
28+
expect(definitionElement).toBeInTheDocument()
29+
expect(definitionElement).toHaveTextContent("Definition 1")
2330
})
2431

2532
it("applies custom className to the <dl> element", () => {

0 commit comments

Comments
 (0)