-
Notifications
You must be signed in to change notification settings - Fork 83
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
Add more functional tests #120
Comments
@joelwurtz how would you feel about writing a full integration test about the user context feature? ddeboer is extracting the testing infrastructure in #122 - with this it should become possible to run tests with varnish in the bundle. given the complexity of the interaction, a functional test that goes end to end could be interesting i think. |
I’m here with @joelwurtz at the SymfonyCon hackday, and we doubt whether it makes sense to introduce Varnish into the bundle for integration tests. After all, we already test our context VCL in the library. @dbu Wdyt? |
For the short controller names I guess we need to create some dummy TestBundle, right? |
yeah, its a lot of additional things and redundance in testing. but we could simulate the requests in a WebTestCase and see if the right answers come back. though we might actually even do that already if there are functional tests for the HttpCache kernel class... what do you mean with the short controller names? is that not covered by the symfony core tests on the request matcher? |
Add more functional tests against the Symfony test app:
The text was updated successfully, but these errors were encountered: