-
Notifications
You must be signed in to change notification settings - Fork 10
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
Use IsoCodes test data for validators #115
Conversation
7875589
to
ba7fb56
Compare
ba7fb56
to
c7cabf9
Compare
*/ | ||
private function getIsoCodesTestInstance() | ||
{ | ||
// Test classes seems not be autoloaded by default. |
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.
Configure autoload-dev
, maybe?
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.
What do you mean?
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.
In composer.json
, you can specify how to generate autoload mechanisms for whatever you want (including those test classes I think). Using autoload-dev
will ensure you don't clutter your production autoload mechanism.
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.
Thank you! ❤️
eeecd1c
to
c5ec93d
Compare
This will reduce a lot the test code length and make new integrations easier.
c5ec93d
to
ccd0342
Compare
Waiting ronanguilloux/IsoCodes#92 to be finished.
composer.json
hackIP
orIsbn
...Update "How to add a constraint documentation" doc accordingly.Documentation does not exist yet.If ronanguilloux/IsoCodes#93 is merged before new stable release of IsoCodes, remove special case from
CreditCardValidatorTest::getValidValues
.