Support adding parent teams

Support adding a parent team to a given team. Also support changing the
privacy of a team.

Author: sigmavirus24

src/github3/orgs.py

@@ -118,20 +118,39 @@ def delete(self):
         return self._boolean(self._delete(self._api), 204, 404)
 
     @requires_auth
-    def edit(self, name, permission=""):
+    def edit(
+        self,
+        name: str,
+        permission: str = "",
+        parent_team_id: t.Optional[int] = None,
+        privacy: t.Optional[str] = None,
+    ):
         """Edit this team.
 
         :param str name:
             (required), the new name of this team
         :param str permission:
             (optional), one of ('pull', 'push', 'admin')
+            .. deprecated:: 3.0.0
+
+                This was deprecated by the GitHub API.
+        :param int parent_team_id:
+            (optional), id of the parent team for this team
+        :param str privacy:
+            (optional), one of "closed" or "secret"
         :returns:
             True if successful, False otherwise
         :rtype:
             bool
         """
         if name:
-            data = {"name": name, "permission": permission}
+            data = {"name": name}
+            if permission:
+                data["permission"] = permission
+            if parent_team_id is not None:
+                data["parent_team_id"] = parent_team_id
+            if privacy in {"closed", "secret"}:
+                data["privacy"] = privacy
             json = self._json(self._patch(self._api, data=dumps(data)), 200)
             if json:
                 self._update_attributes(json)

tests/cassettes/Team_edit.json

@@ -48,14 +48,16 @@ def test_edit(self):
         with self.recorder.use_cassette(cassette_name):
             o = self.get_organization()
             # Create a new team to play with
-            t = o.create_team("edit-me")
+            t = o.create_team("edit-me", privacy="closed")
+            p = o.create_team("parent", privacy="closed")
             assert isinstance(t, github3.orgs.Team)
             # Edit the new team
-            assert t.edit("delete-me") is True
+            assert t.edit("delete-me", parent_team_id=p.id) is True
             # Assert that the name has changed
             assert t.name == "delete-me"
             # Get rid of it, we don't need it.
             assert t.delete() is True
+            assert p.delete() is True
 
     def test_has_repository(self):
         """Show that a user can check of a team has access to a repository."""

tests/integration/test_orgs_team.py

@@ -33,10 +33,15 @@ def test_delete(self):
 
     def test_edit(self):
         """Show that a user can edit a team."""
-        self.instance.edit("name", "admin")
+        self.instance.edit("name", "admin", 1234)
 
         self.patch_called_with(
-            url_for(), data={"name": "name", "permission": "admin"}
+            url_for(),
+            data={
+                "name": "name",
+                "permission": "admin",
+                "parent_team_id": 1234,
+            },
         )
 
     def test_has_repository(self):