-
Notifications
You must be signed in to change notification settings - Fork 20
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
have test for xml bundle configuration #5
Comments
This is already implemented in the IcTestBundle for Configuration. But this is really easy to do, so I think we can safely just add a base test class here. |
in that case would it not be time to integrate IcTestBundle then? |
The ICBaseTestBundle is really good in terms of features, but there are things about it that I don't like (declaring properties to determine test behavior for example), which I have mentioned in this PR: instaclick/ICBaseTestBundle#28. And of course there is no PHPCR-ODM support, although this might be fixed in a forthcoming Doctrine Fixtures release. |
this one is fixed too |
it would be great to have a base test to help with first loading a bundle xml configuration file, then validate it both with the xml schema and the Configuration class to be sure they both expect the same.
The text was updated successfully, but these errors were encountered: