launchpad-reviewers team mailing list archive

[Jürgen Gmach <mp+473381@xxxxxxxxxxxxxxxxxx>]

Jürgen Gmach has proposed merging ~jugmac00/launchpad:fix-permission-issue-for-rockfiles into launchpad:master.

Commit message:
Fix missing permission for rockfile

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~jugmac00/launchpad/+git/launchpad/+merge/473381
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~jugmac00/launchpad:fix-permission-issue-for-rockfiles into launchpad:master.

diff --git a/database/schema/security.cfg b/database/schema/security.cfg
index a906e9a..7f634e6 100644
--- a/database/schema/security.cfg
+++ b/database/schema/security.cfg
@@ -1555,6 +1555,7 @@ public.questionjob                      = SELECT, INSERT
 public.questionsubscription             = SELECT
 public.revisionstatusartifact           = SELECT, INSERT
 public.revisionstatusreport             = SELECT, INSERT, UPDATE
+public.rockfile                         = SELECT, INSERT, UPDATE
 public.section                          = SELECT, INSERT
 public.sectionselection                 = SELECT
 public.sharingjob                       = SELECT, INSERT, UPDATE
diff --git a/lib/lp/archiveuploader/tests/test_rockrecipeupload.py b/lib/lp/archiveuploader/tests/test_rockrecipeupload.py
index 28a12cb..75ae4d1 100644
--- a/lib/lp/archiveuploader/tests/test_rockrecipeupload.py
+++ b/lib/lp/archiveuploader/tests/test_rockrecipeupload.py
@@ -39,7 +39,7 @@ class TestRockRecipeUploads(TestUploadProcessorBase):
             self.layer.txn, builds=True
         )
 
-    def test_sets_build_and_state_123(self):
+    def test_sets_build_and_state(self):
         # The upload processor uploads files and sets the correct status.
         self.assertFalse(self.build.verifySuccessfulUpload())
         upload_dir = os.path.join(