单测问题修复

Author: 王允

tests/channels/feishu.test.ts

@@ -42,10 +42,16 @@ describe('FeishuChannel', () => {
     it('should include correct title with date', () => {
       const repositories: RepositoryInfo[] = [];
       const card = FeishuChannel.buildCard(repositories, []);
+      const card2 = FeishuChannel.buildCard(repositories, [], 'daily');
+      const card3 = FeishuChannel.buildCard(repositories, [], 'weekly');
 
       const title = card.header?.title;
+      const title2 = card2.header?.title;
+      const title3 = card3.header?.title;
       expect(title).toBeDefined();
-      expect(title?.content).toContain('GitHub 热榜推送');
+      expect(title?.content).toContain('GitHub 本月热榜推送');
+      expect(title2?.content).toContain('GitHub 今日热榜推送');
+      expect(title3?.content).toContain('GitHub 本周热榜推送');
     });
 
     it('should build card with both new and seen repositories', () => {
@@ -112,14 +118,16 @@ describe('FeishuChannel', () => {
           url: 'https://github.com/org/desc-repo',
           stars: 100,
           description: 'This is a detailed description with multiple lines. It contains important information about the repository. Lines 3 of description.',
+          ai_summary: 'AI generated summary for this repository with important details about its features and usage.',
           language: 'Rust'
         }
       ];
 
-      const card = FeishuChannel.buildCard(repositories, []);
+      const card = FeishuChannel.buildCard(repositories, [], 'monthly');
       const cardContent = JSON.stringify(card);
 
-      expect(cardContent).toContain('This is a detailed description');
+      // 新项目会显示 AI summary
+      expect(cardContent).toContain('AI generated summary');
     });
 
     it('should include language tag when available', () => {

tests/core/fetcher.test.ts

@@ -111,7 +111,7 @@ describe('GitHubFetcher', () => {
 
       const repos = await fetcher.fetchTrending('daily');
 
-      expect(mockInstance.get).toHaveBeenCalledWith('https://github.com/trending?since=daily', expect.any(Object));
+      expect(mockInstance.get).toHaveBeenCalledWith('https://github.com/trending?since=daily');
       expect(Array.isArray(repos)).toBe(true);
     });
 
@@ -127,7 +127,7 @@ describe('GitHubFetcher', () => {
 
       const repos = await fetcher.fetchTrending('weekly');
 
-      expect(mockInstance.get).toHaveBeenCalledWith('https://github.com/trending?since=weekly', expect.any(Object));
+      expect(mockInstance.get).toHaveBeenCalledWith('https://github.com/trending?since=weekly');
       expect(Array.isArray(repos)).toBe(true);
     });
 
@@ -143,7 +143,7 @@ describe('GitHubFetcher', () => {
 
       const repos = await fetcher.fetchTrending('monthly');
 
-      expect(mockInstance.get).toHaveBeenCalledWith('https://github.com/trending?since=monthly', expect.any(Object));
+      expect(mockInstance.get).toHaveBeenCalledWith('https://github.com/trending?since=monthly');
       expect(Array.isArray(repos)).toBe(true);
     });
 
@@ -160,7 +160,7 @@ describe('GitHubFetcher', () => {
       });
 
       await expect(fetcher.fetchTrending('daily')).rejects.toThrow('Failed to fetch trending page');
-      expect(mockInstance.get).toHaveBeenCalledWith('https://github.com/trending?since=daily', expect.any(Object));
+      expect(mockInstance.get).toHaveBeenCalledWith('https://github.com/trending?since=daily');
     });
 
     it('should throw error for network failure', async () => {

tests/tool.test.ts

@@ -21,7 +21,13 @@ let currentHistoryData: HistoryData | null = null;
 
 // Create the mock history manager - this will be called in beforeEach
 function createMockHistoryManager() {
+  const mockData = {
+    repositories: {},
+    last_updated: new Date().toISOString()
+  };
+
   return {
+    data: mockData,
     categorizeRepositories: jest.fn((repos: RepositoryInfo[], config: any) => {
       const newlySeen: RepositoryInfo[] = [];
       const shouldPush: RepositoryInfo[] = [];
@@ -52,10 +58,17 @@ function createMockHistoryManager() {
     }),
     importData: jest.fn((data: HistoryData) => {
       currentHistoryData = data;
+      mockData.repositories = data.repositories;
     }),
     exportData: jest.fn(),
     updateAiSummary: jest.fn(),
-    clear: jest.fn()
+    clear: jest.fn(),
+    getStats: jest.fn(() => ({
+      total_repositories: 0,
+      total_pushes: 0,
+      oldest_entry: undefined,
+      newest_entry: undefined
+    }))
   };
 }
 
@@ -101,7 +114,7 @@ describe('githubTrendingTool', () => {
 
   describe('tool definition', () => {
     it('should have correct name', () => {
-      expect(toolModule.githubTrendingTool.name).toBe('github-trending');
+      expect(toolModule.githubTrendingTool.name).toBe('openclaw-github-trending');
     });
 
     it('should have correct description', () => {
@@ -116,13 +129,14 @@ describe('githubTrendingTool', () => {
 
       const props = params.properties as any;
       expect(props.since).toBeDefined();
-      expect(props.channel).toBeDefined();
+      expect(props.channels).toBeDefined();
 
       // Verify since enum values
       expect(props.since.enum).toEqual(['daily', 'weekly', 'monthly']);
 
-      // Verify channel enum values
-      expect(props.channel.enum).toEqual(['feishu', 'email']);
+      // Verify channels array items
+      expect(props.channels.type).toBe('array');
+      expect(props.channels.items.enum).toEqual(['feishu', 'email']);
     });
   });
 

tests/utils/logger.test.ts

@@ -1,4 +1,4 @@
-import { Logger } from '../../src/utils/logger';
+import { Logger, LogLevel } from '../../src/utils/logger';
 
 describe('Logger', () => {
   let originalConsole: any;
@@ -21,7 +21,7 @@ describe('Logger', () => {
   describe('constructor', () => {
     it('should create logger with default prefix when none provided', () => {
       const logger = new Logger();
-      expect(logger.getPrefix()).toBe('[Logger]');
+      expect(logger.getPrefix()).toBe('[App]');
     });
 
     it('should create logger with custom prefix', () => {
@@ -33,6 +33,7 @@ describe('Logger', () => {
   describe('debug', () => {
     it('should log debug message with prefix', () => {
       const logger = new Logger('Test');
+      logger.setLogLevel(LogLevel.DEBUG); // Enable debug level
       const message = 'Debug message';
       logger.debug(message);
 
@@ -42,6 +43,7 @@ describe('Logger', () => {
 
     it('should log message with multiple arguments', () => {
       const logger = new Logger();
+      logger.setLogLevel(LogLevel.DEBUG); // Enable debug level
       logger.debug('Object:', { key: 'value' }, 'Array:', [1, 2, 3]);
 
       // Check that console.log was called with arguments containing the expected content
@@ -147,6 +149,10 @@ describe('Logger', () => {
   describe('static methods', () => {
     describe('Logger.debug', () => {
       it('should log static debug message', () => {
+        // Static methods use 'App' logger, set its level to debug
+        const appLogger = Logger.get('App');
+        appLogger.setLogLevel(LogLevel.DEBUG);
+
         Logger.debug('Static debug message');
 
         expect(consoleLogSpy).toHaveBeenCalledWith(expect.stringContaining('[DEBUG]'));