Integrate build and test into workflows

Author: cowboyd

.github/workflows/cli-release.yml

@@ -47,7 +47,7 @@ jobs:
           # note you'll typically need to create a personal access token
           # with permissions to create releases in the other repo
           token: ${{ secrets.FRONTSIDEJACK_GITHUB_TOKEN }}
-          files: build/cli/*
+          files: cli/build/*
           fail_on_unmatched_files: true
         env:
           GITHUB_REPOSITORY: thefrontside/graphgen

.github/workflows/verify.yml

@@ -38,4 +38,4 @@ jobs:
         run: deno lint
 
       - name: test
-        run: deno test
+        run: deno task test

cli/.gitignore

@@ -1,3 +1,5 @@
 /bundled-assets.js
 node_modules
+build
 graphgen
+example/package-lock.json

cli/deno.json

@@ -1,23 +1,34 @@
 {
   "tasks": {
-    "build": "deno run -A --unstable npm:vite build",
+    "test": "deno task build && deno task compile && deno task npm:install:example && deno test --unstable -A",
+    "npm:install": "npm install && cd example && npm install",
+    "npm:install:example": "npm install && cd example && npm install",
+    "build": "npm install && deno run -A --unstable npm:vite build && deno task bundle-assets",
     "dev": "deno run -A ./main.ts --watch ./src -f ./example/factory.ts --app-path=./dist",
     "bundle-assets": "deno run -A tasks/bundle-assets.ts dist",
+    "build:cli": "deno task build && deno task build:cli:windows && deno task build:cli:linux && deno task build:cli:macos-m1 && deno task build:cli:macos-intel",
+    "build:cli:windows": "deno compile --no-check -o build/graphgen-x86_64-pc-windows-msvc.exe --target x86_64-pc-windows-msvc --allow-read --allow-net --allow-env main.ts",
+    "build:cli:linux": "deno compile --no-check -o build/graphgen-x86_64-unknown-linux-gnu --target x86_64-unknown-linux-gnu --allow-read --allow-net --allow-env main.ts",
+    "build:cli:macos-m1": "deno compile --no-check -o build/graphgen-aarch64-apple-darwin --target aarch64-apple-darwin --allow-read --allow-net --allow-env main.ts",
+    "build:cli:macos-intel": "deno compile --no-check -o build/graphgen-x86_64-apple-darwin --target x86_64-apple-darwin --allow-read --allow-net --allow-env main.ts",
     "compile": "deno compile --no-check --allow-read --allow-net --allow-env -o graphgen ./main.ts"
   },
   "lint": {
     "rules": {
       "exclude": ["prefer-const", "require-yield", "no-inner-declarations"]
+    },
+    "files": {
+      "exclude": ["dist", "example", "bundled-assets.js", "node_modules"]
     }
   },
   "fmt": {
     "files": {
-      "exclude": ["README.md", "build/"]
+      "exclude": ["README.md", "build/", "node_modules", "example", "bundled-assets.js", "dist"]
     }
   },
   "test": {
     "files": {
-      "exclude": ["build", "node_modules"]
+      "exclude": ["build", "node_modules", "example"]
     }
   },
   "compilerOptions": {

cli/example/factory.ts

@@ -1,7 +1,7 @@
 const { createGraphGen, weighted } = require("@frontside/graphgen");
 const { fakergen } = require("./fakerGen.ts");
-const fs = require('fs');
-const path = require('path');
+const fs = require("fs");
+const path = require("path");
 
 const lifecycles = weighted([["deprecated", .15], ["experimental", .5], [
   "production",
@@ -16,7 +16,7 @@ const gen = (info) => {
   }
 };
 
-const source = String(fs.readFileSync(path.join(__dirname, 'world.graphql')));
+const source = String(fs.readFileSync(path.join(__dirname, "world.graphql")));
 
 module.exports = createGraphGen({
   seed: "factory",

cli/example/package-lock.json

@@ -3,6 +3,6 @@
   "private": true,
   "dependencies": {
     "@faker-js/faker": "7.5.0",
-    "@frontside/graphgen": "../../build/npm"
+    "@frontside/graphgen": "1.6.0"
   }
 }

cli/example/package.json

@@ -25,7 +25,9 @@ const readdir = async (k: string, v: string) => {
 
 await readdir(Deno.args[0], Deno.args[0]);
 
-const f = await Deno.create(Deno.args.length > 1 ? Deno.args[1] : "bundled-assets.js");
+const f = await Deno.create(
+  Deno.args.length > 1 ? Deno.args[1] : "bundled-assets.js",
+);
 await f.write(new TextEncoder().encode(`var m = {};\n`));
 const l = Object.keys(m);
 for (let i = 0; i < l.length; i++) {

cli/tasks/bundle-assets.ts

@@ -1,12 +1,12 @@
-import { describe, expect, it, beforeEach, afterEach } from "./suite.ts";
+import { afterEach, beforeEach, describe, expect, it } from "./suite.ts";
 
 describe("CLI smoke test", () => {
   let process: Deno.Process;
 
   beforeEach(() => {
     process = Deno.run({
-      cmd: ["./graphgen", "--port", "8900", "--factory", "example/factory.ts"]
-    })
+      cmd: ["./graphgen", "--port", "8900", "--factory", "example/factory.ts"],
+    });
   });
 
   afterEach(async () => {
@@ -15,15 +15,18 @@ describe("CLI smoke test", () => {
 
   it("can start up and we can fetch HTML from it", async () => {
     await eventually(async () => {
-      let response = await fetch('http://localhost:8900');
+      let response = await fetch("http://localhost:8900");
       expect(response.ok).toEqual(true);
       let body = await response.text();
       expect(body).toMatch(/div id="main"/);
     }, 10000);
-  })
+  });
 });
 
-async function eventually(assertion: () => Promise<void>, timeout: number): Promise<void> {
+async function eventually(
+  assertion: () => Promise<void>,
+  timeout: number,
+): Promise<void> {
   let lastError: Error | null = null;
   let done = false;
   let timeoutId = null;
@@ -36,11 +39,14 @@ async function eventually(assertion: () => Promise<void>, timeout: number): Prom
       }
     }
   }
-  
+
   try {
     return await Promise.race([
       new Promise<void>((_, reject) => {
-        timeoutId = setTimeout(() => reject(lastError ?? new Error('timeout')), timeout);
+        timeoutId = setTimeout(
+          () => reject(lastError ?? new Error("timeout")),
+          timeout,
+        );
       }),
       runAssertion(),
     ]);

cli/test/smoke.test.ts

@@ -1,28 +1,25 @@
 {
   "tasks": {
+    "test": "deno test && cd cli && deno task test",
     "build:npm": "deno run -A tasks/build-npm.ts",
-    "build:cli:windows": "deno compile -o build/cli/graphgen-x86_64-pc-windows-msvc.exe --target x86_64-pc-windows-msvc --allow-read --allow-net cli/main.ts",
-    "build:cli:linux": "deno compile -o build/cli/graphgen-x86_64-unknown-linux-gnu --target x86_64-unknown-linux-gnu --allow-read --allow-net cli/main.ts",
-    "build:cli:macos-m1": "deno compile -o build/cli/graphgen-aarch64-apple-darwin --target aarch64-apple-darwin --allow-read --allow-net cli/main.ts",
-    "build:cli:macos-intel": "deno compile -o build/cli/graphgen-x86_64-apple-darwin --target x86_64-apple-darwin --allow-read --allow-net cli/main.ts",
-    "build:cli": "deno task build:cli:windows && deno task build:cli:linux && deno task build:cli:macos-m1 && deno task build:cli:macos-intel"
+    "build:cli": "cd cli && deno task build:cli"
   },
   "lint": {
     "rules": {
       "exclude": ["prefer-const", "require-yield", "no-inner-declarations"]
     },
     "files": {
-      "exclude": ["build"]
+      "exclude": ["build", "cli"]
     }
   },
   "fmt": {
     "files": {
-      "exclude": ["README.md", "build/"]
+      "exclude": ["README.md", "build/", "cli"]
     }
   },
   "test": {
     "files": {
-      "exclude": ["build"]
+      "exclude": ["build", "cli"]
     }
   }
 }