dataloaders tests (#110)

* support data loaders in sub directories of docs/

closes #130

* test data loaders

* Update test/dataloaders-test.ts

Co-authored-by: Mike Bostock <[email protected]>

* ignore all .observablehq/cache

---------

Co-authored-by: Mike Bostock <[email protected]>

Author: Fil

.gitignore

@@ -1,8 +1,7 @@
 .DS_Store
+**/.observablehq/cache/
 dist/
-docs/.observablehq/cache
 node_modules/
-test/input/build/*/.observablehq/cache
 test/output/*-changed.*
 test/output/build/*-changed/
 yarn-error.log

package.json

@@ -25,7 +25,7 @@
     "dev": "tsx watch ./src/preview.ts",
     "build": "rm -rf dist && ./bin/observable.ts build",
     "test": "yarn test:mocha && yarn test:lint",
-    "test:mocha": "tsx ./node_modules/.bin/mocha 'test/**/*-test.*'",
+    "test:mocha": "rm -rf test/.observablehq && tsx ./node_modules/.bin/mocha 'test/**/*-test.*'",
     "test:lint": "eslint src test public"
   },
   "dependencies": {

test/dataloaders-test.ts

@@ -0,0 +1,21 @@
+import {Loader} from "../src/dataloader.js";
+import {readFile} from "node:fs/promises";
+import assert from "node:assert";
+
+describe("data loaders are called with the appropriate command", () => {
+  it("a .js data loader is called with node", async () => {
+    const loader = Loader.find("test", "dataloaders/data1.txt")!;
+    const out = await loader.load();
+    assert.strictEqual(await readFile("test/" + out, "utf-8"), "node\n");
+  });
+  it("a .ts data loader is called with tsx", async () => {
+    const loader = Loader.find("test", "dataloaders/data2.txt");
+    const out = await loader!.load();
+    assert.strictEqual(await readFile("test/" + out, "utf-8"), "tsx\n");
+  });
+  it("a .sh data loader is called as a shell script", async () => {
+    const loader = Loader.find("test", "dataloaders/data3.txt");
+    const out = await loader!.load();
+    assert.strictEqual(await readFile("test/" + out, "utf-8"), "shell\n");
+  });
+});

test/dataloaders/data1.txt.js

@@ -0,0 +1,6 @@
+function mytest() {
+  // TODO: how to make sure this is not called by tsx? (and, does it matter?)
+  return "node";
+}
+
+console.log(mytest());

test/dataloaders/data2.txt.ts

@@ -0,0 +1,5 @@
+function mytest2(): string {
+  return "tsx";
+}
+
+console.log(mytest2());

test/dataloaders/data3.txt.sh

@@ -0,0 +1,3 @@
+#! /usr/bin/env bash
+
+echo "shell"