GitHubClient: Extract team_by_name() method

Author: Turbo87

src/github.rs

@@ -25,6 +25,16 @@ impl GitHubClient {
         self.request("/user", auth)
     }
 
+    pub fn team_by_name(
+        &self,
+        org_name: &str,
+        team_name: &str,
+        auth: &AccessToken,
+    ) -> AppResult<GitHubTeam> {
+        let url = format!("/orgs/{}/teams/{}", org_name, team_name);
+        self.request(&url, auth)
+    }
+
     pub fn team_membership(
         &self,
         org_id: i32,
@@ -106,6 +116,18 @@ pub struct GithubUser {
     pub name: Option<String>,
 }
 
+#[derive(Debug, Deserialize, Copy)]
+pub struct GitHubOrganization {
+    pub id: i32, // unique GH id (needed for membership queries)
+}
+
+#[derive(Debug, Deserialize)]
+pub struct GitHubTeam {
+    pub id: i32,              // unique GH id (needed for membership queries)
+    pub name: Option<String>, // Pretty name
+    pub organization: GitHubOrganization,
+}
+
 #[derive(Debug, Deserialize)]
 pub struct GitHubTeamMembership {
     pub state: String,

src/models/team.rs

@@ -138,23 +138,10 @@ impl Team {
             )));
         }
 
-        #[derive(Deserialize)]
-        struct GithubOrganization {
-            id: i32, // unique GH id (needed for membership queries)
-        }
-
-        #[derive(Deserialize)]
-        struct GithubTeam {
-            id: i32,              // unique GH id (needed for membership queries)
-            name: Option<String>, // Pretty name
-            organization: GithubOrganization,
-        }
-
-        let url = format!("/orgs/{}/teams/{}", org_name, team_name);
         let token = AccessToken::new(req_user.gh_access_token.clone());
         let team = app
             .github
-            .request::<GithubTeam>(&url, &token)
+            .team_by_name(org_name, team_name, &token)
             .map_err(|_| {
                 cargo_err(&format_args!(
                     "could not find the github team {}/{}",