Colin Watson has proposed merging
~cjwatson/launchpad:fix-optimize-determineFilesToSend into launchpad:master.
Commit message:
Fix database permission errors in test_expire_archive_files
Requested reviews:
Launchpad code reviewers (launchpad-reviewers)
For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/451023
`test_expire_archive_files` uses `SourcePackageRelease.files` to check whether
source files have been expired, but that now preloads related
`LibraryFileContent` rows and the "binaryfile-expire" DB user doesn't have
access to that. However, this only affects the test code, so the simplest fix
is just to switch back to the "launchpad" DB user before making expiry
assertions.
--
Your team Launchpad code reviewers is requested to review the proposed merge of
~cjwatson/launchpad:fix-optimize-determineFilesToSend into launchpad:master.
diff --git a/lib/lp/soyuz/scripts/tests/test_expire_archive_files.py b/lib/lp/soyuz/scripts/tests/test_expire_archive_files.py
index e5c1424..271b020 100644
--- a/lib/lp/soyuz/scripts/tests/test_expire_archive_files.py
+++ b/lib/lp/soyuz/scripts/tests/test_expire_archive_files.py
@@ -13,7 +13,7 @@ from lp.services.log.logger import BufferLogger
from lp.soyuz.scripts.expire_archive_files import ArchiveExpirer
from lp.soyuz.tests.test_publishing import SoyuzTestPublisher
from lp.testing import TestCaseWithFactory
-from lp.testing.dbuser import switch_dbuser
+from lp.testing.dbuser import dbuser, switch_dbuser
from lp.testing.layers import LaunchpadZopelessLayer
@@ -49,8 +49,8 @@ class ArchiveExpiryTestBase(TestCaseWithFactory):
def runScript(self):
"""Run the expiry script and return."""
script = self.getScript()
- switch_dbuser(self.dbuser)
- script.main()
+ with dbuser(self.dbuser):
+ script.main()
def _setUpExpirablePublications(self, archive=None):
"""Helper to set up publications and indexes that are all expirable."""
@@ -320,8 +320,8 @@ class ArchiveExpiryCommonTests:
# will remove the test publications we just created.
self.layer.txn.commit()
script = self.getScript(["--dry-run"])
- switch_dbuser(self.dbuser)
- script.main()
+ with dbuser(self.dbuser):
+ script.main()
self.assertSourceNotExpired(source)
self.assertBinaryNotExpired(binary)
self.assertIndexNotExpired(index)
@@ -366,8 +366,8 @@ class TestPPAExpiry(ArchiveExpiryTestBase, ArchiveExpiryCommonTests):
script.never_expire = [
self.archive.owner.name,
]
- switch_dbuser(self.dbuser)
- script.main()
+ with dbuser(self.dbuser):
+ script.main()
self.assertSourceNotExpired(source)
self.assertBinaryNotExpired(binary)
self.assertIndexNotExpired(index)
@@ -381,8 +381,8 @@ class TestPPAExpiry(ArchiveExpiryTestBase, ArchiveExpiryCommonTests):
script.never_expire = [
"%s/%s" % (self.archive.owner.name, self.archive.name)
]
- switch_dbuser(self.dbuser)
- script.main()
+ with dbuser(self.dbuser):
+ script.main()
self.assertSourceNotExpired(source)
self.assertBinaryNotExpired(binary)
self.assertIndexNotExpired(index)
@@ -393,8 +393,8 @@ class TestPPAExpiry(ArchiveExpiryTestBase, ArchiveExpiryCommonTests):
source, binary, index = self._setUpExpirablePublications(archive=p3a)
script = self.getScript()
script.always_expire = ["%s/%s" % (p3a.owner.name, p3a.name)]
- switch_dbuser(self.dbuser)
- script.main()
+ with dbuser(self.dbuser):
+ script.main()
self.assertSourceExpired(source)
self.assertBinaryExpired(binary)
self.assertIndexExpired(index)
_______________________________________________
Mailing list: https://launchpad.net/~launchpad-reviewers
Post to : [email protected]
Unsubscribe : https://launchpad.net/~launchpad-reviewers
More help : https://help.launchpad.net/ListHelp
