From 1e339c1815bc8a7e4bf595eef9b2fc0284490bb6 Mon Sep 17 00:00:00 2001 From: "artur.filippovskii" Date: Mon, 1 Dec 2025 11:20:17 +0200 Subject: [PATCH 1/6] feat: Certificate Status marked up as a heading level 2 --- src/course-home/progress-tab/ProgressTab.test.jsx | 6 ++++++ .../progress-tab/certificate-status/CertificateStatus.jsx | 2 +- webpack.dev-tutor.config.js | 0 3 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 webpack.dev-tutor.config.js diff --git a/src/course-home/progress-tab/ProgressTab.test.jsx b/src/course-home/progress-tab/ProgressTab.test.jsx index be99cab11d..72627e6d42 100644 --- a/src/course-home/progress-tab/ProgressTab.test.jsx +++ b/src/course-home/progress-tab/ProgressTab.test.jsx @@ -5,6 +5,7 @@ import { sendTrackEvent } from '@edx/frontend-platform/analytics'; import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth'; import { breakpoints } from '@openedx/paragon'; import MockAdapter from 'axios-mock-adapter'; +import { within } from '@testing-library/react'; import { fireEvent, initializeMockApp, logUnhandledRequests, render, screen, act, @@ -1243,6 +1244,11 @@ describe('Progress Tab', () => { linkType: 'button', pageName: 'progress', }); + + const certificateStatusComponent = screen.queryByTestId('certificate-status-component'); + expect(certificateStatusComponent).toBeInTheDocument(); + const headerElement = within(certificateStatusComponent).getByRole('heading', { level: 2 }); + expect(headerElement).toBeInTheDocument(); }); it('Displays nothing if audit only', async () => { diff --git a/src/course-home/progress-tab/certificate-status/CertificateStatus.jsx b/src/course-home/progress-tab/certificate-status/CertificateStatus.jsx index bc1cd92039..149dda8480 100644 --- a/src/course-home/progress-tab/certificate-status/CertificateStatus.jsx +++ b/src/course-home/progress-tab/certificate-status/CertificateStatus.jsx @@ -244,7 +244,7 @@ const CertificateStatus = () => {
- + {header}} /> {body} diff --git a/webpack.dev-tutor.config.js b/webpack.dev-tutor.config.js new file mode 100644 index 0000000000..e69de29bb2 From 8106ccb432d00646630cd09a344dad2925b67bc5 Mon Sep 17 00:00:00 2001 From: "artur.filippovskii" Date: Mon, 1 Dec 2025 14:41:36 +0200 Subject: [PATCH 2/6] fix: fix test falls --- src/courseware/course/Course.test.jsx | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/courseware/course/Course.test.jsx b/src/courseware/course/Course.test.jsx index 754ff134b5..f6e9040d8f 100644 --- a/src/courseware/course/Course.test.jsx +++ b/src/courseware/course/Course.test.jsx @@ -58,6 +58,10 @@ describe('Course', () => { global.innerWidth = breakpoints.extraLarge.minWidth; }); + beforeEach(() => { + sessionStorage.clear(); + }); + // This was passing when it shouldn't have been because of improper // waitFor use. With the React 18 upgrade it no longer improperly passes // so we are skipping it. See https://github.com/openedx/frontend-app-learning/issues/1669 From 784f0af6cd22cc073c9a245751d0f2ce9d9e6dd8 Mon Sep 17 00:00:00 2001 From: "artur.filippovskii" Date: Tue, 9 Dec 2025 12:05:41 +0200 Subject: [PATCH 3/6] fix: remove extra file and remove extra lines --- src/courseware/course/Course.test.jsx | 3 --- webpack.dev-tutor.config.js | 0 2 files changed, 3 deletions(-) delete mode 100644 webpack.dev-tutor.config.js diff --git a/src/courseware/course/Course.test.jsx b/src/courseware/course/Course.test.jsx index f6e9040d8f..01d9ae7eaf 100644 --- a/src/courseware/course/Course.test.jsx +++ b/src/courseware/course/Course.test.jsx @@ -58,9 +58,6 @@ describe('Course', () => { global.innerWidth = breakpoints.extraLarge.minWidth; }); - beforeEach(() => { - sessionStorage.clear(); - }); // This was passing when it shouldn't have been because of improper // waitFor use. With the React 18 upgrade it no longer improperly passes diff --git a/webpack.dev-tutor.config.js b/webpack.dev-tutor.config.js deleted file mode 100644 index e69de29bb2..0000000000 From 43de14d13c61de5ffc49ab70d3abbcdc8663c98b Mon Sep 17 00:00:00 2001 From: "artur.filippovskii" Date: Tue, 9 Dec 2025 18:10:03 +0200 Subject: [PATCH 4/6] fix: remove extra line --- src/courseware/course/Course.test.jsx | 1 - 1 file changed, 1 deletion(-) diff --git a/src/courseware/course/Course.test.jsx b/src/courseware/course/Course.test.jsx index 01d9ae7eaf..754ff134b5 100644 --- a/src/courseware/course/Course.test.jsx +++ b/src/courseware/course/Course.test.jsx @@ -58,7 +58,6 @@ describe('Course', () => { global.innerWidth = breakpoints.extraLarge.minWidth; }); - // This was passing when it shouldn't have been because of improper // waitFor use. With the React 18 upgrade it no longer improperly passes // so we are skipping it. See https://github.com/openedx/frontend-app-learning/issues/1669 From 38ef4b78ef30850c372bd0c24ab0208393def779 Mon Sep 17 00:00:00 2001 From: "artur.filippovskii" Date: Thu, 18 Dec 2025 10:56:49 +0200 Subject: [PATCH 5/6] fix: change import method --- src/course-home/progress-tab/ProgressTab.test.jsx | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/course-home/progress-tab/ProgressTab.test.jsx b/src/course-home/progress-tab/ProgressTab.test.jsx index 72627e6d42..3443d1c40a 100644 --- a/src/course-home/progress-tab/ProgressTab.test.jsx +++ b/src/course-home/progress-tab/ProgressTab.test.jsx @@ -5,10 +5,9 @@ import { sendTrackEvent } from '@edx/frontend-platform/analytics'; import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth'; import { breakpoints } from '@openedx/paragon'; import MockAdapter from 'axios-mock-adapter'; -import { within } from '@testing-library/react'; import { - fireEvent, initializeMockApp, logUnhandledRequests, render, screen, act, + fireEvent, initializeMockApp, logUnhandledRequests, render, screen, act, within } from '../../setupTest'; import { appendBrowserTimezoneToUrl, executeThunk } from '../../utils'; import * as thunks from '../data/thunks'; From d16fd2bc12701b47b4ea2a059b36505a2198795c Mon Sep 17 00:00:00 2001 From: "artur.filippovskii" Date: Thu, 18 Dec 2025 10:59:37 +0200 Subject: [PATCH 6/6] fix: fix lint errors --- src/course-home/progress-tab/ProgressTab.test.jsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/course-home/progress-tab/ProgressTab.test.jsx b/src/course-home/progress-tab/ProgressTab.test.jsx index 3443d1c40a..122a2b925c 100644 --- a/src/course-home/progress-tab/ProgressTab.test.jsx +++ b/src/course-home/progress-tab/ProgressTab.test.jsx @@ -7,7 +7,7 @@ import { breakpoints } from '@openedx/paragon'; import MockAdapter from 'axios-mock-adapter'; import { - fireEvent, initializeMockApp, logUnhandledRequests, render, screen, act, within + fireEvent, initializeMockApp, logUnhandledRequests, render, screen, act, within, } from '../../setupTest'; import { appendBrowserTimezoneToUrl, executeThunk } from '../../utils'; import * as thunks from '../data/thunks';