[grid] Fixing warnings for UI tests

diemol · diemol · commit 411d877be364 · 2021-11-23T22:43:09.000+01:00
diff --git a/javascript/grid-ui/src/components/Node/Node.tsx b/javascript/grid-ui/src/components/Node/Node.tsx
@@ -70,7 +70,7 @@ class Node extends React.Component<NodeProps, {}> {
         <CardContent className={classes.paddingContent}>
           <Grid
             container
-            justify='space-between'
+            justifyContent="space-between"
             spacing={1}
           >
             <Grid item xs={10}>
diff --git a/javascript/grid-ui/src/components/Node/NodeLoad.tsx b/javascript/grid-ui/src/components/Node/NodeLoad.tsx
@@ -15,11 +15,7 @@
 // specific language governing permissions and limitations
 // under the License.
 
-import {
-  Box,
-  Grid,
-  Typography
-} from '@material-ui/core'
+import { Box, Grid, Typography } from '@material-ui/core'
 import React, { ReactNode } from 'react'
 import NodeInfo from '../../models/node-info'
 import LinearProgress, { LinearProgressProps } from '@material-ui/core/LinearProgress'
@@ -51,7 +47,7 @@ class NodeLoad extends React.Component<{ node: NodeInfo }, {}> {
       <Grid item xs={12}>
         <Grid
           container
-          justify='space-between'
+          justifyContent="space-between"
           spacing={2}
         >
           <Grid item xs={3}>
diff --git a/javascript/grid-ui/src/components/Node/Stereotypes.tsx b/javascript/grid-ui/src/components/Node/Stereotypes.tsx
@@ -57,7 +57,8 @@ class Stereotypes extends React.Component<StereotypesProps, {}> {
       return (
         <Grid item key={index}>
           <Grid container alignItems="center" spacing={1}>
-            <Tooltip title={JSON.stringify(slotStereotype.rawData.stereotype)}>
+            <Tooltip
+              title={JSON.stringify(slotStereotype.rawData.stereotype) ?? ''}>
               <Badge badgeContent={slotStereotype.slotCount} color={'primary'}
                      className={classes.boxStyle}>
                 <Grid item className={classes.browserVersion}>
diff --git a/javascript/grid-ui/src/components/RunningSessions/RunningSessions.tsx b/javascript/grid-ui/src/components/RunningSessions/RunningSessions.tsx
@@ -182,8 +182,8 @@ function EnhancedTableHead (props: EnhancedTableProps): JSX.Element {
         {headCells.map((headCell) => (
           <TableCell
             key={headCell.id}
-            align='left'
-            padding='default'
+            align="left"
+            padding="normal"
             sortDirection={orderBy === headCell.id ? order : false}
           >
             <TableSortLabel
diff --git a/javascript/grid-ui/src/tests/components/NavBar.test.tsx b/javascript/grid-ui/src/tests/components/NavBar.test.tsx
@@ -25,7 +25,11 @@ it('renders menu options names', () => {
   const history = createMemoryHistory()
   render(
     <Router history={history}>
-      <NavBar />
+      <NavBar open={true}
+              maxSession={10}
+              sessionCount={0}
+              nodeCount={1}
+              sessionQueueSize={0}/>
     </Router>
   )
   expect(screen.getByText('Sessions')).toBeInTheDocument()
@@ -39,7 +43,11 @@ it('overall concurrency is not rendered on root path with a single node',
     history.push('/')
     render(
       <Router history={history}>
-        <NavBar open maxSession={0} sessionCount={0} nodeCount={1} />
+        <NavBar open={true}
+                maxSession={0}
+                sessionCount={0}
+                nodeCount={1}
+                sessionQueueSize={0}/>
       </Router>
     )
     expect(screen.queryByTestId('overall-concurrency')).not.toBeInTheDocument()
@@ -51,7 +59,11 @@ it('overall concurrency is rendered on root path with more than one node',
     history.push('/')
     render(
       <Router history={history}>
-        <NavBar open maxSession={0} sessionCount={0} nodeCount={2} />
+        <NavBar open={true}
+                maxSession={0}
+                sessionCount={0}
+                nodeCount={2}
+                sessionQueueSize={0}/>
       </Router>
     )
     expect(screen.getByTestId('overall-concurrency')).toBeInTheDocument()
@@ -63,7 +75,11 @@ it('overall concurrency is rendered on root path with more than one node',
     history.push('/')
     render(
       <Router history={history}>
-        <NavBar open maxSession={0} sessionCount={0} nodeCount={2} />
+        <NavBar open={true}
+                maxSession={0}
+                sessionCount={0}
+                nodeCount={2}
+                sessionQueueSize={0}/>
       </Router>
     )
     expect(screen.getByTestId('overall-concurrency')).toBeInTheDocument()
@@ -75,7 +91,11 @@ it('overall concurrency is rendered on a path different than and one node',
     history.push('/sessions')
     render(
       <Router history={history}>
-        <NavBar open maxSession={0} sessionCount={0} nodeCount={1} />
+        <NavBar open={true}
+                maxSession={0}
+                sessionCount={0}
+                nodeCount={1}
+                sessionQueueSize={0}/>
       </Router>
     )
     expect(screen.getByTestId('overall-concurrency')).toBeInTheDocument()
diff --git a/javascript/grid-ui/src/tests/components/Node.test.tsx b/javascript/grid-ui/src/tests/components/Node.test.tsx
@@ -33,7 +33,8 @@ const slotStereotype: StereotypeInfo = {
   browserName: 'chrome',
   browserVersion: 'v. 88',
   slotCount: 12,
-  rawData: []
+  rawData: ['stereotype: {"browserName": "chrome"}'],
+  platformName: 'macos'
 }
 
 const node: NodeInfo = {

Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ class Node extends React.Component<NodeProps, {}> {`
`70`	`70`	`<CardContent className={classes.paddingContent}>`
`71`	`71`	`<Grid`
`72`	`72`	`container`
`73`		`- justify='space-between'`
	`73`	`+ justifyContent="space-between"`
`74`	`74`	`spacing={1}`
`75`	`75`	`>`
`76`	`76`	`<Grid item xs={10}>`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,8 @@ const slotStereotype: StereotypeInfo = {`
`33`	`33`	`browserName: 'chrome',`
`34`	`34`	`browserVersion: 'v. 88',`
`35`	`35`	`slotCount: 12,`
`36`		`- rawData: []`
	`36`	`+ rawData: ['stereotype: {"browserName": "chrome"}'],`
	`37`	`+ platformName: 'macos'`
`37`	`38`	`}`
`38`	`39`
`39`	`40`	`const node: NodeInfo = {`