Skip to content

Commit

Permalink
Merge pull request #1211 from weather-gov/dependabot/npm_and_yarn/pla…
Browse files Browse the repository at this point in the history
…ywright/test-1.44.0

Bump @playwright/test from 1.43.1 to 1.44.0
  • Loading branch information
loganmcdonald-noaa authored May 23, 2024
2 parents 86e3ec2 + 272c8f6 commit c3d8c8c
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 15 deletions.
16 changes: 8 additions & 8 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@
},
"devDependencies": {
"@axe-core/playwright": "^4.9.0",
"@playwright/test": "^1.43.1",
"@playwright/test": "^1.44.0",
"@types/node": "^20.12.12",
"axe-core": "^4.9.1",
"chai": "^5.1.1",
Expand Down
19 changes: 13 additions & 6 deletions playwright.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ const { defineConfig, devices } = require("@playwright/test");
/**
* @see https://playwright.dev/docs/test-configuration
*/
module.exports = defineConfig({
const config = {
testDir: "./tests/playwright",
/* Run tests in files in parallel */
fullyParallel: true,
Expand Down Expand Up @@ -53,10 +53,8 @@ module.exports = defineConfig({
name: "Mobile Chrome",
use: { ...devices["Pixel 5"] },
},
{
name: "Mobile Safari",
use: { ...devices["iPhone 12"] },
},
// Mobile Safari is excluded in CI because it tends to fail when it's
// containerized.

/* Test against branded browsers. */
{
Expand All @@ -68,4 +66,13 @@ module.exports = defineConfig({
use: { ...devices["Desktop Chrome"], channel: "chrome" },
},
],
});
};

if (!process.env.CI) {
config.projects.push({
name: "Mobile Safari",
use: { ...devices["iPhone 12"] },
});
}

module.exports = defineConfig(config);

0 comments on commit c3d8c8c

Please sign in to comment.