Update README, add test for parsing

.prettierignore

@@ -1,3 +1,4 @@
 dist/
 lib/
-node_modules/
+node_modules/
+parseJsonSafely.test.ts

README.md

@@ -23,8 +23,8 @@ Optional: JSON string of headers to pass into request. Default `"{}"`.
 uses: dkershner6/post-api-call-action@v1
 with:
   url: ${{ secrets.API_URL }}
-  data: "{'command': 'publish'}"
+  data: "{\"command\": \"publish\"}"
-  headers: "{'Authorization': '${{ secrets.API_KEY }}'}"
+  headers: "{\"Authorization\": \"Bearer ${{ secrets.API_KEY }}\"}"
 ```
 
 

dist/index.js

@@ -21,33 +21,49 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 Object.defineProperty(exports, "__esModule", ({ value: true }));
 const core_1 = __nccwpck_require__(2186);
 const axios_1 = __importDefault(__nccwpck_require__(6545));
-const parseJsonSafely = (jsonString) => {
+const parseJsonSafely_1 = __nccwpck_require__(8521);
-    try {
-        return JSON.parse(jsonString);
-    }
-    catch (error) {
-        return {};
-    }
-};
 function run() {
     return __awaiter(this, void 0, void 0, function* () {
         try {
             const url = core_1.getInput('url');
             core_1.info(`Sending POST request to ${url}`);
-            const data = parseJsonSafely(core_1.getInput('data'));
+            const data = parseJsonSafely_1.parseJsonSafely(core_1.getInput('data'));
-            const headers = parseJsonSafely(core_1.getInput('headers'));
+            const headers = parseJsonSafely_1.parseJsonSafely(core_1.getInput('headers'));
             yield axios_1.default.post(url, data, {
                 headers,
             });
         }
-        catch (error) {
+        catch (err) {
-            core_1.setFailed(error.message);
+            core_1.error(err.message);
+            core_1.setFailed(err.message);
         }
     });
 }
 run();
 
 
+/***/ }),
+
+/***/ 8521:
+/***/ ((__unused_webpack_module, exports, __nccwpck_require__) => {
+
+"use strict";
+
+Object.defineProperty(exports, "__esModule", ({ value: true }));
+exports.parseJsonSafely = void 0;
+const core_1 = __nccwpck_require__(2186);
+const parseJsonSafely = (jsonString) => {
+    try {
+        return JSON.parse(jsonString);
+    }
+    catch (err) {
+        core_1.error(`Parsing error: ${jsonString} - ${err.message}`);
+        return {};
+    }
+};
+exports.parseJsonSafely = parseJsonSafely;
+
+
 /***/ }),
 
 /***/ 7351:

package.json

@@ -1,6 +1,6 @@
 {
   "name": "post-api-call-action",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "private": true,
   "description": "Send a POST request action",
   "main": "lib/main.js",

src/main.ts

@@ -1,13 +1,6 @@
-import { getInput, info, setFailed } from '@actions/core';
+import { getInput, info, error, setFailed } from '@actions/core';
 import axios from 'axios';
-
+import { parseJsonSafely } from './parseJsonSafely';
-const parseJsonSafely = (jsonString: string): any => {
-    try {
-        return JSON.parse(jsonString);
-    } catch (error) {
-        return {};
-    }
-};
 
 async function run(): Promise<void> {
     try {
@@ -18,8 +11,9 @@ async function run(): Promise<void> {
         await axios.post(url, data, {
             headers,
         });
-    } catch (error) {
+    } catch (err) {
-        setFailed(error.message);
+        error(err.message);
+        setFailed(err.message);
     }
 }
 

src/parseJsonSafely.test.ts

@@ -0,0 +1,11 @@
+import { parseJsonSafely } from './parseJsonSafely';
+
+it('Should parse example strings correctly', () => {
+    const testString = "{\"Authorization\": \"Bearer testsdtestdgsdsfgs\"}";
+
+    const result = parseJsonSafely(testString);
+
+    expect(result).toEqual({
+        Authorization: 'Bearer testsdtestdgsdsfgs',
+    });
+});

src/parseJsonSafely.ts

@@ -0,0 +1,10 @@
+import { error } from '@actions/core';
+
+export const parseJsonSafely = (jsonString: string): unknown => {
+    try {
+        return JSON.parse(jsonString);
+    } catch (err) {
+        error(`Parsing error: ${jsonString} - ${err.message}`);
+        return {};
+    }
+};