Skip to content
This repository has been archived by the owner on Jan 30, 2025. It is now read-only.

Commit

Permalink
Merge pull request #425 from bluehost/fix/matrix-workflow
Browse files Browse the repository at this point in the history
Fix/matrix workflow
  • Loading branch information
circlecube authored May 10, 2023
2 parents ec21663 + b5ff3c1 commit 0dc4154
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/cypress-matrix.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ name: Cypress Test Matrix

on:
pull_request:
types: [ opened, edited, reopened, ready_for_review ]
types: [ opened, edited, reopened, ready_for_review, synchronize ]
branches:
- 'master'
workflow_dispatch:
Expand Down
10 changes: 10 additions & 0 deletions cypress.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ module.exports = defineConfig({
}
}

// Exclude onboarding tests for WordPress lower than 6.1
if (semver.satisfies(config.env.wpSemverVersion, '<6.1.0')) {
config.excludeSpecPattern = config.excludeSpecPattern.concat(
[
Expand All @@ -56,6 +57,15 @@ module.exports = defineConfig({
);
}

// Exclude onboarding/ecommerce tests for PHP lower than 7.3 (7.1 and 7.2)
if (semver.satisfies(config.env.phpVersion, '<7.3.0')) {
config.excludeSpecPattern = config.excludeSpecPattern.concat(
[
"tests/cypress/integration/z-newfold-labs/wp-module-onboarding/ecommerce/**",
]
);
}

return config;
},
baseUrl: 'http://localhost:8882',
Expand Down

0 comments on commit 0dc4154

Please sign in to comment.