feat: add support for removing inline registered templates

.eslintrc.json

@@ -1,8 +1,7 @@
 {
   "extends": [
     "plugin:adonis/typescriptPackage",
-    "prettier",
-    "prettier/@typescript-eslint"
+    "prettier"
   ],
   "plugins": [
     "prettier"

src/CacheManager/index.ts

@@ -48,4 +48,15 @@ export class CacheManager implements CacheManagerContract {
 
     this.cacheStore.set(absPath, payload)
   }
+
+  /**
+   * Delete template from the compiled cache
+   */
+  public delete(absPath: string) {
+    if (!this.enabled) {
+      return
+    }
+
+    this.cacheStore.delete(absPath)
+  }
 }

src/Contracts/index.ts

@@ -56,6 +56,11 @@ export interface LoaderContract {
    * Register in memory template and presenter
    */
   register(templatePath: string, contents: LoaderTemplate): void
+
+  /**
+   * Remove the pre-registered template
+   */
+  remove(templatePath: string): void
 }
 
 /**
@@ -95,6 +100,7 @@ export interface CacheManagerContract {
   get(templatePath: string): undefined | LoaderTemplate
   set(templatePath: string, compiledOutput: LoaderTemplate): void
   has(templatePath: string): boolean
+  delete(templatePath: string): void
 }
 
 /**
@@ -320,6 +326,11 @@ export interface EdgeContract {
    */
   registerTemplate(templatePath: string, contents: LoaderTemplate): this
 
+  /**
+   * Remove the template registered using the "registerTemplate" method
+   */
+  removeTemplate(templatePath: string): this
+
   /**
    * Register a global value
    */

src/Edge/index.ts

@@ -223,6 +223,16 @@ export class Edge implements EdgeContract {
     return this
   }
 
+  /**
+   * Remove the template registered using the "registerTemplate" method
+   */
+  public removeTemplate(templatePath: string): this {
+    this.loader.remove(templatePath)
+    this.compiler.cacheManager.delete(templatePath)
+    this.asyncCompiler.cacheManager.delete(templatePath)
+    return this
+  }
+
   /**
    * Returns a new instance of edge. The instance
    * can be used to define locals.

src/Loader/index.ts

@@ -258,4 +258,11 @@ export class Loader implements LoaderContract {
 
     this.preRegistered.set(templatePath, contents)
   }
+
+  /**
+   * Remove registered template
+   */
+  public remove(templatePath: string) {
+    this.preRegistered.delete(templatePath)
+  }
 }

test/edge.spec.ts

@@ -124,6 +124,23 @@ test.group('Edge', (group) => {
     assert.equal(edge.render('hello::foo', { username: 'virk' }).trim(), 'Hello virk')
   })
 
+  test('clear compiled cache when template is removed', async (assert) => {
+    const edge = new Edge({ cache: true })
+
+    edge.registerTemplate('foo', {
+      template: 'Hello {{ username }}',
+    })
+    assert.equal(edge.render('foo', { username: 'virk' }).trim(), 'Hello virk')
+    assert.equal((await edge.renderAsync('foo', { username: 'virk' })).trim(), 'Hello virk')
+
+    edge.removeTemplate('foo')
+    edge.registerTemplate('foo', {
+      template: 'Hi {{ username }}',
+    })
+    assert.equal(edge.render('foo', { username: 'virk' }).trim(), 'Hi virk')
+    assert.equal((await edge.renderAsync('foo', { username: 'virk' })).trim(), 'Hi virk')
+  })
+
   test('pass absolute path of template to lexer errors', async (assert) => {
     assert.plan(1)
     await fs.add('foo.edge', '@if(1 + 1)')

test/loader.spec.ts

@@ -119,6 +119,23 @@ test.group('Loader', (group) => {
     assert.equal(template.trim(), 'Hello world')
   })
 
+  test('remove registered template', async (assert) => {
+    const loader = new Loader()
+    loader.register('my-view', {
+      template: 'Hello world',
+    })
+    loader.mount('default', __dirname)
+
+    const { template } = loader.resolve('my-view')
+    assert.equal(template.trim(), 'Hello world')
+
+    loader.remove('my-view')
+    assert.throw(
+      () => loader.resolve('my-view'),
+      `Cannot resolve "${join(__dirname, 'my-view.edge')}". Make sure the file exists`
+    )
+  })
+
   test('pre registering duplicate templates must raise an error', async (assert) => {
     const loader = new Loader()
     loader.register('my-view', { template: 'Hello world' })