Merge lp:~edwin-grubbs/launchpad/bug-482176-add-team-member-ajax-part3 into lp:launchpad

[Edwin Grubbs]

Add member via ajax link on team +index page.

[Edwin Grubbs (edwin-grubbs)]

Summary
-------

This branch fixes a bunch of tests. Most of the fixes are simple search and
replace, but there are a good 731 lines that are more complicated, so they
need a review instead of a rubber stamp. I have included those diff lines
in this comment so you don't have to read the automated diff.

Implementation details
----------------------

Display the correct informational message in the browser
corresponding to the new behavior for addMember().
    lib/lp/registry/browser/team.py

I changed the owners of several teams in this test so that
it tests the permissions more appropriately. Previously, it
was using foobar for everything, and that is a Launchpad admin,
so it effectively had admin access on all the teams.
    lib/lp/registry/doc/teammembership.txt

I changed the user for these tests, since salgado is also a
Launchpad admin.
    lib/lp/registry/stories/team/xx-team-add-my-teams.txt

Verify that the former members of a team are now only viewable
by the admins of the team on the +members page.
    lib/lp/registry/stories/teammembership/20-managing-members.txt

I changed some of the teams involved in the tests and the logged-in
user, since addMember() will fully add a team if the user is an
admin on both teams. If the user is only an admin on the team adding
another team, then addMember() will set the status to INVITED.
    lib/lp/registry/stories/teammembership/xx-add-member.txt
    lib/lp/registry/stories/teammembership/xx-private-membership.txt
    lib/lp/registry/stories/webservice/xx-private-membership.txt

In the REST API, the logged-in user is passed in as the reviewer
argument. Therefore, the no_priv user has to be used to create a
cyclical invitation as opposed to a cyclical active membership.
    lib/lp/registry/tests/test_teammembership.py

Tests
-----

./bin/test -vv -t registry

