-
-
Couldn't load subscription status.
- Fork 452
PHPUnit: added/updated tests #4758
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I skimmed it.. 😅
That's a lot of coverage now, nice work!
|
Thanks. Total coverage is still below 10%, but starting with around 40 test last year and now having 800 is not so bad. I'll try to add more frequently. |
Description (*)
Added ~100 tests and updated all existing ones.
Zend_Validatewithsymfony/validator#4612 and improved them (there will be some errors now)Related Pull Requests
Zend_Validatewithsymfony/validator#4612Questions or comments
To test controllers i have added this to composer.json
Should we use it for core too? Ping @justinbeaty.