Optimize test performance by reading files once per describe block

Copilot · taylortom · Copilot · commit bcf8ba1e72fd · 2026-02-10T22:33:42.000Z
Co-authored-by: taylortom &lt;1059083+taylortom@users.noreply.github.com&gt;
diff --git a/tests/AbstractMailTransport.spec.js b/tests/AbstractMailTransport.spec.js
@@ -1,4 +1,4 @@
-import { describe, it } from 'node:test'
+import { describe, it, before } from 'node:test'
 import assert from 'node:assert/strict'
 import { readFileSync } from 'node:fs'
 import { join, dirname } from 'node:path'
@@ -7,35 +7,32 @@ import { fileURLToPath } from 'node:url'
 const __dirname = dirname(fileURLToPath(import.meta.url))
 
 describe('AbstractMailTransport', () => {
+  let content
+
+  before(() => {
+    const filePath = join(__dirname, '../lib/AbstractMailTransport.js')
+    content = readFileSync(filePath, 'utf-8')
+  })
+
   describe('module structure', () => {
     it('should have valid JavaScript syntax', () => {
-      const filePath = join(__dirname, '../lib/AbstractMailTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.length > 0)
       assert.ok(content.includes('class AbstractMailTransport'))
     })
 
     it('should export default class', () => {
-      const filePath = join(__dirname, '../lib/AbstractMailTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('export default AbstractMailTransport'))
     })
 
     it('should define send method', () => {
-      const filePath = join(__dirname, '../lib/AbstractMailTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async send'))
     })
 
     it('should define test method', () => {
-      const filePath = join(__dirname, '../lib/AbstractMailTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async test'))
     })
 
     it('should define getConfig method', () => {
-      const filePath = join(__dirname, '../lib/AbstractMailTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('getConfig'))
     })
   })
diff --git a/tests/FilesystemTransport.spec.js b/tests/FilesystemTransport.spec.js
@@ -1,4 +1,4 @@
-import { describe, it } from 'node:test'
+import { describe, it, before } from 'node:test'
 import assert from 'node:assert/strict'
 import { readFileSync } from 'node:fs'
 import { join, dirname } from 'node:path'
@@ -7,47 +7,40 @@ import { fileURLToPath } from 'node:url'
 const __dirname = dirname(fileURLToPath(import.meta.url))
 
 describe('FilesystemTransport', () => {
+  let content
+
+  before(() => {
+    const filePath = join(__dirname, '../lib/transports/FilesystemTransport.js')
+    content = readFileSync(filePath, 'utf-8')
+  })
+
   describe('module structure', () => {
     it('should have valid JavaScript syntax', () => {
-      const filePath = join(__dirname, '../lib/transports/FilesystemTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.length > 0)
       assert.ok(content.includes('class FilesystemTransport'))
     })
 
     it('should export default class', () => {
-      const filePath = join(__dirname, '../lib/transports/FilesystemTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('export default FilesystemTransport'))
     })
 
     it('should extend AbstractMailTransport', () => {
-      const filePath = join(__dirname, '../lib/transports/FilesystemTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('extends AbstractMailTransport'))
     })
 
     it('should define name property as filesystem', () => {
-      const filePath = join(__dirname, '../lib/transports/FilesystemTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes("name = 'filesystem'"))
     })
 
     it('should define outputDir getter', () => {
-      const filePath = join(__dirname, '../lib/transports/FilesystemTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('get outputDir'))
     })
 
     it('should define send method', () => {
-      const filePath = join(__dirname, '../lib/transports/FilesystemTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async send'))
     })
 
     it('should define test method', () => {
-      const filePath = join(__dirname, '../lib/transports/FilesystemTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async test'))
     })
   })
diff --git a/tests/MailerModule.spec.js b/tests/MailerModule.spec.js
@@ -1,4 +1,4 @@
-import { describe, it } from 'node:test'
+import { describe, it, before } from 'node:test'
 import assert from 'node:assert/strict'
 import { readFileSync } from 'node:fs'
 import { join, dirname } from 'node:path'
@@ -7,59 +7,48 @@ import { fileURLToPath } from 'node:url'
 const __dirname = dirname(fileURLToPath(import.meta.url))
 
 describe('MailerModule', () => {
+  let content
+
+  before(() => {
+    const filePath = join(__dirname, '../lib/MailerModule.js')
+    content = readFileSync(filePath, 'utf-8')
+  })
+
   describe('module structure', () => {
     it('should have valid JavaScript syntax', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.length > 0)
       assert.ok(content.includes('class MailerModule'))
     })
 
     it('should export default class', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('export default MailerModule'))
     })
 
     it('should extend AbstractModule', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('extends AbstractModule'))
     })
 
     it('should define init method', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async init'))
     })
 
     it('should define registerTransport method', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('registerTransport'))
     })
 
     it('should define getTransport method', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('getTransport'))
     })
 
     it('should define initTransports method', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async initTransports'))
     })
 
     it('should define send method', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async send'))
     })
 
     it('should define testEmailHandler method', () => {
-      const filePath = join(__dirname, '../lib/MailerModule.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async testEmailHandler'))
     })
   })
diff --git a/tests/SmtpTransport.spec.js b/tests/SmtpTransport.spec.js
@@ -1,4 +1,4 @@
-import { describe, it } from 'node:test'
+import { describe, it, before } from 'node:test'
 import assert from 'node:assert/strict'
 import { readFileSync } from 'node:fs'
 import { join, dirname } from 'node:path'
@@ -7,47 +7,40 @@ import { fileURLToPath } from 'node:url'
 const __dirname = dirname(fileURLToPath(import.meta.url))
 
 describe('SmtpTransport', () => {
+  let content
+
+  before(() => {
+    const filePath = join(__dirname, '../lib/transports/SmtpTransport.js')
+    content = readFileSync(filePath, 'utf-8')
+  })
+
   describe('module structure', () => {
     it('should have valid JavaScript syntax', () => {
-      const filePath = join(__dirname, '../lib/transports/SmtpTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.length > 0)
       assert.ok(content.includes('class SmtpTransport'))
     })
 
     it('should export default class', () => {
-      const filePath = join(__dirname, '../lib/transports/SmtpTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('export default SmtpTransport'))
     })
 
     it('should extend AbstractMailTransport', () => {
-      const filePath = join(__dirname, '../lib/transports/SmtpTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('extends AbstractMailTransport'))
     })
 
     it('should define name property as smtp', () => {
-      const filePath = join(__dirname, '../lib/transports/SmtpTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes("name = 'smtp'"))
     })
 
     it('should define createTransport method', () => {
-      const filePath = join(__dirname, '../lib/transports/SmtpTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('createTransport'))
     })
 
     it('should define send method', () => {
-      const filePath = join(__dirname, '../lib/transports/SmtpTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async send'))
     })
 
     it('should define test method', () => {
-      const filePath = join(__dirname, '../lib/transports/SmtpTransport.js')
-      const content = readFileSync(filePath, 'utf-8')
       assert.ok(content.includes('async test'))
     })
   })
