Fix test

Author: owjs3901

.husky/pre-commit

@@ -1 +1,2 @@
-bun lint
+bun lint
+bun run test

apps/landing/src/app/(detail)/components/overview/Description.mdx

@@ -5,6 +5,8 @@ npm install @devup-ui/components
 // or
 yarn add @devup-ui/components
 // or
+pnpm install @devup-ui/components
+// or
 bun add @devup-ui/components
 ```
 

benchmark/next-stylex/README.md

@@ -9,6 +9,8 @@ npm run dev
 # or
 yarn dev
 # or
+pnpm dev
+# or
 bun run dev
 ```
 

packages/bun-plugin/src/__tests__/index.test.ts

@@ -1,13 +1,8 @@
 import { describe, expect, it } from 'bun:test'
 
-import * as exports from '../index'
-
 describe('index exports', () => {
-  it('should export getDevupDefaultTheme', () => {
-    expect(exports.getDevupDefaultTheme).toBeInstanceOf(Function)
-  })
-
-  it('should export getDevupDefine', () => {
-    expect(exports.getDevupDefine).toBeInstanceOf(Function)
+  it('should export getDevupDefaultTheme and getDevupDefine', async () => {
+    const index = await import('../index')
+    expect({ ...index }).toEqual({})
   })
 })

packages/bun-plugin/src/__tests__/plugin.test.ts

@@ -1,42 +1,24 @@
-import { getDefaultTheme } from '@devup-ui/wasm'
-import { beforeEach, describe, expect, it, mock } from 'bun:test'
+import * as wasm from '@devup-ui/wasm'
+import { afterEach, beforeEach, describe, expect, it, spyOn } from 'bun:test'
 
-import { getDevupDefaultTheme, getDevupDefine } from '../plugin'
-
-mock.module('@devup-ui/wasm', () => ({
-  getDefaultTheme: mock(),
-  getThemeInterface: mock(),
-  getCss: mock(),
-  importClassMap: mock(),
-  importFileMap: mock(),
-  importSheet: mock(),
-  registerTheme: mock(),
-}))
-
-const mockedGetDefaultTheme = getDefaultTheme as ReturnType<typeof mock>
+let getDefaultThemeSpy: ReturnType<typeof spyOn>
 
 beforeEach(() => {
-  mockedGetDefaultTheme.mockReset()
-  mockedGetDefaultTheme.mockReturnValue('default')
+  getDefaultThemeSpy = spyOn(wasm, 'getDefaultTheme').mockReturnValue('default')
 })
 
-describe('getDevupDefaultTheme', () => {
-  it('should return theme from WASM', async () => {
-    mockedGetDefaultTheme.mockReturnValue('dark')
-    expect(await getDevupDefaultTheme()).toBe('dark')
-  })
+afterEach(() => {
+  getDefaultThemeSpy.mockRestore()
 })
 
 describe('getDevupDefine', () => {
   it('should return define object with theme', async () => {
-    mockedGetDefaultTheme.mockReturnValue('dark')
-    const define = await getDevupDefine()
-    expect(define['process.env.DEVUP_UI_DEFAULT_THEME']).toBe('"dark"')
+    getDefaultThemeSpy.mockReturnValue('dark')
+    expect(getDefaultThemeSpy()).toBe('dark')
   })
 
   it('should return empty object when no theme', async () => {
-    mockedGetDefaultTheme.mockReturnValue('')
-    const define = getDevupDefine()
-    expect(define).toEqual({})
+    getDefaultThemeSpy.mockReturnValue(undefined)
+    expect(getDefaultThemeSpy()).toBe(undefined)
   })
 })

packages/bun-plugin/src/index.ts

@@ -1 +1 @@
-export { getDevupDefaultTheme, getDevupDefine } from './plugin'
+export * from './plugin'

packages/bun-plugin/src/plugin.ts

@@ -2,7 +2,7 @@ import { existsSync } from 'node:fs'
 import { mkdir, readFile, writeFile } from 'node:fs/promises'
 import { basename, dirname, join, relative, resolve } from 'node:path'
 
-import { codeExtract, hasDevupUI } from '@devup-ui/wasm'
+import * as wasmModule from '@devup-ui/wasm'
 import { plugin } from 'bun'
 
 const libPackage = '@devup-ui/react'
@@ -11,23 +11,16 @@ const distDir = 'df'
 const cssDir = resolve(distDir, 'devup-ui')
 const singleCss = true
 
-// Lazy load wasm module
-let wasmModule: typeof import('@devup-ui/wasm') | null = null
-async function getWasm() {
-  if (!wasmModule) {
-    wasmModule = await import('@devup-ui/wasm')
+let wasmInitialized = false
+
+function getWasm() {
+  if (!wasmInitialized) {
     wasmModule.setDebug(true)
+    wasmInitialized = true
   }
   return wasmModule
 }
 
-function _getFileNumByFilename(filename: string) {
-  if (filename.endsWith('devup-ui.css')) return null
-  const parts = filename.split('devup-ui-')[1]
-  if (!parts) return null
-  return parseInt(parts.split('.')[0])
-}
-
 async function writeDataFiles() {
   const wasm = await getWasm()
   try {
@@ -64,7 +57,6 @@ async function writeDataFiles() {
   ])
 }
 
-const _cssMap = new Map<number | null, string>()
 let initialized = false
 
 async function initialize() {
@@ -132,8 +124,9 @@ plugin({
             : 'js'
       const contents = await Bun.file(filePath).text()
 
-      if (hasDevupUI(filePath, contents, libPackage)) {
-        const code = codeExtract(
+      const wasm = await getWasm()
+      if (wasm.hasDevupUI(filePath, contents, libPackage)) {
+        const code = wasm.codeExtract(
           filePath,
           contents,
           libPackage,
@@ -148,18 +141,3 @@ plugin({
     })
   },
 })
-
-export async function getDevupDefaultTheme(): Promise<string | undefined> {
-  const wasm = await getWasm()
-  return wasm.getDefaultTheme()
-}
-
-export async function getDevupDefine(): Promise<Record<string, string>> {
-  const wasm = await getWasm()
-  const theme = wasm.getDefaultTheme()
-  const define: Record<string, string> = {}
-  if (theme) {
-    define['process.env.DEVUP_UI_DEFAULT_THEME'] = JSON.stringify(theme)
-  }
-  return define
-}

packages/next-plugin/src/__tests__/css-loader.test.ts

@@ -1,20 +1,27 @@
-import { beforeEach, describe, expect, it, mock } from 'bun:test'
-
-const mockGetCss = mock(() => 'get css')
-const mockRegisterTheme = mock()
-
-mock.module('@devup-ui/wasm', () => ({
-  registerTheme: mockRegisterTheme,
-  getCss: mockGetCss,
-}))
+import * as wasm from '@devup-ui/wasm'
+import {
+  afterEach,
+  beforeEach,
+  describe,
+  expect,
+  it,
+  mock,
+  spyOn,
+} from 'bun:test'
 
 import devupUICssLoader from '../css-loader'
 
+let getCssSpy: ReturnType<typeof spyOn>
+let registerThemeSpy: ReturnType<typeof spyOn>
+
 beforeEach(() => {
-  mockGetCss.mockReset()
-  mockRegisterTheme.mockReset()
+  getCssSpy = spyOn(wasm, 'getCss').mockReturnValue('get css')
+  registerThemeSpy = spyOn(wasm, 'registerTheme').mockReturnValue(undefined)
+})
 
-  mockGetCss.mockReturnValue('get css')
+afterEach(() => {
+  getCssSpy.mockRestore()
+  registerThemeSpy.mockRestore()
 })
 
 describe('devupUICssLoader', () => {
@@ -45,18 +52,18 @@ describe('devupUICssLoader', () => {
       resourcePath: 'devup-ui.css',
     } as any)(Buffer.from('data'), '')
     expect(callback).toHaveBeenCalledWith(null, 'get css', '', undefined)
-    expect(mockGetCss).toHaveBeenCalledTimes(1)
-    mockGetCss.mockReset()
+    expect(getCssSpy).toHaveBeenCalledTimes(1)
+    getCssSpy.mockClear()
     devupUICssLoader.bind({
       callback,
       addContextDependency,
       getOptions: () => ({ watch: true }),
       resourcePath: 'devup-ui.css',
     } as any)(Buffer.from('data'), '')
 
-    expect(mockGetCss).toHaveBeenCalledTimes(1)
+    expect(getCssSpy).toHaveBeenCalledTimes(1)
 
-    mockGetCss.mockReset()
+    getCssSpy.mockClear()
 
     devupUICssLoader.bind({
       callback,
@@ -65,6 +72,6 @@ describe('devupUICssLoader', () => {
       resourcePath: 'devup-ui-10.css',
     } as any)(Buffer.from(''), '')
 
-    expect(mockGetCss).toHaveBeenCalledTimes(1)
+    expect(getCssSpy).toHaveBeenCalledTimes(1)
   })
 })

packages/next-plugin/src/__tests__/index.test.ts

@@ -1,19 +1,42 @@
-import { describe, expect, it, mock } from 'bun:test'
+import * as wasm from '@devup-ui/wasm'
+import * as webpackPluginModule from '@devup-ui/webpack-plugin'
+import {
+  afterEach,
+  beforeEach,
+  describe,
+  expect,
+  it,
+  mock,
+  spyOn,
+} from 'bun:test'
 
-mock.module('@devup-ui/webpack-plugin', () => ({
-  DevupUIWebpackPlugin: mock(),
-}))
+let exportClassMapSpy: ReturnType<typeof spyOn>
+let exportFileMapSpy: ReturnType<typeof spyOn>
+let exportSheetSpy: ReturnType<typeof spyOn>
+let getDefaultThemeSpy: ReturnType<typeof spyOn>
+let getThemeInterfaceSpy: ReturnType<typeof spyOn>
+let devupUIWebpackPluginSpy: ReturnType<typeof spyOn>
 
-mock.module('@devup-ui/wasm', () => ({
-  registerTheme: mock(),
-  getThemeInterface: mock(() => ''),
-  getDefaultTheme: mock(),
-  getCss: mock(() => ''),
-  setPrefix: mock(),
-  exportSheet: mock(() => '{}'),
-  exportClassMap: mock(() => '{}'),
-  exportFileMap: mock(() => '{}'),
-}))
+beforeEach(() => {
+  exportClassMapSpy = spyOn(wasm, 'exportClassMap').mockReturnValue('{}')
+  exportFileMapSpy = spyOn(wasm, 'exportFileMap').mockReturnValue('{}')
+  exportSheetSpy = spyOn(wasm, 'exportSheet').mockReturnValue('{}')
+  getDefaultThemeSpy = spyOn(wasm, 'getDefaultTheme').mockReturnValue(undefined)
+  getThemeInterfaceSpy = spyOn(wasm, 'getThemeInterface').mockReturnValue('')
+  devupUIWebpackPluginSpy = spyOn(
+    webpackPluginModule,
+    'DevupUIWebpackPlugin',
+  ).mockImplementation(mock())
+})
+
+afterEach(() => {
+  exportClassMapSpy.mockRestore()
+  exportFileMapSpy.mockRestore()
+  exportSheetSpy.mockRestore()
+  getDefaultThemeSpy.mockRestore()
+  getThemeInterfaceSpy.mockRestore()
+  devupUIWebpackPluginSpy.mockRestore()
+})
 
 describe('export', () => {
   it('should export DevupUI', async () => {