Add Unit Tests for ListContacts and ContactStats - Githubissues

liberu-crm / crm-laravel

CRM application written in Laravel 11 / PHP 8.3 using Filament 3

https://www.facebook.com/liberusoftware

11 stars 3 forks source link

Add Unit Tests for ListContacts and ContactStats #40

Closed sweep-ai[bot] closed 3 weeks ago

sweep-ai[bot] commented 3 months ago

PR Feedback (click)

[ ] 👍 Sweep Did Well
[ ] 👎 Sweep Needs Improvement
I created this PR to address this rule:

"All new business logic should have corresponding unit tests."

Description

This PR introduces unit tests for the ListContacts and ContactStats classes. The tests cover the new business logic introduced in these classes, ensuring that the application's functionality works as expected.

Summary of Changes

Created ListContactsTest.php in the tests/Unit directory. This test class contains tests for the 'created_at' filter, bulk delete action, and global search functionality in the ListContacts class.
Created ContactStatsTest.php in the tests/Unit directory. This test class contains tests for the total contacts calculation, recent contacts calculation, and contact categorization in the ContactStats class.

These tests help maintain the quality of our codebase by ensuring that new features and changes do not break existing functionality.

sweep-ai[bot] commented 3 months ago

Rollback Files For Sweep

[ ] Rollback changes to tests/Unit/ListContactsTest.php
[ ] Rollback changes to tests/Unit/ListContactsTest.php
[ ] Rollback changes to tests/Unit/ContactStatsTest.php
[ ] Rollback changes to tests/Unit/ContactStatsTest.php