[Behat] Convert the configuration to the PHP format#18847
Closed
loic425 wants to merge 1 commit intoSylius:2.3from
Closed
[Behat] Convert the configuration to the PHP format#18847loic425 wants to merge 1 commit intoSylius:2.3from
loic425 wants to merge 1 commit intoSylius:2.3from
Conversation
❌ Preview Environment deleted from BunnyshellAvailable commands:
|
27 tasks
597bb92 to
ce8d076
Compare
|
Important Review skippedToo many files! This PR contains 300 files, which is 150 over the limit of 150. 📒 Files selected for processing (300)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ce8d076 to
11d74ed
Compare
TheMilek
added a commit
that referenced
this pull request
Feb 24, 2026
| Q | A |-----------------|----- | Branch? | 2.3 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | split from #18847 | License | MIT <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Restructured the Behat test configuration into a centralized, consolidated configuration approach. * Removed legacy Behat configuration and replaced it with a new programmatic config structure. * Updated project ignore patterns to reflect configuration file changes and avoid stale config being tracked. * Reorganized test suite declarations for clearer grouping of API, CLI, domain, hybrid, and UI suites. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
This was referenced Feb 24, 2026
TheMilek
added a commit
that referenced
this pull request
Feb 25, 2026
| Q | A |-----------------|----- | Branch? | 2.3 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | split from #18847 | License | MIT <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Migrated Behat API test suite configurations from YAML to PHP across all API-focused suites (account, addressing, admin, cart, channel, checkout, currency, inventory, locale, order, payment, product, promotion, shipping, taxation, taxon, and user management), removing the legacy YAML aggregators and consolidating suite definitions into PHP-based configuration files. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
TheMilek
added a commit
that referenced
this pull request
Feb 26, 2026
| Q | A |-----------------|----- | Branch? | 2.3 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Related tickets | split from #18847 | License | MIT
Contributor
|
The changes have been split into separate PRs, issues fixed, and all of them have been merged. Cheers 🍻 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Maybe we need the transform attribute first....