use EmptyCell for zero routes, copy tweak

oxidecomputer · Dec 13, 2024 · f761405 · f761405
1 parent 64a7bbe
commit f761405
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/app/pages/project/vpcs/VpcPage/tabs/VpcGatewaysTab.tsx b/app/pages/project/vpcs/VpcPage/tabs/VpcGatewaysTab.tsx
@@ -51,7 +51,8 @@ const GatewayIpPoolCell = (gatewaySelector: PP.VpcInternetGateway) => {
 const GatewayRoutes = ({ project, vpc, gateway }: PP.VpcInternetGateway) => {
   const matchingRoutes = useGatewayRoutes({ project, vpc, gateway })
   const to = pb.vpcInternetGateway({ project, vpc, gateway })
-  return <LinkCell to={to}>{matchingRoutes?.length || 0}</LinkCell>
+  if (!matchingRoutes?.length) return <EmptyCell />
+  return <LinkCell to={to}>{matchingRoutes.length}</LinkCell>
 }
 
 const colHelper = createColumnHelper<InternetGateway>()

diff --git a/app/pages/project/vpcs/internet-gateway-edit.tsx b/app/pages/project/vpcs/internet-gateway-edit.tsx
@@ -164,10 +164,10 @@ export function EditInternetGatewayForm() {
           ))
         ) : (
           <div className="mt-2">
-            {'This internet gateway does not have any specific IP addresses attached. '}
+            {'This internet gateway does not have any IP addresses attached. '}
             {hasAttachedPool
               ? 'It will use an address from the attached IP pool.'
-              : 'Use the CLI to attach an IP Pool or specify an IP address to use with this gateway.'}
+              : 'Use the CLI to attach an IP pool or IP address to this gateway.'}
           </div>
         )}
       </div>

diff --git a/test/e2e/vpcs.e2e.ts b/test/e2e/vpcs.e2e.ts
@@ -335,12 +335,14 @@ test('can view internet gateways', async ({ page }) => {
     description: 'internet gateway 1',
     'Attached IP Address': '123.4.56.3',
     'Attached IP Pool': 'ip-pool-1',
+    Routes: '1',
   })
   await expectRowVisible(table, {
     name: 'internet-gateway-2',
     description: 'internet gateway 2',
     'Attached IP Address': '—',
     'Attached IP Pool': 'ip-pool-2',
+    Routes: '—',
   })
 
   await page.getByRole('link', { name: 'internet-gateway-1' }).click()