chore: Update Node.js version to 20 in action.yml and package.json

ALJAZEERAPLUS · Jul 4, 2024 · b5fa3f8 · b5fa3f8
1 parent 1e5c3f9
commit b5fa3f8
Show file tree

Hide file tree

Showing 10 changed files with 3,312 additions and 2,643 deletions.
diff --git a/README.md b/README.md
@@ -31,7 +31,7 @@ jobs:
       uses: ALJAZEERAPLUS/sauce-labs-storage-action@v2
       with:
         endpoint-action: upload
-        file-path: app/build/outputs/apk/release/app-release.apk
+        upload-file-path: app/build/outputs/apk/release/app-release.apk
         sauce-labs-username: ${{ secrets.SAUCELABS_USERNAME }}
         sauce-labs-access-key: ${{ secrets.SAUCELABS_ACCESS_KEY }}
 ```

diff --git a/__tests__/endpoints/storage/files.test.js → ...ts__/endpoints/storage/fetchFiles.test.js b/__tests__/endpoints/storage/files.test.js → ...ts__/endpoints/storage/fetchFiles.test.js
@@ -1,6 +1,6 @@
 const axios = require('axios');
 const core = require('@actions/core');
-const files = require('../../../src/endpoints/storage/files');
+const fetchFiles = require('../../../src/endpoints/storage/fetchFiles');
 
 jest.mock('axios');
 jest.mock('@actions/core');
@@ -22,15 +22,16 @@ describe('Files Endpoint', () => {
     });
 
     test('sends correct authorization header', async () => {
-      await files();
+      await fetchFiles();
 
-      expect(axios.get).toHaveBeenCalledWith(expect.anything(), {
+      expect(axios.get).toHaveBeenCalledWith('https://api.region.saucelabs.com/v1/storage/files', {
         headers: {
           Authorization: `Basic ${Buffer.from('username:access-key').toString('base64')}`,
         },
         params: {
           kind: 'ios',
           per_page: 100,
+          page: 1,
         },
       });
     });
@@ -42,11 +43,12 @@ describe('Files Endpoint', () => {
           items: [{
             id: '1234',
             kind: 'android',
+            page: 1,
           }],
         },
       });
 
-      await files();
+      await fetchFiles();
 
       expect(axios.get).toHaveBeenCalled();
       expect(core.setOutput).toHaveBeenCalledWith('file-id', '1234');
@@ -61,18 +63,18 @@ describe('Files Endpoint', () => {
         },
       });
 
-      await files();
+      await fetchFiles();
 
       expect(axios.get).toHaveBeenCalled();
-      expect(core.setFailed).toHaveBeenCalledWith('No file found');
+      expect(core.setFailed).toHaveBeenCalledWith('File not found');
     });
 
     test('handles unexpected response code', async () => {
       axios.get.mockResolvedValue({
         status: 404,
       });
 
-      await files();
+      await fetchFiles();
 
       expect(axios.get).toHaveBeenCalled();
       expect(core.setFailed).toHaveBeenCalledWith('Unexpected error: Unexpected response code: 404');
@@ -84,7 +86,7 @@ describe('Files Endpoint', () => {
         data: {},
       });
 
-      await files();
+      await fetchFiles();
 
       expect(axios.get).toHaveBeenCalled();
       expect(core.setFailed).toHaveBeenCalledWith('Unexpected error: Unexpected response structure');
@@ -93,7 +95,7 @@ describe('Files Endpoint', () => {
     test('handles exception', async () => {
       axios.get.mockRejectedValue(new Error('Network error'));
 
-      await files();
+      await fetchFiles();
 
       expect(axios.get).toHaveBeenCalled();
       expect(core.setFailed).toHaveBeenCalledWith('Unexpected error: Network error');
@@ -143,7 +145,7 @@ describe('Files Endpoint', () => {
       mockValues['get-file-id-app-version'] = version;
       core.getInput.mockImplementation((name) => mockValues[name]);
 
-      await files();
+      await fetchFiles();
 
       expect(core.setOutput).toHaveBeenCalledWith('file-id', expected);
     });
@@ -157,7 +159,7 @@ describe('Files Endpoint', () => {
       mockValues['get-file-id-app-build-number'] = build;
       core.getInput.mockImplementation((name) => mockValues[name]);
 
-      await files();
+      await fetchFiles();
 
       expect(core.setOutput).toHaveBeenCalledWith('file-id', expected);
     });
@@ -170,7 +172,7 @@ describe('Files Endpoint', () => {
       mockValues['get-file-id-app-description'] = description;
       core.getInput.mockImplementation((name) => mockValues[name]);
 
-      await files();
+      await fetchFiles();
 
       expect(core.setOutput).toHaveBeenCalledWith('file-id', expected);
     });

diff --git a/__tests__/index.test.js b/__tests__/index.test.js
@@ -1,7 +1,7 @@
 /* eslint-disable global-require */
 const core = require('@actions/core');
 const upload = require('../src/endpoints/storage/upload');
-const files = require('../src/endpoints/storage/files');
+const fetchFiles = require('../src/endpoints/storage/fetchFiles');
 const index = require('../src/index');
 
 jest.mock('@actions/core', () => ({
@@ -10,7 +10,7 @@ jest.mock('@actions/core', () => ({
   info: jest.fn(),
 }));
 jest.mock('../src/endpoints/storage/upload');
-jest.mock('../src/endpoints/storage/files');
+jest.mock('../src/endpoints/storage/fetchFiles');
 
 describe('Action execution', () => {
   afterEach(() => {
@@ -23,15 +23,15 @@ describe('Action execution', () => {
     await index.main();
 
     expect(upload).toHaveBeenCalled();
-    expect(files).not.toHaveBeenCalled();
+    expect(fetchFiles).not.toHaveBeenCalled();
   });
 
   it('should call files when endpoint-action is "get-file-id"', async () => {
     core.getInput.mockReturnValue('get-file-id');
 
     await index.main();
 
-    expect(files).toHaveBeenCalled();
+    expect(fetchFiles).toHaveBeenCalled();
     expect(upload).not.toHaveBeenCalled();
   });
 

diff --git a/action.yml b/action.yml
@@ -41,5 +41,5 @@ branding:
   icon: upload-cloud
   color: red
 runs:
-  using: node16
+  using: node20
   main: dist/index.js