of course I didn't push to main without running the tests, why do you ask

Author: david-crespo

test/e2e/networking.e2e.ts

@@ -57,14 +57,12 @@ test('Create and edit subnet', async ({ page }) => {
   // VPC detail, subnets tab
   await expectVisible(page, [
     'role=heading[name*="mock-vpc"]',
-    'role=tab[name="Subnets"]',
-    // 'role=tab[name="System Routes"]',
-    // 'role=tab[name="Routers"]',
     'role=tab[name="Firewall Rules"]',
-    'role=cell[name="mock-subnet"]',
-    // TODO: assert minitable contents
+    'role=cell[name="allow-icmp"]',
   ])
 
+  await page.getByRole('tab', { name: 'Subnets' }).click()
+
   // Create subnet
   await page.click('role=button[name="New subnet"]')
   await expectVisible(page, [

test/e2e/vpcs.e2e.ts

@@ -9,15 +9,17 @@ import { expect, test } from '@playwright/test'
 
 test('can nav to VpcPage from /', async ({ page }) => {
   await page.goto('/')
-  await page.click('table :text("mock-project")')
-  await page.click('a:has-text("VPCs")')
-  await page.click('a:has-text("mock-vpc")')
-  await expect(page.locator('text=mock-subnet')).toBeVisible()
+  await page.getByRole('table').getByRole('link', { name: 'mock-project' }).click()
+  await page.getByRole('link', { name: 'VPCs' }).click()
+  await page.getByRole('link', { name: 'mock-vpc' }).click()
+  await expect(page.getByRole('tab', { name: 'Firewall rules' })).toBeVisible()
+  await expect(page.getByRole('cell', { name: 'allow-icmp' })).toBeVisible()
   expect(await page.title()).toEqual('mock-vpc / VPCs / mock-project / Oxide Console')
 })
 
 test('can create and delete subnet', async ({ page }) => {
   await page.goto('/projects/mock-project/vpcs/mock-vpc')
+  await page.getByRole('tab', { name: 'Subnets' }).click()
   // only one row in table, the default mock-subnet
   const rows = page.locator('tbody >> tr')
   await expect(rows).toHaveCount(1)