Merge pull request #255 from opensafely-core/iaindillingham/test-in-c…

…heckout-copy-of-repository-permissions-yaml

Test in-checkout copy of `repository-permissions.yaml`

opensafely/check.py

@@ -204,6 +204,13 @@ def get_datasource_permissions(permissions_url):
     return permissions
 
 
+def get_local_permissions():
+    path = Path(Path(PERMISSIONS_URL).name)
+    yaml = YAML()
+    permissions = yaml.load(path.read_text())
+    return permissions
+
+
 def get_repository_name(continue_on_error):
     if "GITHUB_REPOSITORY" in os.environ:
         return os.environ["GITHUB_REPOSITORY"]

tests/test_check.py

@@ -348,16 +348,23 @@ def test_check(
         validate_pass(capsys, continue_on_error)
 
 
-def test_repository_permissions_yaml():
+def get_datasource_permissions():
     try:
         permissions = check.get_datasource_permissions(check.PERMISSIONS_URL)
+        return permissions
     except RequestException as e:
         # This test should always pass on main, but if we've renamed the file
         # on the branch, it will fail before it's merged
         branch = subprocess.run(["git", "rev-parse", "--abbrev-ref", "HEAD"])
         if branch != "main" and "Error 404" in str(e):
             pytest.xfail("Permissions file does not exist on main yet")
 
+
+@pytest.mark.parametrize(
+    "get_permissions", [get_datasource_permissions, check.get_local_permissions]
+)
+def test_repository_permissions_yaml(get_permissions):
+    permissions = get_permissions()
     assert permissions, "empty permissions file"
     assert type(permissions) == CommentedMap, "invalid permissions file"
     for k, v in permissions.items():