Fix pagination logic to use next link (#418)

* Fix pagination logic to use `next` link

Fixes #414

* Fix tests

Author: kevmoo

CHANGELOG.md

@@ -1,7 +1,8 @@
-## 9.25.0-wip
+## 9.25.0
 
 * Require Dart 3.5
 * Require `package:http` `^1.0.0`.
+* Fix pagination logic to use `next` link.
 
 ## 9.24.0
 

lib/src/common/util/pagination.dart

@@ -28,9 +28,6 @@ class PaginationHelper {
       params = Map.from(params);
     }
 
-    var page = params['page'] ?? 1;
-    params['page'] = page;
-
     while (true) {
       http.Response response;
       try {
@@ -70,7 +67,8 @@ class PaginationHelper {
         break;
       }
 
-      params['page'] = ++page;
+      path = next;
+      params = null;
     }
   }
 

pubspec.yaml

@@ -1,5 +1,5 @@
 name: github
-version: 9.25.0-wip
+version: 9.25.0
 description: A high-level GitHub API Client Library that uses Github's v3 API
 homepage: https://github.com/SpinlockLabs/github.dart
 

test/common/repos_service_test.dart

@@ -16,7 +16,7 @@ void main() {
   test('listCommits', () async {
     final repositories = create((request) async {
       expect(request.url.path, '/repos/${slug.fullName}/commits');
-      expect(request.url.query, 'page=1');
+      expect(request.url.query, isEmpty);
 
       return Response(listCommits, StatusCodes.OK);
     });
@@ -29,7 +29,7 @@ void main() {
       expect(request.url.path, '/repos/${slug.fullName}/commits');
       expect(
         request.url.query,
-        'author=octocat&committer=octodog&sha=abc&path=%2Fpath&since=2022-02-22T00%3A00%3A00.000&until=2023-02-22T00%3A00%3A00.000&page=1',
+        'author=octocat&committer=octodog&sha=abc&path=%2Fpath&since=2022-02-22T00%3A00%3A00.000&until=2023-02-22T00%3A00%3A00.000',
       );
       return Response(listCommits, StatusCodes.OK);
     });

test/git_test.dart

@@ -172,7 +172,7 @@ void main() {
   test('code search', () async {
     nock(fakeApiUrl)
         .get(
-            '/search/code?q=search%20repo%3ASpinlockLabs%2Fgithub.dart%20in%3Afile&per_page=20&page=1')
+            '/search/code?q=search%20repo%3ASpinlockLabs%2Fgithub.dart%20in%3Afile&per_page=20')
         .reply(200, nocked.searchResults);
 
     final results = (await github.search