Adding verbose logging option to node script (#1186)

craigatk · Dec 2, 2023 · 03d359b · 03d359b
1 parent 4b60ec6
commit 03d359b
Show file tree

Hide file tree

Showing 23 changed files with 206 additions and 156 deletions.
diff --git a/publishers/node-script/package.json b/publishers/node-script/package.json
@@ -1,6 +1,6 @@
 {
   "name": "projektor-publish",
-  "version": "3.12.0",
+  "version": "3.12.1",
   "homepage": "https://projektor.dev/docs/node-script/",
   "repository": {
     "type": "git",
@@ -24,11 +24,11 @@
     "node-gzip": "1.1.2"
   },
   "devDependencies": {
-    "axios-mock-adapter": "1.21.5",
+    "axios-mock-adapter": "1.22.0",
     "env-cmd": "10.1.0",
     "jest": "29.7.0",
     "jest-junit": "16.0.0",
-    "prettier": "3.0.3",
+    "prettier": "3.1.0",
     "projektor-publish": "3.12.0",
     "wait-for-expect": "3.0.2"
   },

diff --git a/publishers/node-script/src/__tests__/index-coverage.spec.js b/publishers/node-script/src/__tests__/index-coverage.spec.js
@@ -25,21 +25,21 @@ describe("node script index - coverage reports", () => {
       { coverage, serverUrl, compressionEnabled: false },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(mockAxios.history.post.length).toBe(1);
 
     const resultsPostRequest = mockAxios.history.post.find((postRequest) =>
-      postRequest.url.includes("groupedResults")
+      postRequest.url.includes("groupedResults"),
     );
 
     const resultsPostData = resultsPostRequest.data;
 
     const resultsPostBody = JSON.parse(resultsPostData);
     expect(resultsPostBody.coverageFiles.length).toBe(1);
     expect(resultsPostBody.coverageFiles[0].reportContents).toContain(
-      "<coverage"
+      "<coverage",
     );
   });
 });
