Merge pull request #12 from WilliamRagstad/develop

Author: WilliamRagstad

deno.json

@@ -1,6 +1,6 @@
 {
   "name": "@knight/knight",
-  "version": "2.4.1",
+  "version": "2.4.2",
   "license": "MIT",
   "exports": "./mod.ts",
   "imports": {

src/Knight.ts

@@ -218,14 +218,6 @@ export class Knight {
     return path;
   }
 
-  /**
-   * Dynamically import a JavaScript module via a data URL
-   * @param file File path to read as javascript text and import dynamically via data URL
-   */
-  private static async dynamicImportUrl(file: string): Promise<string> {
-    return `data:text/javascript,${await Deno.readTextFile(file)}`;
-  }
-
   /**
    * Find all controllers in the local project
    */
@@ -237,7 +229,9 @@ export class Knight {
         const subControllers = await this.findLocalControllersIn(path);
         controllers.push(...subControllers);
       } else if (file.isFile && file.name.endsWith("Controller.ts")) {
-        const module = await import(await Knight.dynamicImportUrl(path));
+        // Dynamically import a JavaScript module via a data URL
+        const fileContent = await Deno.readTextFile(path);
+        const module = await import(`data:text/javascript,${fileContent}`);
         const defaultController = module.default;
         // Check that the default controller implements IController
         // console.log(module, defaultController);