fix: js bin and add tests (#531)

fixes #529

Author: psteinroe

.github/workflows/pull_request.yml

@@ -172,7 +172,7 @@ jobs:
 
   test-js-bindings:
     name:
-      Test JS Bindings
+      Test JS Packages
       # use the same image we use for compiling
     runs-on: ubuntu-22.04
     services:
@@ -201,12 +201,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

@@ -21,19 +21,19 @@ const PLATFORMS = {
 };
 
 function isMusl() {
-  let stderr;
+  let stdout;
   try {
-    stderr = execSync("ldd --version", {
+    stdout = execSync("ldd --version", {
       stdio: [
         "ignore", // stdin
         "pipe", // stdout – glibc systems print here
         "pipe", // stderr – musl systems print here
       ],
     });
   } catch (err) {
-    stderr = err.stderr;
+    stdout = err.stderr;
   }
-  if (stderr.indexOf("musl") > -1) {
+  if (typeof stdout === 'string' && stdout.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;