diff --git a/publishers/node-script/src/__tests__/index-exit.spec.js b/publishers/node-script/src/__tests__/index-exit.spec.js
@@ -35,7 +35,7 @@ describe("node script index exit with failure", () => {
         "src/__tests__/resultsWithFailure/*.xml",
       ],
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(process.exitCode).toBe(1);
@@ -56,7 +56,7 @@ describe("node script index exit with failure", () => {
         "src/__tests__/resultsDir1/*.xml",
       ],
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(process.exitCode).not.toBe(1);
@@ -75,7 +75,7 @@ describe("node script index exit with failure", () => {
       { resultsFileGlobs, failOnPublishError: true, serverUrl },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(process.exitCode).toBe(1);

diff --git a/publishers/node-script/src/__tests__/index-git.spec.js b/publishers/node-script/src/__tests__/index-git.spec.js
@@ -29,7 +29,7 @@ describe("node script index - Git source control values", () => {
       },
       { VELA_BUILD_REF: "refs/heads/my-branch" },
       null,
-      "src/__tests__/does_not_exist.json"
+      "src/__tests__/does_not_exist.json",
     );
     expect(mockAxios.history.post.length).toBe(1);
 
@@ -54,7 +54,7 @@ describe("node script index - Git source control values", () => {
       },
       { VELA_PULL_REQUEST_SOURCE: "my-branch" },
       null,
-      "src/__tests__/does_not_exist.json"
+      "src/__tests__/does_not_exist.json",
     );
     expect(mockAxios.history.post.length).toBe(1);
 
@@ -84,7 +84,7 @@ describe("node script index - Git source control values", () => {
         DRONE_PULL_REQUEST: "42",
       },
       null,
-      "src/__tests__/does_not_exist.json"
+      "src/__tests__/does_not_exist.json",
     );
     expect(mockAxios.history.post.length).toBe(1);
 
@@ -112,7 +112,7 @@ describe("node script index - Git source control values", () => {
       },
       { VELA_BUILD_PULL_REQUEST: "42" },
       null,
-      "src/__tests__/does_not_exist.json"
+      "src/__tests__/does_not_exist.json",
     );
     expect(mockAxios.history.post.length).toBe(1);
 
@@ -138,7 +138,7 @@ describe("node script index - Git source control values", () => {
       },
       { VELA_BUILD_PULL_REQUEST: "42", VELA_BUILD_NUMBER: "22" },
       null,
-      "src/__tests__/does_not_exist.json"
+      "src/__tests__/does_not_exist.json",
     );
     expect(mockAxios.history.post.length).toBe(1);
 
@@ -163,7 +163,7 @@ describe("node script index - Git source control values", () => {
       },
       { VELA_BUILD_PULL_REQUEST: "42", VELA_BUILD_NUMBER: "22" },
       null,
-      "src/__tests__/does_not_exist.json"
+      "src/__tests__/does_not_exist.json",
     );
     expect(mockAxios.history.post.length).toBe(1);
 

diff --git a/publishers/node-script/src/__tests__/index-performance.spec.js b/publishers/node-script/src/__tests__/index-performance.spec.js
@@ -29,26 +29,26 @@ describe("node script index - performance results", () => {
       { performance, serverUrl, compressionEnabled },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(mockAxios.history.post.length).toBe(1);
 
     const resultsPostRequest = mockAxios.history.post.find((postRequest) =>
-      postRequest.url.includes("groupedResults")
+      postRequest.url.includes("groupedResults"),
     );
 
     const parsedRequestBody = JSON.parse(resultsPostRequest.data.toString());
     expect(parsedRequestBody.groupedTestSuites.length).toBe(0);
 
     expect(parsedRequestBody.performanceResults.length).toBe(2);
     const file1 = parsedRequestBody.performanceResults.find(
-      (file) => file.name === "perf-test-1.json"
+      (file) => file.name === "perf-test-1.json",
     );
     expect(file1.resultsBlob).toBe('{"name":"perf-test-1"}');
 
     const file2 = parsedRequestBody.performanceResults.find(
-      (file) => file.name === "perf-test-2.json"
+      (file) => file.name === "perf-test-2.json",
     );
     expect(file2.resultsBlob).toBe('{"name":"perf-test-2"}');
   });
@@ -62,13 +62,13 @@ describe("node script index - performance results", () => {
       { performance, serverUrl, compressionEnabled },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(mockAxios.history.post.length).toBe(0);
 
     expect(consoleLog).toHaveBeenCalledWith(
-      "No performance results files found in locations does_not_exist/*.json"
+      "No performance results files found in locations does_not_exist/*.json",
     );
   });
 });
diff --git a/publishers/node-script/src/__tests__/index-quality.spec.js b/publishers/node-script/src/__tests__/index-quality.spec.js
@@ -30,7 +30,7 @@ describe("node script index - code quality reports", () => {
       { resultsFileGlobs, codeQuality, serverUrl },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(mockAxios.history.post.length).toBe(1);
@@ -66,7 +66,7 @@ describe("node script index - code quality reports", () => {
       { resultsFileGlobs, codeQuality, serverUrl },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(mockAxios.history.post.length).toBe(1);
@@ -106,7 +106,7 @@ describe("node script index - code quality reports", () => {
       { resultsFileGlobs, codeQuality, serverUrl },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(mockAxios.history.post.length).toBe(1);

diff --git a/publishers/node-script/src/__tests__/index-slack.spec.js b/publishers/node-script/src/__tests__/index-slack.spec.js
@@ -34,7 +34,7 @@ describe("index Slack message file", () => {
       { resultsFileGlobs, serverUrl, projectName, writeSlackMessageFile },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(fs.existsSync(slackMessageFileName)).toBeTruthy();
@@ -44,7 +44,7 @@ describe("index Slack message file", () => {
       .toString();
     const parsedMessage = JSON.parse(messageFileContents);
     expect(parsedMessage.attachments[0].pretext).toBe(
-      `Tests passed in project ${projectName}`
+      `Tests passed in project ${projectName}`,
     );
   });
 
@@ -69,7 +69,7 @@ describe("index Slack message file", () => {
       },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(fs.existsSync(slackMessageFileName)).toBeTruthy();
@@ -79,7 +79,7 @@ describe("index Slack message file", () => {
       .toString();
     const parsedMessage = JSON.parse(messageFileContents);
     expect(parsedMessage.attachments[0].pretext).toBe(
-      `Tests passed in project ${slackProjectName}`
+      `Tests passed in project ${slackProjectName}`,
     );
   });
 
@@ -102,7 +102,7 @@ describe("index Slack message file", () => {
       },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(fs.existsSync(slackMessageFileName)).toBeTruthy();
@@ -112,7 +112,7 @@ describe("index Slack message file", () => {
       .toString();
     const parsedMessage = JSON.parse(messageFileContents);
     expect(parsedMessage.attachments[0].pretext).toBe(
-      `Tests passed in project ${slackProjectName}`
+      `Tests passed in project ${slackProjectName}`,
     );
   });
 
@@ -135,7 +135,7 @@ describe("index Slack message file", () => {
       },
       {},
       null,
-      "projektor.none.json"
+      "projektor.none.json",
     );
 
     expect(fs.existsSync(slackMessageFileName)).toBeTruthy();
@@ -147,7 +147,7 @@ describe("index Slack message file", () => {
     const message = parsedMessage.attachments[0];
     expect(message.pretext).toBe("No test results found");
     expect(message.text).toBe(
-      `No test results found to publish in project ${slackProjectName}`
+      `No test results found to publish in project ${slackProjectName}`,
     );
   });
 });