Make messaging system testable

[Christopher Allan Webber]

We should make it possible to test the messaging system.

If you look at mediagoblin/utils.py you can see what happens when
we \_activate\_testing() for some other things... ie, we insert
template context info like so:

::

        if TESTS_ENABLED:
            TEMPLATE_TEST_CONTEXT[template_path] = context

Maybe we should do something similar with messaging? Either we
should store in a dictionary like TEMPLATE\_TEST\_CONTEXT, maybe
TEST\_MESSAGES (which gets wiped by clear\_test\_buckets()) or we
should store in the session like request.session['test\_messages']
(should also get wiped by clear\_test\_buckets())

I'm assigning this to Caleb Davis who I think is a good fit for
it.

[Christopher Allan Webber]

Assigning to cmoylan!

[Caleb Davis]

go cmoylan, go! :)

[Will Kahn-Greene]

We release 0.0.4, so I'm bumping this to 0.0.5.

[Chris Moylan]

Seems like the best way to do this is to move the messaging
functions into util.py.

[Christopher Allan Webber]

Indeed, though I think Aaron is facing some of this stuff in
`http://bugs.foocorp.net/issues/444 <http://bugs.foocorp.net/issues/444>`_,
so adding him to this bug.

[Christopher Allan Webber]

Ping :)

[Chris Moylan]

I thought about moving messaging into util.py, but you're right -
it's a junk drawer of functionality. So instead I imported util.py
into messages.py to get access to the globals TESTS\_ENABLED and
TEMPLATE\_TEST\_CONTEXT. However, messages.py is already being
imported in util.py, so I'm not sure if there will be any circular
reference problems. I think the best solution would be to move the
test globals into their own file which can be imported wherever.

[Christopher Allan Webber]

Re: util.py, probably
`http://bugs.foocorp.net/issues/444 <http://bugs.foocorp.net/issues/444>`_
is related :)

[Chris Moylan]

So should this wait until 444 is resolved? I don't want to move
messaging into util.py if that's just going to create a headache
when it comes time to refactor it.

[Shawn Khan]

Hi Chris, `#444 </issues/444>`_ is now done, are you still
interested in this ticket?

[Christopher Allan Webber]

Yes it should be possible to move forward with this now. Chris
Moylan, if you're still interested, feel free to ping me on IRC!

[Will Kahn-Greene]

The original url for this bug was ​http://bugs.foocorp.net/issues/436 .
Relations:
#154: related

[Christopher Allan Webber]

No response recently from Chris Moylan on this. I'll send him an email. But if someone wants to just pick this up in the meanwhile they're welcome to. Should be reasonably simple.