eyeling 1.21.1 → 1.21.2

package/HANDBOOK.md

@@ -2141,7 +2141,7 @@ In practice, this means:
 - builtin module loading accepts only the documented export forms
 - the helper API exposed by `__buildBuiltinRegistrationApi()` has a fixed key set
 - builtin handlers should return an array of substitution objects
-- accidental helper drift is caught by `test/builtin-contract.test.js`
+- accidental helper drift is caught by `test/builtins.test.js`
 
 This is only meant to stop silent breakage. It is **not** a promise that Eyeling can never change the builtin API. If the helper surface ever needs to change, that change should be deliberate, documented, and called out in release notes.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eyeling",
-  "version": "1.21.1",
+  "version": "1.21.2",
   "description": "A minimal Notation3 (N3) reasoner in JavaScript.",
   "main": "./index.js",
   "keywords": [
@@ -42,14 +42,14 @@
     "build": "node tools/bundle.js",
     "test:packlist": "node test/packlist.test.js",
     "test:api": "node test/api.test.js",
-    "test:builtin-contract": "node test/builtin-contract.test.js",
+    "test:builtins": "node test/builtins.test.js",
     "test:n3gen": "node test/n3gen.test.js",
     "test:examples": "node test/examples.test.js",
     "test:extra": "node test/extra.test.js",
     "test:manifest": "node test/manifest.test.js",
     "test:playground": "node test/playground.test.js",
     "test:package": "node test/package.test.js",
-    "test:all": "npm run test:api && npm run test:builtin-contract && npm run test:n3gen && npm run test:examples && npm run test:extra && npm run test:manifest && npm run test:playground",
+    "test:all": "npm run test:api && npm run test:builtins && npm run test:n3gen && npm run test:examples && npm run test:extra && npm run test:manifest && npm run test:playground",
     "pretest": "npm run build && npm run test:packlist",
     "test": "npm run test:all",
     "posttest": "npm run test:package",

package/test/{builtin-contract.test.js → builtins.test.js}

@@ -1,7 +1,6 @@
 'use strict';
 
 const assert = require('node:assert/strict');
-const path = require('node:path');
 
 const TTY = process.stdout.isTTY;
 const C = TTY
@@ -18,7 +17,6 @@ function fail(msg) {
   console.error(`${C.r}FAIL${C.n} ${msg}`);
 }
 
-const fixtures = path.join(__dirname, 'fixtures', 'builtins');
 const builtins = require('../lib/builtins');
 require('../lib/engine');
 
@@ -62,8 +60,43 @@ const expectedTermsKeys = [
   'Triple',
   'Rule',
 ].sort();
+
 const expectedNsKeys = ['RDF_NS', 'XSD_NS', 'CRYPTO_NS', 'MATH_NS', 'TIME_NS', 'LIST_NS', 'LOG_NS', 'STRING_NS'].sort();
 
+function makeOkMapModule() {
+  return {
+    'http://example.org/test#ok': ({ subst }) => [subst],
+  };
+}
+
+function makeOkRegisterModule() {
+  return {
+    register(api) {
+      api.registerBuiltin('http://example.org/test#ok-register', ({ subst }) => [subst]);
+    },
+  };
+}
+
+function makeOkBuiltinsModule() {
+  return {
+    builtins: {
+      'http://example.org/test#ok-builtins': ({ subst }) => [subst],
+    },
+  };
+}
+
+function makeOkDefaultMapModule() {
+  return {
+    default: {
+      'http://example.org/test#ok-default-map': ({ subst }) => [subst],
+    },
+  };
+}
+
+function makeBadExportModule() {
+  return 42;
+}
+
 const cases = [
   {
     name: 'builtin helper API stays stable and frozen',
@@ -80,23 +113,17 @@ const cases = [
   {
     name: 'registerBuiltinModule accepts supported module export forms',
     run() {
-      assert.doesNotThrow(() => builtins.registerBuiltinModule(require(path.join(fixtures, 'ok-map.js')), 'ok-map'));
-      assert.doesNotThrow(() =>
-        builtins.registerBuiltinModule(require(path.join(fixtures, 'ok-register.js')), 'ok-register'),
-      );
-      assert.doesNotThrow(() =>
-        builtins.registerBuiltinModule(require(path.join(fixtures, 'ok-builtins.js')), 'ok-builtins'),
-      );
-      assert.doesNotThrow(() =>
-        builtins.registerBuiltinModule(require(path.join(fixtures, 'ok-default-map.js')), 'ok-default-map'),
-      );
+      assert.doesNotThrow(() => builtins.registerBuiltinModule(makeOkMapModule(), 'ok-map'));
+      assert.doesNotThrow(() => builtins.registerBuiltinModule(makeOkRegisterModule(), 'ok-register'));
+      assert.doesNotThrow(() => builtins.registerBuiltinModule(makeOkBuiltinsModule(), 'ok-builtins'));
+      assert.doesNotThrow(() => builtins.registerBuiltinModule(makeOkDefaultMapModule(), 'ok-default-map'));
     },
   },
   {
     name: 'registerBuiltinModule rejects unsupported module exports',
     run() {
       assert.throws(
-        () => builtins.registerBuiltinModule(require(path.join(fixtures, 'bad-export.js')), 'bad-export'),
+        () => builtins.registerBuiltinModule(makeBadExportModule(), 'bad-export'),
         /must export a function, a \{ register\(\) \} object, or an object mapping predicate IRIs to handlers/,
       );
     },

package/test/fixtures/builtins/bad-export.js

@@ -1,3 +0,0 @@
-'use strict';
-
-module.exports = 42;

package/test/fixtures/builtins/ok-builtins.js

@@ -1,7 +0,0 @@
-'use strict';
-
-module.exports = {
-  builtins: {
-    'http://example.org/test#ok-builtins': ({ subst }) => [subst],
-  },
-};

package/test/fixtures/builtins/ok-default-map.js

@@ -1,7 +0,0 @@
-'use strict';
-
-module.exports = {
-  default: {
-    'http://example.org/test#ok-default-map': ({ subst }) => [subst],
-  },
-};

package/test/fixtures/builtins/ok-map.js

@@ -1,5 +0,0 @@
-'use strict';
-
-module.exports = {
-  'http://example.org/test#ok': ({ subst }) => [subst],
-};

package/test/fixtures/builtins/ok-register.js

@@ -1,7 +0,0 @@
-'use strict';
-
-module.exports = {
-  register(api) {
-    api.registerBuiltin('http://example.org/test#ok-register', ({ subst }) => [subst]);
-  },
-};