SuccessMessageMixinTests fail if MESSAGE_STORAGE is not CookieStorage

[Matt Leach]

The TestCase messages_tests.text_mixins.SuccessMessageMixinTestschecks that a cookie has been created. This means it will fail if the settings object MESSAGE_STORAGE is set to django.contrib.messages.storage.session.SessionStorage or anything that isn't CookieStorage.

Probably best to fix using an override_settings(MESSAGE_STORAGE='django.contrib.messages.storage.cookie.CookieStorage') decorator.

[Tim Graham]

I don't think there's a need to specify default settings in override_settings() everywhere we rely on them. Did you have some use case in mind?

[Matt Leach]

Fair enough. I didn't have a use case except that it means the tests will fail if you change the MESSAGE_STORAGE variable.

[Matt Leach]

Have just noticed that in tests/messages_tests/urls.py ContactFormViewWithMsg should use reverse_lazy (as below) rather than passing in the view object to success url (success_url = show). Is that worth fixing?

class ContactFormViewWithMsg(SuccessMessageMixin, FormView):
    form_class = ContactForm
    success_url = reverse_lazy('show_message')
    success_message = "%(name)s was created successfully"

[Tim Graham]

Sure, I guess it might be nice if the test actually verified that value since even success_url = 'foo' doesn't result in a test failure. Feel free to submit a PR.