Merge pull request #1496 from weather-gov/mgwalker/edge

Rely on Playwright's built-in tools for downloading browsers
weather-gov · Jul 31, 2024 · 9e14e0f · 9e14e0f
2 parents 7600348 + fd8d053
commit 9e14e0f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/.github/actions/setup-playwright/action.yml b/.github/actions/setup-playwright/action.yml
@@ -1,8 +1,6 @@
 runs:
   using: composite
   steps:
-    - uses: browser-actions/setup-chrome@v1
-    # - uses: browser-actions/setup-edge@v1
     - uses: actions/setup-node@v4
       with:
         node-version: 18

diff --git a/playwright.config.js b/playwright.config.js
@@ -19,7 +19,7 @@ const config = {
   /* Retry on CI only */
   retries: process.env.CI ? 2 : 0,
   /* Reporter to use. See https://playwright.dev/docs/test-reporters */
-  reporter: process.env.CI ? 'blob' : 'html',
+  reporter: process.env.CI ? "blob" : "html",
   /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
   use: {
     /* Base URL to use in actions like `await page.goto('/')`. */
@@ -55,10 +55,11 @@ const config = {
     // containerized.
 
     /* Test against branded browsers. */
-    // {
-    //   name: "Microsoft Edge",
-    //   use: { ...devices["Desktop Edge"], channel: "msedge" },
-    // },
+    {
+      name: "Microsoft Edge",
+      use: { ...devices["Desktop Edge"], channel: "msedge" },
+    },
+
     {
       name: "Google Chrome",
       use: { ...devices["Desktop Chrome"], channel: "chrome" },