refactor: resolve the instructor dashboard link with frontend-base

The back link substituted the course ID into the route and checked
whether the result was a path in this site or an external URL by hand;
frontend-base now owns that.

Co-Authored-By: Claude <noreply@anthropic.com>

Author: arbrandes

package-lock.json

@@ -62,7 +62,7 @@
   "sass": "^1.49.0"
  },
  "peerDependencies": {
-  "@openedx/frontend-base": "^2.0.0-alpha.2 || 0.0.0-dev",
+  "@openedx/frontend-base": "^2.0.0-alpha.15 || 0.0.0-dev",
   "@openedx/paragon": "^23",
   "@tanstack/react-query": "^5",
   "react": "^18",

package.json

@@ -1,6 +1,6 @@
 import { Link } from 'react-router-dom';
 
-import { getUrlByRouteRole, useIntl } from '@openedx/frontend-base';
+import { resolveRouteByRole, useIntl } from '@openedx/frontend-base';
 import { Button } from '@openedx/paragon';
 
 import { instructorDashboardUrl } from '@src/data/services/lms/urls';
@@ -22,8 +22,7 @@ export const GradebookHeader = () => {
   // Prefer the instructor dashboard route if the running site provides one,
   // so navigation stays within the SPA; otherwise fall back to a full page
   // load of the legacy LMS dashboard.
-  const dashboardRoute = getUrlByRouteRole(instructorDashboardRole)?.replace(':courseId', courseId);
-  const isInternalRoute = !!dashboardRoute && !/^[a-z][a-z0-9+.-]*:/i.test(dashboardRoute);
+  const dashboardRoute = resolveRouteByRole(instructorDashboardRole, { courseId });
   const backLinkContent = (
     <>
       <span aria-hidden="true">{'<< '}</span>
@@ -32,10 +31,10 @@ export const GradebookHeader = () => {
   );
   return (
     <div className="gradebook-header">
-      {isInternalRoute ? (
-        <Link to={dashboardRoute} className="mb-3">{backLinkContent}</Link>
+      {dashboardRoute?.isInternal ? (
+        <Link to={dashboardRoute.url} className="mb-3">{backLinkContent}</Link>
       ) : (
-        <a href={dashboardRoute ?? instructorDashboardUrl()} className="mb-3">{backLinkContent}</a>
+        <a href={dashboardRoute?.url ?? instructorDashboardUrl()} className="mb-3">{backLinkContent}</a>
       )}
       <h1>{formatMessage(messages.gradebook)}</h1>
       <div className="subtitle-row d-flex justify-content-between align-items-center">

src/components/GradebookHeader/index.jsx

@@ -2,7 +2,7 @@ import { renderWithAllProviders, initializeMocks } from '@src/testUtils';
 import { screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 
-import { getUrlByRouteRole } from '@openedx/frontend-base';
+import { resolveRouteByRole } from '@openedx/frontend-base';
 import { instructorDashboardUrl } from '@src/data/services/lms/urls';
 
 import { GradebookHeader } from './index';
@@ -11,7 +11,7 @@ import messages from './messages';
 
 jest.mock('@openedx/frontend-base', () => ({
   ...jest.requireActual('@openedx/frontend-base'),
-  getUrlByRouteRole: jest.fn(),
+  resolveRouteByRole: jest.fn(),
 }));
 jest.mock('@src/data/services/lms/urls', () => ({
   instructorDashboardUrl: jest.fn(),
@@ -25,7 +25,7 @@ describe('GradebookHeader', () => {
 
   beforeEach(() => {
     jest.clearAllMocks();
-    getUrlByRouteRole.mockReturnValue(null);
+    resolveRouteByRole.mockReturnValue(null);
     instructorDashboardUrl.mockReturnValue('https://example.com/dashboard');
   });
 
@@ -90,8 +90,15 @@ describe('GradebookHeader', () => {
     });
 
     it('renders an SPA link when the site provides an instructor dashboard route', () => {
-      getUrlByRouteRole.mockReturnValue('/instructor-dashboard/:courseId');
+      resolveRouteByRole.mockReturnValue({
+        url: '/instructor-dashboard/course-v1:TestU+CS101+2024',
+        isInternal: true,
+      });
       renderWithAllProviders(<GradebookHeader />);
+      expect(resolveRouteByRole).toHaveBeenCalledWith(
+        'org.openedx.frontend.role.instructorDashboard',
+        { courseId: 'course-v1:TestU+CS101+2024' },
+      );
       const dashboardLink = screen.getByRole('link');
       expect(dashboardLink).toHaveAttribute(
         'href',
@@ -101,7 +108,10 @@ describe('GradebookHeader', () => {
     });
 
     it('renders a plain anchor when the instructor dashboard route is external', () => {
-      getUrlByRouteRole.mockReturnValue('https://other.example.com/dashboard');
+      resolveRouteByRole.mockReturnValue({
+        url: 'https://other.example.com/dashboard',
+        isInternal: false,
+      });
       renderWithAllProviders(<GradebookHeader />);
       const dashboardLink = screen.getByRole('link');
       expect(dashboardLink).toHaveAttribute(