[mrrobot47]

Adds a Behat suite for log-command (features/: 52 scenarios, outline rows counted) and a Behat workflow that runs it on every pull request, on pushes to develop and on demand.

It covers ee log show for a site (type filters, line counts, following), --cli and --global, site resolution and validation errors. features/COVERAGE.md maps each command, flag and behaviour to its scenarios.

• The workflow runs the suite inside a fresh core checkout: core develop is cloned, every easyengine/* package is set to dev-develop, and this package is installed from the pull request's checkout through a Composer path repository, so the code under test is the PR's, not Packagist's.
• It runs on PHP 7.4 and 8.5.
• Scenarios written for open bugs are left out for now: they assert the fixed behaviour, so each comes back with its fix. Every scenario in this PR gates, and COVERAGE.md lists the bugs that aren't covered yet.

To run it locally (as root, on a test machine, since the suite creates and deletes *.test sites): replace a core checkout's features/ with this package's (rm -rf features && cp -R <package>/features features, as the workflow does; copying over it would keep core's own cli.feature, whose scenario runs ee cli self-uninstall --yes) and run sudo -E vendor/bin/behat from the core root.

Green run on my fork: https://github.com/mrrobot47/log-command/actions/runs/39292416544

[Copilot]

🟡 Changes recommended

Local runs delete pre-existing PHAR artifacts and fail directory-resolution scenarios when checkout paths contain spaces.

2 open issues

• Hook deletes pre-existing PHAR build artifacts · New
• Executable path is not shell-quoted · New

What changed in this PR

Adds integration coverage for ee log using Behat inside an EasyEngine core checkout.

Changes:

• Adds scenarios for site, CLI and global logs, validation, filtering and following.
• Introduces fixture helpers and a coverage inventory.
• Adds PHP 7.4/8.5 CI runs against the checked-out package, with uploaded results.

File	Description
features/​log.feature	Command surface and site resolution scenarios.
features/​log-site.feature	Site log filtering, following and safety scenarios.
features/​log-global.feature	CLI and global logging scenarios.
features/​COVERAGE.md	Coverage map and deferred bugs.
features/​bootstrap/​support.php	Shared assertion utilities.
features/​bootstrap/​FeatureContext.php	Steps, fixtures, timeouts and cleanup.
.github/​workflows/​behat.yml	Core-based test matrix and result collection.

_{Review effort: Balanced}

[Copilot]

🔵 Needs a closer look

The CSV assertion helper can incorrectly pass when duplicate actual rows compensate for missing expected rows.

No open issues 🎉

1 previously missed issue

In code that hasn't changed since last review

Prevent duplicate actual rows from masking missing expected rows

features/​bootstrap/​support.php:197

The shared match counter lets duplicate actual rows compensate for missing expected rows. For example, actual CSV name\nfoo\nfoo returns true for expected rows [['name'], ['foo'], ['bar']], although bar is absent. Track a match separately for each expected row and stop after its first match to prevent false-positive assertions.

2 resolved since last review

• Executable path is not shell-quoted
• Hook deletes pre-existing PHAR build artifacts

_{Review effort: Balanced}

[Copilot]

🟢 Approval recommended

No unresolved blocking issues were identified in the scenarios, test harness or CI integration.

No open issues 🎉

_{Review effort: Balanced}