Skip to content

Feature/cosmetics#745

Merged
mihaileu merged 7 commits into
masterfrom
feature/cosmetics
Mar 6, 2026
Merged

Feature/cosmetics#745
mihaileu merged 7 commits into
masterfrom
feature/cosmetics

Conversation

@mihaileu

@mihaileu mihaileu commented Mar 6, 2026

Copy link
Copy Markdown
Collaborator

This pull request refactors the test suite to use the Pest testing framework instead of PHPUnit, resulting in more concise and modern test files. The main changes include removing the abstract BaseCommandTest class, converting test classes to Pest's function-based syntax, and moving setup logic into beforeEach() hooks. This improves test readability and maintainability.

Test suite modernization and migration to Pest:

  • Removed the abstract BaseCommandTest class, as Pest handles shared setup via beforeEach() hooks instead of inheritance. (Tests/Command/BaseCommandTest.php)
  • Converted command test classes (ConsumerCommandTest, DynamicConsumerCommandTest, MultipleConsumerCommandTest, PurgeCommandTest) from PHPUnit to Pest, replacing class-based tests and setUp() with beforeEach() and Pest's function-based test() syntax. (Tests/Command/ConsumerCommandTest.php, Tests/Command/DynamicConsumerCommandTest.php, Tests/Command/MultipleConsumerCommandTest.php, Tests/Command/PurgeCommandTest.php) [1] [2] [3] [4]
  • Updated event test files (AfterProcessingMessageEventTest, BeforeProcessingMessageEventTest, OnIdleEventTest) to Pest syntax, removing class wrappers and using test() functions for clarity and brevity. (Tests/Event/AfterProcessingMessageEventTest.php, Tests/Event/BeforeProcessingMessageEventTest.php, Tests/Event/OnIdleEventTest.php) [1] [2] [3]

@mihaileu mihaileu merged commit 56305f1 into master Mar 6, 2026
12 of 13 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.

1 participant