restored check for private repos -- was using wrong attribute earlier

api/gitlab_classes.py

@@ -348,7 +348,7 @@ class GitlabProject:
 
     def get_visibility(self) -> str:
         """
-        :return: 'private', etc.
+        :return: 'private', 'public', or 'internal'.
         """
         return self.git_project.visibility
 
@@ -433,14 +433,6 @@ class GitlabProject:
         return commits
 
 
-    # repository_access_level,
-    def get_access_level(self) -> str:
-        """
-        :return: The access level of the repository ('private' is private; 'enabled' is public)
-        """
-        return self.git_project.repository_access_level
-
-
     def get_labels(self) -> Set[str]:
         """
         :return: set of label names
@@ -464,6 +456,7 @@ class GitlabProject:
     #                                                        repo_branches, labels, events, members
     # empty_repo
     # archived
+    # repository_access_level,
     # resolve_outdated_diff_discussions
     # container_registry_enabled
     # issues_enabled

prep-code-review-and-response.py

@@ -81,10 +81,9 @@ def validate_forks():
         project_names: List[str] = list(map(lambda p: p.get_name(), projects))
         if old_group_name in project_names:
             project: GitlabProject = list(filter(lambda p: p.get_name() == old_group_name, projects))[0]
-            # # NOT WORKING AS EXPECTED
-            # if project.get_access_level() != 'private':
-            #     print(f'\t{student.get_canvas_user().get_name()} has not set {old_group_name}.git to "private" '
-            #           f'(access level is {project.get_access_level()}).')
+            if project.get_visibility() != 'private':
+                print(f'\t{student.get_canvas_user().get_name()} has not set {old_group_name}.git to "private" '
+                      f'(access level is {project.get_visibility()}).')
             users: List[GitlabUser] = project.get_all_users()
             usernames: List[str] = list(map(lambda u: u.get_username(), users))
             for TA in TAs: