Skip to content
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

Open
3 of 6 tasks
ddeboer opened this issue Jul 31, 2014 · 4 comments
Open
3 of 6 tasks

Add more functional tests #120

ddeboer opened this issue Jul 31, 2014 · 4 comments
Labels

Comments

@ddeboer
Copy link
Member

ddeboer commented Jul 31, 2014

Add more functional tests against the Symfony test app:

@dbu
Copy link
Contributor

dbu commented Aug 6, 2014

@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.

@ddeboer
Copy link
Member Author

ddeboer commented Nov 29, 2014

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?

@ddeboer
Copy link
Member Author

ddeboer commented Nov 29, 2014

For the short controller names I guess we need to create some dummy TestBundle, right?

@dbu
Copy link
Contributor

dbu commented Nov 30, 2014

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants