chore: use Deno from all ci scripts

cmoog · cmoog · commit 1f928210a6f7 · 2020-12-24T20:34:03.000-06:00
diff --git a/.github/workflows/integration.yaml b/.github/workflows/integration.yaml
@@ -23,4 +23,4 @@ jobs:
         with:
           go-version: '^1.14'
       - name: integration tests
-        run: ./ci/steps/integration.sh
+        run: ./ci/steps/integration.ts
diff --git a/.github/workflows/test.yaml b/.github/workflows/test.yaml
@@ -5,6 +5,7 @@ jobs:
   fmt:
     runs-on: ubuntu-latest
     steps:
+      - run: apt-get update && apt-get install deno
       - uses: actions/checkout@v1
       - uses: actions/cache@v1
         with:
@@ -15,10 +16,11 @@ jobs:
       - name: fmt
         uses: ./ci/image
         with:
-          args: ./ci/steps/fmt.sh
+          args: ./ci/steps/fmt.ts
   lint:
     runs-on: ubuntu-latest
     steps:
+      - run: apt-get update && apt-get install deno
       - uses: actions/checkout@v2
       - name: golangci-lint
         uses: golangci/golangci-lint-action@v2
@@ -28,6 +30,7 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
+      - run: apt-get update && apt-get install deno
       - uses: actions/checkout@v1
       - uses: actions/cache@v1
         with:
@@ -38,10 +41,11 @@ jobs:
       - name: test
         uses: ./ci/image
         with:
-          args: ./ci/steps/unit_test.sh
+          args: ./ci/steps/unit_test.ts
   gendocs:
     runs-on: ubuntu-latest
     steps:
+      - run: apt-get update && apt-get install deno
       - uses: actions/checkout@v1
       - uses: actions/cache@v1
         with:
@@ -52,4 +56,4 @@ jobs:
       - name: generate-docs
         uses: ./ci/image
         with:
-          args: ./ci/steps/gendocs.sh
+          args: ./ci/steps/gendocs.ts
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -0,0 +1,3 @@
+{
+  "deno.enable": true
+}
diff --git a/ci/steps/fmt.sh b/ci/steps/fmt.sh
diff --git a/ci/steps/fmt.ts b/ci/steps/fmt.ts
@@ -0,0 +1,13 @@
+#!/usr/bin/env -S deno run --allow-all
+import { root, execInherit, requireNoFilesChanged, isCI } from "./lib.ts"
+
+await root()
+
+console.log("--- formatting")
+await execInherit("go mod tidy")
+await execInherit("gofmt -w -s .")
+await execInherit(`goimports -w "-local=$$(go list -m)" .`)
+
+if (isCI()) {
+  await requireNoFilesChanged()
+}
diff --git a/ci/steps/gendocs.sh b/ci/steps/gendocs.sh
diff --git a/ci/steps/gendocs.ts b/ci/steps/gendocs.ts
@@ -0,0 +1,13 @@
+#!/usr/bin/env -S deno run --allow-all
+import { root, execInherit, requireNoFilesChanged, isCI } from "./lib.ts"
+
+await root()
+
+console.log("--- regenerating documentation")
+await Deno.remove("./docs", { recursive: true })
+await Deno.mkdir("./docs")
+
+await execInherit("go run ./cmd/coder gen-docs ./docs")
+if (isCI()) {
+  await requireNoFilesChanged()
+}
diff --git a/ci/steps/integration.sh b/ci/steps/integration.sh
diff --git a/ci/steps/integration.ts b/ci/steps/integration.ts
@@ -0,0 +1,12 @@
+#!/usr/bin/env -S deno run --allow-all
+import { root, execInherit } from "./lib.ts"
+
+await root()
+
+console.info("--- building integration test image")
+await execInherit(
+  "docker build -f ./ci/integration/Dockerfile -t coder-cli-integration:latest ."
+)
+
+console.info("--- run go tests")
+await execInherit("go test ./ci/integration -count=1")
diff --git a/ci/steps/lib.ts b/ci/steps/lib.ts
@@ -1,9 +1,94 @@
-import { exec } from "https://cdn.depjs.com/exec/mod.ts"
+export const root = async () =>
+  Deno.chdir(await exec("git rev-parse --show-toplevel"))
 
-export const root = async () => Deno.chdir(await exec("git rev-parse --show-toplevel"))
 export const string = (a: Uint8Array): string => new TextDecoder().decode(a)
+
 export const bytes = (a: string): Uint8Array => new TextEncoder().encode(a)
