Skip to content

IBX-12374: Converted the Behat suites to PHP configuration - #15

Merged
ViniTou merged 2 commits into
6.0from
IBX-12374-php-behat-suites
Sep 10, 2026
Merged

IBX-12374: Converted the Behat suites to PHP configuration#15
ViniTou merged 2 commits into
6.0from
IBX-12374-php-behat-suites

Conversation

@ViniTou

@ViniTou ViniTou commented Sep 10, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue IBX-12374

Description:

This repository was missed by the Behat 4 migration sweep (no local checkout on the machine running the org scans; found by an API sweep of all 483 org repos). Behat 4 (ibexa/behat#200) only loads PHP configuration, and the YAML root this file imported was deleted in ibexa/behat#201 — so the YAML suites were dead.

  • Added behat_suites.php — a faithful conversion (same suite, contexts and paths), importing the PHP root shipped by ibexa/behat since #195.
  • Updated the browser-tests workflow --config where present.
  • Deleted the YAML file.
  • Where this repository has minimum-stability: stable, added behat/behat: ^4.0@alpha + friends-of-behat/mink-extension: ^3.0@alpha to require-dev — stability flags are root-only, so ibexa/behat's alpha pin does not propagate and composer install fails on cache-miss without them.

Same treatment as the earlier missed-repo wave (e.g. ibexa/integrated-help#76).

For QA:

Browser CI (where present) runs the suite through the PHP configuration on Behat 4.

Documentation:

N/A

@mikadamczyk mikadamczyk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this behat_suites file used?

@ViniTou
ViniTou requested a review from mikadamczyk September 10, 2026 12:50
@ViniTou
ViniTou merged commit 0ef7792 into 6.0 Sep 10, 2026
7 checks passed
@ViniTou
ViniTou deleted the IBX-12374-php-behat-suites branch September 10, 2026 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants