test: update related tests

Author: diana-villalvazo-wgu

src/components/SpecifyLearnerField.test.tsx

@@ -214,7 +214,7 @@ describe('SpecifyLearnerField', () => {
       (useCourseInfo as jest.Mock).mockReturnValue({ data: { permissions: { admin: true, dataResearcher: false } } });
       (useLearner as jest.Mock).mockReturnValue({
         data: {},
-        refetch: jest.fn().mockResolvedValue({ data: {} }),
+        refetch: jest.fn().mockResolvedValue({ data: {}, error: { isAxiosError: true, response: { status: 404 } } }),
         error: { isAxiosError: true, response: { status: 404 } },
       });
     });

src/enrollments/components/UpdateBetaTesterModal.test.tsx

@@ -181,8 +181,8 @@ describe('UpdateBetaTesterModal', () => {
     it('shows alert with failed usernames when some users fail (remove action)', async () => {
       const mockSuccessData = {
         results: [
-          { identifier: 'failed-user', userDoesNotExist: true },
-          { identifier: 'jane.doe', userDoesNotExist: false },
+          { identifier: 'failed-user', userDoesNotExist: true, isActive: true },
+          { identifier: 'jane.doe', userDoesNotExist: false, isActive: true },
         ]
       };
 
@@ -206,7 +206,7 @@ describe('UpdateBetaTesterModal', () => {
     it('shows alert with failed usernames when some users fail (add action)', async () => {
       const mockSuccessData = {
         results: [
-          { identifier: 'failed-user', userDoesNotExist: true },
+          { identifier: 'failed-user', userDoesNotExist: true, isActive: null },
         ]
       };
 
@@ -231,10 +231,39 @@ describe('UpdateBetaTesterModal', () => {
       });
     });
 
+    it('shows alert with inactive usernames when some users are inactive', async () => {
+      const mockSuccessData = {
+        results: [
+          { identifier: 'inactive-user', userDoesNotExist: false, isActive: false },
+          { identifier: 'jane.doe', userDoesNotExist: false, isActive: true },
+        ]
+      };
+
+      const mutateWithSuccess = (_params: UpdateBetaTestersParams, callbacks: any) => {
+        callbacks.onSuccess(mockSuccessData);
+      };
+      mutateMock.mockImplementation(mutateWithSuccess);
+
+      const nonBetaTesterProps = {
+        ...defaultProps,
+        learner: learnerNonBetaTester,
+      };
+
+      renderWithAlertAndIntl(
+        <UpdateBetaTesterModal {...nonBetaTesterProps} />
+      );
+
+      expect(mockAddAlert).toHaveBeenCalledWith({
+        type: 'warning',
+        message: messages.inactiveUsers.defaultMessage,
+        extraContent: expect.any(Array),
+      });
+    });
+
     it('does not show alert when all users succeed', async () => {
       const mockSuccessData = {
         results: [
-          { identifier: 'jane.doe', userDoesNotExist: false },
+          { identifier: 'jane.doe', userDoesNotExist: false, isActive: true },
         ]
       };
 

src/grading/components/GradingLearnerContent.tsx

@@ -165,6 +165,9 @@ const GradingLearnerContent = ({ toolType, onShowTasks }: GradingLearnerContentP
 
   const handleLearnerChange = (usernameOrEmail: string): void => {
     setUsernameOrEmail(usernameOrEmail);
+    // Reset problem field when learner changes due to progress and attempts change for every learner
+    setBlockId('');
+    problemFieldRef.current?.reset();
   };
 
   return (

src/specialExams/data/api.test.ts

@@ -176,6 +176,38 @@ describe('specialExams api', () => {
         'https://test-lms.com/api/instructor/v2/courses/course-v1:edX+Special%20Course+2023/special_exams/attempts?page=1&page_size=20&search=user%40example.com'
       );
     });
+
+    it('handles ordering parameter correctly', async () => {
+      const courseId = 'course-v1:edX+Test+2023';
+      const paramsWithOrdering: AttemptsParams = {
+        page: 0,
+        pageSize: 20,
+        emailOrUsername: '',
+        ordering: '-examType'
+      };
+
+      await getAttempts(courseId, paramsWithOrdering);
+
+      expect(mockHttpClient.get).toHaveBeenCalledWith(
+        'https://test-lms.com/api/instructor/v2/courses/course-v1:edX+Test+2023/special_exams/attempts?page=1&page_size=20&ordering=-exam_type'
+      );
+    });
+
+    it('handles complex ordering parameter with multiple parts', async () => {
+      const courseId = 'course-v1:edX+Test+2023';
+      const paramsWithComplexOrdering: AttemptsParams = {
+        page: 1,
+        pageSize: 10,
+        emailOrUsername: 'test@example.com',
+        ordering: 'proctoringExam.examName'
+      };
+
+      await getAttempts(courseId, paramsWithComplexOrdering);
+
+      expect(mockHttpClient.get).toHaveBeenCalledWith(
+        'https://test-lms.com/api/instructor/v2/courses/course-v1:edX+Test+2023/special_exams/attempts?page=2&page_size=10&search=test%40example.com&ordering=proctoring_exam.exam_name'
+      );
+    });
   });
 
   describe('getAllowances', () => {

src/specialExams/data/api.ts

@@ -15,8 +15,11 @@ const getQueryParams = (params: AttemptsParams) => {
   }
 
   if (params.ordering) {
+    const hasNegativePrefix = params.ordering.startsWith('-');
+
     const orderingParam = params.ordering.split('.').map((part) => snakeCase(part)).join('.');
-    queryParams.append('ordering', orderingParam);
+
+    queryParams.append('ordering', `${hasNegativePrefix ? '-' : ''}${orderingParam}`);
   }
 
   return queryParams;