Enable setting of the new global_write= permissions.

Author: LTLA

src/pygobbler/fetch_permissions.py

@@ -41,6 +41,11 @@ def fetch_permissions(project: str, registry: str, url: str, force_remote: bool
           authorization does not expire.
         - ``trusted`` (optional), whether the uploader is trusted. If not
           provided, defaults to false.
+
+        The top-level dictionary may also contain `global_write`, a boolean
+        indicating whether global writes are supported. If true, any user may
+        create a new asset in this project, and each user can upload new
+        versions to any asset they created under this mode.
     """
     if os.path.exists(registry) and not force_remote:
         with open(os.path.join(registry, project, "..permissions"), "r") as f:

src/pygobbler/set_permissions.py

@@ -3,7 +3,15 @@
 from . import fetch_permissions
 
 
-def set_permissions(project: str, registry: str, staging: str, url: str, owners: Optional[List] = None, uploaders: Optional[Dict] = None, append: bool = True):
+def set_permissions(
+    project: str,
+    registry: str,
+    staging: str,
+    url: str,
+    owners: Optional[List] = None,
+    uploaders: Optional[Dict] = None,
+    global_write: Optional[bool] = None,
+    append: bool = True):
     """
     Set the owner and uploader permissions for a project.
 
@@ -30,6 +38,12 @@ def set_permissions(project: str, registry: str, staging: str, url: str, owners:
             :py:func:`~.fetch_permissions` for  the expected format. If None,
             no change is made to the existing uploaders.
 
+        global_write:
+            Whether to enable global writes for this project, see the
+            ``global_write`` field in the return value of
+            :py:func:`~.fetch_permissions` for more details. If None, no change
+            is made to the global write status.
+
         append:
             Whether ``owners`` and ``uploaders`` should be appended to the
             existing owners and uploaders, respectively. If False, the
@@ -51,4 +65,7 @@ def set_permissions(project: str, registry: str, staging: str, url: str, owners:
         if uploaders is not None:
             perms["uploaders"] = uploaders
 
+    if global_write is not None:
+        perms["global_write"] = global_write
+
     ut.dump_request(staging, url, "set_permissions", { "project": project, "permissions": perms })

src/pygobbler/start_gobbler.py

@@ -9,7 +9,13 @@
 test_port = None
 test_process = None
 
-def start_gobbler(staging: Optional[str] = None, registry: Optional[str] = None, port: Optional[int] = None, wait: float = 1, version: str = "0.3.2", overwrite: bool = False) -> Tuple[bool, str, str, str]:
+def start_gobbler(
+    staging: Optional[str] = None,
+    registry: Optional[str] = None,
+    port: Optional[int] = None,
+    wait: float = 1,
+    version: str = "0.3.7",
+    overwrite: bool = False) -> Tuple[bool, str, str, str]:
     """
     Start a test Gobbler service.
 

tests/test_set_permissions.py

@@ -20,6 +20,7 @@ def test_set_permissions():
     assert len(perms["uploaders"]) == 1
     assert perms["uploaders"][0]["id"] == "lawremi"
     assert perms["uploaders"][0]["until"] == until
+    assert not "global_write" in perms
 
     # Checking uploader appending, while also checking owners=NULL.
     pyg.set_permissions("test-perms", uploaders=[ { "id": "ArtifactDB-bot", "trusted": True } ], staging=staging, url=url, registry=registry)
@@ -29,6 +30,7 @@ def test_set_permissions():
     assert perms["uploaders"][0]["id"] == "lawremi"
     assert perms["uploaders"][1]["id"] == "ArtifactDB-bot"
     assert perms["uploaders"][1]["trusted"]
+    assert not "global_write" in perms
 
     # Checking union of owners, and also that uploaders=NULL works.
     pyg.set_permissions("test-perms", owners=[ "PeteHaitch", "LTLA" ], staging=staging, url=url, registry=registry)
@@ -47,3 +49,8 @@ def test_set_permissions():
     perms = pyg.fetch_permissions("test-perms", registry=registry, url=url)
     assert perms["owners"] == [ "LTLA" ]
     assert len(perms["uploaders"]) == 0
+
+    # Checking that it works with global writes enabled.
+    pyg.set_permissions("test-perms", global_write=True, staging=staging, url=url, registry=registry)
+    perms = pyg.fetch_permissions("test-perms", registry=registry, url=url)
+    assert perms["global_write"]