@zhin.js/core 1.0.5 → 1.0.6

package/tests/command.test.ts

@@ -34,54 +34,56 @@ vi.mock('segment-matcher', () => {
   return { SegmentMatcher, MatchResult }
 })
 
-// Mock Plugin
-const mockPlugin = {
-  name: 'test-plugin',
-  getPermit: vi.fn((permission: string) => {
-    // Mock permit checker
-    return {
-      check: vi.fn(async (perm: string, message: any) => {
-        if (permission === 'adapter(discord)') {
-          return message.$adapter === 'discord'
-        }
-        if (permission === 'adapter(telegram)') {
-          return message.$adapter === 'telegram'
-        }
-        if (permission === 'adapter(email)') {
-          return message.$adapter === 'email'
-        }
-        if (permission === 'adapter(test)') {
-          return message.$adapter === 'test'
-        }
-        return true
-      })
-    }
-  })
+// Mock App with permissions
+const mockApp = {
+  permissions: {
+    get: vi.fn((permission: string) => {
+      // Mock permit checker
+      return {
+        check: vi.fn(async (perm: string, message: any) => {
+          if (permission === 'adapter(discord)') {
+            return message.$adapter === 'discord'
+          }
+          if (permission === 'adapter(telegram)') {
+            return message.$adapter === 'telegram'
+          }
+          if (permission === 'adapter(email)') {
+            return message.$adapter === 'email'
+          }
+          if (permission === 'adapter(test)') {
+            return message.$adapter === 'test'
+          }
+          return true
+        })
+      }
+    })
+  }
 } as any
 
-// 为多个权限测试创建特殊的 mock plugin
-const multiPermitMockPlugin = {
-  name: 'test-plugin',
-  getPermit: vi.fn((permission: string) => {
-    return {
-      check: vi.fn(async (perm: string, message: any) => {
-        // 对于多个权限，只要有一个匹配就返回 true
-        if (permission === 'adapter(discord)' && message.$adapter === 'discord') {
-          return true
-        }
-        if (permission === 'adapter(telegram)' && message.$adapter === 'telegram') {
-          return true
-        }
-        if (permission === 'adapter(email)' && message.$adapter === 'email') {
-          return true
-        }
-        if (permission === 'adapter(test)' && message.$adapter === 'test') {
-          return true
-        }
-        return false
-      })
-    }
-  })
+// 为多个权限测试创建特殊的 mock app
+const multiPermitMockApp = {
+  permissions: {
+    get: vi.fn((permission: string) => {
+      return {
+        check: vi.fn(async (perm: string, message: any) => {
+          // 对于多个权限，只要有一个匹配就返回 true
+          if (permission === 'adapter(discord)' && message.$adapter === 'discord') {
+            return true
+          }
+          if (permission === 'adapter(telegram)' && message.$adapter === 'telegram') {
+            return true
+          }
+          if (permission === 'adapter(email)' && message.$adapter === 'email') {
+            return true
+          }
+          if (permission === 'adapter(test)' && message.$adapter === 'test') {
+            return true
+          }
+          return false
+        })
+      }
+    })
+  }
 } as any
 
 describe('Command系统测试', () => {
@@ -135,8 +137,8 @@ describe('Command系统测试', () => {
         $raw: 'hello world'
       }
 
-      const discordResult = await command.handle(discordMessage, mockPlugin)
-      const telegramResult = await command.handle(telegramMessage, mockPlugin)
+      const discordResult = await command.handle(discordMessage, mockApp)
+      const telegramResult = await command.handle(telegramMessage, mockApp)
 
       expect(discordResult).toBe('Hello from Discord!')
       expect(telegramResult).toBeUndefined()
@@ -183,9 +185,9 @@ describe('Command系统测试', () => {
         $raw: 'hello'
       }
 
-      const discordResult = await command.handle(discordMessage, mockPlugin)
-      const telegramResult = await command.handle(telegramMessage, mockPlugin)
-      const emailResult = await command.handle(emailMessage, mockPlugin)
+      const discordResult = await command.handle(discordMessage, mockApp)
+      const telegramResult = await command.handle(telegramMessage, mockApp)
+      const emailResult = await command.handle(emailMessage, mockApp)
 
       expect(discordResult).toBe('Hello!')
       expect(telegramResult).toBeUndefined()
@@ -212,7 +214,7 @@ describe('Command系统测试', () => {
         $raw: 'test message'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
 
       expect(actionSpy).toHaveBeenCalledWith(message, expect.any(Object))
       expect(result).toBe('Action executed!')
@@ -240,7 +242,7 @@ describe('Command系统测试', () => {
         $raw: 'test'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
 
       expect(action1).toHaveBeenCalled()
       expect(action2).toHaveBeenCalled()
@@ -266,7 +268,7 @@ describe('Command系统测试', () => {
         $raw: 'async test'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
 
       expect(asyncAction).toHaveBeenCalled()
       expect(result).toBe('Async result')
@@ -290,7 +292,7 @@ describe('Command系统测试', () => {
         $raw: 'echo hello world'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
 
       expect(actionSpy).toHaveBeenCalledWith(
         message,
@@ -320,7 +322,7 @@ describe('Command系统测试', () => {
         $raw: 'goodbye'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
       expect(result).toBeUndefined()
     })
 
@@ -340,7 +342,7 @@ describe('Command系统测试', () => {
         $raw: ''
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
       expect(result).toBeUndefined()
     })
 
@@ -362,7 +364,7 @@ describe('Command系统测试', () => {
         $raw: '[图片]'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
       expect(result).toBeUndefined()
     })
   })
@@ -412,9 +414,9 @@ describe('Command系统测试', () => {
         $raw: 'hello world'
       }
 
-      const validResult = await command.handle(validMessage, mockPlugin)
-      const wrongAdapterResult = await command.handle(wrongAdapterMessage, mockPlugin)
-      const nonMatchingResult = await command.handle(nonMatchingMessage, mockPlugin)
+      const validResult = await command.handle(validMessage, mockApp)
+      const wrongAdapterResult = await command.handle(wrongAdapterMessage, mockApp)
+      const nonMatchingResult = await command.handle(nonMatchingMessage, mockApp)
 
       expect(validResult).toBe('Admin command executed')
       expect(wrongAdapterResult).toBeUndefined()
@@ -443,7 +445,7 @@ describe('Command系统测试', () => {
         $raw: 'error test'
       }
 
-      await expect(command.handle(message, mockPlugin)).rejects.toThrow('Action failed')
+      await expect(command.handle(message, mockApp)).rejects.toThrow('Action failed')
     })
 
     it('应该正确处理动作中的异步错误', async () => {
@@ -464,7 +466,7 @@ describe('Command系统测试', () => {
         $raw: 'async-error test'
       }
 
-      await expect(command.handle(message, mockPlugin)).rejects.toThrow('Async action failed')
+      await expect(command.handle(message, mockApp)).rejects.toThrow('Async action failed')
     })
   })
 
@@ -504,7 +506,7 @@ describe('Command系统测试', () => {
 
       const startTime = Date.now()
       const results = await Promise.all(
-        messages.map(message => command.handle(message, mockPlugin))
+        messages.map(message => command.handle(message, mockApp))
       )
       const endTime = Date.now()
 
@@ -539,7 +541,7 @@ describe('Command系统测试', () => {
         $raw: 'say hello world from bot'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
 
       expect(actionSpy).toHaveBeenCalledWith(
         message,
@@ -587,8 +589,8 @@ describe('Command系统测试', () => {
         $raw: 'multi test'
       }
 
-      const privateResult = await command.handle(privateMessage, mockPlugin)
-      const groupResult = await command.handle(groupMessage, mockPlugin)
+      const privateResult = await command.handle(privateMessage, mockApp)
+      const groupResult = await command.handle(groupMessage, mockApp)
 
       expect(privateResult).toBe('私人消息响应')
       expect(groupResult).toBe('群组消息响应')
@@ -613,7 +615,7 @@ describe('Command系统测试', () => {
         $raw: 'admin test'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
       expect(result).toBeUndefined()
     })
 
@@ -634,7 +636,7 @@ describe('Command系统测试', () => {
         $raw: 'admin test'
       }
 
-      const result = await command.handle(message, mockPlugin)
+      const result = await command.handle(message, mockApp)
       expect(result).toBe('Admin command')
     })
   })

package/tests/plugin.test.ts

@@ -5,6 +5,7 @@ import { MessageCommand } from '../src/command'
 import { Component, defineComponent, ComponentContext } from '../src/component'
 import { Message } from '../src/message'
 import { PluginError, MessageError } from '../src/errors'
+import * as path from 'path'
 
 describe('Plugin系统测试', () => {
   let app: App
@@ -12,7 +13,8 @@ describe('Plugin系统测试', () => {
 
   beforeEach(() => {
     app = new App()
-    plugin = app.createDependency('test-plugin', 'test-plugin.ts')
+    // 使用 mock 文件路径，不依赖真实文件
+    plugin = app.createDependency('test-plugin', '/mock/test-plugin.ts')
   })
 
   afterEach(async () => {
@@ -23,13 +25,13 @@ describe('Plugin系统测试', () => {
     it('应该正确初始化Plugin实例', () => {
       expect(plugin).toBeInstanceOf(Plugin)
       expect(plugin.name).toBe('test-plugin')
-      expect(plugin.filename).toBe('test-plugin.ts')
+      expect(plugin.filename).toContain('test-plugin.ts')
       expect(plugin.app).toBe(app)
       expect(plugin.commands).toEqual([])
       expect(plugin.components).toBeInstanceOf(Map)
       expect(plugin.components.size).toBe(0)
-      // Plugin有默认的命令处理中间件，所以不为空
-      expect(plugin.middlewares.length).toBeGreaterThan(0)
+      // Plugin没有默认的中间件
+      expect(plugin.middlewares.length).toBe(0)
     })
 
     it('应该正确获取logger实例', () => {
@@ -89,8 +91,8 @@ describe('Plugin系统测试', () => {
         $raw: 'test message'
       }
 
-      // 触发消息处理
-      await plugin.emit('message.receive', mockMessage)
+      // 触发消息处理 - 通过 App 的 receiveMessage 方法
+      await app.receiveMessage(mockMessage)
 
       // 由于有默认的命令中间件，执行顺序可能会不同
       // 但我们的中间件应该被调用
@@ -120,9 +122,10 @@ describe('Plugin系统测试', () => {
         $raw: 'test message'
       }
 
-      await plugin.emit('message.receive', mockMessage)
-
-      // 验证错误处理逻辑 - 中间件异常会被处理但可能不会回复给用户
+      // 中间件异常应该被抛出
+      await expect(app.receiveMessage(mockMessage)).rejects.toThrow('中间件测试错误')
+      
+      // 验证中间件被调用
       expect(errorMiddleware).toHaveBeenCalled()
     })
   })
@@ -154,10 +157,10 @@ describe('Plugin系统测试', () => {
         $raw: 'test message'
       }
 
-      await plugin.emit('message.receive', mockMessage)
+      await app.receiveMessage(mockMessage)
 
-      // 验证命令被调用（handle 现在需要传入 plugin 作为第二个参数）
-      expect(handleSpy).toHaveBeenCalledWith(mockMessage, expect.any(Plugin))
+      // 验证命令被调用（handle 现在需要传入 app 作为第二个参数）
+      expect(handleSpy).toHaveBeenCalledWith(mockMessage, expect.any(App))
     })
   })
 
@@ -302,7 +305,8 @@ describe('Plugin系统测试', () => {
         $raw: 'error message'
       }
 
-      await plugin.emit('message.receive', mockMessage)
+      // 中间件异常应该被抛出
+      await expect(app.receiveMessage(mockMessage)).rejects.toThrow('处理失败')
 
       // 验证错误中间件被调用
       expect(errorMiddleware).toHaveBeenCalled()
@@ -358,7 +362,7 @@ describe('Plugin系统测试', () => {
       }
 
       // 触发消息处理
-      await plugin.emit('message.receive', mockMessage)
+      await app.receiveMessage(mockMessage)
 
       // 验证中间件被调用
       expect(middlewareExecuted).toHaveBeenCalled()