launchpad-reviewers team mailing list archive

[Andrey Fedoseev <mp+430130@xxxxxxxxxxxxxxxxxx>]

Andrey Fedoseev has proposed merging ~andrey-fedoseev/launchpad:packaging-security into launchpad:master.

Commit message:
Fix another failing test caused by the recent changes in `Packaging` security

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~andrey-fedoseev/launchpad/+git/launchpad/+merge/430130
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~andrey-fedoseev/launchpad:packaging-security into launchpad:master.

diff --git a/lib/lp/code/model/tests/test_branchtarget.py b/lib/lp/code/model/tests/test_branchtarget.py
index 1b74d7a..b73a527 100644
--- a/lib/lp/code/model/tests/test_branchtarget.py
+++ b/lib/lp/code/model/tests/test_branchtarget.py
@@ -434,9 +434,10 @@ class TestProductBranchTarget(TestCaseWithFactory, BaseBranchTargetTests):
         # Packages that are linked to the products are mergeable.
         branch = self.factory.makePackageBranch()
         # Link it up.
-        branch.sourcepackage.setPackaging(
-            self.original.development_focus, branch.owner
-        )
+        with person_logged_in(branch.sourcepackage.owner):
+            branch.sourcepackage.setPackaging(
+                self.original.development_focus, branch.owner
+            )
         self.assertTrue(self.target.areBranchesMergeable(branch.target))
 
     def test_default_merge_target(self):