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.
On Fri, 2010-01-15 at 11:12 +0000, Henning Eggers wrote: current_ browser_ request_ no_interaction current_ browser_ request
> Review: Approve code
> Thank you for this fix. Only suggestion I'd have is to split up the test into two methods.
>
> test_get_
> test_get_
>
Yeah, that'd be a nice and trivial change, so, done.
Thanks!