Merge pull request #22 from chengpeiquan/develop

Develop

Author: chengpeiquan

package.json

@@ -1,7 +1,7 @@
 {
   "name": "vite-plugin-banner",
   "description": "A banner plugin for Vite. Adds a banner to the top of each generated chunk.",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "author": "chengpeiquan <[email protected]>",
   "license": "MIT",
   "homepage": "https://github.com/chengpeiquan/vite-plugin-banner",

src/libs/formatConfig.ts

@@ -25,9 +25,11 @@ export default function formatConfig(
   const type: string = Object.prototype.toString.call(options)
 
   // Block illegal types
-  if (!['[object String]', '[object Object]', '[object Function]'].includes(type)) {
+  if (
+    !['[object String]', '[object Object]', '[object Function]'].includes(type)
+  ) {
     throw new Error(
-      '[vite-plugin-banner] The options must be a string, an object or a callback function.'
+      '[vite-plugin-banner] The options must be a string, an object or a function.'
     )
   }
 

test/formatConfig.test.ts

@@ -0,0 +1,57 @@
+import formatConfig from '../src/libs/formatConfig'
+
+describe('formatConfig.ts', () => {
+  test('Illegal options type', () => {
+    expect(() => {
+      formatConfig(undefined as any)
+    }).toThrow(
+      '[vite-plugin-banner] The options must be a string, an object or a function.'
+    )
+  })
+})
+
+describe('formatConfig.ts', () => {
+  test('Empty content', () => {
+    expect(() => {
+      formatConfig({
+        content: '',
+        outDir: '',
+      })
+    }).toThrow('[vite-plugin-banner] The banner content can not be empty.')
+  })
+})
+
+describe('formatConfig.ts', () => {
+  test('Valid options type', () => {
+    expect(
+      formatConfig({
+        content: 'Hello World',
+        outDir: '',
+      })
+    ).toStrictEqual({
+      content: 'Hello World',
+      outDir: '',
+      debug: false,
+      verify: true,
+    })
+  })
+})
+
+describe('formatConfig.ts', () => {
+  test('Illegal options type', () => {
+    function callback(fileName: string) {
+      return fileName.endsWith('.js')
+        ? 'This message will inject into `js` files.'
+        : 'This message will inject into other files.'
+    }
+
+    expect(() => {
+      formatConfig(callback)
+    }).toStrictEqual({
+      content: expect.any(Function),
+      outDir: '',
+      debug: false,
+      verify: true,
+    })
+  })
+})

test/getConfig.test.ts

@@ -14,12 +14,16 @@ describe('verifyBanner.ts', () => {
 
 describe('verifyBanner.ts', () => {
   test('Illegal content type', () => {
-    expect(verifyBanner(null)).toBe('The banner content must be a string.')
+    expect(verifyBanner(null as any)).toBe(
+      'The banner content must be a string.'
+    )
   })
 })
 
 describe('verifyBanner.ts', () => {
   test('Illegal content type', () => {
-    expect(verifyBanner(undefined)).toBe('The banner content must be a string.')
+    expect(verifyBanner(undefined as any)).toBe(
+      'The banner content must be a string.'
+    )
   })
 })