Merge ~jugmac00/launchpad:add-username-for-oauth-generation-notification into launchpad:master

[Jürgen Gmach]

Add username to OAuth generation notification

[Cristian Gonzalez (cristiangsp)]

Looks good, little comment about the test.

review: Needs Fixing

[Colin Watson (cjwatson)]

review: Approve

[Jürgen Gmach (jugmac00)]

Thank you for the review! I applied all the suggested changes.

[Colin Watson (cjwatson)]

review: Approve

[Cristian Gonzalez (cristiangsp)]

review: Approve

diff --git a/lib/lp/services/oauth/model.py b/lib/lp/services/oauth/model.py
index 469cbad..c7b44ad 100644
--- a/lib/lp/services/oauth/model.py
+++ b/lib/lp/services/oauth/model.py
@@ -394,9 +394,12 @@ class OAuthRequestToken(OAuthBase, StormBase):
 
         # We want to notify the user that this oauth token has been generated
         # for them for security reasons.
+        subject = (
+            'OAuth token generated for %s in Launchpad' % self.person.name
+        )
         self.person.security_field_changed(
-            "OAuth token generated in Launchpad",
-            "A new OAuth token consumer was enabled in Launchpad.")
+            subject, 'A new OAuth token consumer was enabled in Launchpad.'
+        )
 
         self._getStore().remove(self)
         return access_token, secret
diff --git a/lib/lp/services/oauth/tests/test_tokens.py b/lib/lp/services/oauth/tests/test_tokens.py
index c683786..d36f125 100644
--- a/lib/lp/services/oauth/tests/test_tokens.py
+++ b/lib/lp/services/oauth/tests/test_tokens.py
@@ -38,6 +38,7 @@ from lp.testing import (
     verifyObject,
     )
 from lp.testing.layers import DatabaseFunctionalLayer
+from lp.testing.mail_helpers import pop_notifications
 
 
 class TestOAuth(TestCaseWithFactory):
@@ -379,6 +380,18 @@ class TestAccessTokens(TestOAuth):
         access_token.date_expires = self.a_long_time_ago
         self.assertEqual(person.oauth_access_tokens.count(), 0)
 
+    def test_email_notification_message_for_generated_tokens(self):
+        token = self.factory.makeOAuthRequestToken(
+            reviewed_by=self.person,
+        )
+        token.createAccessToken()
+        notifications = pop_notifications()
+
+        expected = (
+            'OAuth token generated for %s in Launchpad' % self.person.name
+        )
+        self.assertEqual(expected, notifications[0]['subject'])
+
 
 class TestHelperFunctions(TestOAuth):