launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:py3-scoping into launchpad:master.

Commit message:
Fix TestProjectGroupExcludeConjoinedMasterSearch for Python 3

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/398378

This failed due to tighter scoping rules.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:py3-scoping into launchpad:master.

diff --git a/lib/lp/bugs/tests/test_bugsearch_conjoined.py b/lib/lp/bugs/tests/test_bugsearch_conjoined.py
index eb7fc01..faf1e7f 100644
--- a/lib/lp/bugs/tests/test_bugsearch_conjoined.py
+++ b/lib/lp/bugs/tests/test_bugsearch_conjoined.py
@@ -237,7 +237,7 @@ class TestProjectGroupExcludeConjoinedMasterSearch(TestSearchBase):
         unexcluded_count = 0
         for bugtask in masters:
             unexcluded_count += 1
-            with person_logged_in(product.owner):
+            with person_logged_in(bugtask.target.owner):
                 bugtask.transitionToStatus(
                     BugTaskStatus.WONTFIX, bugtask.target.owner)
             self.assertEqual(