Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make test data providers in multiple files static #48

Merged
merged 2 commits into from
Feb 4, 2024
Merged

Conversation

SmetDenis
Copy link
Member

Changed the test data providers in EmailTest, EnvTest, FilterTest, and TimerTest classes to be static. This approach allows these methods to be called without creating the test class instances, improving overall testing performance.

Changed the test data providers in EmailTest, EnvTest, FilterTest, and TimerTest classes to be static. This approach allows these methods to be called without creating the test class instances, improving overall testing performance.
@coveralls
Copy link

coveralls commented Feb 4, 2024

Coverage Status

coverage: 92.758%. remained the same
when pulling f799cfa on fix-linter
into 55ddbe0 on master.

The test methods in EmailTest.php, EnvTest.php, and FilterTest.php have been repositioned for better organization. It is important to note that no significant changes were made to the functionalities of these methods; they were simply moved to a different part in their respective files for a more streamlined and logical order.
@SmetDenis SmetDenis merged commit 0b9863c into master Feb 4, 2024
20 checks passed
@SmetDenis SmetDenis deleted the fix-linter branch February 4, 2024 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants