test: remove redundant API request in dashboard tests

fityannugroho · Nov 12, 2024 · 83bcfff · 83bcfff
1 parent a7e90a1
commit 83bcfff
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 8 deletions.
diff --git a/app/__tests__/dashboard/area-selector.spec.ts b/app/__tests__/dashboard/area-selector.spec.ts
@@ -2,9 +2,6 @@ import { config } from '@/lib/config'
 import { expect, test } from '@playwright/test'
 
 test.beforeEach(async ({ page }) => {
-  const res = await page.request.get(config.dataSource.area.url)
-  await expect(res).toBeOK()
-
   await page.goto('/')
 })
 

diff --git a/app/__tests__/dashboard/boundary-settings.spec.ts b/app/__tests__/dashboard/boundary-settings.spec.ts
@@ -2,9 +2,6 @@ import { config } from '@/lib/config'
 import test, { expect } from '@playwright/test'
 
 test.beforeEach(async ({ page }) => {
-  const res = await page.request.get(config.dataSource.area.url)
-  await expect(res).toBeOK()
-
   await page.goto('/')
 })
 

diff --git a/app/__tests__/home.spec.ts b/app/__tests__/home.spec.ts
@@ -1,14 +1,16 @@
 import { config } from '@/lib/config'
 import { expect, test } from '@playwright/test'
 
+test.beforeEach(async ({ page }) => {
+  await page.goto('/')
+})
+
 test.describe('meta tags', () => {
   test('has title', async ({ page }) => {
-    await page.goto('/')
     await expect(page).toHaveTitle(config.appName)
   })
 
   test('has description', async ({ page }) => {
-    await page.goto('/')
     await expect(
       page.locator('head').locator('meta[name="description"]'),
     ).toHaveAttribute('content', config.appDescription)