Merge lp:~jamesh/storm/django-support into lp:storm

[James Henstridge]

[Gustavo Niemeyer (niemeyer)]

Looks great! +1!

review: Approve

[Gustavo Niemeyer (niemeyer)]

Argh!! That review was for a different merge proposal.

review: Abstain

[Gustavo Niemeyer (niemeyer)]

Well, it actually does look good too. :-)

Just a few hints about Mocker use, which may be safely ignored if you don't want to change these now:

[1]

+ self.expect(begin()).result(None)
(...)
+ self.expect(abort()).result(None)

The expect+result isn't strictly necessary in these cases. Just doing

    begin()
    self.mocker.replay()

for instance, would work.

[2]

+ # We test three request methods
+ self.expect(commit()).result(None)
+ self.expect(commit()).result(None)
+ self.expect(commit()).result(None)

Same thing here. This might be written as:

    self.expect(commit()).count(3)

review: Approve

[Jamu Kakar (jkakar)]

[1]

+__all__ = ["get_store"]

"get_store_uri" should be defined here too, in storm/django/stores.py.

[2]

+ def tearDown(self):
+ transaction.abort()
+ self.drop_tables()
+ settings._target = None
+ global_zstorm._reset()
+ stores.have_configured_stores = False
+ transaction.manager.free(transaction.get())
+ super(DjangoBackendTests, self).tearDown()

It might be worth making ZStorm._reset public so that we can access
one less private member here.

This is great. I'm looking forward to using it. +1!

review: Approve

[lp:~jamesh/storm/django-support]

270. By James Henstridge on 2008-09-11

Fix __all__ for storm.django.stores

271. By James Henstridge on 2008-09-11

merge from trunk

272. By James Henstridge on 2008-09-11

Add NEWS entry for storm.django.

273. By James Henstridge on 2008-09-11

Remove unused import.