Skip to content

Update PHPUnit test filter syntax in documentation and Taskfile - #24

Merged
roble merged 2 commits into
mainfrom
dev
Mar 10, 2026
Merged

Update PHPUnit test filter syntax in documentation and Taskfile#24
roble merged 2 commits into
mainfrom
dev

Conversation

@roble

@roble roble commented Mar 10, 2026

Copy link
Copy Markdown
Collaborator

This pull request updates the way PHPUnit tests are filtered for the Settings module to use a more specific filter string. This change ensures that only the intended tests are run when executing tests for the Settings module.

Testing command updates:

  • Updated the PHPUnit filter argument in both the Taskfile.yml and the documentation (CLAUDE.md) to use 'Modules.Settings' instead of Settings, providing more accurate test selection. [1] [2]

Copilot AI review requested due to automatic review settings March 10, 2026 20:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Settings module test-running commands to use a more specific PHPUnit --filter value, improving how module-specific tests are selected when running the Modules test suite.

Changes:

  • Updated php artisan test filter in Taskfile.yml to Modules.Settings.
  • Updated the documented PHPUnit command in CLAUDE.md to match.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
Taskfile.yml Adjusts the module test task’s PHPUnit filter string.
CLAUDE.md Updates the Testing docs to use the same new PHPUnit filter string.

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

Comment thread Taskfile.yml Outdated
Comment thread Taskfile.yml Outdated
Comment thread CLAUDE.md Outdated
Comment thread CLAUDE.md Outdated
@roble
roble merged commit 8438727 into main Mar 10, 2026
2 checks passed
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.

2 participants