373-disclosure: Fix toggle and add test

indiana-university · Oct 6, 2023 · e0f172e · e0f172e
1 parent 4d916dd
commit e0f172e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/components/PageContent/Disclosure/Disclosure.jsx b/src/components/PageContent/Disclosure/Disclosure.jsx
@@ -84,7 +84,7 @@ const Disclosure = ({
       <button
         className="rvt-disclosure__toggle"
         onClick={toggleDisclosure}
-        aria-expanded="false"
+        aria-expanded={isOpenState ? "true" : "false"}
       >
         {title}
       </button>

diff --git a/src/components/PageContent/Disclosure/Disclosure.test.jsx b/src/components/PageContent/Disclosure/Disclosure.test.jsx
@@ -52,12 +52,15 @@ describe("<Disclosure />", () => {
       render(<Disclosure title={title}>{child}</Disclosure>);
 
       await user.click(screen.getByRole("button", {}));
+      const button = screen.getByRole("button", {});
       const children = screen.queryByTestId(TestUtils.Disclosure.testId, {});
       expect(children).toBeVisible();
       expect(children).toHaveClass("rvt-disclosure__content");
+      expect(button).toHaveAttribute("aria-expanded", "true");
       expect(screen.queryByText(child, {})).toBeVisible();
 
       await user.click(screen.getByRole("button", {}));
+      expect(button).toHaveAttribute("aria-expanded", "false");
       expect(
         screen.queryByTestId(TestUtils.Disclosure.testId, {})
       ).not.toBeInTheDocument();