fix(ui/tasks): updated reducer test to more accurately reflect the cu…

…rrent reducer functionality and updated the parameter name for consistency

ui/src/tasks/actions/index.ts

@@ -352,11 +352,11 @@ export const selectTaskByID = (id: string) => async (
   }
 }
 
-export const setAllTaskOptionsByID = (id: string) => async (
+export const setAllTaskOptionsByID = (taskID: string) => async (
   dispatch
 ): Promise<void> => {
   try {
-    const task = await client.tasks.get(id)
+    const task = await client.tasks.get(taskID)
     dispatch(setAllTaskOptions(task))
   } catch (e) {
     console.error(e)

ui/src/tasks/reducers/tasks.test.ts

@@ -6,11 +6,12 @@ import {setTaskOption} from 'src/tasks/actions'
 import {TaskSchedule} from 'src/utils/taskOptionsToFluxScript'
 
 // skipping this test since the cron and interval values should not be reset when toggling between schedule tasks
-describe.skip('tasksReducer', () => {
+describe('tasksReducer', () => {
   describe('setTaskOption', () => {
-    it('clears the cron property from the task options when interval is selected', () => {
+    it('should not clear the cron property from the task options when interval is selected', () => {
       const initialState = defaultState
-      initialState.taskOptions = {...defaultTaskOptions, cron: '0 2 * * *'}
+      const cron = '0 2 * * *'
+      initialState.taskOptions = {...defaultTaskOptions, cron }
 
       const actual = tasksReducer(
         initialState,
@@ -22,16 +23,17 @@ describe.skip('tasksReducer', () => {
         taskOptions: {
           ...defaultTaskOptions,
           taskScheduleType: TaskSchedule.interval,
-          cron: '',
+          cron,
         },
       }
 
       expect(actual).toEqual(expected)
     })
 
-    it('clears the interval property from the task options when cron is selected', () => {
+    it('should not clear the interval property from the task options when cron is selected', () => {
       const initialState = defaultState
-      initialState.taskOptions = {...defaultTaskOptions, interval: '24h'} // todo(docmerlin): allow for time units larger than 1d, right now h is the longest unit our s
+      const interval = '24h'
+      initialState.taskOptions = {...defaultTaskOptions, interval } // todo(docmerlin): allow for time units larger than 1d, right now h is the longest unit our s
 
       const actual = tasksReducer(
         initialState,
@@ -43,7 +45,7 @@ describe.skip('tasksReducer', () => {
         taskOptions: {
           ...defaultTaskOptions,
           taskScheduleType: TaskSchedule.cron,
-          interval: '',
+          interval,
         },
       }