Code review comment for lp:~salgado/lazr.restful/bug-507447

Revision history for this message
Guilherme Salgado (salgado) wrote :

On Fri, 2010-01-15 at 11:12 +0000, Henning Eggers wrote:
> Review: Approve code
> Thank you for this fix. Only suggestion I'd have is to split up the test into two methods.
>
> test_get_current_browser_request_no_interaction
> test_get_current_browser_request
>

Yeah, that'd be a nice and trivial change, so, done.

Thanks!

« Back to merge proposal