fixup 4f4ba

Author: LinuxMercedes

assigner/backends/gitlab.py

@@ -282,10 +282,11 @@ def add_member(self, user_id, level):
 
     def edit_member(self, user_id, level):
         payload = {
-            "id": self.id,
-            "user_id": user_id,
+            #"id": self.id,
+            #"user_id": user_id,
             "access_level": level.value
         }
+        logging.debug(payload)
         return self._gl_put(
             "/projects/{}/members/{}".format(self.id, user_id), payload
         )
@@ -358,10 +359,10 @@ def unprotect(self, branch="master"):
                             .format(self.id, branch))
 
     def unlock(self, student_id: str) -> None:
-        self.edit_member(student_id, Access.reporter)
+        self.edit_member(student_id, Access.developer)
 
     def lock(self, student_id: str) -> None:
-        self.edit_member(student_id, Access.developer)
+        self.edit_member(student_id, Access.reporter)
 
     def _gl_get(self, path, params={}):
         return self.__class__._cls_gl_get(

assigner/backends/mock.py

@@ -213,10 +213,10 @@ def unarchive(self):
         return MagicMock()
 
     def unlock(self, student_id: str) -> None:
-        self.edit_member(student_id, Access.reporter)
+        self.edit_member(student_id, Access.developer)
 
     def lock(self, student_id: str) -> None:
-        self.edit_member(student_id, Access.developer)
+        self.edit_member(student_id, Access.reporter)
 
     # NOTE: these are not the same defaults that Mock uses
     def protect(self, branch="master", developer_push=True, developer_merge=True):

assigner/commands/lock.py

@@ -14,7 +14,7 @@ def lock(args):
     #pylint: disable=no-value-for-parameter
     return manage_repos(
         args,
-        lambda repo, student: repo.unlock(student["id"])
+        lambda repo, student: repo.lock(student["id"])
     )
 
 

assigner/commands/unlock.py

@@ -13,7 +13,7 @@ def unlock(args):
     #pylint: disable=no-value-for-parameter
     return manage_repos(
         args,
-        lambda repo, student: repo.lock(student)
+        lambda repo, student: repo.unlock(student["id"])
     )