.

src/index.ts

@@ -14,7 +14,7 @@ import {
   formatShellDiff,
   formatShellDiffBrillig,
 } from "./format/program";
-import { computeProgramDiffs } from "./report";
+import { computeProgramDiffs, parseReport } from "./report";
 import { DiffBrillig, DiffCircuit, WorkspaceReport } from "./types";
 
 const token = process.env.GITHUB_TOKEN || core.getInput("token");
@@ -102,11 +102,11 @@ function loadReports(referenceContent: string): [WorkspaceReport, WorkspaceRepor
   const compareContent = fs.readFileSync(localReportPath, "utf8");
 
   core.info(`Mapping compared gas reports`);
-  const compareReports: WorkspaceReport = JSON.parse(compareContent);
+  const compareReports = parseReport(compareContent);
   core.info(`Got ${compareReports.programs.length} compare programs`);
 
   core.info(`Mapping reference gas reports`);
-  const referenceReports: WorkspaceReport = JSON.parse(referenceContent);
+  const referenceReports = parseReport(referenceContent);
   core.info(`Got ${compareReports.programs.length} reference programs`);
   core.endGroup();
 

src/report.ts

@@ -12,6 +12,10 @@ import {
   DiffBrillig,
 } from "./types";
 
+export const parseReport = (content: string): WorkspaceReport => {
+  return JSON.parse(content);
+};
+
 export const variation = (current: number, previous: number) => {
   const delta = current - previous;
 

tests/contract_report.test.ts

@@ -1,13 +1,13 @@
 import * as fs from "fs";
 
 import { formatMarkdownDiff, formatShellDiff } from "../src/format/contract";
-import { computeContractDiffs } from "../src/report";
+import { computeContractDiffs, parseReport } from "../src/report";
 
 const srcContent = fs.readFileSync("tests/mocks/gas_report.2.json", "utf8");
 const cmpContent = fs.readFileSync("tests/mocks/gas_report.1.json", "utf8");
 
-const srcContractReports = JSON.parse(srcContent).contracts;
-const cmpContractReports = JSON.parse(cmpContent).contracts;
+const srcContractReports = parseReport(srcContent).contracts;
+const cmpContractReports = parseReport(cmpContent).contracts;
 
 describe("Markdown format", () => {
   // shows how the runner will run a javascript action with env / stdout protocol

tests/diff.test.ts

@@ -1,14 +1,14 @@
 import * as fs from "fs";
 
-import { computeProgramDiffs } from "../src/report";
+import { computeProgramDiffs, parseReport } from "../src/report";
 import { DiffBrillig, DiffCircuit, ProgramReport } from "../src/types";
 
 const srcContent = fs.readFileSync("tests/mocks/gas_report.2.json", "utf8");
 const cmpContent = fs.readFileSync("tests/mocks/gas_report.1.json", "utf8");
 
 describe("Program diffs", () => {
-  const srcProgramReports: ProgramReport[] = JSON.parse(srcContent).programs;
-  const cmpProgramReports: ProgramReport[] = JSON.parse(cmpContent).programs;
+  const srcProgramReports: ProgramReport[] = parseReport(srcContent).programs;
+  const cmpProgramReports: ProgramReport[] = parseReport(cmpContent).programs;
 
   it("should diff 1 and 2 successfully", () => {
     const expectedDiffCircuits: DiffCircuit[] = [

tests/loading.test.ts

@@ -1,14 +1,16 @@
 import * as fs from "fs";
 
+import { parseReport } from "../src/report";
+
 const srcContent = fs.readFileSync("tests/mocks/gas_report.2.json", "utf8");
 const cmpContent = fs.readFileSync("tests/mocks/gas_report.1.json", "utf8");
 
 describe("Report Loading", () => {
   it("should load 1 successfully", () => {
-    console.log(JSON.parse(srcContent));
+    console.log(parseReport(srcContent));
   });
 
   it("should load 2 successfully", () => {
-    console.log(JSON.parse(cmpContent));
+    console.log(parseReport(cmpContent));
   });
 });

tests/program_report.test.ts

@@ -9,14 +9,14 @@ import {
   formatShellDiff,
   formatShellDiffBrillig,
 } from "../src/format/program";
-import { computeProgramDiffs } from "../src/report";
+import { computeProgramDiffs, parseReport } from "../src/report";
 import { ProgramReport } from "../src/types";
 
 const srcContent = fs.readFileSync("tests/mocks/gas_report.2.json", "utf8");
 const cmpContent = fs.readFileSync("tests/mocks/gas_report.1.json", "utf8");
 
-const srcContractReports: ProgramReport[] = JSON.parse(srcContent).programs;
-const cmpContractReports: ProgramReport[] = JSON.parse(cmpContent).programs;
+const srcContractReports: ProgramReport[] = parseReport(srcContent).programs;
+const cmpContractReports: ProgramReport[] = parseReport(cmpContent).programs;
 
 describe("Markdown format", () => {
   // shows how the runner will run a javascript action with env / stdout protocol