[Mailman-Developers] [Bug 1418276] Re: Deleting a user via the REST API does not delete their user preferences

[Barry Warsaw]

On Mar 20, 2015, at 05:41 PM, Abhishek wrote:

>In rest/tests/test_users.py you have used
>config.db.store.query(Preferences) inside with. But there is another
>usage of it in line 221 and also in the file model/tests/test_user.py
>which are not inside with block. Any specific reason for the
>difference..?
>
>230        with transaction():
>231            preferences = config.db.store.query(Preferences).filter_by(
>232                id=anne.preferences.id)
>233            self.assertEqual(preferences.count(), 0)

Yes.  The ids don't get assigned until after the commit, so they have to be
within the with-statement in order for the subsequent query to work.  But
after that, the delete does not require a commit for the query to succeed.

I don't think this was the case with Storm, but it appears to be so with
SQLAlchemy.