build: Upgrade to Node 24

Author: PKulkoRaccoonGang

.github/workflows/validate.yml

@@ -12,7 +12,6 @@ jobs:
     strategy:
       matrix:
         node: [20, 24]
-    continue-on-error: ${{ matrix.node == 24 }}
     steps:
     - uses: actions/checkout@v5
     - uses: actions/setup-node@v4

.nvmrc

@@ -1 +1 @@
-20
+24

jest.config.js

@@ -22,22 +22,4 @@ const config = createConfig('jest', {
   testEnvironment: 'jsdom',
 });
 
-// delete config.testURL;
-
-config.reporters = [...(config.reporters || []), ["jest-console-group-reporter", {
-  // change this setting if need to see less details for each test
-  // reportType: "summary" | "details",
-  // enable: true | false,
-  afterEachTest: {
-    enable: true,
-    filePaths: false,
-    reportType: "details",
-  },
-  afterAllTests: {
-    reportType: "summary",
-    enable: true,
-    filePaths: true,
-  },
-}]];
-
 module.exports = config;

package-lock.json

@@ -26,8 +26,8 @@ const SidebarProvider: React.FC<Props> = ({
   const { verifiedMode } = useModel('courseHomeMeta', courseId);
   const topic = useModel('discussionTopics', unitId);
   const windowWidth = useWindowSize().width ?? window.innerWidth;
-  const shouldDisplayFullScreen = windowWidth < breakpoints.large.minWidth;
-  const shouldDisplaySidebarOpen = windowWidth > breakpoints.medium.minWidth;
+  const shouldDisplayFullScreen = windowWidth < (breakpoints.large?.minWidth ?? 992);
+  const shouldDisplaySidebarOpen = windowWidth > (breakpoints.medium?.minWidth ?? 768);
   const query = new URLSearchParams(window.location.search);
   const isInitiallySidebarOpen = shouldDisplaySidebarOpen || query.get('sidebar') === 'true';
   const sidebarKey = `sidebar.${courseId}`;

src/courseware/course/new-sidebar/SidebarContextProvider.tsx

@@ -44,7 +44,7 @@ describe('NotificationsWidget', () => {
   }
 
   beforeEach(async () => {
-    global.innerWidth = breakpoints.large.minWidth;
+    global.innerWidth = breakpoints.large?.minWidth ?? 992;
     store = initializeStore();
     axiosMock = new MockAdapter(getAuthenticatedHttpClient());
     axiosMock.onGet(courseMetadataUrl).reply(200, defaultMetadata);

src/courseware/course/new-sidebar/sidebars/discussions-notifications/notifications/NotificationsWidget.test.tsx

@@ -39,6 +39,46 @@ jest.mock('@src/generic/plugin-store', () => ({
   usePluginsCallback: jest.fn((_, cb) => cb),
 }));
 
+/* eslint-disable react/prop-types */
+/* eslint-disable global-require */
+jest.mock('@openedx/paragon', () => {
+  const ReactLib = require('react');
+  const actual = jest.requireActual('@openedx/paragon');
+
+  return {
+    ...actual,
+    Modal: ({
+      isOpen, title, body, children, onClose,
+    }) => {
+      if (!isOpen) {
+        return null;
+      }
+
+      let content = null;
+      if (body) {
+        content = ReactLib.createElement('div', null, body);
+      } else if (title) {
+        content = ReactLib.createElement('iframe', {
+          title,
+          src: 'test-url',
+        });
+      }
+
+      return ReactLib.createElement(
+        'div',
+        { role: 'dialog' },
+        ReactLib.createElement('div', {
+          'data-testid': 'modal-backdrop',
+          onClick: onClose || (() => {}),
+        }),
+        content,
+        children,
+      );
+    },
+  };
+});
+/* eslint-enable react/prop-types */
+
 class MockLoggingService {
   // eslint-disable-next-line no-console
   logInfo = jest.fn(infoString => console.log(infoString));