From a701d4ceeccaf2fe305312d49a487cc621528819 Mon Sep 17 00:00:00 2001
From: Paul Adamsen <117673433+pauladamseniii@users.noreply.github.com>
Date: Tue, 23 May 2023 09:30:08 -0400
Subject: [PATCH] CEC-4341 - Fix Can signal export select all (#340)

* CEC-4341 - Fix Can signal export select all

* fix code smell
---
 src/components/CANSelfServe/SelfServe/index.jsx     | 10 ++++++----
 src/components/Contexts/CANSignalsExportContext.jsx | 10 ++++++++--
 src/services/CANSignalAPI.js                        |  7 +++----
 src/services/__mocks__/CANSignalAPI.js              |  8 ++++----
 4 files changed, 21 insertions(+), 14 deletions(-)

diff --git a/src/components/CANSelfServe/SelfServe/index.jsx b/src/components/CANSelfServe/SelfServe/index.jsx
index 3de5740..d67a531 100644
--- a/src/components/CANSelfServe/SelfServe/index.jsx
+++ b/src/components/CANSelfServe/SelfServe/index.jsx
@@ -16,7 +16,7 @@ const MainForm = ({ id }) => {
   const [selectedStartDate, setSelectedStartDate] = useState(new Date(Date.now() - 24 * 60 * 60 * 1000));
   const [selectedEndDate, setSelectedEndDate] = useState(new Date());
   const [selectedCanSignals, setSelectedCanSignals] = useState([]);
-
+  const [selectAllCanSignals, setSelectAllCanSignals] = useState(false);
 
   const {
     token: {
@@ -29,8 +29,8 @@ const MainForm = ({ id }) => {
     let timestamp_start = Date.parse(selectedStartDate.toUTCString()) / 1000
     let timestamp_end = Date.parse(selectedEndDate.toUTCString()) / 1000
     try {
-      await getDynamicColumnCANSignals(id, timestamp_start, timestamp_end, selectedCanSignals, token)
-    } catch(e){
+      await getDynamicColumnCANSignals(id, timestamp_start, timestamp_end, selectedCanSignals, selectAllCanSignals, token)
+    } catch (e) {
       setMessage(e.message);
       logger.error(e.stack)
     }
@@ -74,12 +74,14 @@ const MainForm = ({ id }) => {
   const handleSelectedItemsChange = (event) => {
     const { value } = event.target;
     if (value.some(item => item === "Select All")) {
+      setSelectAllCanSignals(true);
       if (selectedCanSignals.length === canSignals.length) {
         setSelectedCanSignals([]);
       } else {
         setSelectedCanSignals(canSignals.map(signal => signal.signal_name));
       }
     } else {
+      setSelectAllCanSignals(false);
       setSelectedCanSignals(value);
     }
   };
@@ -156,7 +158,7 @@ const MainForm = ({ id }) => {
         </Grid>
         <Grid item xs={12}>
           <FormControl fullWidth required>
-          <InputLabel id="select-can-signals-label">Select CAN signals</InputLabel>
+            <InputLabel id="select-can-signals-label">Select CAN signals</InputLabel>
             <Select
               labelId="select-can-signals-label"
               id="select-can-signals"
diff --git a/src/components/Contexts/CANSignalsExportContext.jsx b/src/components/Contexts/CANSignalsExportContext.jsx
index 175c851..b09f9c0 100644
--- a/src/components/Contexts/CANSignalsExportContext.jsx
+++ b/src/components/Contexts/CANSignalsExportContext.jsx
@@ -23,14 +23,20 @@ export const CANSignalsExportProvider = ({ children }) => {
     }
   };
 
-  const getDynamicColumnCANSignals = async (vin, timestart, timeend, cansingals, token) => {
+  const getDynamicColumnCANSignals = async (vin, timestart, timeend, cansignals, selectall, token) => {
     try {
       setBusy(true)
       if (!vin) return;
 
       validateVIN(vin);
       if (timestart > timeend) throw new Error("Start time cannot be after end time");
-      const result = await api.getCanSignalsVin(vin, timestart, timeend, cansingals, token);
+
+      let result
+      if (selectall === true) {
+        result = await api.getCanSignalsVin(vin, timestart, timeend, [], selectall, token);
+      } else {
+        result = await api.getCanSignalsVin(vin, timestart, timeend, cansignals, false, token);
+      }
       if (result.error || !result.ok)
         throw new Error(`Get CAN signals error. ${result.message}`);
 
diff --git a/src/services/CANSignalAPI.js b/src/services/CANSignalAPI.js
index 6035dab..757418e 100644
--- a/src/services/CANSignalAPI.js
+++ b/src/services/CANSignalAPI.js
@@ -6,8 +6,8 @@ const API_ENDPOINT = process.env.REACT_APP_OTA_SERVICE_URL;
 
 const canSignalAPI = {
 
-  getCanSignalsVin: async (vin, timestamp_start, timestamp_end, can_signals, token) =>
-    fetch(addQueryParams(`${API_ENDPOINT}/can_signals_export`, { vin, timestamp_start, timestamp_end, can_signals }), {
+  getCanSignalsVin: async (vin, timestamp_start, timestamp_end, can_signals, select_all, token) =>
+    fetch(addQueryParams(`${API_ENDPOINT}/can_signals_export`, { vin, timestamp_start, timestamp_end, can_signals, select_all }), {
       method: "GET",
       headers: Object.assign(
         { "Content-Type": "application/json" },
@@ -15,8 +15,7 @@ const canSignalAPI = {
       ),
       responseType: "blob"
     })
-    .catch(errorHandler),
-
+      .catch(errorHandler),
 
   getCanSignalList: async (token) =>
     fetch(addQueryParams(`${API_ENDPOINT}/can_signals_list`), {
diff --git a/src/services/__mocks__/CANSignalAPI.js b/src/services/__mocks__/CANSignalAPI.js
index e088c01..9391f4e 100644
--- a/src/services/__mocks__/CANSignalAPI.js
+++ b/src/services/__mocks__/CANSignalAPI.js
@@ -1,11 +1,11 @@
 const canSignalList = [
-  { signal_name: "123"},
-  { signal_name: "456"},
-  { signal_name: "789"}
+  { signal_name: "123" },
+  { signal_name: "456" },
+  { signal_name: "789" }
 ];
 
 const canSignalAPI = {
-  getCanSignalsVin: async (vin, timestamp_start, timestamp_end, can_signals, token) => {
+  getCanSignalsVin: async (vin, timestamp_start, timestamp_end, can_signals, select_all, token) => {
     return { data: "fake data" };
   },
   getCanSignalList: async (token) => {