Merge lp:~jml/launchpad/launchbag-test-bug-58 into lp:launchpad

[Jonathan Lange]

[Jonathan Lange (jml)]

This branch simply adds a test to show that ILaunchBag.user is None when no-one is logged in. It fixes a two-digit bug.

[Michael Hudson-Doyle (mwhudson)]

Haha!

review: Approve

[Tim Penhey (thumper)]

> This branch simply adds a test to show that ILaunchBag.user is None when no-
> one is logged in. It fixes a two-digit bug.

It would be good to update the doctest to follow the new guidelines :)

[Jonathan Lange (jml)]

On Thu, Sep 3, 2009 at 10:23 AM, Tim Penhey<email address hidden> wrote:
>> This branch simply adds a test to show that ILaunchBag.user is None when no-
>> one is logged in. It fixes a two-digit bug.
>
> It would be good to update the doctest to follow the new guidelines :)

I'd love to, but since there's no clear, up-to-date document
explaining the guidelines, I won't.

https://dev.launchpad.net/TestsStyleGuide is very much out of date,
and has way too much prose for someone trying to figure out correct
indentation & markup.

jml

=== modified file 'lib/canonical/launchpad/doc/launchbag.txt'
--- lib/canonical/launchpad/doc/launchbag.txt	2009-05-12 08:10:20 +0000
+++ lib/canonical/launchpad/doc/launchbag.txt	2009-09-03 00:08:57 +0000
@@ -57,6 +57,11 @@
 >>> print launchbag.login
 None
 
+'user' will also be set to None:
+
+>>> print launchbag.user
+None
+
 Let's do a cookie auth principal identification.  In this case, the login
 will be [e-mail].