fix: js bin and add tests

Author: psteinroe

.github/workflows/pull_request.yml

@@ -170,11 +170,18 @@ jobs:
       - name: Run tests
         run: cargo test --workspace
 
-  test-js-bindings:
+  test-js-packages:
     name:
-      Test JS Bindings
+      Test JS Packages
       # use the same image we use for compiling
-    runs-on: ubuntu-22.04
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        include:
+          # use the same images we use for compiling
+          - os: windows-2022
+          - os: ubuntu-22.04
+          - os: macos-14
     services:
       postgres:
         image: postgres:latest
@@ -201,12 +208,15 @@ jobs:
         uses: oven-sh/setup-bun@v2
       - name: Install JS dependencies
         run: bun install
-      - name: Build TypeScript code
+      - name: Build backend-jsonrpc
         working-directory: packages/@postgrestools/backend-jsonrpc
         run: bun run build
-      - name: Run JS tests
+      - name: Run backend-jsonrpc test
         working-directory: packages/@postgrestools/backend-jsonrpc
         run: bun run test
+      - name: Run cli test
+        working-directory: packages/@postgrestools/postgrestools
+        run: bun run test
 
   codegen:
     name: Check Codegen

packages/@postgrestools/postgrestools/bin/postgrestools

@@ -33,7 +33,7 @@ function isMusl() {
   } catch (err) {
     stderr = err.stderr;
   }
-  if (stderr.indexOf("musl") > -1) {
+  if (stderr?.indexOf("musl") > -1) {
     return true;
   }
   return false;

packages/@postgrestools/postgrestools/package.json

@@ -40,5 +40,8 @@
     "@postgrestools/cli-x86_64-linux-gnu": "<placeholder>",
     "@postgrestools/cli-aarch64-linux-gnu": "<placeholder>",
     "@postgrestools/cli-x86_64-linux-musl": "<placeholder>"
+  },
+  "scripts": {
+    "test": "bun test"
   }
 }

packages/@postgrestools/postgrestools/test/bin.test.js

@@ -0,0 +1,62 @@
+import { describe, it, expect } from "bun:test";
+import { spawn } from "child_process";
+import { join, dirname } from "path";
+import { fileURLToPath } from "url";
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const binPath = join(__dirname, "../bin/postgrestools");
+const testSqlPath = join(__dirname, "test.sql");
+
+describe("postgrestools bin", () => {
+
+  it("should check a SQL file successfully", async () => {
+    const result = await new Promise((resolve) => {
+      const proc = spawn("node", [binPath, "check", testSqlPath], {
+        env: { ...process.env },
+      });
+
+      let stdout = "";
+      let stderr = "";
+
+      proc.stdout.on("data", (data) => {
+        stdout += data.toString();
+      });
+
+      proc.stderr.on("data", (data) => {
+        stderr += data.toString();
+      });
+
+      proc.on("close", (code) => {
+        resolve({ code, stdout, stderr });
+      });
+    });
+
+    expect(result.code).toBe(0);
+    expect(result.stderr).toBe("");
+  });
+
+  it("should fail when file doesn't exist", async () => {
+    const result = await new Promise((resolve) => {
+      const proc = spawn("node", [binPath, "check", "nonexistent.sql"], {
+        env: { ...process.env },
+      });
+
+      let stdout = "";
+      let stderr = "";
+
+      proc.stdout.on("data", (data) => {
+        stdout += data.toString();
+      });
+
+      proc.stderr.on("data", (data) => {
+        stderr += data.toString();
+      });
+
+      proc.on("close", (code) => {
+        resolve({ code, stdout, stderr });
+      });
+    });
+
+    expect(result.code).not.toBe(0);
+  });
+});

packages/@postgrestools/postgrestools/test/test.sql

@@ -0,0 +1 @@
+select 1;