From 3302aae1fcd53efff62c70e7fd170ae566bf209a Mon Sep 17 00:00:00 2001
From: alipov_d <dmytro.alipov@raccoongang.com>
Date: Tue, 14 Nov 2023 16:35:19 +0200
Subject: [PATCH] fix: request error for empty user list responses

---
 src/containers/ListView/TableAction.jsx         |  1 +
 .../__snapshots__/TableAction.test.jsx.snap     |  1 +
 src/data/redux/thunkActions/grading.js          |  8 +++++---
 src/data/redux/thunkActions/grading.test.js     | 17 +++++++++++------
 4 files changed, 18 insertions(+), 9 deletions(-)

diff --git a/src/containers/ListView/TableAction.jsx b/src/containers/ListView/TableAction.jsx
index 44886d3ff..40a800e1d 100644
--- a/src/containers/ListView/TableAction.jsx
+++ b/src/containers/ListView/TableAction.jsx
@@ -11,6 +11,7 @@ export const TableAction = ({ tableInstance, handleClick }) => (
     onClick={handleClick(tableInstance.rows)}
     variant="primary"
     className="view-all-responses-btn"
+    disabled={tableInstance.rows.length === 0}
   >
     <FormattedMessage {...messages.viewAllResponses} />
   </Button>
diff --git a/src/containers/ListView/__snapshots__/TableAction.test.jsx.snap b/src/containers/ListView/__snapshots__/TableAction.test.jsx.snap
index 5b3144673..0d3eb462b 100644
--- a/src/containers/ListView/__snapshots__/TableAction.test.jsx.snap
+++ b/src/containers/ListView/__snapshots__/TableAction.test.jsx.snap
@@ -3,6 +3,7 @@
 exports[`TableAction component snapshots 1`] = `
 <Button
   className="view-all-responses-btn"
+  disabled={false}
   onClick={[MockFunction]}
   variant="primary"
 >
diff --git a/src/data/redux/thunkActions/grading.js b/src/data/redux/thunkActions/grading.js
index e0b7e7be2..d26dae918 100644
--- a/src/data/redux/thunkActions/grading.js
+++ b/src/data/redux/thunkActions/grading.js
@@ -35,9 +35,11 @@ export const loadPrev = () => (dispatch) => {
  * @param {string[]} submissionUUIDs - ordered list of submissionUUIDs for selected submissions
  */
 export const loadSelectionForReview = (submissionUUIDs) => (dispatch) => {
-  dispatch(actions.grading.updateSelection(submissionUUIDs));
-  dispatch(actions.app.setShowReview(true));
-  dispatch(module.loadSubmission());
+  if (submissionUUIDs && submissionUUIDs.length > 0) {
+    dispatch(actions.grading.updateSelection(submissionUUIDs));
+    dispatch(actions.app.setShowReview(true));
+    dispatch(module.loadSubmission());
+  }
 };
 
 export const loadSubmission = () => (dispatch, getState) => {
diff --git a/src/data/redux/thunkActions/grading.test.js b/src/data/redux/thunkActions/grading.test.js
index 493495149..efd024a2b 100644
--- a/src/data/redux/thunkActions/grading.test.js
+++ b/src/data/redux/thunkActions/grading.test.js
@@ -107,13 +107,13 @@ describe('grading thunkActions', () => {
       });
     });
     describe('loadSelectionForReview', () => {
-      const submissionUUIDs = [
-        'submission-id-0',
-        'submission-id-1',
-        'submission-id-2',
-        'submission-id-3',
-      ];
       test('dispatches actions.grading.updateSelection, actions.app.setShowReview(true), and then loadSubmission', () => {
+        const submissionUUIDs = [
+          'submission-id-0',
+          'submission-id-1',
+          'submission-id-2',
+          'submission-id-3',
+        ];
         thunkActions.loadSelectionForReview(submissionUUIDs)(dispatch, getState);
         expect(dispatch.mock.calls).toEqual([
           [actions.grading.updateSelection(submissionUUIDs)],
@@ -121,6 +121,11 @@ describe('grading thunkActions', () => {
           [thunkActions.loadSubmission()],
         ]);
       });
+      test('with empty submissionUUIDs does not dispatch any action', () => {
+        const submissionUUIDs = [];
+        thunkActions.loadSelectionForReview(submissionUUIDs)(dispatch, getState);
+        expect(dispatch).not.toHaveBeenCalled();
+      });
     });
   });