From 2d55ba4ccc0e9f902db83ad911cabe5d6d7b59be Mon Sep 17 00:00:00 2001 From: Jillian Date: Fri, 25 Apr 2025 10:28:37 +0930 Subject: [PATCH] fix: allow units sourced from libraries to update their settings (#1863) --- src/course-unit/header-title/HeaderTitle.jsx | 1 - src/course-unit/header-title/HeaderTitle.test.jsx | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/course-unit/header-title/HeaderTitle.jsx b/src/course-unit/header-title/HeaderTitle.jsx index 1c318c207..f87199e16 100644 --- a/src/course-unit/header-title/HeaderTitle.jsx +++ b/src/course-unit/header-title/HeaderTitle.jsx @@ -89,7 +89,6 @@ const HeaderTitle = ({ className="flex-shrink-0" iconAs={SettingsIcon} onClick={openConfigureModal} - disabled={readOnly} /> ', () => { expect(getByRole('button', { name: messages.altButtonSettings.defaultMessage })).toBeEnabled(); }); - it('Units sourced from upstream show a disabled edit form and config menu', async () => { + it('Units sourced from upstream show a disabled edit button', async () => { // Override mock unit with one sourced from an upstream library axiosMock = new MockAdapter(getAuthenticatedHttpClient()); axiosMock @@ -92,7 +92,7 @@ describe('', () => { const { getByRole } = renderComponent(); expect(getByRole('button', { name: messages.altButtonEdit.defaultMessage })).toBeDisabled(); - expect(getByRole('button', { name: messages.altButtonSettings.defaultMessage })).toBeDisabled(); + expect(getByRole('button', { name: messages.altButtonSettings.defaultMessage })).toBeEnabled(); }); it('calls toggle edit title form by clicking on Edit button', () => {