fix: using relative path now

Author: marslanabdulrauf

src/optimizer-page/scan-results/BrokenLinkTable.test.tsx

@@ -717,7 +717,7 @@ describe('BrokenLinkTable', () => {
 
       await waitFor(() => {
         expect(window.open).toHaveBeenCalledWith(
-          'http://localhost/course/course-v1:TestX+Test101+2024/container/unit-1#block-1',
+          '/course/course-v1:TestX+Test101+2024/container/unit-1#block-1',
           '_blank',
         );
       });

src/optimizer-page/scan-results/BrokenLinkTable.tsx

@@ -211,7 +211,11 @@ const BrokenLinkTable: FC<BrokenLinkTableProps> = ({
             return {
               Links: (
                 <LinksCol
-                  block={{ url: buildBlockContainerUrl(courseId, unit.id, block.id), displayName: block.displayName || 'Go to block', id: block.id }}
+                  block={{
+                    url: buildBlockContainerUrl(courseId, unit.id, block.id),
+                    displayName: block.displayName || 'Go to block',
+                    id: block.id,
+                  }}
                   href={displayLink}
                   showIcon={false}
                   showUpdateButton
@@ -240,7 +244,10 @@ const BrokenLinkTable: FC<BrokenLinkTableProps> = ({
         const blockBrokenLinks = block.brokenLinks.map((link) => ({
           Links: (
             <LinksCol
-              block={{ url: buildBlockContainerUrl(courseId, unit.id, block.id), displayName: block.displayName || 'Go to block' }}
+              block={{
+                url: buildBlockContainerUrl(courseId, unit.id, block.id),
+                displayName: block.displayName || 'Go to block',
+              }}
               href={link}
               linkType={BROKEN}
             />
@@ -256,7 +263,10 @@ const BrokenLinkTable: FC<BrokenLinkTableProps> = ({
         const blockLockedLinks = block.lockedLinks.map((link) => ({
           Links: (
             <LinksCol
-              block={{ url: buildBlockContainerUrl(courseId, unit.id, block.id), displayName: block.displayName || 'Go to block' }}
+              block={{
+                url: buildBlockContainerUrl(courseId, unit.id, block.id),
+                displayName: block.displayName || 'Go to block',
+              }}
               href={link}
               linkType={LOCKED}
             />
@@ -273,7 +283,10 @@ const BrokenLinkTable: FC<BrokenLinkTableProps> = ({
         const externalForbiddenLinks = block.externalForbiddenLinks.map((link) => ({
           Links: (
             <LinksCol
-              block={{ url: buildBlockContainerUrl(courseId, unit.id, block.id), displayName: block.displayName || 'Go to block' }}
+              block={{
+                url: buildBlockContainerUrl(courseId, unit.id, block.id),
+                displayName: block.displayName || 'Go to block',
+              }}
               href={link}
               linkType={MANUAL}
             />

src/optimizer-page/utils.test.js

@@ -114,21 +114,7 @@ describe('buildBlockContainerUrl', () => {
     const result = buildBlockContainerUrl(courseId, unitId, blockId);
 
     expect(result).toBe(
-      `${window.location.origin}/course/${courseId}/container/${unitId}#${blockId}`,
+      `/course/${courseId}/container/${unitId}#${blockId}`,
     );
   });
-
-  it('should return only the internal path when window is undefined', () => {
-    const courseId = 'course-v1:Test+Course+2024';
-    const unitId = 'unit123';
-    const blockId = 'block456';
-
-    Object.defineProperty(globalThis, 'window', { value: undefined, configurable: true, writable: true });
-
-    const result = buildBlockContainerUrl(courseId, unitId, blockId);
-
-    Object.defineProperty(globalThis, 'window', { value: globalThis, configurable: true, writable: true });
-
-    expect(result).toBe(`/course/${courseId}/container/${unitId}#${blockId}`);
-  });
 });

src/optimizer-page/utils.ts

@@ -8,9 +8,7 @@ export const buildBlockContainerUrl = (
 ): string => {
   const path = `/course/${courseId}/container/${unitId}#${blockId}`;
   const internalPath = createCorrectInternalRoute(path);
-  return typeof window !== 'undefined'
-    ? `${window.location.origin}${internalPath}`
-    : internalPath;
+  return internalPath;
 };
 
 export const countBrokenLinks = (