Get 100 latest formatter releases

By default, Github's API returns 30 results per page for 'list'
requests; this cuts off old Google Java Format releases, including
any that run on Java 8.

100 is the maxmimum. There are 42 releases at time of writing, so
this happens to let them all through.

Author: j-emmel

__tests__/releases.test.ts

@@ -105,6 +105,7 @@ function mockOctokitReturnRelease(releaseData: ReleaseData) {
 }
 
 const URL_BASE = "https://api.github.com/repos/google/google-java-format/releases";
+const URL_TAIL = "?per_page=100";
 
 describe('get all release data', () => {
     test('get all release data with API', async () => {
@@ -113,7 +114,7 @@ describe('get all release data', () => {
         const results = await releases.getAllReleaseData();
         expect(results).toEqual(allReleases);
         // IMPORTANT: should not have a trailing slash
-        expectLastCurlCallForUrl(URL_BASE);
+        expectLastCurlCallForUrl(URL_BASE + URL_TAIL);
     });
 
     test('get all release data with API and call to API fails', async () => {
@@ -124,7 +125,7 @@ describe('get all release data', () => {
             .rejects
             .toThrow(error)
         // IMPORTANT: should not have a trailing slash
-        expectLastCurlCallForUrl(URL_BASE);
+        expectLastCurlCallForUrl(URL_BASE + URL_TAIL);
     });
 
     test('get all release data with octokit', async () => {
@@ -148,14 +149,14 @@ describe('get latest release data', () => {
             const result = await releases.getLatestReleaseData(javaVersion);
             expect(result).toEqual(expectedRelease);
             // IMPORTANT: should not have a trailing slash
-            expectLastCurlCallForUrl(URL_BASE);
+            expectLastCurlCallForUrl(URL_BASE + URL_TAIL);
         });
 
         test('when java version is 21, then return release latest', async () => {
             mockApiReturnRelease(dummyReleaseData);
             const result = await releases.getLatestReleaseData(21);
             expect(result).toEqual(dummyReleaseData);
-            expectLastCurlCallForUrl(URL_BASE + "/latest");
+            expectLastCurlCallForUrl(URL_BASE + "/latest" + URL_TAIL);
         });
     });
 
@@ -166,7 +167,7 @@ describe('get latest release data', () => {
         expect(() => releases.getLatestReleaseData(21))
             .rejects
             .toThrow(error);
-        expectLastCurlCallForUrl(URL_BASE + "/latest");
+        expectLastCurlCallForUrl(URL_BASE + "/latest" + URL_TAIL);
     });
 
     describe('get latest release data with octokit', () => {
@@ -195,7 +196,7 @@ describe('get release by name', () => {
         const result = await releases.getReleaseDataByName('dummy-release-data');
         expect(result).toEqual(dummyReleaseData);
         // IMPORTANT: should not have a trailing slash
-        expectLastCurlCallForUrl(URL_BASE);
+        expectLastCurlCallForUrl(URL_BASE + URL_TAIL);
     });
 
     test('get release by name (non-existing)', async () => {
@@ -204,6 +205,6 @@ describe('get release by name', () => {
         const result = await releases.getReleaseDataByName('non-existing-data');
         expect(result).toBeUndefined();
         // IMPORTANT: should not have a trailing slash
-        expectLastCurlCallForUrl(URL_BASE);
+        expectLastCurlCallForUrl(URL_BASE + URL_TAIL);
     });
 });

dist/index.js

@@ -32713,15 +32713,15 @@ class Releases {
         this.octokit = octokit;
     }
     async callReleasesApi(pathParameter) {
-        const url = `${Releases.apiReleases}${pathParameter || ''}`;
+        const url = `${Releases.apiReleases}${pathParameter || ''}?per_page=100`;
         const response = await this.execute('curl', ['-sL', url], { ignoreReturnCode: false });
         return JSON.parse(response.stdOut);
     }
     async getAllReleaseData() {
         if (!this.octokit) {
             return this.callReleasesApi();
         }
-        const params = { owner: const_1.repositoryOwner, repo: const_1.repositoryName };
+        const params = { owner: const_1.repositoryOwner, repo: const_1.repositoryName, per_page: 100 };
         const response = await this.octokit.rest.repos.listReleases(params);
         return response.data;
     }

src/releases.ts

@@ -21,7 +21,7 @@ export class Releases {
     private async callReleasesApi(pathParameter: string | number): Promise<ReleaseData>;
     private async callReleasesApi(pathParameter?: number): Promise<ReleaseData | ReleaseData[]>;
     private async callReleasesApi(pathParameter?: string | number): Promise<ReleaseData | ReleaseData[]> {
-        const url = `${Releases.apiReleases}${pathParameter || ''}`;
+        const url = `${Releases.apiReleases}${pathParameter || ''}?per_page=100`;
         const response = await this.execute('curl', ['-sL', url], { ignoreReturnCode: false });
         return JSON.parse(response.stdOut);
     }
@@ -30,7 +30,7 @@ export class Releases {
         if (!this.octokit) {
             return this.callReleasesApi();
         }
-        const params = { owner: GJF_REPO_OWNER, repo: GJF_REPO_NAME };
+        const params = { owner: GJF_REPO_OWNER, repo: GJF_REPO_NAME, per_page: 100 };
         const response = await this.octokit.rest.repos.listReleases(params);
         return response.data;
     }