+
 export const read = async (path: string): Promise<string> =>
   string(await Deno.readFile(path))
+
 export const write = async (path: string, data: string): Promise<void> =>
   Deno.writeFile(path, bytes(data))
+
+const removeTrailingLineBreak = (str: string) => {
+  return str.replace(/\n$/, "")
+}
+
+export const execInherit = async (
+  cmd: string | string[] | ExecOptions
+): Promise<void> => {
+  let opts: Deno.RunOptions
+  if (typeof cmd === "string") {
+    opts = {
+      cmd: ["sh", "-c", cmd],
+    }
+  } else if (Array.isArray(cmd)) {
+    opts = {
+      cmd,
+    }
+  } else {
+    opts = cmd
+  }
+
+  opts.stdout = "inherit"
+  opts.stderr = "inherit"
+
+  const process = Deno.run(opts)
+  const { success } = await process.status()
+  if (!success) {
+    process.close()
+    throw new Error("exec: failed to execute command")
+  }
+}
+
+export type ExecOptions = Omit<Deno.RunOptions, "stdout" | "stderr">
+
+export const exec = async (
+  cmd: string | string[] | ExecOptions
+): Promise<string> => {
+  let opts: Deno.RunOptions
+
+  if (typeof cmd === "string") {
+    opts = {
+      cmd: ["sh", "-c", cmd],
+    }
+  } else if (Array.isArray(cmd)) {
+    opts = {
+      cmd,
+    }
+  } else {
+    opts = cmd
+  }
+
+  opts.stdout = "piped"
+  opts.stderr = "piped"
+
+  const process = Deno.run(opts)
+  const decoder = new TextDecoder()
+  const { success } = await process.status()
+
+  if (!success) {
+    const msg = removeTrailingLineBreak(
+      decoder.decode(await process.stderrOutput())
+    )
+
+    process.close()
+
+    throw new Error(msg || "exec: failed to execute command")
+  }
+
+  return removeTrailingLineBreak(decoder.decode(await process.output()))
+}
+
+export const requireNoFilesChanged = async (): Promise<void> => {
+  const changed = await exec(
+    "git ls-files --other --modified --exclude-standard"
+  )
+  if (changed !== "") {
+    throw Error(`Files needs generation or formatting:
+${changed}`)
+  }
+}
+
+export const isCI = (): boolean => !!Deno.env.get("CI")
diff --git a/ci/steps/lint.sh b/ci/steps/lint.sh
diff --git a/ci/steps/lint.ts b/ci/steps/lint.ts
@@ -0,0 +1,7 @@
+#!/usr/bin/env -S deno run --allow-all
+import { root, execInherit } from "./lib.ts"
+
+await root()
+
+console.info("--- golangci-lint")
+await execInherit("golangci-lint run -c .golangci.yml")
diff --git a/ci/steps/unit_test.sh b/ci/steps/unit_test.sh
diff --git a/ci/steps/unit_test.ts b/ci/steps/unit_test.ts
@@ -0,0 +1,9 @@
+#!/usr/bin/env -S deno run --allow-all
+import { root, execInherit } from "./lib.ts"
+
+await root()
+
+console.info("--- running unit tests")
+await execInherit(
+  "go test $(go list ./... | grep -v pkg/tcli | grep -v ci/integration | grep -v coder-sdk)"
+)
diff --git a/ci/steps/update_brew.ts b/ci/steps/update_brew.ts
@@ -9,7 +9,7 @@ interface Params {
 }
 
 const template = ({ sha, version }: Params) => `class Coder < Formula
-  desc "a command line tool for the Coder remote development platorm"
+  desc "A command-line tool for the Coder remote development platform"
   homepage "https://github.com/cdr/coder-cli"
   url "https://github.com/cdr/coder-cli/releases/download/${version}/coder-cli-darwin-amd64-${version}.zip"
   sha256 "${sha}"

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ interface Params {`
`9`	`9`	`}`
`10`	`10`
`11`	`11`	const template = ({ sha, version }: Params) => `class Coder < Formula
`12`		`- desc "a command line tool for the Coder remote development platorm"`
	`12`	`+ desc "A command-line tool for the Coder remote development platform"`
`13`	`13`	`homepage "https://github.com/cdr/coder-cli"`
`14`	`14`	`url "https://github.com/cdr/coder-cli/releases/download/${version}/coder-cli-darwin-amd64-${version}.zip"`
`15`	`15`	`sha256 "${sha}"`