@zhin.js/core 1.0.5 → 1.0.7

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实例', () => {
@@ -44,121 +46,29 @@ describe('Plugin系统测试', () => {
   describe('中间件系统测试', () => {
     it('应该正确添加中间件', () => {
       const middleware = vi.fn(async (message, next) => {
-        await next()
-      })
+        await next();
+      });
 
-      const initialCount = plugin.middlewares.length
-      const unsubscribe = plugin.addMiddleware(middleware)
-      expect(plugin.middlewares).toContain(middleware)
-      expect(plugin.middlewares.length).toBe(initialCount + 1) // 增加1个中间件
-      expect(typeof unsubscribe).toBe('function')
+      const initialCount = plugin.middlewares.length;
+      const unsubscribe = plugin.addMiddleware(middleware);
+      expect(plugin.middlewares).toContain(middleware);
+      expect(plugin.middlewares.length).toBe(initialCount + 1); // 增加1个中间件
+      expect(typeof unsubscribe).toBe('function');
 
       // 测试移除中间件
-      unsubscribe()
-      expect(plugin.middlewares).not.toContain(middleware)
-    })
-
-    it('应该按顺序执行中间件', async () => {
-      const executionOrder: number[] = []
-      
-      const middleware1 = vi.fn(async (message, next) => {
-        executionOrder.push(1)
-        await next()
-        executionOrder.push(4)
-      })
-      
-      const middleware2 = vi.fn(async (message, next) => {
-        executionOrder.push(2)
-        await next()
-        executionOrder.push(3)
-      })
-
-      plugin.addMiddleware(middleware1)
-      plugin.addMiddleware(middleware2)
-
-      // 模拟消息对象
-      const mockMessage: Message = {
-        $id: '1',
-        $adapter: 'test',
-        $bot: 'test-bot',
-        $content: [{ type: 'text', data: { text: 'test' } }],
-        $sender: { id: 'user1', name: 'Test User' },
-        $reply: vi.fn(),
-        $channel: { id: 'test-channel', type: 'private' },
-        $timestamp: Date.now(),
-        $raw: 'test message'
-      }
-
-      // 触发消息处理
-      await plugin.emit('message.receive', mockMessage)
-
-      // 由于有默认的命令中间件，执行顺序可能会不同
-      // 但我们的中间件应该被调用
-      expect(middleware1).toHaveBeenCalledWith(mockMessage, expect.any(Function))
-      expect(middleware2).toHaveBeenCalledWith(mockMessage, expect.any(Function))
-      // 至少应该执行了我们的中间件的前半部分
-      expect(executionOrder).toContain(1)
-      expect(executionOrder).toContain(2)
-    })
-
-    it('应该正确处理中间件异常', async () => {
-      const errorMiddleware = vi.fn(async () => {
-        throw new Error('中间件测试错误')
-      })
-
-      plugin.addMiddleware(errorMiddleware)
-
-      const mockMessage: Message = {
-        $id: '1',
-        $adapter: 'test',
-        $bot: 'test-bot',
-        $content: [{ type: 'text', data: { text: 'test' } }],
-        $sender: { id: 'user1', name: 'Test User' },
-        $reply: vi.fn(),
-        $channel: { id: 'test-channel', type: 'private' },
-        $timestamp: Date.now(),
-        $raw: 'test message'
-      }
-
-      await plugin.emit('message.receive', mockMessage)
-
-      // 验证错误处理逻辑 - 中间件异常会被处理但可能不会回复给用户
-      expect(errorMiddleware).toHaveBeenCalled()
-    })
+      unsubscribe();
+      expect(plugin.middlewares).not.toContain(middleware);
+    });
   })
 
   describe('命令系统测试', () => {
     it('应该正确添加命令', () => {
-      const mockCommand = new MessageCommand('test')
-      plugin.addCommand(mockCommand)
-
-      expect(plugin.commands).toContain(mockCommand)
-      expect(plugin.commands.length).toBe(1)
-    })
-
-    it('应该通过默认中间件处理命令', async () => {
-      const mockCommand = new MessageCommand('test')
-      const handleSpy = vi.spyOn(mockCommand, 'handle').mockResolvedValue(undefined)
-      
-      plugin.addCommand(mockCommand)
-
-      const mockMessage: Message = {
-        $id: '1',
-        $adapter: 'test',
-        $bot: 'test-bot',
-        $content: [{ type: 'text', data: { text: 'test' } }],
-        $sender: { id: 'user1', name: 'Test User' },
-        $reply: vi.fn(),
-        $channel: { id: 'test-channel', type: 'private' },
-        $timestamp: Date.now(),
-        $raw: 'test message'
-      }
+      const mockCommand = new MessageCommand('test');
+      plugin.addCommand(mockCommand);
 
-      await plugin.emit('message.receive', mockMessage)
-
-      // 验证命令被调用（handle 现在需要传入 plugin 作为第二个参数）
-      expect(handleSpy).toHaveBeenCalledWith(mockMessage, expect.any(Plugin))
-    })
+      expect(plugin.commands).toContain(mockCommand);
+      expect(plugin.commands.length).toBe(1);
+    });
   })
 
   describe('函数式组件系统测试', () => {
@@ -223,22 +133,34 @@ describe('Plugin系统测试', () => {
 
   describe('消息发送测试', () => {
     it('应该正确发送消息', async () => {
-      const appSendSpy = vi.spyOn(app, 'sendMessage').mockResolvedValue()
-      
-      const sendOptions = { content: 'Hello World' }
-      await plugin.sendMessage(sendOptions)
+      const appSendSpy = vi.spyOn(app, 'sendMessage').mockResolvedValue('message-id');
 
-      expect(appSendSpy).toHaveBeenCalledWith(sendOptions)
-    })
+      const sendOptions = {
+        content: 'Hello World',
+        context: 'test-context',
+        bot: 'test-bot',
+        id: 'test-id',
+        type: 'text' // 确保类型符合 MessageType
+      };
+      await plugin.sendMessage(sendOptions);
 
-    it('应该正确处理发送消息失败', async () => {
-      const sendError = new Error('发送失败')
-      vi.spyOn(app, 'sendMessage').mockRejectedValue(sendError)
+      expect(appSendSpy).toHaveBeenCalledWith(sendOptions);
+    });
 
-      const sendOptions = { content: 'Hello World' }
-      
-      await expect(plugin.sendMessage(sendOptions)).rejects.toThrow(MessageError)
-    })
+    it('应该正确处理发送消息失败', async () => {
+      const sendError = new Error('发送失败');
+      vi.spyOn(app, 'sendMessage').mockRejectedValue(sendError);
+
+      const sendOptions = {
+        content: 'Hello World',
+        context: 'test-context',
+        bot: 'test-bot',
+        id: 'test-id',
+        type: 'text' // 确保类型符合 MessageType
+      };
+
+      await expect(plugin.sendMessage(sendOptions)).rejects.toThrow(MessageError);
+    });
   })
 
   describe('Prompt系统测试', () => {
@@ -285,30 +207,6 @@ describe('Plugin系统测试', () => {
     })
   })
 
-  describe('错误处理测试', () => {
-    it('应该正确处理消息处理异常', async () => {
-      const errorMiddleware = vi.fn().mockRejectedValue(new Error('处理失败'))
-      plugin.addMiddleware(errorMiddleware)
-
-      const mockMessage: Message = {
-        $id: 'error-msg',
-        $adapter: 'test',
-        $bot: 'test-bot',
-        $content: [{ type: 'text', data: { text: 'test' } }],
-        $sender: { id: 'user1', name: 'Test User' },
-        $reply: vi.fn(),
-        $channel: { id: 'error-channel', type: 'private' },
-        $timestamp: Date.now(),
-        $raw: 'error message'
-      }
-
-      await plugin.emit('message.receive', mockMessage)
-
-      // 验证错误中间件被调用
-      expect(errorMiddleware).toHaveBeenCalled()
-    })
-  })
-
   describe('资源清理测试', () => {
     it('应该正确销毁插件', () => {
       const middleware = vi.fn()
@@ -333,35 +231,4 @@ describe('Plugin系统测试', () => {
     })
   })
 
-  describe('集成测试', () => {
-    it('应该完整处理消息流程', async () => {
-      // 设置测试环境
-      const middlewareExecuted = vi.fn()
-      
-      // 添加测试中间件
-      plugin.addMiddleware(async (message, next) => {
-        middlewareExecuted(message.$content)
-        await next()
-      })
-
-      // 模拟消息
-      const mockMessage: Message = {
-        $id: 'integration-test',
-        $adapter: 'test',
-        $bot: 'test-bot',
-        $content: [{ type: 'text', data: { text: 'hello' } }],
-        $sender: { id: 'user1', name: 'Test User' },
-        $reply: vi.fn(),
-        $channel: { id: 'test-channel', type: 'private' },
-        $timestamp: Date.now(),
-        $raw: 'hello'
-      }
-
-      // 触发消息处理
-      await plugin.emit('message.receive', mockMessage)
-
-      // 验证中间件被调用
-      expect(middlewareExecuted).toHaveBeenCalled()
-    })
-  })
 })