Don't show TPI stats if they're all zero (#3869)

bcgov · Aug 21, 2024 · f5d63d9 · f5d63d9
1 parent 4be25e9
commit f5d63d9
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 1 deletion.
diff --git a/web/src/features/fba/components/infoPanel/FireZoneUnitSummary.tsx b/web/src/features/fba/components/infoPanel/FireZoneUnitSummary.tsx
@@ -37,7 +37,7 @@ const FireZoneUnitSummary = ({
             <FuelSummary selectedFireZoneUnit={selectedFireZoneUnit} fuelTypeInfo={fuelTypeInfo} />
           </Grid>
           <Grid item sx={{ width: '95%' }}>
-            { isNull(fireZoneTPIStats) ? (
+            { isNull(fireZoneTPIStats) || fireZoneTPIStats.valley_bottom + fireZoneTPIStats.mid_slope + fireZoneTPIStats.upper_slope === 0 ? (
               <Typography>
                 No elevation information available.
               </Typography>

diff --git a/web/src/features/fba/components/infoPanel/fireZoneUnitSummary.test.tsx b/web/src/features/fba/components/infoPanel/fireZoneUnitSummary.test.tsx
@@ -85,4 +85,27 @@ describe('FireZoneUnitSummary', () => {
     const fireZoneUnitInfo = getByTestId('elevation-status')
     expect(fireZoneUnitInfo).toBeInTheDocument()
   })
+
+  it('should not render TPI stats all zero', () => {
+    const fireShape: FireShape = {
+      fire_shape_id: 1,
+      mof_fire_zone_name: 'foo',
+      mof_fire_centre_name: 'fizz',
+      area_sqm: 10
+    }
+    const { queryByTestId } = render(
+      <FireZoneUnitSummary
+        fuelTypeInfo={[]}
+        fireZoneTPIStats={{
+          fire_zone_id: 0, 
+          valley_bottom: 0,
+          mid_slope: 0,
+          upper_slope: 0
+        }}
+        selectedFireZoneUnit={fireShape}
+      />
+    )
+    const fireZoneUnitInfo = queryByTestId('elevation-status')
+    expect(fireZoneUnitInfo).not.toBeInTheDocument()
+  })
 })