test: use testBin in all the tests

src/brew/__tests__/brew.test.ts

@@ -1,5 +1,5 @@
 import { setupBrew } from "../brew"
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 
 jest.setTimeout(200000)
 describe("setup-brew", () => {
@@ -7,11 +7,7 @@ describe("setup-brew", () => {
     if (process.platform !== "darwin") {
       return
     }
-    await setupBrew("", "", "")
+    setupBrew("", "", "")
-
+    await testBin("brew")
-    const { status } = spawn("brew", ["--version"], {
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/ccache/__tests__/ccache.test.ts

@@ -1,14 +1,11 @@
 import { setupCcache } from "../ccache"
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 
 jest.setTimeout(200000)
 describe("setup-ccache", () => {
   it("should setup ccache", async () => {
     await setupCcache("", "", "")
 
-    const { status } = spawn("ccache", ["--version"], {
+    await testBin("ccache")
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/chocolatey/__tests__/chocolatey.test.ts

@@ -1,5 +1,5 @@
+import { testBin } from "../../utils/tests/test-helpers"
 import { setupChocolatey } from "../chocolatey"
-import { spawnSync as spawn } from "child_process"
 
 jest.setTimeout(200000)
 describe("setup-chocolatey", () => {
@@ -8,10 +8,6 @@ describe("setup-chocolatey", () => {
       return
     }
     await setupChocolatey("", "", "")
-
+    await testBin("choco")
-    const { status } = spawn("choco", ["--version"], {
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/conan/__tests__/conan.test.ts

@@ -1,14 +1,11 @@
 import { setupConan } from "../conan"
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 
 jest.setTimeout(200000)
 describe("setup-conan", () => {
   it("should setup conan", async () => {
     await setupConan("1.40.1", "", "")
 
-    const { status } = spawn("conan", ["--version"], {
+    await testBin("conan")
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/cppcheck/__tests__/cppcheck.test.ts

@@ -1,14 +1,11 @@
 import { setupCppcheck } from "../cppcheck"
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 
 jest.setTimeout(200000)
 describe("setup-cppcheck", () => {
   it("should setup cppcheck", async () => {
     await setupCppcheck("", "", "")
 
-    const { status } = spawn("cppcheck", ["--version"], {
+    await testBin("cppcheck")
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/doxygen/__tests__/doxygen.test.ts

@@ -1,14 +1,11 @@
 import { setupDoxygen } from "../doxygen"
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 
 jest.setTimeout(200000)
 describe("setup-doxygen", () => {
   it("should setup doxygen", async () => {
     await setupDoxygen("", "", "")
 
-    const { status } = spawn("doxygen", ["--version"], {
+    await testBin("doxygen")
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/gcc/__tests__/gcc.test.ts

@@ -1,4 +1,4 @@
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 import { setupGcc } from "../gcc"
 
 jest.setTimeout(200000)
@@ -6,9 +6,6 @@ describe("setup-gcc", () => {
   it("should setup gcc", async () => {
     await setupGcc("", "", "")
 
-    const { status } = spawn("g++", ["--version"], {
+    await testBin("g++")
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/gcovr/__tests__/gcovr.test.ts

@@ -1,14 +1,11 @@
 import { setupGcovr } from "../gcovr"
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 
 jest.setTimeout(200000)
 describe("setup-gcovr", () => {
   it("should setup gcovr", async () => {
     await setupGcovr("5.0", "", "")
 
-    const { status } = spawn("gcovr", ["--version"], {
+    await testBin("gcovr")
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/llvm/__tests__/llvm.test.ts

@@ -1,11 +1,7 @@
 import { setupLLVM, VERSIONS, getUrl } from "../llvm"
 import { getSpecificVersionAndUrl } from "../../utils/setup/version"
 import { isValidUrl } from "../../utils/http/validate_url"
-import { setupTmpDir, cleanupTmpDir } from "../../utils/tests/test-helpers"
+import { setupTmpDir, cleanupTmpDir, testBin } from "../../utils/tests/test-helpers"
-import { addBinExtension } from "../../utils/setup/setupBin"
-import { join } from "path"
-import { spawnSync as spawn } from "child_process"
-import { which } from "@actions/io"
 
 jest.setTimeout(200000)
 async function testUrl(version: string) {
@@ -47,17 +43,7 @@ describe("setup-llvm", () => {
 
   it("should setup LLVM", async () => {
     const { binDir } = await setupLLVM("11.0.0", directory, "")
-    expect(binDir).toBeDefined()
+    await testBin("clang++", ["--version"], binDir)
-    expect(binDir).not.toHaveLength(0)
-
-    const clangBin = join(binDir, addBinExtension("clang"))
-
-    const { status } = spawn(clangBin, ["--version"], {
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
-
-    expect(await which("clang", true)).toBe(clangBin)
   })
 
   afterAll(async () => {

src/meson/__tests__/meson.test.ts

@@ -1,14 +1,10 @@
 import { setupMeson } from "../meson"
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 
 jest.setTimeout(200000)
 describe("setup-meson", () => {
   it("should setup meson", async () => {
     await setupMeson("0.59.1", "", "")
-
+    await testBin("meson")
-    const { status } = spawn("meson", ["--version"], {
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })

src/msvc/__tests__/msvc.test.ts

@@ -1,5 +1,5 @@
+import { testBin } from "../../utils/tests/test-helpers"
 import { setupMSVC } from "../msvc"
-import { spawnSync as spawn } from "child_process"
 
 jest.setTimeout(200000)
 describe("setup-msvc", () => {
@@ -9,10 +9,10 @@ describe("setup-msvc", () => {
     }
     await setupMSVC("2019", "", "")
 
-    spawn("cl", {
+    try {
-      encoding: "utf8",
+      await testBin("cl", [])
-    })
+    } catch (err) {
       // TODO see #1
-    // expect(status).toBe(0)
+    }
   })
 })

src/opencppcoverage/__tests__/opencppcoverage.test.ts

@@ -1,5 +1,5 @@
 import { setupOpencppcoverage } from "../opencppcoverage"
-import { spawnSync as spawn } from "child_process"
+import { testBin } from "../../utils/tests/test-helpers"
 
 jest.setTimeout(200000)
 describe("setup-OpenCppCoverage", () => {
@@ -9,9 +9,6 @@ describe("setup-OpenCppCoverage", () => {
     }
     await setupOpencppcoverage("", "", "")
 
-    const { status } = spawn("OpenCppCoverage", ["--version"], {
+    await testBin("OpenCppCoverage")
-      encoding: "utf8",
-    })
-    expect(status).toBe(0)
   })
 })