Re: [Snowdrift-dev] [PATCH] Port Snowdrift tests from HSpec to Tasty

2017-07-08 Thread Aaron Wolf
On 07/08/2017 01:41 AM, fr33domlo...@riseup.net wrote: > From: fr33domlover > > --- > website/Snowdrift.cabal | 12 +++- > website/test/AlertsSpec.hs| 6 +++--- > website/test/AuthSiteSpec.hs | 8 > website/test/DiscourseSpec.hs | 35

[Snowdrift-dev] [PATCH] Port Snowdrift tests from HSpec to Tasty

2017-07-08 Thread fr33domlover
From: fr33domlover --- website/Snowdrift.cabal | 12 +++- website/test/AlertsSpec.hs| 6 +++--- website/test/AuthSiteSpec.hs | 8 website/test/DiscourseSpec.hs | 35 --- website/test/HandlerSpec.hs | 6

[Snowdrift-dev] [PATCH] crowdmatch: Use (SqlPersistT env) monad to allow combining queries

2017-07-08 Thread fr33domlover
From: fr33domlover Before this commit, crowdmatch actions run database queries separately inside them like a black box. That means in practice many applications of `runDB`, one per query, so a transaction per query. This is an unnecessary waste, and doesn't gain much (or

Re: [Snowdrift-dev] [PATCH] crowdmatch: Use (SqlPersistT env) monad to allow combining queries

2017-07-08 Thread Bryan Richter
On Sun, Jul 09, 2017 at 04:56:29AM +0300, fr33domlover wrote: > > Before this commit, crowdmatch actions run database queries separately > inside them like a black box. That means in practice many applications > of `runDB`, one per query, so a transaction per query. This is an > unnecessary