Pruned incremental diff
-----------------------
{{{
=== modified file 'lib/lp/registry/browser/team.py'
--- lib/lp/registry/browser/team.py 2009-12-10 18:11:38 +0000
+++ lib/lp/registry/browser/team.py 2010-01-04 17:12:48 +0000
@@ -993,9 +993,11 @@
         assert newmember != self.context, (
             "Can't add team to itself: %s" % newmember)

- self.context.addMember(newmember, reviewer=self.user,
- status=TeamMembershipStatus.APPROVED)
- if newmember.isTeam():
+ changed, new_status = self.context.addMember(
+ newmember, reviewer=self.user,
+ status=TeamMembershipStatus.APPROVED)
+
+ if new_status == TeamMembershipStatus.INVITED:
             msg = "%s has been invited to join this team." % (
                   newmember.unique_displayname)
         else:

=== modified file 'lib/lp/registry/doc/teammembership.txt'
--- lib/lp/registry/doc/teammembership.txt 2009-12-16 02:22:29 +0000
+++ lib/lp/registry/doc/teammembership.txt 2010-01-04 17:12:47 +0000
@@ -22,23 +22,28 @@
     >>> from zope.component import getUtility
     >>> from canonical.launchpad.interfaces import IPersonSet
     >>> personset = getUtility(IPersonSet)
- >>> foobar = personset.getByName('name16')
- >>> reviewer = foobar
+ >>> jblack = personset.getByName('jblack')
+ >>> nop...

[Aaron Bentley (abentley)]

The documentation is inverted here:

+First invite ubuntu-team to be member of the name20 team.

- >>> browser.open('http://launchpad.dev/~name21/+addmember')
- >>> browser.getControl('New member:').value = 'ubuntu-team'
+ >>> browser = setupBrowser(
+ ... auth='Basic <email address hidden>:test')
+ >>> browser.open('http://launchpad.dev/~ubuntu-team/+addmember')
+ >>> browser.getControl('New member:').value = 'name20'
     >>> browser.getControl('Add Member').click()

(name20 is added to ubuntu-team)

Other than that, this looks okay to land.

review: Approve (code)

=== modified file 'lib/canonical/launchpad/pagetests/standalone/xx-beta-testers-redirection.txt'
--- lib/canonical/launchpad/pagetests/standalone/xx-beta-testers-redirection.txt	2009-09-16 08:11:22 +0000
+++ lib/canonical/launchpad/pagetests/standalone/xx-beta-testers-redirection.txt	2010-01-05 17:55:34 +0000
@@ -239,7 +239,7 @@
     >>> foo_bar = getUtility(IPersonSet).getByName('name16')
     >>> launchpad_beta_testers = getUtility(
     ...     ILaunchpadCelebrities).launchpad_beta_testers
-    >>> launchpad_beta_testers.addMember(foo_bar, reviewer=foo_bar)
+    >>> ignored = launchpad_beta_testers.addMember(foo_bar, reviewer=foo_bar)
     >>> foo_bar.inTeam(launchpad_beta_testers)
     True
 
 
=== modified file 'lib/lp/answers/doc/person.txt'
--- lib/lp/answers/doc/person.txt	2009-04-17 10:32:16 +0000
+++ lib/lp/answers/doc/person.txt	2010-01-05 17:55:34 +0000
@@ -279,7 +279,7 @@
 contact through his team membership.
 
     >>> landscape_team = personset.getByName("landscape-developers")
-    >>> landscape_team.addMember(no_priv_raw, foo_bar_raw)
+    >>> ignored = landscape_team.addMember(no_priv_raw, foo_bar_raw)
     >>> no_priv_raw.inTeam(landscape_team)
     True
 
@@ -301,7 +301,7 @@
     >>> translator_team = personset.getByName('ubuntu-translators')
     >>> no_priv_raw.inTeam(translator_team)
     False
-    >>> translator_team.addMember(landscape_team, carlos_raw)
+    >>> ignored = translator_team.addMember(landscape_team, carlos_raw)
 
     # We need to accept the invitation sent by the addMember() call in
     # order to make landscape_team an actual member of translator_team.
 
=== modified file 'lib/lp/bugs/browser/tests/bug-views.txt'
--- lib/lp/bugs/browser/tests/bug-views.txt	2009-06-12 16:36:02 +0000
+++ lib/lp/bugs/browser/tests/bug-views.txt	2010-01-05 17:55:34 +0000
@@ -710,7 +710,7 @@
 People in the team get to see the comments.
 
     >>> login('[e-mail]')
-    >>> syncing_team.addMember(no_priv, no_priv)
+    >>> ignored = syncing_team.addMember(no_priv, no_priv)
 
     >>> login('[e-mail]')
     >>> view = BugTaskView(bug.bugtasks[0], LaunchpadTestRequest())
 
=== modified file 'lib/lp/bugs/doc/bugnotification-comment-syncing-team.txt'
--- lib/lp/bugs/doc/bugnotification-comment-syncing-team.txt	2009-06-12 16:36:02 +0000
+++ lib/lp/bugs/doc/bugnotification-comment-syncing-team.txt	2010-01-05 17:55:34 +0000
@@ -64,7 +64,7 @@
 Members of the syncing team will get the notifications.
 
     >>> LaunchpadZopelessLayer.switchDbUser('launchpad')
-    >>> syncing_team.addMember(no_priv, no_priv)
+    >>> ignored = syncing_team.addMember(no_priv, no_priv)
     >>> transaction.commit()
 
     >>> LaunchpadZopelessLayer.switchDbUser('bugnotification')
 
=== modified file 'lib/lp/bugs/doc/bugnotification-sending.txt'
--- lib/lp/bugs/doc/bugnotification-sending.txt	2009-12-05 18:37:28 +0000
+++ lib/lp/bugs/doc/bugnotification-sending.txt	2010-01-05 17:55:34 +0000
@@ -1280,7 +1280,8 @@
     ...     displayname='Concise Team Person',
     ...     email='[e-mail]')
     >>> concise_team_person.verbose_bugnotifications = True
-    >>> concise_team.addMember(concise_team_person, concise_team_person)
+    >>> ignored = concise_team.addMember(
+    ...     concise_team_person, concise_team_person)
     >>> bug.subscribe(concise_team, concise_team_person)
     <BugSubscription...>
 
@@ -1294,7 +1295,8 @@
     ...     displayname='Verbose Team Person',
     ...     email='[e-mail]')
     >>> verbose_team_person.verbose_bugnotifications = False
-    >>> verbose_team.addMember(verbose_team_person, verbose_team_person)
+    >>> ignored = verbose_team.addMember(
+    ...     verbose_team_person, verbose_team_person)
     >>> bug.subscribe(verbose_team, verbose_team_person)
     <BugSubscription...>
 
 
=== modified file 'lib/lp/bugs/doc/bugtask.txt'
--- lib/lp/bugs/doc/bugtask.txt	2009-08-27 07:58:37 +0000
+++ lib/lp/bugs/doc/bugtask.txt	2010-01-05 17:55:34 +0000
@@ -959,7 +959,8 @@
 Team *is* subscribed to the bug, No Privileges Person will see the private bug.
 
     >>> login("[e-mail]")
-    >>> ubuntu_team.addMember(no_priv, reviewer=ubuntu_team.teamowner)
+    >>> ignored = ubuntu_team.addMember(
+    ...     no_priv, reviewer=ubuntu_team.teamowner)
 
     >>> login("[e-mail]")
     >>> params = BugTaskSearchParams(
 
=== modified file 'lib/lp/bugs/stories/bugwatches/xx-bugwatch-comments.txt'
--- lib/lp/bugs/stories/bugwatches/xx-bugwatch-comments.txt	2009-09-02 12:33:41 +0000
+++ lib/lp/bugs/stories/bugwatches/xx-bugwatch-comments.txt	2010-01-05 17:55:34 +0000
@@ -55,7 +55,7 @@
     >>> foo_bar = getUtility(IPersonSet).getByName('name16')
     >>> launchpad_beta_testers = getUtility(
     ...     ILaunchpadCelebrities).launchpad_beta_testers
-    >>> launchpad_beta_testers.addMember(mark, reviewer=foo_bar)
+    >>> ignored = launchpad_beta_testers.addMember(mark, reviewer=foo_bar)
     >>> mark.inTeam(launchpad_beta_testers)
     True
 
 
=== modified file 'lib/lp/code/stories/branches/xx-branch-deletion.txt'
--- lib/lp/code/stories/branches/xx-branch-deletion.txt	2009-09-23 14:40:53 +0000
+++ lib/lp/code/stories/branches/xx-branch-deletion.txt	2010-01-05 17:55:34 +0000
@@ -84,7 +84,7 @@
     >>> login(ANONYMOUS)
     >>> name12 = getUtility(IPersonSet).getByName('name12')
     >>> ubuntu_branches = getUtility(ILaunchpadCelebrities).ubuntu_branches
-    >>> removeSecurityProxy(ubuntu_branches).addMember(
+    >>> ignored = removeSecurityProxy(ubuntu_branches).addMember(
     ...     name12, ubuntu_branches.teamowner)
     >>> login_person(name12)
     >>> branch = factory.makePackageBranch(owner=name12)
 
=== modified file 'lib/lp/code/stories/feeds/xx-revision-atom.txt'
--- lib/lp/code/stories/feeds/xx-revision-atom.txt	2009-07-09 05:12:34 +0000
+++ lib/lp/code/stories/feeds/xx-revision-atom.txt	2010-01-05 17:55:34 +0000
@@ -54,7 +54,7 @@
 
     >>> login_person(mike)
     >>> team = factory.makeTeam(mike, 'The M Team', name='m-team')
-    >>> team.addMember(mary, mike)
+    >>> ignored = team.addMember(mary, mike)
     >>> from lp.code.interfaces.revision import IRevisionSet
     >>> from zope.component import getUtility
     >>> revision_set = getUtility(IRevisionSet)
 
=== modified file 'lib/lp/registry/browser/team.py'
--- lib/lp/registry/browser/team.py	2009-12-10 18:11:38 +0000
+++ lib/lp/registry/browser/team.py	2010-01-05 17:55:34 +0000
@@ -993,9 +993,11 @@
         assert newmember != self.context, (
             "Can't add team to itself: %s" % newmember)
 
-        self.context.addMember(newmember, reviewer=self.user,
-                               status=TeamMembershipStatus.APPROVED)
-        if newmember.isTeam():
+        changed, new_status = self.context.addMember(
+            newmember, reviewer=self.user,
+            status=TeamMembershipStatus.APPROVED)
+
+        if new_status == TeamMembershipStatus.INVITED:
             msg = "%s has been invited to join this team." % (
                   newmember.unique_displayname)
         else:
 
=== modified file 'lib/lp/registry/browser/tests/peoplemerge-views.txt'
--- lib/lp/registry/browser/tests/peoplemerge-views.txt	2009-07-27 20:53:27 +0000
+++ lib/lp/registry/browser/tests/peoplemerge-views.txt	2010-01-05 17:55:34 +0000
@@ -38,7 +38,7 @@
     >>> parent_team = factory.makeTeam()
     >>> child_team = factory.makeTeam(name='child-team')
     >>> random_team = factory.makeTeam()
-    >>> parent_team.addMember(
+    >>> ignored = parent_team.addMember(
     ...     child_team, reviewer=parent_team.teamowner, force_team_add=True)
     >>> form = {'field.dupe_person': child_team.name,
     ...         'field.target_person': random_team.name,
 
=== modified file 'lib/lp/registry/browser/tests/person-views.txt'
--- lib/lp/registry/browser/tests/person-views.txt	2009-12-23 16:17:12 +0000
+++ lib/lp/registry/browser/tests/person-views.txt	2010-01-05 17:55:34 +0000
@@ -911,7 +911,7 @@
 
     >>> login("[e-mail]")
     >>> team = factory.makeTeam()
-    >>> team.addMember(sample_person, sample_person)
+    >>> ignored = team.addMember(sample_person, sample_person)
     >>> ppa = factory.makeArchive(distribution=ubuntu, owner=team)
 
     >>> login(ANONYMOUS)
 
=== modified file 'lib/lp/registry/browser/tests/product-views.txt'
--- lib/lp/registry/browser/tests/product-views.txt	2009-11-23 14:52:25 +0000
+++ lib/lp/registry/browser/tests/product-views.txt	2010-01-05 17:55:34 +0000
@@ -142,7 +142,7 @@
     >>> commercial_member = getUtility(IPersonSet).getByEmail(
     ...     '[e-mail]')
     >>> registry_experts = getUtility(IPersonSet).getByName('registry')
-    >>> registry_experts.addMember(commercial_member, reviewer=mark)
+    >>> ignored = registry_experts.addMember(commercial_member, reviewer=mark)
     >>> login('[e-mail]')
     >>> view = create_initialized_view(firefox, name='+review-license')
     >>> print view.label
 
=== modified file 'lib/lp/registry/browser/tests/projectgroupset-views.txt'
--- lib/lp/registry/browser/tests/projectgroupset-views.txt	2009-09-12 06:00:05 +0000
+++ lib/lp/registry/browser/tests/projectgroupset-views.txt	2010-01-05 17:55:34 +0000
@@ -44,7 +44,7 @@
 
     >>> registry_member = factory.makePerson()
     >>> login('[e-mail]')
-    >>> registry.addMember(registry_member, registry.teamowner)
+    >>> ignored = registry.addMember(registry_member, registry.teamowner)
     >>> registry_member.inTeam(registry)
     True
 
 
=== modified file 'lib/lp/registry/browser/tests/team-views.txt'
--- lib/lp/registry/browser/tests/team-views.txt	2009-11-25 23:20:08 +0000
+++ lib/lp/registry/browser/tests/team-views.txt	2010-01-05 17:55:34 +0000
@@ -33,7 +33,7 @@
     >>> salgado = person_set.getByName('salgado')
     >>> mark = person_set.getByName('mark')
     >>> login('[e-mail]')
-    >>> ubuntu_team.addMember(salgado, reviewer=mark)
+    >>> ignored = ubuntu_team.addMember(salgado, reviewer=mark)
 
     >>> team_home = create_initialized_view(ubuntu_team, '+index')
     >>> for member in team_home.recently_approved_members:
@@ -125,7 +125,7 @@
 
     >>> london_member = factory.makePerson(
     ...     latitude=51.49, longitude=-0.13, time_zone='Europe/London')
-    >>> context.addMember(london_member, mark)
+    >>> ignored = context.addMember(london_member, mark)
     >>> context.mapped_participants_count
     1
     >>> view = create_initialized_view(context, '+map')
@@ -139,7 +139,7 @@
 
     >>> brazil_member = factory.makePerson(
     ...     latitude=-23.60, longitude=-46.64, time_zone='America/Sao_Paulo')
-    >>> context.addMember(brazil_member, mark)
+    >>> ignored = context.addMember(brazil_member, mark)
     >>> context.mapped_participants_count
     2
     >>> view = create_initialized_view(context, '+map')
@@ -153,7 +153,7 @@
 
     >>> london_member2 = factory.makePerson(
     ...     latitude=51.49, longitude=-0.13, time_zone='Europe/London')
-    >>> context.addMember(london_member2, mark)
+    >>> ignored = context.addMember(london_member2, mark)
     >>> context.mapped_participants_count
     3
     >>> view = create_initialized_view(context, '+map')
 
=== modified file 'lib/lp/registry/browser/tests/teammembership-views.txt'
--- lib/lp/registry/browser/tests/teammembership-views.txt	2010-01-05 17:55:30 +0000
+++ lib/lp/registry/browser/tests/teammembership-views.txt	2010-01-05 17:55:34 +0000
@@ -62,8 +62,7 @@
     >>> from lp.registry.browser.person import TeamInvitationView
 
     >>> login_person(team_owner)
-    >>> super_team.addMember(team, team_owner)
-    True
+    >>> ignored = super_team.addMember(team, team_owner)
     >>> membership = membership_set.getByPersonAndTeam(team, super_team)
     >>> login_person(team.teamowner)
     >>> view = TeamInvitationView(membership, request)
 
=== modified file 'lib/lp/registry/doc/announcement.txt'
--- lib/lp/registry/doc/announcement.txt	2009-08-13 19:03:36 +0000
+++ lib/lp/registry/doc/announcement.txt	2010-01-05 17:55:34 +0000
@@ -32,7 +32,7 @@
     >>> login('[e-mail]')
     >>> from canonical.launchpad.interfaces import ILaunchpadCelebrities
     >>> beta = getUtility(ILaunchpadCelebrities).launchpad_beta_testers
-    >>> beta.addMember(mark, mark)
+    >>> ignored = beta.addMember(mark, mark)
     >>> from canonical.database.sqlbase import flush_database_updates
     >>> flush_database_updates()
     >>> logout()
 
=== modified file 'lib/lp/registry/doc/commercialsubscription.txt'
--- lib/lp/registry/doc/commercialsubscription.txt	2009-09-16 20:08:36 +0000
+++ lib/lp/registry/doc/commercialsubscription.txt	2010-01-05 17:55:34 +0000
@@ -550,7 +550,7 @@
     >>> registry_member = factory.makePerson()
     >>> celebs = getUtility(ILaunchpadCelebrities)
     >>> registry = celebs.registry_experts
-    >>> registry.addMember(registry_member, registry.teamowner)
+    >>> ignored = registry.addMember(registry_member, registry.teamowner)
     >>> logout()
 
     >>> login_person(registry_member)
 
=== modified file 'lib/lp/registry/doc/mailinglist-email-notification.txt'
--- lib/lp/registry/doc/mailinglist-email-notification.txt	2009-05-12 08:11:06 +0000
+++ lib/lp/registry/doc/mailinglist-email-notification.txt	2010-01-05 17:55:34 +0000
@@ -10,7 +10,7 @@
     >>> salgado = getUtility(IPersonSet).getByName('salgado')
 
     >>> from lp.registry.tests.mailinglists_helper import (
-    ...		new_team, new_list_for_team)
+    ...     new_team, new_list_for_team)
 
     # login() as an admin so that we can call IPerson.join() on any
     # person/team we want.
@@ -33,7 +33,8 @@
     >>> list_one.status.name
     'ACTIVE'
 
-    >>> transaction.commit()	# Send the emails.
+    # Send the emails.
+    >>> transaction.commit()
 
 Both Anne and No Privileges Person receive the notifications, Anne as a
 newly subscribed member and No Privileges Person as the owner of the
@@ -89,7 +90,7 @@
 
     >>> super_team = new_team('super-team', with_list=False)
     >>> sub_team = new_team('sub-team', with_list=False)
-    >>> super_team.addMember(sub_team, salgado, force_team_add=True)
+    >>> ignored = super_team.addMember(sub_team, salgado, force_team_add=True)
     >>> anne.join(super_team)
     >>> lars = factory.makePersonByName('Lars')
     >>> lars.join(sub_team)
@@ -120,7 +121,8 @@
     >>> super_list.status.name
     'ACTIVE'
 
-    >>> transaction.commit()	# Send the emails.
+    # Send the emails.
+    >>> transaction.commit()
     >>> len(stub.test_emails)
     4
     >>> print_emails(include_reply_to=True)
@@ -220,6 +222,7 @@
     >>> renewed_list.status.name
     'ACTIVE'
 
-    >>> transaction.commit()	# Send any outstanding emails.
+    # Send any outstanding emails.
+    >>> transaction.commit()
     >>> len(stub.test_emails)
     0
 
=== modified file 'lib/lp/registry/doc/mailinglist-subscriptions.txt'
--- lib/lp/registry/doc/mailinglist-subscriptions.txt	2009-05-06 15:13:39 +0000
+++ lib/lp/registry/doc/mailinglist-subscriptions.txt	2010-01-05 17:55:34 +0000
@@ -293,7 +293,7 @@
     ...     'sub-team', team_owner)
     >>> team_names.extend((super_team.name, sub_team.name))
 
-    >>> super_team.addMember(sub_team, salgado, force_team_add=True)
+    >>> ignored = super_team.addMember(sub_team, salgado, force_team_add=True)
     >>> lars = factory.makePersonByName('Lars')
     >>> lars.join(super_team)
     >>> lars.join(sub_team)
@@ -333,7 +333,7 @@
     ...     'team-three', team_owner)
     >>> team_names.append(team_three.name)
 
-    >>> team_one.addMember(team_three, salgado, force_team_add=True)
+    >>> ignored = team_one.addMember(team_three, salgado, force_team_add=True)
     >>> dirk = factory.makePersonByName('Dirk')
     >>> dirk.join(team_two)
     >>> dirk.join(team_three)
@@ -714,7 +714,7 @@
     >>> team_five = factory.makeTeam(name='team-five', owner=team_owner)
     >>> team_names.append(team_five.name)
 
-    >>> team_one.addMember(team_five, salgado, force_team_add=True)
+    >>> ignored = team_one.addMember(team_five, salgado, force_team_add=True)
     >>> gwen = factory.makePersonByName('Gwen')
     >>> hank = factory.makePersonByName('Hank')
     >>> iona = factory.makePersonByName('Iona')
 
=== modified file 'lib/lp/registry/doc/mailinglists.txt'
--- lib/lp/registry/doc/mailinglists.txt	2009-12-03 20:28:54 +0000
+++ lib/lp/registry/doc/mailinglists.txt	2010-01-05 17:55:34 +0000
@@ -82,7 +82,8 @@
     >>> from canonical.launchpad.interfaces import TeamMembershipStatus
     >>> login('[e-mail]')
     >>> bart = factory.makePersonByName('Bart')
-    >>> team_two.addMember(anne, bart, status=TeamMembershipStatus.ADMIN)
+    >>> ignored = team_two.addMember(
+    ...     anne, bart, status=TeamMembershipStatus.ADMIN)
     >>> login(ANONYMOUS)
     >>> list_two = list_set.new(team_two, anne)
     >>> list_two
 
=== modified file 'lib/lp/registry/doc/person-merge.txt'
--- lib/lp/registry/doc/person-merge.txt	2009-12-07 16:07:43 +0000
+++ lib/lp/registry/doc/person-merge.txt	2010-01-05 17:55:34 +0000
@@ -42,9 +42,9 @@
     >>> ubuntu_team.teamowner = marilize
 
     >>> ubuntu_translators = personset.getByName('ubuntu-translators')
-    >>> ubuntu_translators.addMember(marilize, marilize)
+    >>> ignored = ubuntu_translators.addMember(marilize, marilize)
     >>> rosetta_admins = personset.getByName('rosetta-admins')
-    >>> rosetta_admins.addMember(marilize, marilize)
+    >>> ignored = rosetta_admins.addMember(marilize, marilize)
 
 Karma gets reassigned to the person we merge into. Let's assign karma to
 Marilize and save it for later comparison.
@@ -348,7 +348,7 @@
     >>> from canonical.launchpad.interfaces import IPollSubset, PollSecrecy
     >>> test_team = personset.newTeam(sample, 'test-team', 'Test team')
     >>> launchpad_devs = personset.getByName('launchpad')
-    >>> launchpad_devs.addMember(
+    >>> ignored = launchpad_devs.addMember(
     ...     test_team, reviewer=launchpad_devs.teamowner, force_team_add=True)
     >>> today = datetime.now(pytz.timezone('UTC'))
     >>> tomorrow = today + timedelta(days=1)
 
=== modified file 'lib/lp/registry/doc/person.txt'
--- lib/lp/registry/doc/person.txt	2009-12-03 20:00:43 +0000
+++ lib/lp/registry/doc/person.txt	2010-01-05 17:55:34 +0000
@@ -482,7 +482,7 @@
 And we can even add other members to our new team!
 
     >>> login('[e-mail]')
-    >>> not_a_person.addMember(lifeless, reviewer=ddaa)
+    >>> ignored = not_a_person.addMember(lifeless, reviewer=ddaa)
     >>> login(ANONYMOUS)
     >>> [member.name for member in not_a_person.activemembers]
     [u'ddaa', u'lifeless']
@@ -1125,7 +1125,7 @@
     >>> registry_member = factory.makePerson()
     >>> celebs = getUtility(ILaunchpadCelebrities)
     >>> registry = celebs.registry_experts
-    >>> registry.addMember(registry_member, registry.teamowner)
+    >>> ignored = registry.addMember(registry_member, registry.teamowner)
     >>> logout()
 
     >>> login_person(registry_member)
 
=== modified file 'lib/lp/registry/doc/private-team-visibility.txt'
--- lib/lp/registry/doc/private-team-visibility.txt	2010-01-04 23:17:19 +0000
+++ lib/lp/registry/doc/private-team-visibility.txt	2010-01-05 17:55:34 +0000
@@ -17,7 +17,7 @@
     >>> priv_team = factory.makeTeam(owner=priv_owner, name="priv-team",
     ...                              visibility=PersonVisibility.PRIVATE)
     >>> login_person(priv_owner)
-    >>> priv_team.addMember(priv_member, reviewer=priv_owner)
+    >>> ignored = priv_team.addMember(priv_member, reviewer=priv_owner)
 
 The team owner can see the membership.
 
@@ -57,7 +57,7 @@
     >>> pub_member = factory.makePerson(name="pub-member")
     >>> pub_team = factory.makeTeam(owner=pub_owner, name="pubteam")
     >>> login_person(pub_owner)
-    >>> pub_team.addMember(pub_member, reviewer=pub_owner)
+    >>> ignored = pub_team.addMember(pub_member, reviewer=pub_owner)
 
 At this point the public team owner cannot see the priv-team's bits.
 
@@ -67,7 +67,7 @@
     Unauthorized: (<Person at ... priv-team (Priv Team)>, 'name', 'launchpad.View')
 
     >>> login_person(priv_owner)
-    >>> priv_team.addMember(pub_team, reviewer=priv_owner)
+    >>> ignored = priv_team.addMember(pub_team, reviewer=priv_owner)
 
 The public team is not yet a member of the priv-team.
 
 
=== modified file 'lib/lp/registry/doc/teammembership-email-notification.txt'
--- lib/lp/registry/doc/teammembership-email-notification.txt	2010-01-05 17:55:30 +0000
+++ lib/lp/registry/doc/teammembership-email-notification.txt	2010-01-05 17:55:34 +0000
@@ -227,8 +227,7 @@
 
     >>> cprov = personset.getByName('cprov')
     >>> marilize = personset.getByName('marilize')
-    >>> ubuntu_team.addMember(marilize, reviewer=cprov)
-    True
+    >>> ignored = ubuntu_team.addMember(marilize, reviewer=cprov)
     >>> transaction.commit()
     >>> len(stub.test_emails)
     6
@@ -277,8 +276,7 @@
     >>> mirror_admins = personset.getByName('ubuntu-mirror-admins')
     >>> mirror_admins.getTeamAdminsEmailAddresses()
     ['[e-mail]']
-    >>> ubuntu_team.addMember(mirror_admins, reviewer=cprov)
-    True
+    >>> ignored = ubuntu_team.addMember(mirror_admins, reviewer=cprov)
     >>> transaction.commit()
     >>> len(stub.test_emails)
     1
@@ -329,8 +327,7 @@
 Similarly, a notification is sent if the invitation is declined.
 
     >>> landscape = personset.getByName('landscape-developers')
-    >>> ubuntu_team.addMember(landscape, reviewer=cprov)
-    True
+    >>> ignored = ubuntu_team.addMember(landscape, reviewer=cprov)
 
     # Reset stub.test_emails as we don't care about the notification triggered
     # by the addMember() call.
@@ -363,8 +360,8 @@
 passing force_team_add=True to the addMember() method.
 
     >>> launchpad = personset.getByName('launchpad')
-    >>> ubuntu_team.addMember(launchpad, reviewer=cprov, force_team_add=True)
-    True
+    >>> ignored = ubuntu_team.addMember(
+    ...     launchpad, reviewer=cprov, force_team_add=True)
     >>> flush_database_updates()
     >>> transaction.commit()
     >>> len(stub.test_emails)
@@ -854,8 +851,7 @@
     >>> dummy = pop_notifications()
     >>> member = factory.makePerson(
     ...     name='team-member', email='[e-mail]')
-    >>> team_one.addMember(member, owner)
-    True
+    >>> ignored = team_one.addMember(member, owner)
     >>> print_distinct_emails()
     From: Team One ...
     ----------------------------------------
@@ -881,8 +877,7 @@
 
     >>> team_two = factory.makeTeam(
     ...     name='team-two', email='[e-mail]', owner=owner)
-    >>> team_one.addMember(team_two, owner, force_team_add=True)
-    True
+    >>> ignored = team_one.addMember(team_two, owner, force_team_add=True)
     >>> print_distinct_emails()
     From: Team One ...
     ----------------------------------------
 
=== modified file 'lib/lp/registry/doc/teammembership.txt'
--- lib/lp/registry/doc/teammembership.txt	2010-01-05 17:55:30 +0000
+++ lib/lp/registry/doc/teammembership.txt	2010-01-05 17:55:34 +0000
@@ -22,23 +22,28 @@
     >>> from zope.component import getUtility
     >>> from canonical.launchpad.interfaces import IPersonSet
     >>> personset = getUtility(IPersonSet)
-    >>> foobar = personset.getByName('name16')
-    >>> reviewer = foobar
+    >>> jblack = personset.getByName('jblack')
+    >>> nopriv = personset.getByName('no-priv')
+    >>> jdub = personset.getByName('jdub')
+    >>> reviewer = nopriv
     >>> t1 = personset.newTeam(
-    ...     foobar, 't1', 't1',
+    ...     jblack, 't1', 't1',
     ...     subscriptionpolicy=TeamSubscriptionPolicy.OPEN)
     >>> t2 = personset.newTeam(
-    ...     foobar, 't2', 't2',
+    ...     nopriv, 't2', 't2',
     ...     subscriptionpolicy=TeamSubscriptionPolicy.OPEN)
     >>> t3 = personset.newTeam(
-    ...     foobar, 't3', 't3',
+    ...     jdub, 't3', 't3',
     ...     subscriptionpolicy=TeamSubscriptionPolicy.MODERATED)
     >>> t4 = personset.newTeam(
-    ...     foobar, 't4', 't4',
+    ...     nopriv, 't4', 't4',
     ...     subscriptionpolicy=TeamSubscriptionPolicy.OPEN)
     >>> t5 = personset.newTeam(
-    ...     foobar, 't5', 't5',
+    ...     nopriv, 't5', 't5',
     ...     subscriptionpolicy=TeamSubscriptionPolicy.OPEN)
+    >>> t6 = personset.newTeam(
+    ...     jdub, 't6', 't6',
+    ...     subscriptionpolicy=TeamSubscriptionPolicy.MODERATED)
 
     # Make sure the teams have predictable (and different) creation dates as
     # some of our tests depend on that.
@@ -66,15 +71,15 @@
 member of that team --somebody has to approve his membership first:
 
     >>> [m.displayname for m in t4.allmembers]
-    [u'Foo Bar', u'Guilherme Salgado']
+    [u'Guilherme Salgado', u'No Privileges Person']
 
     >>> [m.displayname for m in t3.allmembers]
-    [u'Foo Bar']
+    [u'Jeff Waugh']
     >>> login_person(t3.teamowner)
     >>> t3.setMembershipData(salgado, TeamMembershipStatus.APPROVED, reviewer)
     >>> flush_database_updates()
     >>> [m.displayname for m in t3.allmembers]
-    [u'Foo Bar', u'Guilherme Salgado']
+    [u'Guilherme Salgado', u'Jeff Waugh']
 
 The join() method is not allowed for teams whose subscription policy is
 RESTRICTED. And it'll be a no-op in case the user has already joined the
@@ -141,12 +146,12 @@
     # Log in as the team owner.
     >>> login_person(t3.teamowner)
 
-addMember returns True if the member got added (i.e. he wasn't already a
-member of the team).
+If the member was added (i.e. he wasn't already a member of the team),
+addMember returns a tuple with True plus the new membership status.
 
     >>> t3.addMember(
     ...     salgado, reviewer=mark, status=TeamMembershipStatus.ADMIN)
-    True
+    (True, <DBItem TeamMembershipStatus.ADMIN...)
     >>> from canonical.launchpad.interfaces import ITeamMembershipSet
     >>> membershipset = getUtility(ITeamMembershipSet)
     >>> flush_database_updates()
@@ -158,26 +163,26 @@
     >>> salgado in t3.activemembers
     True
 
-addMember returns True also when the member is added as a proposed
-member.
+addMember returns (True, PROPOSED) also when the member is added as a
+proposed member.
 
     >>> marilize = personset.getByName('marilize')
     >>> t3.addMember(
     ...     marilize, reviewer=mark, status=TeamMembershipStatus.PROPOSED)
-    True
+    (True, <DBItem TeamMembershipStatus.PROPOSED...)
     >>> flush_database_updates()
     >>> marilize in t3.activemembers
     False
 
 If addMember is called with a person that is already a member, it
-returns False.
+returns a tuple with False and the current status of the membership.
 
     >>> t3.addMember(
     ...     salgado, reviewer=mark, status=TeamMembershipStatus.ADMIN)
-    False
+    (False, <DBItem TeamMembershipStatus.ADMIN...)
     >>> t3.addMember(
     ...     marilize, reviewer=mark, status=TeamMembershipStatus.PROPOSED)
-    False
+    (False, <DBItem TeamMembershipStatus.PROPOSED...)
 
 As expected, the membership object implements ITeamMembership.
 
@@ -188,29 +193,37 @@
 
 Note that, by default, the ITeam.addMember() API works slightly different
 when the added member is a team. In that case the team will actually be
-invited to be a member and one of the team's admins will have to accpet the
+invited to be a member and one of the team's admins will have to accept the
 invitation before the team is made a member.
 
-    >>> t1.addMember(t2, reviewer)
+    >>> login_person(t1.teamowner)
+
+    # If the reviewer were also an admin of the team being added,
+    # the status would go to APPROVED instead of INVITED.
+    >>> t2.teamowner != t1.teamowner
     True
+    >>> t1.addMember(t2, reviewer=t1.teamowner)
+    (True, <DBItem TeamMembershipStatus.INVITED...)
     >>> membership = membershipset.getByPersonAndTeam(t2, t1)
     >>> membership.status == TeamMembershipStatus.INVITED
     True
     >>> [m.displayname for m in t1.allmembers]
-    [u'Foo Bar']
+    [u'James Blackwell']
 
 Once one of the t2 admins approve the membership, t2 is shown as a member
-of t1.
+of t1 and the owner of t2 is an indirect member.
 
     >>> login_person(t2.teamowner)
     >>> t2.acceptInvitationToBeMemberOf(t1, comment='something')
+    >>> [m.displayname for m in t1.activemembers]
+    [u'James Blackwell', u't2']
     >>> [m.displayname for m in t1.allmembers]
-    [u'Foo Bar', u't2']
+    [u'James Blackwell', u'No Privileges Person', u't2']
 
 A team admin can also decline an invitation made to his team.
 
     >>> t2.addMember(t3, reviewer=mark)
-    True
+    (True, <DBItem TeamMembershipStatus.INVITED...)
     >>> login_person(t3.teamowner)
     >>> t3.declineInvitationToBeMemberOf(t2, comment='something')
     >>> membership = membershipset.getByPersonAndTeam(t3, t2)
@@ -222,21 +235,40 @@
 force_team_add=True to addMember(). We'll use that to add t3 as a member of
 t2, thus making all t3 members be considered members of t2 as well.
 
-    >>> login_person(t3.teamowner)
-    >>> t2.addMember(t3, reviewer=mark, force_team_add=True)
+    >>> login_person(t2.teamowner)
+
+    # If the reviewer is also an admin of the team being added,
+    # force_team_add is unnecessary, and we can't prove that that
+    # argument works.
+    >>> t3.teamowner != t2.teamowner
     True
+    >>> t2.addMember(t3, reviewer=t2.teamowner, force_team_add=True)
+    (True, <DBItem TeamMembershipStatus.APPROVED...)
     >>> [m.displayname for m in t2.allmembers]
-    [u'Foo Bar', u'Guilherme Salgado', u't3']
+    [u'Guilherme Salgado', u'Jeff Waugh', u'No Privileges Person', u't3']
 
 And members of t1 as well, since t2 is a member of t1.
 
     >>> [m.displayname for m in t1.allmembers]
-    [u'Foo Bar', u'Guilherme Salgado', u't2', u't3']
+    [u'Guilherme Salgado', u'James Blackwell', u'Jeff Waugh',
+    u'No Privileges Person', u't2', u't3']
+
+
+Passing in force_team_add=True is not necessary if the reviewer is the
+admin of the team being added.
+
+    >>> login_person(t3.teamowner)
+    >>> t6.addMember(t3, reviewer=t3.teamowner)
+    (True, <DBItem TeamMembershipStatus.APPROVED...)
+    >>> [m.displayname for m in t6.allmembers]
+    [u'Guilherme Salgado', u'Jeff Waugh', u't3']
 
 Can we add t2 as a member of t3? No, we prevent this kind of loop, and users
 can't do this because our vocabularies won't allow members that would cause
 loops.
 
+    >>> foobar = personset.getByEmail('[e-mail]')
+    >>> login_person(foobar)
     >>> t3.addMember(t2, reviewer)
     Traceback (most recent call last):
     ...
@@ -246,19 +278,21 @@
 Adding t2 as a member of t5 will add all t2 members as t5 members too.
 
     >>> t5.addMember(t2, reviewer, force_team_add=True)
-    True
+    (True, <DBItem TeamMembershipStatus.APPROVED...)
     >>> [m.displayname for m in t5.allmembers]
-    [u'Foo Bar', u'Guilherme Salgado', u't2', u't3']
+    [u'Guilherme Salgado', u'Jeff Waugh', u'No Privileges Person',
+    u't2', u't3']
 
 Adding t5 and t1 as members of t4 will add all t5 and t1 members as t4
 members too.
 
     >>> t4.addMember(t5, reviewer, force_team_add=True)
-    True
+    (True, <DBItem TeamMembershipStatus.APPROVED...)
     >>> t4.addMember(t1, reviewer, force_team_add=True)
-    True
+    (True, <DBItem TeamMembershipStatus.APPROVED...)
     >>> [m.displayname for m in t4.allmembers]
-    [u'Foo Bar', u'Guilherme Salgado', u't1', u't2', u't3', u't5']
+    [u'Guilherme Salgado', u'James Blackwell', u'Jeff Waugh',
+    u'No Privileges Person', u't1', u't2', u't3', u't5']
 
     >>> flush_database_updates()
 
@@ -305,18 +339,20 @@
 member anymore, it doesn't have any members apart from its owner.
 
     >>> [m.displayname for m in t5.allmembers]
-    [u'Foo Bar']
+    [u'No Privileges Person']
 
 Removing t2 from t5 won't remove it from t4, because t2 is also a member of
 t1, which is a member of t4.
 
     >>> [m.displayname for m in t4.allmembers]
-    [u'Foo Bar', u'Guilherme Salgado', u't1', u't2', u't3', u't5']
+    [u'Guilherme Salgado', u'James Blackwell', u'Jeff Waugh',
+    u'No Privileges Person', u't1', u't2', u't3', u't5']
 
 Nothing changes in t1, because t5 wasn't one of its members.
 
     >>> [m.displayname for m in t1.allmembers]
-    [u'Foo Bar', u'Guilherme Salgado', u't2', u't3']
+    [u'Guilherme Salgado', u'James Blackwell', u'Jeff Waugh',
+    u'No Privileges Person', u't2', u't3']
 
 If 'Guilherme Salgado' decides to leave t3, he'll also be removed from t1
 and t2, but not from t4, because he's a direct member of t4.
@@ -350,31 +386,31 @@
 
     >>> cprov = getUtility(IPersonSet).getByName('cprov')
     >>> t3.addMember(cprov, reviewer)
-    True
+    (True, <DBItem TeamMembershipStatus.APPROVED...)
     >>> [m.displayname for m in t3.allmembers]
-    [u'Celso Providelo', u'Foo Bar']
+    [...u'Celso Providelo'...
 
     >>> [m.displayname for m in t2.allmembers]
-    [u'Celso Providelo', u'Foo Bar', u't3']
+    [...u'Celso Providelo'...
 
     >>> [m.displayname for m in t1.allmembers]
-    [u'Celso Providelo', u'Foo Bar', u't2', u't3']
+    [...u'Celso Providelo'...
 
     >>> [m.displayname for m in t4.allmembers]
-    [u'Celso Providelo', u'Foo Bar', u'Guilherme Salgado', u't1', u't2',
-     u't3', u't5']
-
-
-It's important to note that even if Foo Bar leaves the team he's the owner,
-he'll retains his rights over the team because he's the team's owner. This
-ensures we'll never have teams which can't be managed.
-
-    >>> login_person(foobar)
-    >>> foobar.leave(t5)
+    [...u'Celso Providelo'...
+
+
+It's important to note that even if the owner leaves the team, which
+removes his membership, he will still be the team's owner and retain his
+rights over it. This ensures we'll never have teams which can't be
+managed.
+
+    >>> login_person(t5.teamowner)
+    >>> t5.teamowner.leave(t5)
     >>> flush_database_updates()
     >>> [m.displayname for m in t5.allmembers]
     []
-    >>> foobar.inTeam(t5)
+    >>> t5.teamowner.inTeam(t5)
     True
 
 
@@ -390,7 +426,7 @@
 
     # Foo Bar is a launchpad admin, but even so he can't change a membership's
     # status/expiry-date by hand.
-    >>> login('[e-mail]')
+    >>> login_person(foobar)
     >>> membership = foobar.myactivememberships[0]
     >>> membership.status = None
     Traceback (most recent call last):
@@ -709,7 +745,7 @@
 
     >>> [(membership.person.name, membership.status.title)
     ...  for membership in t3.member_memberships]
-    [(u'cprov', 'Approved'), (u'name16', 'Administrator')]
+    [(u'cprov', 'Approved'), (u'jdub', 'Administrator')]
 
 A team has a number of other methods that return the people which are members
 of it, all based on Person.getMembersByStatus:
@@ -748,7 +784,7 @@
 admin members plus the owner in case he is not one of the admin members.
 
     >>> [admin.unique_displayname for admin in t3.adminmembers]
-    [u'Foo Bar (name16)']
+    [u'Jeff Waugh (jdub)']
     >>> list(t3.getDirectAdministrators()) == list(t3.adminmembers)
     True
 
@@ -823,7 +859,7 @@
     >>> admins = getUtility(IPersonSet).getByName('admins')
     >>> login_person(t1.teamowner)
     >>> t1.addMember(admins, reviewer=t1.teamowner, force_team_add=True)
-    True
+    (True, <DBItem TeamMembershipStatus.APPROVED...)
     >>> flush_database_updates()
     >>> print '\n'.join(sorted(
     ...     team.name for team in salgado.teams_participated_in))
@@ -834,11 +870,12 @@
     t1
     t4
 
-On the other hand, making t2 a member of admins won't change anything
+On the other hand, making t3 a member of admins won't change anything
 for Salgado.
 
-    >>> admins.addMember(t2, reviewer=admins.teamowner, force_team_add=True)
-    True
+    >>> login_person(foobar)
+    >>> admins.addMember(t3, reviewer=admins.teamowner, force_team_add=True)
+    (True, <DBItem TeamMembershipStatus.APPROVED...)
     >>> flush_database_updates()
     >>> print '\n'.join(sorted(
     ...     team.name for team in salgado.teams_participated_in))
@@ -855,8 +892,8 @@
 Person instances have membership caches (_inTeam_cache) to avoid the
 need to reissue identical membership queries repeatedly. (This test uses
 Person directly to allow us access to an unproxied object, since
-_inTeam_cache is private, and TeamMembershipSet directly to allow us to
-call TeamMembership.setStatus, which is private).
+_inTeam_cache is private, and it uses TeamMembershipSet directly to allow
+us to call TeamMembership.setStatus, which is private).
 
     >>> from lp.registry.model.person import Person
     >>> from lp.registry.model.teammembership import TeamMembershipSet
@@ -864,7 +901,7 @@
     >>> no_priv.inTeam(admins)
     False
     >>> no_priv._inTeam_cache
-    {25: False}
+    {...25: False...}
 
 This cache is cleared when memberships are created:
 
@@ -920,7 +957,7 @@
     >>> transaction.commit()
 
     >>> [m.displayname for m in t3.allmembers]
-    [u'Bad-user', u'Foo Bar']
+    [u'Bad-user', u'Jeff Waugh']
     >>> bad_user.is_valid_person
     False
     >>> print bad_user.preferredemail
 
=== modified file 'lib/lp/registry/doc/vocabularies.txt'
--- lib/lp/registry/doc/vocabularies.txt	2009-12-07 16:07:43 +0000
+++ lib/lp/registry/doc/vocabularies.txt	2010-01-05 17:55:34 +0000
@@ -255,8 +255,7 @@
 
     >>> from canonical.launchpad.interfaces import PersonVisibility
     >>> otherteam = person_set.getByName('otherteam')
-    >>> otherteam.addMember(foo_bar, foo_bar)
-
+    >>> ignored = otherteam.addMember(foo_bar, foo_bar)
     >>> otherteam.visibility = PersonVisibility.PRIVATE_MEMBERSHIP
     >>> for term in person_active_membership:
     ...     print term.token, term.value.displayname, term.title
@@ -1398,7 +1397,7 @@
     >>> registry_member = factory.makePerson()
     >>> celebs = getUtility(ILaunchpadCelebrities)
     >>> registry = celebs.registry_experts
-    >>> registry.addMember(registry_member, registry.teamowner)
+    >>> ignored = registry.addMember(registry_member, registry.teamowner)
     >>> logout()
 
     >>> login_person(registry_member)
 
=== modified file 'lib/lp/registry/stories/mailinglists/lifecycle.txt'
--- lib/lp/registry/stories/mailinglists/lifecycle.txt	2009-12-03 21:19:49 +0000
+++ lib/lp/registry/stories/mailinglists/lifecycle.txt	2010-01-05 17:55:34 +0000
@@ -363,7 +363,7 @@
     >>> person_set = getUtility(IPersonSet)
     >>> test = person_set.getByName('name12')
     >>> experts = getUtility(ILaunchpadCelebrities).mailing_list_experts
-    >>> experts.addMember(test, reviewer=experts.teamowner)
+    >>> ignored = experts.addMember(test, reviewer=experts.teamowner)
     >>> logout()
     >>> transaction.commit()
 
 
=== modified file 'lib/lp/registry/stories/mailinglists/subscriptions.txt'
--- lib/lp/registry/stories/mailinglists/subscriptions.txt	2009-12-03 20:28:54 +0000
+++ lib/lp/registry/stories/mailinglists/subscriptions.txt	2010-01-05 17:55:34 +0000
@@ -195,7 +195,7 @@
     >>> person_set = getUtility(IPersonSet)
     >>> jdub = person_set.getByName('jdub')
     >>> mark = person_set.getByName('mark')
-    >>> beta.addMember(jdub, reviewer=mark)
+    >>> ignored = beta.addMember(jdub, reviewer=mark)
     >>> from canonical.database.sqlbase import flush_database_updates
     >>> flush_database_updates()
     >>> logout()
@@ -424,9 +424,9 @@
     >>> salgado = person_set.getByName('salgado')
     >>> jordi = person_set.getByName('jordi')
     >>> rosetta_admins = person_set.getByName('rosetta-admins')
-    >>> rosetta_admins.addMember(salgado, reviewer=mark)
+    >>> ignored = rosetta_admins.addMember(salgado, reviewer=mark)
     >>> rosetta_admins.mailing_list.subscribe(salgado)
-    >>> rosetta_admins.addMember(jordi, reviewer=mark)
+    >>> ignored = rosetta_admins.addMember(jordi, reviewer=mark)
     >>> rosetta_admins.mailing_list.subscribe(jordi)
     >>> logout()
     >>> browser.reload()
 
=== modified file 'lib/lp/registry/stories/person/xx-admin-person-review.txt'
--- lib/lp/registry/stories/person/xx-admin-person-review.txt	2010-01-04 10:24:29 +0000
+++ lib/lp/registry/stories/person/xx-admin-person-review.txt	2010-01-05 17:55:34 +0000
@@ -103,7 +103,8 @@
     >>> from canonical.launchpad.interfaces import ILaunchpadCelebrities
     >>> from zope.component import getUtility
     >>> registry_team = getUtility(ILaunchpadCelebrities).registry_experts
-    >>> registry_team.addMember(registry_expert, registry_team.teamowner)
+    >>> ignored = registry_team.addMember(
+    ...     registry_expert, registry_team.teamowner)
     >>> print registry_expert.inTeam(registry_team)
     True
     >>> logout()
 
=== modified file 'lib/lp/registry/stories/product/xx-product-edit.txt'
--- lib/lp/registry/stories/product/xx-product-edit.txt	2009-10-08 16:29:45 +0000
+++ lib/lp/registry/stories/product/xx-product-edit.txt	2010-01-05 17:55:34 +0000
@@ -292,7 +292,7 @@
     ...     name='reggie', email='[e-mail]', password='test')
     >>> celebs = getUtility(ILaunchpadCelebrities)
     >>> registry = celebs.registry_experts
-    >>> registry.addMember(registry_member, registry.teamowner)
+    >>> ignored = registry.addMember(registry_member, registry.teamowner)
     >>> logout()
 
     >>> registry_browser = setupBrowser(
 
=== modified file 'lib/lp/registry/stories/product/xx-productset.txt'
--- lib/lp/registry/stories/product/xx-productset.txt	2009-09-18 17:38:50 +0000
+++ lib/lp/registry/stories/product/xx-productset.txt	2010-01-05 17:55:34 +0000
@@ -23,7 +23,7 @@
     ...     name='reggie', email='[e-mail]', password='test')
     >>> celebs = getUtility(ILaunchpadCelebrities)
     >>> registry = celebs.registry_experts
-    >>> registry.addMember(registry_member, registry.teamowner)
+    >>> ignored = registry.addMember(registry_member, registry.teamowner)
     >>> logout()
 
     >>> registry_browser = setupBrowser(
 
=== modified file 'lib/lp/registry/stories/team/xx-team-add-my-teams.txt'
--- lib/lp/registry/stories/team/xx-team-add-my-teams.txt	2009-11-22 15:43:16 +0000
+++ lib/lp/registry/stories/team/xx-team-add-my-teams.txt	2010-01-05 17:55:34 +0000
@@ -6,8 +6,9 @@
 This is done from the +add-my-teams page, which is linked from a team's
 home page.
 
-    >>> browser = setupBrowser(auth='Basic [e-mail]:zeca')
-    >>> browser.open('http://launchpad.dev/~ubuntu-team')
+    >>> browser = setupBrowser(
+    ...     auth='Basic [e-mail]:test')
+    >>> browser.open('http://launchpad.dev/~name21')
     >>> browser.getLink('Add one of my teams').click()
     >>> browser.title
     'Propose/add one of your teams to another one...
@@ -18,47 +19,48 @@
     >>> print extract_text(find_tag_by_id(browser.contents, 'candidates'))
     This is a moderated team, so one of its administrators will have
     to review any memberships you propose.
-    Launchpad Administrators
-    Mailing List Experts
-    Rosetta Administrators
+    GuadaMen
+    Ubuntu Security Team
+    Ubuntu Team
     or Cancel
 
-We'll now propose Launchpad Administrators as a member of Ubuntu Team.
+We'll now propose Ubuntu Team as a member of the Hoary Gnome Team (name21).
 
-    >>> browser.getControl(name='field.teams').value = ['admins']
+    >>> browser.open('http://launchpad.dev/~name21')
+    >>> link = browser.getLink('Add one of my teams')
+    >>> link.click()
+    >>> browser.getControl(name='field.teams').value = ['ubuntu-team']
     >>> browser.getControl('Continue').click()
     >>> browser.title
-    'Ubuntu Team in Launchpad'
+    'Hoary Gnome Team in Launchpad'
     >>> print get_feedback_messages(browser.contents)
-    [u'Launchpad Administrators has been proposed to this team.']
+    [u'Ubuntu Team has been proposed to this team.']
     >>> print extract_text(
-    ...     find_tag_by_id(browser.contents, 'recently-applied'))
+    ...     find_tag_by_id(browser.contents, 'recently-proposed'))
     Pending approval
-    Launchpad Administrators
-    ...
+    Ubuntu Team...
 
 If it were an OPEN team, we'd be able to directly add any of our teams as
 members.
 
-    >>> admin_browser.open('http://launchpad.dev/~ubuntu-team/+edit')
+    >>> admin_browser.open('http://launchpad.dev/~name21/+edit')
     >>> admin_browser.getControl(
     ...     name='field.subscriptionpolicy').displayValue = ['Open Team']
     >>> admin_browser.getControl('Save').click()
     >>> admin_browser.title
-    'Ubuntu Team in Launchpad'
+    'Hoary Gnome Team in Launchpad'
 
-    >>> browser.open('http://launchpad.dev/~ubuntu-team/+add-my-teams')
-    >>> browser.getControl(name='field.teams').value = ['rosetta-admins']
+    >>> browser.open('http://launchpad.dev/~name21/+add-my-teams')
+    >>> browser.getControl(name='field.teams').value = ['ubuntu-team']
     >>> browser.getControl('Continue').click()
     >>> browser.title
-    'Ubuntu Team in Launchpad'
+    'Hoary Gnome Team in Launchpad'
     >>> print get_feedback_messages(browser.contents)
-    [u'Rosetta Administrators has been added to this team.']
+    [u'Ubuntu Team has been added to this team.']
     >>> print extract_text(
     ...     find_tag_by_id(browser.contents, 'recently-approved'))
-    Recently approved
-    Rosetta Administrators
-    ...
+    Latest members
+    Ubuntu Team...
 
 In the case of restricted teams, though, there is no way to propose any of
 your teams as members.
@@ -88,38 +90,38 @@
     Traceback (most recent call last):
     Unauthorized:...
 
-You also can't propose a team to itself. Here although Salgado is usually
-allowed to propose Launchpad Administrators in other team, it doesn't
-appear in the list when proposing a team for the Launchpad
-Administrators team. Likewise Mailing List Experts isn't shown because
-the Launchpad Administrators are a member of Mailing List Experts.
-Adding Mailing List Experts would create a cycle.
+You also can't propose a team to itself. Here although Colin Watson is
+usually allowed to propose Guadamen in other team, it doesn't appear in
+the list when proposing a team for the Guadamen team. Likewise Mailing
+List Experts isn't shown because the Launchpad Administrators are a
+member of Mailing List Experts.  Adding Mailing List Experts would
+create a cycle.
 
-    >>> browser.open('http://launchpad.dev/~admins/+add-my-teams')
+    >>> browser.open('http://launchpad.dev/~guadamen/+add-my-teams')
     >>> browser.getControl(name='field.teams').options
-    ['rosetta-admins']
+    ['ubuntu-security']
 
 Teams that are already member of the team can't be proposed or added.
-For example, Rosetta Administrators is not in the list of choices
-anymore of the Ubuntu Team:
+For example, Ubuntu Team is not in the list of choices
+anymore of the Hoary Gnome Team:
 
-    >>> admin_browser.open('http://launchpad.dev/~ubuntu-team/+edit')
+    >>> admin_browser.open('http://launchpad.dev/~name21/+edit')
     >>> admin_browser.getControl(
     ...     name='field.subscriptionpolicy').displayValue = ['Open']
     >>> admin_browser.getControl('Save').click()
 
-    >>> browser.open('http://launchpad.dev/~ubuntu-team/+add-my-teams')
+    >>> browser.open('http://launchpad.dev/~name21/+members')
+    >>> browser.open('http://launchpad.dev/~name21/+add-my-teams')
     >>> browser.getControl(name='field.teams').options
-    ['admins', 'mailing-list-experts']
+    ['guadamen', 'ubuntu-security']
     >>> browser.getControl(name='field.teams').value = [
-    ...     'admins', 'mailing-list-experts']
+    ...     'guadamen', 'ubuntu-security']
     >>> browser.getControl('Continue').click()
     >>> print "\n".join(get_feedback_messages(browser.contents))
-    Launchpad Administrators, Mailing List Experts have been added to this
-    team.
+    GuadaMen, Ubuntu Security Team have been added to this team.
 
 And when no teams can be added, a message is displayed:
 
-    >>> browser.open('http://launchpad.dev/~ubuntu-team/+add-my-teams')
+    >>> browser.open('http://launchpad.dev/~name21/+add-my-teams')
     >>> print extract_text(find_tag_by_id(browser.contents, 'no-candidates'))
     None of the teams you administer can be added to this team.
 
=== modified file 'lib/lp/registry/stories/teammembership/20-managing-members.txt'
--- lib/lp/registry/stories/teammembership/20-managing-members.txt	2009-09-18 21:24:09 +0000
+++ lib/lp/registry/stories/teammembership/20-managing-members.txt	2010-01-05 17:55:34 +0000
@@ -28,7 +28,15 @@
     >>> print_members(browser.contents, 'proposedmembers')
     Foo Bar
 
-    >>> print_members(browser.contents, 'inactivemembers')
+Former members are only viewable by admins of the team.
+
+    >>> print find_tag_by_id(browser.contents, 'inactivemembers')
+    None
+
+    >>> name12_browser = setupBrowser(
+    ...     auth="Basic [e-mail]:test")
+    >>> name12_browser.open(browser.url)
+    >>> print_members(name12_browser.contents, 'inactivemembers')
     Karl Tilbury
     No Privileges Person
 
@@ -57,12 +65,13 @@
     ... [launchpad]
     ... default_batch_size: 2
     ... """)
-    >>> browser.open('http://launchpad.dev/~admins/+members?inactive_batch=2')
-    >>> print_members(browser.contents, 'inactivemembers')
+    >>> admin_browser.open(
+    ...     'http://launchpad.dev/~admins/+members?inactive_batch=2')
+    >>> print_members(admin_browser.contents, 'inactivemembers')
     Celso Providelo
     David Allouche
-    >>> browser.getLink('Next', index=2).click()
-    >>> print_members(browser.contents, 'inactivemembers')
+    >>> admin_browser.getLink('Next', index=2).click()
+    >>> print_members(admin_browser.contents, 'inactivemembers')
     James Blackwell
     >>> config_data = config.pop('default-batch-size')
 
 
=== modified file 'lib/lp/registry/stories/teammembership/private-team.txt'
--- lib/lp/registry/stories/teammembership/private-team.txt	2009-10-26 21:12:49 +0000
+++ lib/lp/registry/stories/teammembership/private-team.txt	2010-01-05 17:55:34 +0000
@@ -17,7 +17,7 @@
     >>> person_set = getUtility(IPersonSet)
     >>> cprov = person_set.getByName('cprov')
     >>> login_person(owner)
-    >>> priv_team.addMember(cprov, reviewer=owner)
+    >>> ignored = priv_team.addMember(cprov, reviewer=owner)
     >>> logout()
 
     >>> admin_browser.open('http://launchpad.dev/~private-team')
 
=== modified file 'lib/lp/registry/stories/teammembership/xx-add-member.txt'
--- lib/lp/registry/stories/teammembership/xx-add-member.txt	2009-10-26 14:37:31 +0000
+++ lib/lp/registry/stories/teammembership/xx-add-member.txt	2010-01-05 17:55:34 +0000
@@ -49,7 +49,7 @@
     >>> from canonical.launchpad.interfaces import TeamMembershipStatus
     >>> from canonical.launchpad.ftests import ANONYMOUS, login, logout
     >>> login(ANONYMOUS) # login() because we need a zope interaction.
-    >>> cprov_landscape_membership.setStatus(
+    >>> ignored = cprov_landscape_membership.setStatus(
     ...     TeamMembershipStatus.DEACTIVATED, landscape_team.teamowner)
     >>> logout()
     >>> cprov_landscape_membership.syncUpdate()
@@ -171,44 +171,47 @@
 same. When an admin accepts or declines an invitation, the other admin can't
 take action on that invitation anymore.
 
-First invite ubuntu-team to be member of the name21 team.
+First invite name20 to be a member of ubuntu-team.
 
-    >>> browser.open('http://launchpad.dev/~name21/+addmember')
-    >>> browser.getControl('New member:').value = 'ubuntu-team'
+    >>> browser = setupBrowser(
+    ...     auth='Basic [e-mail]:test')
+    >>> browser.open('http://launchpad.dev/~ubuntu-team/+addmember')
+    >>> browser.getControl('New member:').value = 'name20'
     >>> browser.getControl('Add Member').click()
 
     >>> for tag in find_tags_by_class(browser.contents,
     ...                               'informational message'):
     ...     print tag.renderContents()
-    Ubuntu Team (ubuntu-team) has been invited to join this team.
+    Warty Security Team (name20) has been invited to join this team.
 
 Open the invitations page with one admin browser.
 
-    >>> browser.open('http://launchpad.dev/~ubuntu-team/+invitation/name21')
+    >>> browser = setupBrowser(auth='Basic [e-mail]:test')
+    >>> browser.open('http://launchpad.dev/~name20/+invitation/ubuntu-team')
 
 Open the same page with another admin browser.
 
-    >>> second_browser = setupBrowser(auth='Basic [e-mail]:test')
+    >>> second_browser = setupBrowser(auth='Basic [e-mail]:test')
     >>> second_browser.open(
-    ...     'http://launchpad.dev/~ubuntu-team/+invitation/name21')
+    ...     'http://launchpad.dev/~name20/+invitation/ubuntu-team')
 
 Accept the invitation in the first browser.
 
     >>> browser.getControl('Accept').click()
     >>> browser.url
-    'http://launchpad.dev/~ubuntu-team'
+    'http://launchpad.dev/~name20'
 
     >>> for tag in find_tags_by_class(browser.contents,
     ...                               'informational message'):
     ...     print tag.renderContents()
-    This team is now a member of Hoary Gnome Team
+    This team is now a member of Ubuntu Team
 
 Accepting the invitation in the second browser, redirects to the team page
 and a message is displayed.
 
     >>> second_browser.getControl('Accept').click()
     >>> second_browser.url
-    'http://launchpad.dev/~ubuntu-team'
+    'http://launchpad.dev/~name20'
 
     >>> for tag in find_tags_by_class(second_browser.contents,
     ...                               'informational message'):
@@ -233,7 +236,7 @@
     >>> login(ANONYMOUS) # login() because we need a zope interaction.
     >>> carlos_translators_membership = TeamMembership.selectOneBy(
     ...     personID=carlos.id, teamID=ubuntu_translators.id)
-    >>> carlos_translators_membership.setStatus(
+    >>> ignored = carlos_translators_membership.setStatus(
     ...     TeamMembershipStatus.DEACTIVATED, ubuntu_translators.teamowner)
     >>> logout()
     >>> carlos_translators_membership.syncUpdate()
 
=== modified file 'lib/lp/registry/stories/teammembership/xx-expire-subscription.txt'
--- lib/lp/registry/stories/teammembership/xx-expire-subscription.txt	2009-08-13 19:03:36 +0000
+++ lib/lp/registry/stories/teammembership/xx-expire-subscription.txt	2010-01-05 17:55:34 +0000
@@ -32,7 +32,7 @@
     ...     personID=sampleperson.id, teamID=team.id)
     >>> from canonical.launchpad.ftests import ANONYMOUS, login, logout
     >>> login(ANONYMOUS) # login() because we need a zope interaction.
-    >>> tm.setStatus(TeamMembershipStatus.EXPIRED, team.teamowner)
+    >>> ignored = tm.setStatus(TeamMembershipStatus.EXPIRED, team.teamowner)
     >>> logout()
 
 Persist the change:
 
=== modified file 'lib/lp/registry/stories/teammembership/xx-member-renewed-membership.txt'
--- lib/lp/registry/stories/teammembership/xx-member-renewed-membership.txt	2009-12-10 16:15:21 +0000
+++ lib/lp/registry/stories/teammembership/xx-member-renewed-membership.txt	2010-01-05 17:55:34 +0000
@@ -141,7 +141,7 @@
     >>> login('[e-mail]')
     >>> mirror_admins = personset.getByName('ubuntu-mirror-admins')
     >>> landscape_devs = personset.getByName('landscape-developers')
-    >>> mirror_admins.addMember(
+    >>> ignored = mirror_admins.addMember(
     ...     landscape_devs, mirror_admins.teamowner, force_team_add=True)
     >>> membership = getUtility(ITeamMembershipSet).getByPersonAndTeam(
     ...     landscape_devs, mirror_admins)
 
=== modified file 'lib/lp/registry/stories/teammembership/xx-private-membership.txt'
--- lib/lp/registry/stories/teammembership/xx-private-membership.txt	2009-12-01 22:09:05 +0000
+++ lib/lp/registry/stories/teammembership/xx-private-membership.txt	2010-01-05 17:55:34 +0000
@@ -43,7 +43,7 @@
     <...<a href=".../~member" class="sprite person">Gold Member</a>...
 
 
-    >>> find_tag_by_id(browser.contents, 'recently-applied')
+    >>> find_tag_by_id(browser.contents, 'recently-proposed')
     <...<a href=".../~no-priv" class="sprite person">No Privileges Person</a>...
 
     >>> print extract_text(
@@ -257,9 +257,9 @@
 team's home page, they see the private team show up in the "Subteam of"
 section.
 
-    >>> admin_browser.open('http://launchpad.dev/~myteam/+addmember')
-    >>> admin_browser.getControl('New member').value = 'guadamen'
-    >>> admin_browser.getControl('Add Member').click()
+    >>> owner_browser.open('http://launchpad.dev/~myteam/+addmember')
+    >>> owner_browser.getControl('New member').value = 'guadamen'
+    >>> owner_browser.getControl('Add Member').click()
     >>> admin_browser.open(
     ...     'http://launchpad.dev/~guadamen/+invitation/myteam')
     >>> admin_browser.getControl('Accept').click()
 
=== modified file 'lib/lp/registry/stories/vouchers/xx-voucher-redemption.txt'
--- lib/lp/registry/stories/vouchers/xx-voucher-redemption.txt	2009-09-08 15:58:38 +0000
+++ lib/lp/registry/stories/vouchers/xx-voucher-redemption.txt	2010-01-05 17:55:34 +0000
@@ -117,7 +117,8 @@
     >>> commercial_member = getUtility(IPersonSet).getByEmail(
     ...     '[e-mail]')
     >>> celebs = getUtility(ILaunchpadCelebrities)
-    >>> celebs.registry_experts.addMember(commercial_member, commercial_member)
+    >>> ignored = celebs.registry_experts.addMember(
+    ...     commercial_member, commercial_member)
     >>> logout()
 
     >>> browser.open(
 
=== modified file 'lib/lp/registry/stories/webservice/xx-private-membership.txt'
--- lib/lp/registry/stories/webservice/xx-private-membership.txt	2009-10-01 12:30:32 +0000
+++ lib/lp/registry/stories/webservice/xx-private-membership.txt	2010-01-05 17:55:34 +0000
@@ -35,9 +35,10 @@
 Similarly, when a public team is a sub-team of a private team, non-members
 cannot see the private team in the public team's super_team's attribute.
 
-    >>> admin_browser.open('http://launchpad.dev/~myteam/+addmember')
-    >>> admin_browser.getControl('New member').value = 'guadamen'
-    >>> admin_browser.getControl('Add Member').click()
+    >>> owner_browser = setupBrowser(auth="Basic [e-mail]:test")
+    >>> owner_browser.open('http://launchpad.dev/~myteam/+addmember')
+    >>> owner_browser.getControl('New member').value = 'guadamen'
+    >>> owner_browser.getControl('Add Member').click()
     >>> admin_browser.open(
     ...     'http://launchpad.dev/~guadamen/+invitation/myteam')
     >>> admin_browser.getControl('Accept').click()
 
=== modified file 'lib/lp/registry/stories/webservice/xx-source-package.txt'
--- lib/lp/registry/stories/webservice/xx-source-package.txt	2009-12-05 18:37:28 +0000
+++ lib/lp/registry/stories/webservice/xx-source-package.txt	2010-01-05 17:55:34 +0000
@@ -67,7 +67,7 @@
     >>> login('[e-mail]')
     >>> person = factory.makePerson()
     >>> ubuntu_branches = getUtility(ILaunchpadCelebrities).ubuntu_branches
-    >>> ubuntu_branches.addMember(person, ubuntu_branches.teamowner)
+    >>> ignored = ubuntu_branches.addMember(person, ubuntu_branches.teamowner)
     >>> logout()
 
 Then we set the branch on the evolution hoary package:
 
=== modified file 'lib/lp/registry/stories/webservice/xx-structuralsubscription.txt'
--- lib/lp/registry/stories/webservice/xx-structuralsubscription.txt	2009-12-05 18:37:28 +0000
+++ lib/lp/registry/stories/webservice/xx-structuralsubscription.txt	2010-01-05 17:55:34 +0000
@@ -8,7 +8,7 @@
     >>> eric_db = factory.makePerson(name='eric')
     >>> michael_db = factory.makePerson(name='michael')
     >>> pythons_db = factory.makeTeam(name='pythons', owner=michael_db)
-    >>> pythons_db.addMember(eric_db, michael_db)
+    >>> ignored = pythons_db.addMember(eric_db, michael_db)
 
     >>> fooix_db = factory.makeProduct(name='fooix', owner=eric_db)
     >>> fooix01_db = fooix_db.newSeries(eric_db, '0.1', 'Series 0.1')
 
=== modified file 'lib/lp/registry/tests/bug-249185.txt'
--- lib/lp/registry/tests/bug-249185.txt	2009-04-17 10:32:16 +0000
+++ lib/lp/registry/tests/bug-249185.txt	2010-01-05 17:55:34 +0000
@@ -9,18 +9,19 @@
 
     >>> login_person(person)
     >>> dev1 = factory.makePerson()
-    >>> devs.addMember(dev1, person)
+    >>> ignored = devs.addMember(dev1, person)
 
 Beta testers has lp-devs and adjutants as members.
 
-    >>> beta_testers.addMember(devs, person, force_team_add=True)
-    >>> beta_testers.addMember(adjutants, person, force_team_add=True)
+    >>> ignored = beta_testers.addMember(devs, person, force_team_add=True)
+    >>> ignored = beta_testers.addMember(
+    ...     adjutants, person, force_team_add=True)
     >>> dev1.hasParticipationEntryFor(beta_testers)
     True
 
 Adjutants has lp-devs as member.
 
-    >>> adjutants.addMember(devs, person, force_team_add=True)
+    >>> ignored = adjutants.addMember(devs, person, force_team_add=True)
     >>> dev1.hasParticipationEntryFor(adjutants)
     True
 
 
=== modified file 'lib/lp/registry/tests/test_teammembership.py'
--- lib/lp/registry/tests/test_teammembership.py	2009-10-19 16:14:18 +0000
+++ lib/lp/registry/tests/test_teammembership.py	2010-01-05 17:55:34 +0000
@@ -546,8 +546,8 @@
         # Invite team2 as member of team1 and team1 as member of team2. This
         # is not a problem because that won't make any team an active member
         # of the other.
-        self.team1.addMember(self.team2, self.foobar)
-        self.team2.addMember(self.team1, self.foobar)
+        self.team1.addMember(self.team2, self.no_priv)
+        self.team2.addMember(self.team1, self.no_priv)
         team1_on_team2 = getUtility(ITeamMembershipSet).getByPersonAndTeam(
             self.team1, self.team2)
         team2_on_team1 = getUtility(ITeamMembershipSet).getByPersonAndTeam(
@@ -580,7 +580,7 @@
         """No status change can create cyclical participation."""
         # Invite team1 as a member of team3 and forcibly add team2 as member
         # of team1 and team3 as member of team2.
-        self.team3.addMember(self.team1, self.foobar)
+        self.team3.addMember(self.team1, self.no_priv)
         self.team1.addMember(self.team2, self.foobar, force_team_add=True)
         self.team2.addMember(self.team3, self.foobar, force_team_add=True)
 
@@ -593,7 +593,7 @@
             TeamMembershipStatus.APPROVED, self.foobar)
 
     def test_invited_member_can_be_made_admin(self):
-        self.team2.addMember(self.team1, self.foobar)
+        self.team2.addMember(self.team1, self.no_priv)
         team1_on_team2 = getUtility(ITeamMembershipSet).getByPersonAndTeam(
             self.team1, self.team2)
         self.assertEqual(team1_on_team2.status, TeamMembershipStatus.INVITED)
 
=== modified file 'lib/lp/soyuz/browser/tests/archive-views.txt'
--- lib/lp/soyuz/browser/tests/archive-views.txt	2009-11-09 17:59:18 +0000
+++ lib/lp/soyuz/browser/tests/archive-views.txt	2010-01-05 17:55:34 +0000
@@ -957,7 +957,7 @@
 buildd_secret would leak through the public buildlogs.
 
     >>> login('[e-mail]')
-    >>> a_team.addMember(cprov, mark)
+    >>> ignored = a_team.addMember(cprov, mark)
     >>> login('[e-mail]')
 
     >>> view = create_initialized_view(
@@ -1186,7 +1186,7 @@
 continue to be denied in the UI.
 
     >>> login('[e-mail]')
-    >>> ubuntu.main_archive.owner.addMember(cprov, cprov)
+    >>> ignored = ubuntu.main_archive.owner.addMember(cprov, cprov)
     >>> login('[e-mail]')
 
     >>> view = create_initialized_view(
 
=== modified file 'lib/lp/soyuz/browser/tests/build-views.txt'
--- lib/lp/soyuz/browser/tests/build-views.txt	2009-12-11 12:59:08 +0000
+++ lib/lp/soyuz/browser/tests/build-views.txt	2010-01-05 17:55:34 +0000
@@ -153,7 +153,7 @@
     ...     "launchpad-buildd-admins")
     >>> nopriv = getUtility(IPersonSet).getByName("no-priv")
     >>> login("[e-mail]")
-    >>> buildd_admins.addMember(nopriv, nopriv)
+    >>> ignored = buildd_admins.addMember(nopriv, nopriv)
 
     >>> login("[e-mail]")
     >>> failed_build_view = getMultiAdapter(
 
=== modified file 'lib/lp/soyuz/doc/archive.txt'
--- lib/lp/soyuz/doc/archive.txt	2009-12-25 07:54:31 +0000
+++ lib/lp/soyuz/doc/archive.txt	2010-01-05 17:55:34 +0000
@@ -1653,7 +1653,8 @@
 
 But if we make him part of the uploader_team he'll gain access:
 
-    >>> uploader_team.addMember(indirect_uploader, indirect_uploader)
+    >>> ignored = uploader_team.addMember(
+    ...     indirect_uploader, indirect_uploader)
     >>> for ppa in archive_set.getPPAsForUser(indirect_uploader):
     ...     print ppa.displayname
     PPA for Celso Providelo
@@ -2249,7 +2250,7 @@
     >>> login('[e-mail]')
     >>> ubuntu_security = getUtility(IPersonSet).getByName(
     ...     'ubuntu-security')
-    >>> ubuntu_security.addMember(carlos, cprov)
+    >>> ignored = ubuntu_security.addMember(carlos, cprov)
 
     >>> login('[e-mail]')
     >>> check_permission('launchpad.Append', primary)
 
=== modified file 'lib/lp/soyuz/doc/archivepermission.txt'
--- lib/lp/soyuz/doc/archivepermission.txt	2009-09-18 09:29:49 +0000
+++ lib/lp/soyuz/doc/archivepermission.txt	2010-01-05 17:55:34 +0000
@@ -297,7 +297,7 @@
 Now add "[e-mail]" to the techboard team and log in as him.
 
     >>> person = personset.getByEmail("[e-mail]")
-    >>> techboard.addMember(
+    >>> ignored = techboard.addMember(
     ...     person, reviewer=person, status=TeamMembershipStatus.APPROVED,
     ...     force_team_add=True)
     >>> login_person(person)
 
=== modified file 'lib/lp/soyuz/doc/archivesubscriber.txt'
--- lib/lp/soyuz/doc/archivesubscriber.txt	2009-08-13 19:03:36 +0000
+++ lib/lp/soyuz/doc/archivesubscriber.txt	2010-01-05 17:55:34 +0000
@@ -448,7 +448,8 @@
 indirect members that are not themselves groups are included:
 
     >>> launchpad_devs = getUtility(IPersonSet).getByName('launchpad')
-    >>> launchpad_devs.addMember(team_cprov, mark, force_team_add=True)
+    >>> ignored = launchpad_devs.addMember(
+    ...     team_cprov, mark, force_team_add=True)
     >>> subscription = mark.archive.newSubscription(
     ...     launchpad_devs, mark, description=u"LP team too")
     >>> for person in subscription.getNonActiveSubscribers():
 
=== modified file 'lib/lp/soyuz/doc/build-notification.txt'
--- lib/lp/soyuz/doc/build-notification.txt	2009-08-13 19:03:36 +0000
+++ lib/lp/soyuz/doc/build-notification.txt	2010-01-05 17:55:34 +0000
@@ -530,7 +530,7 @@
   >>> LaunchpadZopelessLayer.switchDbUser('launchpad')
   >>> team = factory.makeTeam(owner=cprov, email="[e-mail]")
   >>> mark = failed_candidate.sourcepackagerelease.creator
-  >>> team.addMember(mark, mark)
+  >>> ignored = team.addMember(mark, mark)
   >>> discard = pop_notifications() # Discard team join email.
   >>> team_ppa = factory.makeArchive(owner=team)
   >>> naked_candidate = removeSecurityProxy(failed_candidate)
 
=== modified file 'lib/lp/soyuz/doc/build.txt'
--- lib/lp/soyuz/doc/build.txt	2009-11-24 07:02:21 +0000
+++ lib/lp/soyuz/doc/build.txt	2010-01-05 17:55:34 +0000
@@ -837,7 +837,7 @@
     >>> buildd_admin = factory.makePerson()
     >>> buildd_admins = getUtility(
     ...     IPersonSet).getByName('launchpad-buildd-admins')
-    >>> buildd_admins.addMember(buildd_admin, buildd_admin)
+    >>> ignored = buildd_admins.addMember(buildd_admin, buildd_admin)
     >>> bob_builds = bob.getBuildRecords(user=buildd_admin)
     >>> print_build_details(bob_builds)
     ubuntu-team: hppa build of mozilla-firefox 0.9 in ubuntu warty RELEASE
@@ -845,9 +845,9 @@
     ubuntu-team: i386 build of cdrkit 1.0 in ubuntu breezy-autotest RELEASE
     ...
     ubuntu-team: i386 build of mozilla-firefox 0.9 in ubuntu warty RELEASE
-    ubuntu-team: i386 build of mozilla-firefox 0.9 in ubuntu breezy-autotest 
+    ubuntu-team: i386 build of mozilla-firefox 0.9 in ubuntu breezy-autotest
     RELEASE
-    
+
     >>> bob_builds.count()
     13
 
 
=== modified file 'lib/lp/soyuz/doc/distroseriesqueue-dist-upgrader.txt'
--- lib/lp/soyuz/doc/distroseriesqueue-dist-upgrader.txt	2009-08-31 02:52:41 +0000
+++ lib/lp/soyuz/doc/distroseriesqueue-dist-upgrader.txt	2010-01-05 17:55:34 +0000
@@ -279,7 +279,7 @@
   >>> beta_testers = getUtility(ILaunchpadCelebrities).launchpad_beta_testers
   >>> admin = getUtility(ILaunchpadCelebrities).admin
   >>> name16 = getUtility(IPersonSet).getByName("name16")
-  >>> beta_testers.addMember(name16, admin)
+  >>> ignored = beta_testers.addMember(name16, admin)
 
   >>> foobar_archive = getUtility(IArchiveSet).new(
   ...     distribution=ubuntutest, purpose=ArchivePurpose.PPA,
 
=== modified file 'lib/lp/soyuz/doc/packageset.txt'
--- lib/lp/soyuz/doc/packageset.txt	2009-08-13 15:12:16 +0000
+++ lib/lp/soyuz/doc/packageset.txt	2010-01-05 17:55:34 +0000
@@ -581,7 +581,7 @@
 
 Now add "[e-mail]" to the techboard team and log in as him.
 
-    >>> techboard.addMember(
+    >>> ignored = techboard.addMember(
     ...     person2, reviewer=person2, status=TeamMembershipStatus.APPROVED,
     ...     force_team_add=True)
     >>> login_person(person2)
 
=== modified file 'lib/lp/soyuz/stories/ppa/xx-private-ppa-subscription-stories.txt'
--- lib/lp/soyuz/stories/ppa/xx-private-ppa-subscription-stories.txt	2009-09-14 07:56:54 +0000
+++ lib/lp/soyuz/stories/ppa/xx-private-ppa-subscription-stories.txt	2010-01-05 17:55:34 +0000
@@ -273,7 +273,7 @@
     >>> login('[e-mail]')
     >>> foobar = getUtility(IPersonSet).getByName('name16')
     >>> mark = getUtility(IPersonSet).getByName('mark')
-    >>> launchpad.addMember(mark, foobar)
+    >>> ignored = launchpad.addMember(mark, foobar)
     >>> import transaction
     >>> transaction.commit()
     >>> logout()
 
=== modified file 'lib/lp/soyuz/stories/soyuz/xx-private-builds.txt'
--- lib/lp/soyuz/stories/soyuz/xx-private-builds.txt	2009-09-17 14:45:15 +0000
+++ lib/lp/soyuz/stories/soyuz/xx-private-builds.txt	2010-01-05 17:55:34 +0000
@@ -51,7 +51,7 @@
     ...     IPersonSet).getByName("launchpad-buildd-admins")
     >>> from lp.registry.interfaces.person import (
     ...     TeamMembershipStatus)
-    >>> buildd_admins.addMember(
+    >>> ignored = buildd_admins.addMember(
     ...     name12, reviewer=name12, status=TeamMembershipStatus.APPROVED)
 
     >>> flush_database_updates()
 
=== modified file 'lib/lp/translations/stories/standalone/xx-sourcepackage-export.txt'
--- lib/lp/translations/stories/standalone/xx-sourcepackage-export.txt	2009-09-19 08:30:29 +0000
+++ lib/lp/translations/stories/standalone/xx-sourcepackage-export.txt	2010-01-05 17:55:34 +0000
@@ -97,7 +97,7 @@
     >>> oofy = factory.makePerson(email='[e-mail]',
     ...     name='oofy', password='test', displayname='Oofy Prosser')
     >>> rosetta_admins = personset.getByName('rosetta-admins')
-    >>> rosetta_admins.addMember(oofy, carlos)
+    >>> ignored = rosetta_admins.addMember(oofy, carlos)
 
     >>> gussie = factory.makePerson(email='[e-mail]',
     ...     name='gussie', password='test', displayname='Gussie Fink-Nottle')