Adjustments

Author: JamesIves

.github/workflows/integration.yml

@@ -23,7 +23,7 @@ jobs:
           persist-credentials: false
 
       - name: Fetch Data 📦
-        uses: JamesIves/fetch-api-data-action@releases/v2
+        uses: JamesIves/fetch-api-data-action@v2
         with:
           endpoint: https://jsonplaceholder.typicode.com/todos/1
           save-location: fetch-api-data-custom
@@ -37,3 +37,29 @@ jobs:
           folder: fetch-api-data-custom
           target-folder: data
           ssh-key: ${{ secrets.DEPLOY_KEY }}
+
+  refresh-api-data-modified-formatting-encoding:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout 🛎️
+        uses: actions/checkout@v2.4.0
+        with:
+          persist-credentials: false
+
+      - name: Fetch Data 📦
+        uses: JamesIves/fetch-api-data-action@v2
+        with:
+          endpoint: https://jsonplaceholder.typicode.com/todos/1
+          save-location: fetch-api-data-custom-encoding
+          save-name: todo2
+          encoding: hex
+          format: txt
+          retry: true
+
+      - name: Build and Deploy Repo 🚀
+        uses: JamesIves/github-pages-deploy-action@v4.2.2
+        with:
+          branch: gh-pages
+          folder: fetch-api-data-custom-encoding
+          target-folder: encoding
+          ssh-key: ${{ secrets.DEPLOY_KEY }}

.github/workflows/version.yml

@@ -10,4 +10,4 @@ jobs:
     name: Update Major Version Tag
     runs-on: ubuntu-latest
     steps:
-      - uses: nowactions/update-majorver@v1
+      - uses: nowactions/update-majorver@v1.1.0

README.md

@@ -186,7 +186,7 @@ The following configuration options should be set.
 | `save-location`       | By default the save location of the file is `fetch-api-data-action/data.json`, if you'd like to override the directory you can do so by specifying a new one with this variable.                                                                                                                                                                                                                                                                               | `with`           | **No**   |
 | `save-name`           | You can override the name of the exported `.json` file by specifying a new one here. You should _not_ include the file extension in your name.                                                                                                                                                                                                                                                                                                                 | `with`           | **No**   |
 | `format`              | Allows you to modify the extension of the file saved from the API response, for example you can set this field to `json` or `txt`. This field defaults to `json`.                                                                                                                                                                                                                                                                                              | `with`           | **No**   |
-| `encoding`            | Allows you to modify the encoding of the file saved from the API response, for example you can set this field to `utf8` or `hex`. This field defaults to `hex`. Choose from `ascii` or `utf8` or `utf-8` or `utf16le` or `ucs2` or `ucs-2` or `base64` or `latin1` or `binary` or `hex`                                                                                                                                                                        | `with`           | **No**   |
+| `encoding`            | Allows you to modify the encoding of the file saved from the API response, for example you can set this field to `utf8` or `hex`. This field defaults to `utf8`. Choose from `ascii`, `utf8`, `utf-8`, `utf16le`, `ucs2`, `ucs-2`, `base64`, `latin1`, `binary` or `hex`.                                                                                                                                                                                      | `with`           | **No**   |
 | `debug`               | If set to `true` the action will log the API responses it receives in the terminal.                                                                                                                                                                                                                                                                                                                                                                            | `with`           | **No**   |
 
 ---

__tests__/fetch.test.ts

@@ -54,7 +54,7 @@ describe('fetch', () => {
         })
       } catch (error) {
         expect(error instanceof Error && error.message).toBe(
-          "There was an error fetching from the API: TypeError: Cannot read property 'cat' of null"
+          "There was an error fetching from the API: TypeError: Cannot read property 'cat' of null ❌"
         )
       }
     })
@@ -77,7 +77,7 @@ describe('fetch', () => {
         })
       } catch (error) {
         expect(error instanceof Error && error.message).toBe(
-          'There was an error fetching from the API: Error: {"a":1}'
+          'There was an error fetching from the API: Error: {"a":1} ❌'
         )
       }
     })
@@ -101,7 +101,7 @@ describe('fetch', () => {
         })
       } catch (error) {
         expect(error instanceof Error && error.message).toBe(
-          'There was an error fetching from the API: FetchError: invalid json response body at https://jives.dev/ reason: Unexpected token < in JSON at position 0'
+          'There was an error fetching from the API: FetchError: invalid json response body at https://jives.dev/ reason: Unexpected token < in JSON at position 0 ❌'
         )
       }
     })
@@ -143,5 +143,20 @@ describe('fetch', () => {
       })
       expect(process.env['fetch-api-data']).toBe('68656C6C6F21')
     })
+
+    it('should fail if invalid encoding is used', async () => {
+      try {
+        await generateExport({
+          data: '68656C6C6F21',
+          encoding: 'hexxxxx' as BufferEncoding,
+          format: 'txt',
+          saveName: 'hex-data'
+        })
+      } catch (error) {
+        expect(error instanceof Error && error.message).toBe(
+          `There was an error generating the export file: TypeError [ERR_INVALID_OPT_VALUE_ENCODING]: The value "hexxxxx" is invalid for option "encoding" ❌`
+        )
+      }
+    })
   })
 })

src/fetch.ts

@@ -58,7 +58,7 @@ export async function retrieveData({
       }
     )
   } catch (error) {
-    throw new Error(`There was an error fetching from the API: ${error}`)
+    throw new Error(`There was an error fetching from the API: ${error} ❌`)
   }
 }
 
@@ -75,12 +75,19 @@ export async function generateExport({
     saveName ? saveName : 'data'
   }.${format ? format : 'json'}`
   const dataEncoding = encoding ? encoding : 'utf8'
-  await mkdirP(`${saveLocation ? saveLocation : 'fetch-api-data-action'}`)
-  await fs.writeFile(file, data, dataEncoding)
 
-  info(`Saved ${file} 💾`)
+  try {
+    await mkdirP(`${saveLocation ? saveLocation : 'fetch-api-data-action'}`)
+    await fs.writeFile(file, data, dataEncoding)
+
+    info(`Saved ${file} 💾`)
 
-  exportVariable('fetch-api-data', data)
+    exportVariable('fetch-api-data', data)
 
-  return Status.SUCCESS
+    return Status.SUCCESS
+  } catch (error) {
+    throw new Error(
+      `There was an error generating the export file: ${error} ❌`
+    )
+  }
 }