fix: update tests for link role after href change

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

Author: arbrandes

src/containers/CourseCard/components/CourseCardActions/BeginCourseButton.test.jsx

@@ -63,22 +63,22 @@ describe('BeginCourseButton', () => {
       it('should be disabled', () => {
         useActionDisabledState.mockReturnValueOnce({ disableBeginCourse: true });
         renderComponent();
-        const button = screen.getByRole('button', { name: 'Begin Course' });
+        const button = screen.getByRole('link', { name: 'Begin Course' });
         expect(button).toHaveClass('disabled');
         expect(button).toHaveAttribute('aria-disabled', 'true');
       });
     });
     describe('enabled', () => {
       it('should be enabled', () => {
         renderComponent();
-        const button = screen.getByRole('button', { name: 'Begin Course' });
+        const button = screen.getByRole('link', { name: 'Begin Course' });
         expect(button).not.toHaveClass('disabled');
         expect(button).not.toHaveAttribute('aria-disabled', 'true');
       });
       it('should track enter course clicked event on click, with exec ed param', () => {
         renderComponent();
         const user = userEvent.setup();
-        const button = screen.getByRole('button', { name: 'Begin Course' });
+        const button = screen.getByRole('link', { name: 'Begin Course' });
         user.click(button);
         expect(useCourseTrackingEvent).toHaveBeenCalledWith(
           track.course.enterCourseClicked,

src/containers/CourseCard/components/CourseCardActions/ResumeButton.test.jsx

@@ -63,23 +63,23 @@ describe('ResumeButton', () => {
       });
       it('should be disabled', () => {
         render(<IntlProvider locale="en"><ResumeButton {...props} /></IntlProvider>);
-        const button = screen.getByRole('button', { name: 'Resume' });
+        const button = screen.getByRole('link', { name: 'Resume' });
         expect(button).toHaveClass('disabled');
         expect(button).toHaveAttribute('aria-disabled', 'true');
       });
     });
     describe('enabled', () => {
       it('should be enabled', () => {
         render(<IntlProvider locale="en"><ResumeButton {...props} /></IntlProvider>);
-        const button = screen.getByRole('button', { name: 'Resume' });
+        const button = screen.getByRole('link', { name: 'Resume' });
         expect(button).toBeInTheDocument();
         expect(button).not.toHaveClass('disabled');
         expect(button).not.toHaveAttribute('aria-disabled', 'true');
       });
       it('should track enter course clicked event on click, with exec ed param', async () => {
         render(<IntlProvider locale="en"><ResumeButton {...props} /></IntlProvider>);
         const user = userEvent.setup();
-        const button = screen.getByRole('button', { name: 'Resume' });
+        const button = screen.getByRole('link', { name: 'Resume' });
         user.click(button);
         expect(useCourseTrackingEvent).toHaveBeenCalledWith(
           track.course.enterCourseClicked,

src/containers/CourseCard/components/CourseCardActions/ViewCourseButton.test.jsx

@@ -32,7 +32,7 @@ const homeUrl = 'homeUrl';
 describe('ViewCourseButton', () => {
   it('learner can view course', async () => {
     render(<IntlProvider locale="en"><ViewCourseButton {...defaultProps} /></IntlProvider>);
-    const button = screen.getByRole('button', { name: 'View Course' });
+    const button = screen.getByRole('link', { name: 'View Course' });
     expect(button).toBeInTheDocument();
     expect(button).not.toHaveClass('disabled');
     expect(button).not.toHaveAttribute('aria-disabled', 'true');
@@ -42,7 +42,7 @@ describe('ViewCourseButton', () => {
     useCourseTrackingEvent.mockReturnValue(mockedTrackCourseEvent);
     render(<IntlProvider locale="en"><ViewCourseButton {...defaultProps} /></IntlProvider>);
     const user = userEvent.setup();
-    const button = screen.getByRole('button', { name: 'View Course' });
+    const button = screen.getByRole('link', { name: 'View Course' });
     await user.click(button);
     expect(useCourseTrackingEvent).toHaveBeenCalledWith(
       track.course.enterCourseClicked,
@@ -54,7 +54,7 @@ describe('ViewCourseButton', () => {
   it('learner cannot view course', () => {
     useActionDisabledState.mockReturnValueOnce({ disableViewCourse: true });
     render(<IntlProvider locale="en"><ViewCourseButton {...defaultProps} /></IntlProvider>);
-    const button = screen.getByRole('button', { name: 'View Course' });
+    const button = screen.getByRole('link', { name: 'View Course' });
     expect(button).toHaveClass('disabled');
     expect(button).toHaveAttribute('aria-disabled', 'true');
   });