fix: invalid transformer export (#75)

svelteness · Sep 14, 2021 · 4729f0b · 4729f0b
1 parent 1d6057c
commit 4729f0b
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 12 deletions.
diff --git a/rollup.config.cjs b/rollup.config.cjs
@@ -14,7 +14,7 @@ module.exports = [
     output: {
       file: 'dist/transformer.cjs',
       format: 'cjs',
-      exports: 'named'
+      exports: 'default'
     },
     external,
     plugins: [

diff --git a/src/__tests__/transformer.test.cjs b/src/__tests__/transformer.test.cjs
@@ -1,11 +1,11 @@
 const { readFileSync } = require('fs')
 const { resolve } = require('path')
-const { createTransformer } = require('../../dist/transformer.cjs')
+const transformer = require('../../dist/transformer.cjs')
 
 const runTransformer = (filename, options) => {
   const path = require.resolve(`./fixtures/${filename}.svelte`)
   const source = readFileSync(path).toString()
-  const result = createTransformer().process(source, path, { transformerConfig: options })
+  const result = transformer.process(source, path, { transformerConfig: options })
   expect(result.code).toBeDefined()
   expect(result.code).toContain('SvelteComponent')
   expect(result.map).toBeDefined()

diff --git a/src/__tests__/transformer.test.js b/src/__tests__/transformer.test.js
@@ -3,15 +3,15 @@ import { dirname } from 'path'
 import { fileURLToPath } from 'url'
 import { jest } from '@jest/globals'
 
-import { processAsync } from '../../dist/transformer.mjs'
+import transformer from '../../dist/transformer.mjs'
 
 // Node API __dirname is missing in ESM
 export const __dirname = dirname(fileURLToPath(import.meta.url))
 
 const runTransformer = async (filename, options) => {
   const path = `${__dirname}/fixtures/${filename}.svelte`
   const source = readFileSync(path).toString()
-  const result = await processAsync(source, path, { transformerConfig: options })
+  const result = await transformer.processAsync(source, path, { transformerConfig: options })
   expect(result.code).toBeDefined()
   expect(result.code).toContain('SvelteComponent')
   expect(result.map).toBeDefined()

diff --git a/src/transformer.js b/src/transformer.js
@@ -10,7 +10,7 @@ const dynamicImport = async (filename) => import(pathToFileURL(filename))
 /**
  * Jest will only call this method when running in ESM mode.
  */
-export const processAsync = async (source, filename, jestOptions) => {
+const processAsync = async (source, filename, jestOptions) => {
   const options = jestOptions?.transformerConfig ?? {}
   const { preprocess, rootMode } = options
 
@@ -33,7 +33,7 @@ export const processAsync = async (source, filename, jestOptions) => {
  * Starts a new process, so is higher overhead than processAsync.
  * However, Jest calls this method in CJS mode.
  */
-const processSync = (source, filename, jestOptions) => {
+const process = (source, filename, jestOptions) => {
   const options = jestOptions?.transformerConfig ?? {}
   const { preprocess, rootMode, maxBuffer, showConsoleLog } = options
   if (!preprocess) {
@@ -80,9 +80,7 @@ const compiler = (format, options = {}, filename, processedCode, processedMap) =
   }
 }
 
-export const createTransformer = () => ({
-  process: processSync,
+export default {
+  process,
   processAsync
-})
-
-export default createTransformer
+}