fix: popArgument drops duplicate arguments breaking --parallel --exclude-gropup=

[flap152]

What:

• Bug Fix

Description:

popArgument() in HandleArguments uses array_flip() twice, which silently
drops duplicate values. This breaks --parallel when multiple --exclude-group
flags are passed, since the parallel argument pipeline stores --exclude-group
and the group name as separate array entries (creating duplicate keys on flip).

The Coverage plugin had the same inline array_flip pattern.

Fix: Replace array_flip with array_search + unset in popArgument(),
and refactor Coverage to use the HandleArguments trait instead of inline logic.

Tests: Unit tests for popArgument() proving duplicate values are preserved.
Visual test for parallel with multiple --exclude-group flags.

Related:

Fixes #1437
Supersedes #1620, #1468

[Copilot]

Pull request overview

Fixes argument handling in Pest plugins so duplicate argument values are not silently dropped (notably in --parallel scenarios with repeated --exclude-group), and adds coverage to prevent regressions.

Changes:

• Reworked HandleArguments::popArgument() to remove an element via array_search + unset instead of array_flip (preserves duplicate values).
• Refactored the Coverage plugin’s argument stripping to reuse the HandleArguments trait.
• Added unit + visual tests for the duplicate-argument scenario and updated snapshots/expected counts accordingly.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/Plugins/Concerns/HandleArguments.php	Changes popArgument() implementation to avoid dropping duplicate values.
src/Plugins/Coverage.php	Uses the shared HandleArguments trait to strip coverage-related CLI args safely.
tests/Unit/Plugins/Concerns/HandleArguments.php	Adds unit tests proving duplicates are preserved.
tests/Visual/Parallel.php	Adds a visual regression test for multiple --exclude-group under --parallel; updates expected output counts.
tests/.snapshots/success.txt	Snapshot updates to include new tests and updated totals.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.