Make use of assertTemplateUsed and assertTemplateNotUsed as ContextManagers

[Gregor Müllegger]

It can be useful to use the assertTemplateUsed assertion on a limited block of code via a context manager instead on a request:

with self.assertTemplateUsed('template_used/base.html'):
    render_to_string('template_used/base.html')

This is something that already proofed useful in the soc2011/form-rendering branch. It's implemented there and is well tested: ​https://github.com/gregmuellegger/django/blob/soc2011/form-rendering/django/test/testcases.py#L720

Same is true for assertTemplateNotUsed.

[Gregor Müllegger]

The feature is implemented here: ​https://github.com/gregmuellegger/django/tree/t17604-assertTemplateUsed
The compare view: ​https://github.com/gregmuellegger/django/compare/master...t17604-assertTemplateUsed

Would be good if anyone could in particular review the docs, since my english is not the best at the moment.

[Gregor Müllegger]

merged with trunk

[Carl Meyer]

I'm seeing three errors and one failure in the tests with the latest patch: ​http://paste.pocoo.org/show/543635/

[Gregor Müllegger]

Did you applied the patch with "patch -p1 < t17604-assertTemplateUsed.2.diff"? I tried so and had the same problem, since it doesn't create the two empty files:

tests/regressiontests/test_utils/templates/template_used/base.html
tests/regressiontests/test_utils/templates/template_used/alternative.html

Use git apply t17604-assertTemplateUsed.2.diff instead or create those empty files by hand.
Sorry, I don't know how to make the patch work otherwise :-(

[Carl Meyer]

In [17412]:

Fixed #17604 - Added context-manager capability to assertTemplateUsed and assertTemplateNotUsed. Thanks Greg Müllegger.

[Carl Meyer]

Oops, my bad :-) Thanks for the pointer on getting the patch applied properly.