Fix some unit tests

Author: hgreebe

frontend/src/old-pages/Clusters/Costs/__tests__/EnableCostMonitoringButton.test.tsx

@@ -59,7 +59,7 @@ describe('given a component to activate cost monitoring for the account', () =>
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {full: '3.2.0'},
+          wizard: {version: '3.2.0'},
         },
         aws: {
           region: 'us-west-1',
@@ -92,7 +92,7 @@ describe('given a component to activate cost monitoring for the account', () =>
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {full: '3.2.0'},
+          wizard: {version: '3.2.0'},
         },
         aws: {
           region: 'us-west-1',
@@ -121,7 +121,7 @@ describe('given a component to activate cost monitoring for the account', () =>
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {full: '3.1.5'},
+          wizard: {version: '3.1.5'},
         },
         aws: {
           region: 'us-west-1',

frontend/src/old-pages/Configure/Queues/Queues.test.tsx

@@ -241,10 +241,8 @@ describe('Given a list of queues', () => {
             subnets: [],
           },
           app: {
-            version: {
-              full: version,
-            },
             wizard: {
+              version: version,
               config: {
                 Scheduling: {
                   SlurmQueues: [
@@ -380,10 +378,8 @@ describe('Given a queue', () => {
       subnets: [],
     },
     app: {
-      version: {
-        full: '3.4.0',
-      },
       wizard: {
+        version: '3.4.0',
         config: {
           HeadNode: {
             Networking: {
@@ -474,10 +470,8 @@ describe('Given a queue', () => {
           subnets: [],
         },
         app: {
-          version: {
-            full: '3.4.0',
-          },
           wizard: {
+            version: '3.6.0',
             config: {
               Scheduling: {
                 SlurmQueues: [

frontend/src/old-pages/Configure/Version/ClusterVersionField.tsx

@@ -43,7 +43,7 @@ export function ClusterVersionField({ hideLabel = false }: ClusterVersionFieldPr
   const editing = !!useState(editingPath)
   const versions = useState(['app', 'version', 'full'])
 
-  const options = versions.map((version: any) => ({
+  const options = (versions || []).map((version: string) => ({
     label: version,
     value: version
   }))

frontend/src/old-pages/Configure/__tests__/Configure.test.tsx

@@ -30,6 +30,7 @@ describe('Given a configure component (wizard)', () => {
             source: {
               loading: true,
             },
+            version: '3.12.0'
           },
         },
       })
@@ -57,6 +58,7 @@ describe('Given a configure component (wizard)', () => {
             source: {
               loading: false,
             },
+            version: '3.12.0'
           },
         },
       })

frontend/src/old-pages/Configure/__tests__/dynamicFS.test.tsx

@@ -12,10 +12,11 @@ describe('Given a PCluster version', () => {
       mockStore.getState.mockReturnValue({
         app: {
           version: {
-            full: '3.3.0',
+            full: ['3.3.0'],
           },
           wizard: {
             editing: true,
+            version: '3.3.0',
           },
         },
       })
@@ -33,11 +34,9 @@ describe('Given a PCluster version', () => {
       it('should let the user add or remove filesystems', () => {
         mockStore.getState.mockReturnValue({
           app: {
-            version: {
-              full: '3.2.0',
-            },
             wizard: {
               editing: false,
+              version: '3.2.0',
             },
           },
         })
@@ -55,11 +54,9 @@ describe('Given a PCluster version', () => {
       it('should block the user from creating or removing filesystems', () => {
         mockStore.getState.mockReturnValue({
           app: {
-            version: {
-              full: '3.2.0',
-            },
             wizard: {
               editing: true,
+              version: '3.2.0',
             },
           },
         })

frontend/src/old-pages/Configure/__tests__/storage.test.tsx

@@ -198,8 +198,8 @@ describe('Given a Lustre storage component', () => {
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {
-            full: '3.3.0',
+          wizard: {
+            version: '3.3.0',
           },
         },
       })
@@ -238,8 +238,8 @@ describe('Given a Lustre storage component', () => {
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {
-            full: '3.2.0',
+          wizard: {
+            version: '3.2.0',
           },
         },
       })
@@ -284,8 +284,8 @@ describe('given a component to display an Efs storage instance', () => {
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {
-            full: '3.3.0',
+          wizard: {
+            version: '3.3.0',
           },
         },
       })
@@ -324,8 +324,8 @@ describe('given a component to display an Efs storage instance', () => {
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {
-            full: '3.2.0',
+          wizard: {
+            version: '3.2.0',
           },
         },
       })
@@ -370,8 +370,8 @@ describe('given a component to display an Ebs storage instance', () => {
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {
-            full: '3.2.0',
+          wizard: {
+            version: '3.2.0',
           },
         },
       })
@@ -410,8 +410,8 @@ describe('given a component to display an Ebs storage instance', () => {
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {
-            full: '3.1.0',
+          wizard: {
+            version: '3.1.0',
           },
         },
       })
@@ -448,10 +448,8 @@ describe('given a component to display an Ebs storage instance', () => {
     beforeEach(() => {
       mockStore.getState.mockReturnValue({
         app: {
-          version: {
-            full: '3.2.0',
-          },
           wizard: {
+            version: '3.2.0',
             config: {
               SharedStorage: [
                 {