Keep up with the API

sigmavirus24 · sigmavirus24 · commit 7568e0829d40 · 2012-12-05T16:57:09.000-05:00
github/developer.github.com@f4889b9 github/developer.github.com@8d1db91
diff --git a/github3/api.py b/github3/api.py
@@ -296,6 +296,7 @@ def search_issues(owner, repo, state, keyword):
     :param str repo: (required)
     :param str state: (required), accepted values: ('open', 'closed')
     :param str keyword: (required), what to search for
+    :param int start_page: (optional), page to get (results come 100/page)
     :returns: list of :class:`LegacyIssue <github3.legacy.LegacyIssue>`\ s
     """
     return gh.search_issues(owner, repo, state, keyword)
@@ -304,10 +305,9 @@ def search_issues(owner, repo, state, keyword):
 def search_repos(keyword, **params):
     """Search all repositories by keyword.
 
-    :param keyword: (required)
-    :type keyword: str
-    :param params: (optional), filter by language and/or start_page
-    :type params: dict
+    :param str keyword: (required)
+    :param str language: (optional), language to filter by
+    :param int start_page: (optional), page to get (results come 100/page)
     :returns: list of :class:`LegacyRepo <github3.legacy.LegacyRepo>`\ s
     """
     return gh.search_repos(keyword, **params)
@@ -317,6 +317,7 @@ def search_users(keyword):
     """Search all users by keyword.
 
     :param str keyword: (required)
+    :param int start_page: (optional), page to get (results come 100/page)
     :returns: list of :class:`LegacyUser <github3.legacy.LegacyUser>`\ s
     """
     return gh.search_users(keyword)
diff --git a/github3/gists.py b/github3/gists.py
@@ -154,7 +154,7 @@ def fork(self):
 
         :returns: :class:`Gist <Gist>` if successful, ``None`` otherwise
         """
-        url = self._api + '/fork'
+        url = self._build_url('forks', base_url=self._api)
         json = self._json(self._post(url), 201)
         return Gist(json, self) if json else None
 
diff --git a/github3/github.py b/github3/github.py
@@ -760,41 +760,51 @@ def repository(self, owner, repository):
             json = self._json(self._get(url), 200)
         return Repository(json, self) if json else None
 
-    def search_issues(self, owner, repo, state, keyword):
+    def search_issues(self, owner, repo, state, keyword, start_page=0):
         """Find issues by state and keyword.
 
         :param str owner: (required)
         :param str repo: (required)
         :param str state: (required), accepted values: ('open', 'closed')
         :param str keyword: (required), what to search for
+        :param int start_page: (optional), page to get (results come 100/page)
         :returns: list of :class:`LegacyIssue <github3.legacy.LegacyIssue>`\ s
         """
+        params = {} if int(start_page) > 0 else {'start_page': int(start_page)}
         url = self._build_url('legacy', 'issues', 'search', owner, repo,
                               state, keyword)
-        json = self._json(self._get(url), 200)
+        json = self._json(self._get(url, params=params), 200)
         issues = json.get('issues', [])
         return [LegacyIssue(l, self) for l in issues]
 
-    def search_repos(self, keyword, **params):
+    def search_repos(self, keyword, language='', start_page=0):
         """Search all repositories by keyword.
 
         :param str keyword: (required)
-        :param dict params: (optional), filter by language and/or start_page
+        :param str language: (optional), language to filter by
+        :param int start_page: (optional), page to get (results come 100/page)
         :returns: list of :class:`LegacyRepo <github3.legacy.LegacyRepo>`\ s
         """
         url = self._build_url('legacy', 'repos', 'search', keyword)
+        params = {}
+        if language:
+            params['language'] = language
+        if start_page > 0:
+            params['start_page'] = start_page
         json = self._json(self._get(url, params=params), 200)
         repos = json.get('repositories', [])
         return [LegacyRepo(r, self) for r in repos]
 
-    def search_users(self, keyword):
+    def search_users(self, keyword, start_page=0):
         """Search all users by keyword.
 
         :param str keyword: (required)
+        :param int start_page: (optional), page to get (results come 100/page)
         :returns: list of :class:`LegacyUser <github3.legacy.LegacyUser>`\ s
         """
+        params = {} if int(start_page) > 0 else {'start_page': int(start_page)}
         url = self._build_url('legacy', 'user', 'search', str(keyword))
-        json = self._json(self._get(url), 200)
+        json = self._json(self._get(url, params=params), 200)
         users = json.get('users', [])
         return [LegacyUser(u, self) for u in users]
 
diff --git a/github3/repos.py b/github3/repos.py
@@ -1521,7 +1521,8 @@ def test(self):
 
         :returns: bool
         """
-        return self._boolean(self._post(self._api + '/test'), 204, 404)
+        url = self._build_url('tests', base_url=self._api)
+        return self._boolean(self._post(url), 204, 404)
 
 
 class RepoTag(GitHubObject):
