launchpad-reviewers team mailing list archive

[Kristian Glass <kristian.glass@xxxxxxxxxxxxx>]

Kristian Glass has proposed merging ~doismellburning/launchpad:ogp into launchpad:master.

Commit message:
Update expected email address counts to account for OpenGraph tags
    
Fixes test failure introduced in
62f99f9768eae07feb0bdc4c24e4d6e5fe1a00dc - OpenGraph metadata tags mean
there's an additional instance of a mentioned email address on a bug
page


Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~doismellburning/launchpad/+git/launchpad/+merge/385741
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~doismellburning/launchpad:ogp into launchpad:master.

diff --git a/lib/lp/bugs/browser/tests/test_bug_views.py b/lib/lp/bugs/browser/tests/test_bug_views.py
index d6c4eaf..149bf51 100644
--- a/lib/lp/bugs/browser/tests/test_bug_views.py
+++ b/lib/lp/bugs/browser/tests/test_bug_views.py
@@ -186,7 +186,7 @@ class TestEmailObfuscated(BrowserTestCase):
     def test_user_sees_email_address(self):
         """A logged-in user can see the email address on the page."""
         browser = self.getBrowserForBugWithEmail(no_login=False)
-        self.assertEqual(7, browser.contents.count(self.email_address))
+        self.assertEqual(8, browser.contents.count(self.email_address))
 
     def test_anonymous_sees_not_email_address(self):
         """The anonymous user cannot see the email address on the page."""