jaypie 0.1.4 → 0.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jaypie",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "author": "Finlayson Studio",
   "type": "module",
   "main": "src/index.js",
@@ -17,7 +17,7 @@
     "test:spec:mongoose.package": "vitest run ./src/__tests__/mongoose.package.spec.js"
   },
   "dependencies": {
-    "@jaypie/core": "^0.4.0"
+    "@jaypie/core": "^0.4.1"
   },
   "devDependencies": {
     "@jaypie/testkit": "^1.0.0",
@@ -34,7 +34,7 @@
   },
   "peerDependencies": {
     "@jaypie/aws": "^0.1.1",
-    "@jaypie/lambda": "^0.1.1",
+    "@jaypie/lambda": "^0.1.3",
     "@jaypie/mongoose": "^0.1.1"
   },
   "peerDependenciesMeta": {

package/src/__tests__/dynamicExport.function.spec.js

@@ -73,13 +73,24 @@ describe("Dynamic Export Function", () => {
     it("Returns an object", () => {
       expect(dynamicExport({ moduleImport: MOCK.MODULE })).toBeObject();
     });
-    it("Scalars are undefined when module is not found", async () => {
+    it("If module is not found and you try to get a scalar it will throw", async () => {
       const vars = ["scalar"];
       const result = await dynamicExport({
         vars,
         moduleImport: MOCK.MODULE,
       });
-      expect(result.scalar).toBeUndefined();
+      await expect(() => {
+        console.log("result.scalar :>> ", result.scalar);
+        try {
+          console.log("result.scalar.taco :>> ", result.scalar.taco);
+        } catch (error) {
+          console.log("error :>> ", error);
+        }
+        if (result.scalar) {
+          return true;
+        }
+        return false;
+      }).toThrow();
     });
     it("Exported functions throw if the real module is not installed", async () => {
       const functions = ["default"];
@@ -87,7 +98,17 @@ describe("Dynamic Export Function", () => {
         functions,
         moduleImport: MOCK.MODULE,
       });
-      await expect(result.default).toThrow();
+      await expect(() => {
+        result.default();
+      }).toThrow();
+    });
+    it("Scalars that never existed are undefined", async () => {
+      const vars = ["scalar"];
+      const result = await dynamicExport({
+        vars,
+        moduleImport: MOCK.MODULE,
+      });
+      expect(result.bogus).toBeUndefined();
     });
   });
   describe("Features", () => {

package/src/dynamicExport.function.js

@@ -56,37 +56,28 @@ export default async ({
       "Either `functions` or `vars` must be provided",
     );
   }
-  // Setup
-  let imported;
-  const returning = {};
-  // Preprocess
+  // Process
   try {
-    imported = await dynamicImport(moduleImport);
+    // Attempt to import the module
+    return await dynamicImport(moduleImport);
   } catch (error) {
     moduleLogger.trace(
       `[jaypie] ${moduleImport} could not be imported; continuing`,
     );
   }
-  // Process
-  for (const key of functions) {
-    if (imported) {
-      returning[key] = imported[key];
-    } else {
-      returning[key] = () => {
-        throw new ConfigurationError(
-          `Function ${key} cannot be called because ${moduleImport} is not installed`,
-        );
-      };
-    }
-  }
-  for (const key of vars) {
-    if (imported) {
-      returning[key] = imported[key];
-    } else {
-      // TODO: make the not found vars throw only _when accessed_
-      returning[key] = undefined;
-    }
-  }
   // Return
-  return returning;
+  // * Returns a proxy that throws if you call a functions or get a vars
+  return new Proxy(
+    {},
+    {
+      get: (target, prop) => {
+        if (functions.includes(prop) || vars.includes(prop)) {
+          throw new ConfigurationError(
+            `Attempted to access ${prop} from ${moduleImport}, but it is not installed`,
+          );
+        }
+        return undefined;
+      },
+    },
+  );
 };