fix: edge integration (#91)

* fix: edge integration * test: remove outdated test
adonisjs · Dec 10, 2023 · 362a3ca · 362a3ca
1 parent 0cb4f15
commit 362a3ca
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 3 deletions.
diff --git a/providers/mail_provider.ts b/providers/mail_provider.ts
@@ -38,7 +38,11 @@ export default class MailProvider {
   protected async defineTemplateEngine() {
     if (this.app.usingEdgeJS) {
       const edge = await import('edge.js')
-      Message.templateEngine = edge.default
+      Message.templateEngine = {
+        render(templatePath, helpers, data) {
+          return edge.default.share(helpers).render(templatePath, data)
+        },
+      }
     }
   }
 

diff --git a/tests/integration/mail_provider.spec.ts b/tests/integration/mail_provider.spec.ts
@@ -81,7 +81,7 @@ test.group('Mail Provider', () => {
     await app.container.make('mail.manager')
   }).throws('Invalid "config/mail.ts" file. Make sure you are using the "defineConfig" method')
 
-  test('share edge template engine with Message class', async ({ assert }) => {
+  test('correctly share helpers and view data with edge', async ({ assert }) => {
     const ignitor = new IgnitorFactory()
       .merge({
         rcFileContents: {
@@ -106,6 +106,15 @@ test.group('Mail Provider', () => {
     await app.init()
     await app.boot()
 
-    assert.strictEqual(Message.templateEngine, edge)
+    const message = new Message()
+    edge.registerTemplate('foo/bar', {
+      template: `Hello {{ username }} <img src="{{ embedImage('./foo.jpg') }}" />`,
+    })
+
+    message.htmlView('foo/bar', { username: 'jul' })
+    await message.computeContents()
+
+    assert.isDefined(message.nodeMailerMessage.attachments![0])
+    assert.include(message.nodeMailerMessage.html, 'Hello jul')
   })
 })