launchpad-reviewers team mailing list archive
-
launchpad-reviewers team
-
Mailing list archive
-
Message #00942
Re: [Merge] lp:~sinzui/launchpad/inteam-db-calls-0 into lp:launchpad/devel
On Fri, Sep 10, 2010 at 12:50 AM, Curtis Hovey
<curtis.hovey@xxxxxxxxxxxxx> wrote:
> The cache is populated with TeamParticipation information. We know the db was not hit if no information from the db request was cached
This is a bit of a surrogate. Can I suggest the following:
add
Store.of(self.user).invalidate()
before your call that shouldn't do db (makes the storm cache invalid
and also (because we hook in) ICachedProperties are cleared.
and use
self.assertFalse(
self.assertStatementCount(0, self.user.inTeam, other_user))
which directly checks that no db statements happen.
-Rob
--
https://code.launchpad.net/~sinzui/launchpad/inteam-db-calls-0/+merge/34932
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~sinzui/launchpad/inteam-db-calls-0 into lp:launchpad/devel.
Follow ups
References