CEC-4883: update api shape and display trex version on fleets (#418)

This commit is contained in:
Tristan Timblin
2023-08-17 14:08:57 -07:00
committed by GitHub
parent 0c05663443
commit f4d021b658
7 changed files with 96 additions and 14 deletions

View File

@@ -113,18 +113,20 @@ export const FleetProvider = ({ children }) => {
throw new Error(`Get fleet vehicles error. ${result.message}`);
}
const connectionsResult = await vehiclesAPI.getConnections({ "VINs": result.data }, token)
const vins = result.data.map(vehicle => vehicle.vin);
const connectionsResult = await vehiclesAPI.getConnections({ "VINs": vins }, token)
if (result.error) {
setFleetVehicles([])
throw new Error(`Get vehicles connections error. ${result.message}`);
}
var cars = []
result.data.forEach((vin) => {
result.data.forEach((vehicle) => {
cars.push({
vin: vin,
connected: connectionsResult[vin] || false,
connectedHMI: connectionsResult[`2:${vin}`] || false
vin: vehicle.vin,
connected: connectionsResult[vehicle.vin] || false,
connectedHMI: connectionsResult[`2:${vehicle.vin}`] || false,
trex_version: vehicle.carstate?.trex_version || "",
})
})

View File

@@ -347,8 +347,7 @@ describe("FleetContext", () => {
describe("getFleetVehicles", () => {
beforeEach(() => {
const TestComp = () => {
const { busy, error, fleetVehicles, getFleetVehicles } =
useFleetContext();
const { busy, error, fleetVehicles, getFleetVehicles } = useFleetContext();
return (
<>
@@ -805,16 +804,19 @@ const expectedFleetVehiclesData = [
vin: "USWESTVIN12345678",
connected: true,
connectedHMI: false,
trex_version: "",
},
{
vin: "USWESTVIN12345679",
connected: true,
connectedHMI: false,
trex_version: "",
},
{
vin: "USWESTVIN12345670",
connected: true,
connectedHMI: false,
trex_version: "",
},
];

View File

@@ -67,17 +67,20 @@ export const useFleetContext = () => ({
{
vin: "USWESTVIN12345678",
connected: false,
connectedHMI: false
connectedHMI: false,
trex_version: "",
},
{
vin: "USWESTVIN12345679",
connected: true,
connectedHMI: true
connectedHMI: true,
trex_version: "",
},
{
vin: "USWESTVIN12345670",
connected: false,
connectedHMI: false
connectedHMI: false,
trex_version: "",
},
];
return Promise.resolve(result);

View File

@@ -127,6 +127,29 @@ exports[`FleetVehiclesTable Render 1`] = `
</svg>
</span>
</th>
<th
class="MuiTableCell-root MuiTableCell-head MuiTableCell-alignCenter"
scope="col"
>
<span
aria-disabled="false"
class="MuiButtonBase-root MuiTableSortLabel-root"
role="button"
tabindex="0"
>
T.REX Version
<svg
aria-hidden="true"
class="MuiSvgIcon-root MuiTableSortLabel-icon MuiTableSortLabel-iconDirectionAsc"
focusable="false"
viewBox="0 0 24 24"
>
<path
d="M20 12l-1.41-1.41L13 16.17V4h-2v12.17l-5.58-5.59L4 12l8 8 8-8z"
/>
</svg>
</span>
</th>
<th
class="MuiTableCell-root MuiTableCell-head MuiTableCell-alignCenter"
scope="col"

View File

@@ -34,6 +34,10 @@ const tableColumns = [
id: "vin",
label: "VIN",
},
{
id: "trex_version",
label: "T.REX Version",
},
{
id: "",
label: "Actions",
@@ -204,7 +208,8 @@ const MainForm = ({ name }) => {
}
<Link key={"link" + car.vin} to={`/vehicle-status/${car.vin}`}>{car.vin}</Link>
</TableCell>
<TableCell key={"cell2" + car.vin} align="center">{Actions(car.vin)}</TableCell>
<TableCell key={"cell2" + car.vin} align="center">{car.trex_version}</TableCell>
<TableCell key={"cell3" + car.vin} align="center">{Actions(car.vin)}</TableCell>
</TableRow>
)
))}

View File

@@ -126,6 +126,29 @@ exports[`VehiclesTab Render 1`] = `
</svg>
</span>
</th>
<th
class="MuiTableCell-root MuiTableCell-head MuiTableCell-alignCenter"
scope="col"
>
<span
aria-disabled="false"
class="MuiButtonBase-root MuiTableSortLabel-root"
role="button"
tabindex="0"
>
T.REX Version
<svg
aria-hidden="true"
class="MuiSvgIcon-root MuiTableSortLabel-icon MuiTableSortLabel-iconDirectionAsc"
focusable="false"
viewBox="0 0 24 24"
>
<path
d="M20 12l-1.41-1.41L13 16.17V4h-2v12.17l-5.58-5.59L4 12l8 8 8-8z"
/>
</svg>
</span>
</th>
<th
class="MuiTableCell-root MuiTableCell-head MuiTableCell-alignCenter"
scope="col"