Fix version

Author: owjs3901

.github/workflows/CI.yml

@@ -24,7 +24,7 @@ jobs:
       - uses: actions/checkout@v6
       - uses: oven-sh/setup-bun@v2
         with:
-          bun-version: 1.3.9
+          bun-version: latest
       - name: Install dependencies
         run: bun install
       - name: Lint

src/commands/devup/utils/__tests__/download-devup-xlsx.test.ts

@@ -1,15 +1,11 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { describe, expect, mock, test } from 'bun:test'
 import { downloadDevupXlsx } from '../download-devup-xlsx'
 
 describe('downloadDevupXlsx', () => {
-  let showUIMock: ReturnType<typeof mock>
-  let postMessageMock: ReturnType<typeof mock>
-  let onmessageHandler: ((message: unknown) => void) | null = null
-
-  beforeEach(() => {
-    showUIMock = mock(() => {})
-    postMessageMock = mock(() => {})
-    onmessageHandler = null
+  function createMockFigma() {
+    const showUIMock = mock(() => {})
+    const postMessageMock = mock(() => {})
+    let onmessageHandler: ((message: unknown) => void) | null = null
 
     const uiObj: {
       onmessage?: (message: unknown) => void
@@ -26,18 +22,26 @@ describe('downloadDevupXlsx', () => {
 
     uiObj.postMessage = postMessageMock
 
-    ;(globalThis as { figma?: unknown }).figma = {
+    const ctx = {
       showUI: showUIMock,
       ui: uiObj,
     } as unknown as typeof figma
-  })
 
-  afterEach(() => {
-    ;(globalThis as { figma?: unknown }).figma = undefined
-  })
+    return {
+      ctx,
+      showUIMock,
+      postMessageMock,
+      getHandler: () => onmessageHandler,
+    }
+  }
 
   test('should call showUI with correct HTML string and visible false', () => {
-    downloadDevupXlsx('test.xlsx', '{"theme":{"colors":{},"typography":{}}}')
+    const { ctx, showUIMock } = createMockFigma()
+    downloadDevupXlsx(
+      'test.xlsx',
+      '{"theme":{"colors":{},"typography":{}}}',
+      ctx,
+    )
     expect(showUIMock).toHaveBeenCalledWith(
       expect.stringContaining('xlsx-0.20.3'),
       { visible: false },
@@ -49,8 +53,13 @@ describe('downloadDevupXlsx', () => {
   })
 
   test('should set onmessage handler and post message', () => {
-    downloadDevupXlsx('test.xlsx', '{"theme":{"colors":{},"typography":{}}}')
-    expect(onmessageHandler).not.toBeNull()
+    const { ctx, getHandler, postMessageMock } = createMockFigma()
+    downloadDevupXlsx(
+      'test.xlsx',
+      '{"theme":{"colors":{},"typography":{}}}',
+      ctx,
+    )
+    expect(getHandler()).not.toBeNull()
     expect(postMessageMock).toHaveBeenCalledWith({
       type: 'download',
       fileName: 'test.xlsx',
@@ -59,26 +68,31 @@ describe('downloadDevupXlsx', () => {
   })
 
   test('should return a promise that resolves when onmessage is called', async () => {
+    const { ctx, getHandler, postMessageMock } = createMockFigma()
     const promise = downloadDevupXlsx(
       'test.xlsx',
       '{"theme":{"colors":{},"typography":{}}}',
+      ctx,
     )
 
     // Simulate message from UI
-    if (onmessageHandler) {
-      onmessageHandler(undefined)
+    const handler = getHandler()
+    if (handler) {
+      handler(undefined)
     }
 
     await promise
     expect(postMessageMock).toHaveBeenCalled()
   })
 
   test('should handle different file names and data', () => {
+    const { ctx, postMessageMock } = createMockFigma()
     downloadDevupXlsx(
       'devup.xlsx',
       JSON.stringify({
         theme: { colors: { light: { primary: '#000' } }, typography: {} },
       }),
+      ctx,
     )
     expect(postMessageMock).toHaveBeenCalledWith({
       type: 'download',

src/commands/devup/utils/__tests__/upload-devup-xlsx.test.ts

@@ -1,16 +1,12 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { describe, expect, mock, test } from 'bun:test'
 import type { Devup } from '../../types'
 import { uploadDevupXlsx } from '../upload-devup-xlsx'
 
 describe('uploadDevupXlsx', () => {
-  let showUIMock: ReturnType<typeof mock>
-  let closeMock: ReturnType<typeof mock>
-  let onmessageHandler: ((message: string) => void) | null = null
-
-  beforeEach(() => {
-    showUIMock = mock(() => {})
-    closeMock = mock(() => {})
-    onmessageHandler = null
+  function createMockFigma() {
+    const showUIMock = mock(() => {})
+    const closeMock = mock(() => {})
+    let onmessageHandler: ((message: string) => void) | null = null
 
     const uiObj: {
       onmessage?: (message: string) => void
@@ -27,18 +23,17 @@ describe('uploadDevupXlsx', () => {
 
     uiObj.close = closeMock
 
-    ;(globalThis as { figma?: unknown }).figma = {
+    const ctx = {
       showUI: showUIMock,
       ui: uiObj,
     } as unknown as typeof figma
-  })
 
-  afterEach(() => {
-    ;(globalThis as { figma?: unknown }).figma = undefined
-  })
+    return { ctx, showUIMock, closeMock, getHandler: () => onmessageHandler }
+  }
 
   test('should call showUI with correct HTML string', () => {
-    uploadDevupXlsx()
+    const { ctx, showUIMock } = createMockFigma()
+    uploadDevupXlsx(ctx)
     expect(showUIMock).toHaveBeenCalledWith(
       expect.stringContaining('accept=".xlsx"'),
     )
@@ -48,12 +43,14 @@ describe('uploadDevupXlsx', () => {
   })
 
   test('should resolve with parsed JSON when message is received', async () => {
+    const { ctx, closeMock, getHandler } = createMockFigma()
     const testData = { theme: { colors: {}, typography: {} } }
-    const promise = uploadDevupXlsx()
+    const promise = uploadDevupXlsx(ctx)
 
     // Simulate message from UI
-    if (onmessageHandler) {
-      onmessageHandler(JSON.stringify(testData))
+    const handler = getHandler()
+    if (handler) {
+      handler(JSON.stringify(testData))
     }
 
     const result = await promise
@@ -62,6 +59,7 @@ describe('uploadDevupXlsx', () => {
   })
 
   test('should handle message with colors and typography', async () => {
+    const { ctx, getHandler } = createMockFigma()
     const testData = {
       theme: {
         colors: {
@@ -77,10 +75,11 @@ describe('uploadDevupXlsx', () => {
         },
       },
     }
-    const promise = uploadDevupXlsx()
+    const promise = uploadDevupXlsx(ctx)
 
-    if (onmessageHandler) {
-      onmessageHandler(JSON.stringify(testData))
+    const handler = getHandler()
+    if (handler) {
+      handler(JSON.stringify(testData))
     }
 
     const result = await promise

src/commands/devup/utils/download-devup-xlsx.ts

@@ -3,16 +3,20 @@
  * @param fileName
  * @param data
  */
-export async function downloadDevupXlsx(fileName: string, data: string) {
-  figma.showUI(downloadFileUi(), {
+export async function downloadDevupXlsx(
+  fileName: string,
+  data: string,
+  ctx: typeof figma = figma,
+) {
+  ctx.showUI(downloadFileUi(), {
     visible: false,
   })
 
   const pro = new Promise((resolve) => {
-    figma.ui.onmessage = resolve
+    ctx.ui.onmessage = resolve
   })
 
-  figma.ui.postMessage({
+  ctx.ui.postMessage({
     type: 'download',
     fileName,
     data,

src/commands/devup/utils/upload-devup-xlsx.ts

@@ -5,11 +5,13 @@ import type { Devup } from '../types'
  * @param fileName
  * @param data
  */
-export async function uploadDevupXlsx(): Promise<Devup> {
-  figma.showUI(uploadFileUi('.xlsx'))
+export async function uploadDevupXlsx(
+  ctx: typeof figma = figma,
+): Promise<Devup> {
+  ctx.showUI(uploadFileUi('.xlsx'))
   return new Promise((resolve) => {
-    figma.ui.onmessage = (message) => {
-      figma.ui.close()
+    ctx.ui.onmessage = (message) => {
+      ctx.ui.close()
       resolve(JSON.parse(message))
     }
   })