agent-messenger 2.10.1 → 2.10.2

package/src/platforms/slack/commands/activity.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, mock, test } from 'bun:test'
+import { describe, expect, mock, it } from 'bun:test'
 
 import { Command } from 'commander'
 
@@ -6,7 +6,7 @@ import { activityCommand } from '@/platforms/slack/commands/activity'
 
 describe('activity command', () => {
   describe('list subcommand', () => {
-    test('returns activity items', async () => {
+    it('returns activity items', async () => {
       const mockActivityItems = [
         {
           id: 'act_1',
@@ -36,7 +36,7 @@ describe('activity command', () => {
       expect(result[1].type).toBe('message_reaction')
     })
 
-    test('respects limit option', async () => {
+    it('respects limit option', async () => {
       const mockActivityItems = [
         {
           id: 'act_1',
@@ -56,7 +56,7 @@ describe('activity command', () => {
       expect(result).toHaveLength(1)
     })
 
-    test('respects unread-only mode', async () => {
+    it('respects unread-only mode', async () => {
       const mockActivityItems = [
         {
           id: 'act_1',
@@ -76,7 +76,7 @@ describe('activity command', () => {
       expect(result).toHaveLength(1)
     })
 
-    test('respects types filter', async () => {
+    it('respects types filter', async () => {
       const mockActivityItems = [
         {
           id: 'act_1',
@@ -96,7 +96,7 @@ describe('activity command', () => {
       expect(result).toHaveLength(1)
     })
 
-    test('handles empty activity feed', async () => {
+    it('handles empty activity feed', async () => {
       const mockGetActivityFeed = mock(() => Promise.resolve([]))
       const result = await mockGetActivityFeed()
 
@@ -105,36 +105,36 @@ describe('activity command', () => {
   })
 
   describe('command structure', () => {
-    test('activity command exists', () => {
+    it('activity command exists', () => {
       expect(activityCommand).toBeInstanceOf(Command)
     })
 
-    test('activity command has correct name', () => {
+    it('activity command has correct name', () => {
       expect(activityCommand.name()).toBe('activity')
     })
 
-    test('activity command has description', () => {
+    it('activity command has description', () => {
       expect(activityCommand.description()).toBe('Activity feed commands')
     })
 
-    test('list subcommand exists', () => {
+    it('list subcommand exists', () => {
       const listCmd = activityCommand.commands.find((cmd) => cmd.name() === 'list')
       expect(listCmd).toBeDefined()
     })
 
-    test('list subcommand has --unread option', () => {
+    it('list subcommand has --unread option', () => {
       const listCmd = activityCommand.commands.find((cmd) => cmd.name() === 'list')
       const unreadOption = listCmd?.options.find((opt) => opt.long === '--unread')
       expect(unreadOption).toBeDefined()
     })
 
-    test('list subcommand has --limit option', () => {
+    it('list subcommand has --limit option', () => {
       const listCmd = activityCommand.commands.find((cmd) => cmd.name() === 'list')
       const limitOption = listCmd?.options.find((opt) => opt.long === '--limit')
       expect(limitOption).toBeDefined()
     })
 
-    test('list subcommand has --types option', () => {
+    it('list subcommand has --types option', () => {
       const listCmd = activityCommand.commands.find((cmd) => cmd.name() === 'list')
       const typesOption = listCmd?.options.find((opt) => opt.long === '--types')
       expect(typesOption).toBeDefined()

package/src/platforms/slack/commands/auth.test.ts

@@ -1,4 +1,4 @@
-import { afterAll, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterAll, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { mkdirSync, rmSync } from 'node:fs'
 import { homedir } from 'node:os'
 import { join } from 'node:path'
@@ -27,7 +27,7 @@ describe('TokenExtractor', () => {
   })
 
   describe('getSlackDir', () => {
-    test('returns correct path for darwin', () => {
+    it('returns correct path for darwin', () => {
       // Given: Platform is darwin
       extractor = new TokenExtractor('darwin')
 
@@ -49,7 +49,7 @@ describe('TokenExtractor', () => {
       expect([directPath, sandboxedPath]).toContain(dir)
     })
 
-    test('returns correct path for linux', () => {
+    it('returns correct path for linux', () => {
       // Given: Platform is linux
       extractor = new TokenExtractor('linux')
 
@@ -60,7 +60,7 @@ describe('TokenExtractor', () => {
       expect(dir).toBe(join(homedir(), '.config', 'Slack'))
     })
 
-    test('returns correct path for win32', () => {
+    it('returns correct path for win32', () => {
       // Given: Platform is win32
       extractor = new TokenExtractor('win32')
 
@@ -71,7 +71,7 @@ describe('TokenExtractor', () => {
       expect(dir).toContain('Slack')
     })
 
-    test('throws error for unsupported platform', () => {
+    it('throws error for unsupported platform', () => {
       // Given: Platform is unsupported
       // When/Then: Constructor should throw
       expect(() => new TokenExtractor('freebsd' as NodeJS.Platform)).toThrow('Unsupported platform')
@@ -79,7 +79,7 @@ describe('TokenExtractor', () => {
   })
 
   describe('extract', () => {
-    test('returns empty array when Slack directory does not exist (falls back to browser)', async () => {
+    it('returns empty array when Slack directory does not exist (falls back to browser)', async () => {
       // given
       const nonExistentPath = `/tmp/nonexistent-slack-${Date.now()}-${Math.random()}`
       extractor = new TokenExtractor('darwin', nonExistentPath)
@@ -91,7 +91,7 @@ describe('TokenExtractor', () => {
       expect(result).toEqual([])
     })
 
-    test('returns empty array when no tokens found', async () => {
+    it('returns empty array when no tokens found', async () => {
       // Given: Slack directory exists but has no tokens
       mkdirSync(join(testSlackDir, 'storage'), { recursive: true })
       extractor = new TokenExtractor('darwin', testSlackDir)
@@ -105,7 +105,7 @@ describe('TokenExtractor', () => {
   })
 
   describe('extractTokensFromLevelDB', () => {
-    test('extracts xoxc tokens from LevelDB', async () => {
+    it('extracts xoxc tokens from LevelDB', async () => {
       // Given: LevelDB with xoxc token data
       // This test requires mocking LevelDB - we'll test the integration
       extractor = new TokenExtractor('darwin', testSlackDir)
@@ -117,7 +117,7 @@ describe('TokenExtractor', () => {
   })
 
   describe('extractCookieFromSQLite', () => {
-    test('extracts xoxd cookie from SQLite', async () => {
+    it('extracts xoxd cookie from SQLite', async () => {
       // Given: SQLite Cookies database with d cookie
       // This test requires mocking SQLite - we'll test the integration
       extractor = new TokenExtractor('darwin', testSlackDir)
@@ -143,7 +143,7 @@ describe('Auth Commands Integration', () => {
   })
 
   describe('auth extract', () => {
-    test('stores extracted workspaces in credential manager', async () => {
+    it('stores extracted workspaces in credential manager', async () => {
       // Given: Extracted workspaces
       const workspaces: ExtractedWorkspace[] = [
         {
@@ -174,7 +174,7 @@ describe('Auth Commands Integration', () => {
       expect(config.workspaces.T456.token).toBe('xoxc-789-012')
     })
 
-    test('sets first workspace as current if none exists', async () => {
+    it('sets first workspace as current if none exists', async () => {
       // Given: No current workspace
       const workspace: ExtractedWorkspace = {
         workspace_id: 'T789',
@@ -198,7 +198,7 @@ describe('Auth Commands Integration', () => {
   })
 
   describe('auth logout', () => {
-    test('removes workspace by id', async () => {
+    it('removes workspace by id', async () => {
       // Given: A workspace exists
       await credManager.setWorkspace({
         workspace_id: 'T-logout',
@@ -215,7 +215,7 @@ describe('Auth Commands Integration', () => {
       expect(ws).toBeNull()
     })
 
-    test('removes current workspace and clears current', async () => {
+    it('removes current workspace and clears current', async () => {
       // Given: Current workspace is set
       await credManager.setWorkspace({
         workspace_id: 'T-current',
@@ -233,7 +233,7 @@ describe('Auth Commands Integration', () => {
       expect(config.current_workspace).toBeNull()
     })
 
-    test('throws error when workspace not found', async () => {
+    it('throws error when workspace not found', async () => {
       // Given: Workspace does not exist
       // When: Trying to get non-existent workspace
       const ws = await credManager.getWorkspace('nonexistent')
@@ -244,7 +244,7 @@ describe('Auth Commands Integration', () => {
   })
 
   describe('auth status', () => {
-    test('returns current workspace info', async () => {
+    it('returns current workspace info', async () => {
       // Given: A current workspace is set
       await credManager.setWorkspace({
         workspace_id: 'T-status',
@@ -263,7 +263,7 @@ describe('Auth Commands Integration', () => {
       expect(ws?.workspace_name).toBe('status-test')
     })
 
-    test('returns null when no workspace configured', async () => {
+    it('returns null when no workspace configured', async () => {
       // Given: No workspace configured
       // When: Getting current workspace
       const ws = await credManager.getWorkspace()
@@ -272,7 +272,7 @@ describe('Auth Commands Integration', () => {
       expect(ws).toBeNull()
     })
 
-    test('validates token with Slack API', async () => {
+    it('validates token with Slack API', async () => {
       // Given: A workspace with valid credentials
       // This would require mocking SlackClient.testAuth()
       // We test the integration pattern here
@@ -299,7 +299,7 @@ describe('Auth Commands Integration', () => {
 })
 
 describe('Platform Detection', () => {
-  test('auto-detects current platform', () => {
+  it('auto-detects current platform', () => {
     // Given: Running on current platform
     const platform = process.platform
 
@@ -312,7 +312,7 @@ describe('Platform Detection', () => {
 })
 
 describe('Output Formatting', () => {
-  test('formats extract output correctly', () => {
+  it('formats extract output correctly', () => {
     // Given: Extracted workspaces
     const output = {
       workspaces: ['T123/acme-corp', 'T456/side-project'],
@@ -328,7 +328,7 @@ describe('Output Formatting', () => {
     expect(pretty).toContain('\n')
   })
 
-  test('formats status output correctly', () => {
+  it('formats status output correctly', () => {
     // Given: Status info
     const output = {
       workspace_id: 'T123',
@@ -345,7 +345,7 @@ describe('Output Formatting', () => {
     expect(JSON.parse(json)).toEqual(output)
   })
 
-  test('formats logout output correctly', () => {
+  it('formats logout output correctly', () => {
     // Given: Logout result
     const output = {
       removed: 'T123',
@@ -369,7 +369,7 @@ describe('formatCredentialDebug', () => {
     cookie: 'xoxd-abc123def456ghi789jkl012mno345pqr678stu901vwx234yz',
   }
 
-  test('truncates token and hides cookie by default', () => {
+  it('truncates token and hides cookie by default', () => {
     const result = formatCredentialDebug(ws)
 
     expect(result).toContain('T123ABC')
@@ -379,7 +379,7 @@ describe('formatCredentialDebug', () => {
     expect(result).not.toContain(ws.cookie)
   })
 
-  test('shows full token and cookie when showSecrets is true', () => {
+  it('shows full token and cookie when showSecrets is true', () => {
     const result = formatCredentialDebug(ws, true)
 
     expect(result).toContain(ws.token)
@@ -388,14 +388,14 @@ describe('formatCredentialDebug', () => {
     expect(result).not.toContain('present')
   })
 
-  test('shows cookie=missing when cookie is empty and secrets hidden', () => {
+  it('shows cookie=missing when cookie is empty and secrets hidden', () => {
     const wsNoCookie = { ...ws, cookie: '' }
     const result = formatCredentialDebug(wsNoCookie)
 
     expect(result).toContain('cookie=missing')
   })
 
-  test('shows empty cookie value when cookie is empty and secrets shown', () => {
+  it('shows empty cookie value when cookie is empty and secrets shown', () => {
     const wsNoCookie = { ...ws, cookie: '' }
     const result = formatCredentialDebug(wsNoCookie, true)
 
@@ -405,35 +405,35 @@ describe('formatCredentialDebug', () => {
 })
 
 describe('getExtractionErrorMessage', () => {
-  test('returns cookie failure message for missing_cookie', () => {
+  it('returns cookie failure message for missing_cookie', () => {
     const message = getExtractionErrorMessage(['missing_cookie'])
 
     expect(message).toContain('Cookie extraction failed')
     expect(message).toContain('desktop app or a supported Chromium browser')
   })
 
-  test('returns session expired message for invalid_auth', () => {
+  it('returns session expired message for invalid_auth', () => {
     const message = getExtractionErrorMessage(['invalid_auth'])
 
     expect(message).toContain('session has expired')
     expect(message).toContain('desktop app or a supported Chromium browser')
   })
 
-  test('prioritizes missing_cookie over invalid_auth', () => {
+  it('prioritizes missing_cookie over invalid_auth', () => {
     const message = getExtractionErrorMessage(['invalid_auth', 'missing_cookie'])
 
     expect(message).toContain('Cookie extraction failed')
     expect(message).toContain('desktop app or a supported Chromium browser')
   })
 
-  test('returns generic message for unknown error codes', () => {
+  it('returns generic message for unknown error codes', () => {
     const message = getExtractionErrorMessage(['unknown_error'])
 
     expect(message).toContain('Extracted tokens are invalid')
     expect(message).toContain('desktop app or a supported Chromium browser')
   })
 
-  test('returns generic message for empty failure list', () => {
+  it('returns generic message for empty failure list', () => {
     const message = getExtractionErrorMessage([])
 
     expect(message).toContain('Extracted tokens are invalid')
@@ -442,7 +442,7 @@ describe('getExtractionErrorMessage', () => {
 })
 
 describe('getNoWorkspacesFoundMessage', () => {
-  test('mentions desktop app and browser fallback', () => {
+  it('mentions desktop app and browser fallback', () => {
     expect(getNoWorkspacesFoundMessage()).toContain('desktop app or a supported Chromium browser')
   })
 })
@@ -457,7 +457,7 @@ describe('Error Handling', () => {
     rmSync(testSlackDir, { recursive: true, force: true })
   })
 
-  test('handles missing Slack installation gracefully', async () => {
+  it('handles missing Slack installation gracefully', async () => {
     // given — Slack is not installed
     const nonExistentPath = `/tmp/nonexistent-slack-${Date.now()}-${Math.random()}`
     const extractor = new TokenExtractor('darwin', nonExistentPath)
@@ -467,7 +467,7 @@ describe('Error Handling', () => {
     expect(result).toEqual([])
   })
 
-  test('handles empty Slack directory gracefully', async () => {
+  it('handles empty Slack directory gracefully', async () => {
     // Given: Slack directory exists but has no data
     const extractor = new TokenExtractor('darwin', testSlackDir)
 
@@ -478,7 +478,7 @@ describe('Error Handling', () => {
     expect(result).toEqual([])
   })
 
-  test('handles missing Cookies database gracefully', async () => {
+  it('handles missing Cookies database gracefully', async () => {
     // Given: No Cookies database
     mkdirSync(join(testSlackDir, 'storage'), { recursive: true })
     const extractor = new TokenExtractor('darwin', testSlackDir)

package/src/platforms/slack/commands/bookmark.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import type { SlackBookmark } from '@/platforms/slack/types'
@@ -27,13 +27,13 @@ describe('Bookmark Commands', () => {
   })
 
   describe('bookmark add', () => {
-    test('adds a bookmark successfully', async () => {
+    it('adds a bookmark successfully', async () => {
       const result = await (mockClient as SlackClient).addBookmark('C001', 'Test Bookmark', 'https://example.com')
       expect(result.id).toBe('Bm001')
       expect(result.title).toBe('Test Bookmark')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.addBookmark = mock(async () => {
         throw new Error('channel_not_found')
       })
@@ -44,12 +44,12 @@ describe('Bookmark Commands', () => {
   })
 
   describe('bookmark edit', () => {
-    test('edits a bookmark successfully', async () => {
+    it('edits a bookmark successfully', async () => {
       const result = await (mockClient as SlackClient).editBookmark('C001', 'Bm001', { title: 'Updated Title' })
       expect(result.title).toBe('Updated Title')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.editBookmark = mock(async () => {
         throw new Error('bookmark_not_found')
       })
@@ -60,12 +60,12 @@ describe('Bookmark Commands', () => {
   })
 
   describe('bookmark remove', () => {
-    test('removes a bookmark successfully', async () => {
+    it('removes a bookmark successfully', async () => {
       await (mockClient as SlackClient).removeBookmark('C001', 'Bm001')
       expect(mockClient.removeBookmark).toHaveBeenCalledWith('C001', 'Bm001')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.removeBookmark = mock(async () => {
         throw new Error('bookmark_not_found')
       })
@@ -74,14 +74,14 @@ describe('Bookmark Commands', () => {
   })
 
   describe('bookmark list', () => {
-    test('lists bookmarks in a channel', async () => {
+    it('lists bookmarks in a channel', async () => {
       const bookmarks = await (mockClient as SlackClient).listBookmarks('C001')
       expect(bookmarks).toHaveLength(1)
       expect(bookmarks[0].title).toBe('Test Bookmark')
       expect(bookmarks[0].link).toBe('https://example.com')
     })
 
-    test('throws error when API fails', async () => {
+    it('throws error when API fails', async () => {
       mockClient.listBookmarks = mock(async () => {
         throw new Error('channel_not_found')
       })

package/src/platforms/slack/commands/channel.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 import type { SlackChannel, SlackUser } from '@/platforms/slack/types'
@@ -124,7 +124,7 @@ describe('Channel Commands', () => {
   })
 
   describe('channel list', () => {
-    test('lists all channels', async () => {
+    it('lists all channels', async () => {
       // Given: SlackClient returns channels
       // When: Listing channels
       const channels = await mockClient.listChannels()
@@ -136,7 +136,7 @@ describe('Channel Commands', () => {
       expect(channels[2].name).toBe('archived-channel')
     })
 
-    test('excludes archived channels by default', async () => {
+    it('excludes archived channels by default', async () => {
       // Given: SlackClient returns channels including archived
       const channels = await mockClient.listChannels()
 
@@ -148,7 +148,7 @@ describe('Channel Commands', () => {
       expect(activeChannels.every((c) => !c.is_archived)).toBe(true)
     })
 
-    test('includes archived channels when requested', async () => {
+    it('includes archived channels when requested', async () => {
       // Given: SlackClient returns channels including archived
       const channels = await mockClient.listChannels()
 
@@ -158,7 +158,7 @@ describe('Channel Commands', () => {
       expect(channels.some((c) => c.is_archived)).toBe(true)
     })
 
-    test('filters channels by type=public', async () => {
+    it('filters channels by type=public', async () => {
       // Given: SlackClient returns mixed channels
       // When: Filtering by public type
       const channels = await mockClient.listChannels()
@@ -169,7 +169,7 @@ describe('Channel Commands', () => {
       expect(publicChannels.every((c) => !c.is_private)).toBe(true)
     })
 
-    test('filters channels by type=private', async () => {
+    it('filters channels by type=private', async () => {
       // Given: SlackClient returns mixed channels
       // When: Filtering by private type
       const channels = await mockClient.listChannels()
@@ -182,7 +182,7 @@ describe('Channel Commands', () => {
   })
 
   describe('channel info', () => {
-    test('gets channel by ID', async () => {
+    it('gets channel by ID', async () => {
       // Given: Channel ID C001
       // When: Getting channel info
       const channel = await mockClient.getChannel('C001')
@@ -193,7 +193,7 @@ describe('Channel Commands', () => {
       expect(channel.is_private).toBe(false)
     })
 
-    test('throws error for non-existent channel', async () => {
+    it('throws error for non-existent channel', async () => {
       // Given: Non-existent channel ID
       // When: Getting channel info
       // Then: Should throw error
@@ -205,7 +205,7 @@ describe('Channel Commands', () => {
       }
     })
 
-    test('returns channel with topic and purpose', async () => {
+    it('returns channel with topic and purpose', async () => {
       // Given: Channel with topic and purpose
       // When: Getting channel info
       const channel = await mockClient.getChannel('C001')
@@ -217,7 +217,7 @@ describe('Channel Commands', () => {
   })
 
   describe('channel history (alias for message list)', () => {
-    test('history command should be alias for message list', () => {
+    it('history command should be alias for message list', () => {
       // Given: channel history command
       // When: Checking command structure
       // Then: Should be alias for message list
@@ -226,7 +226,7 @@ describe('Channel Commands', () => {
   })
 
   describe('channel open', () => {
-    test('opens a DM channel with a single user', async () => {
+    it('opens a DM channel with a single user', async () => {
       // Given: A user ID
       // When: Opening a conversation
       const result = await mockClient.openConversation('U001')
@@ -237,7 +237,7 @@ describe('Channel Commands', () => {
       expect(mockClient.openConversation).toHaveBeenCalledWith('U001')
     })
 
-    test('returns already_open when DM exists', async () => {
+    it('returns already_open when DM exists', async () => {
       // Given: An existing DM conversation
       ;(mockClient.openConversation as any).mockImplementation(async () => ({
         channel_id: 'D0ABC123',
@@ -252,7 +252,7 @@ describe('Channel Commands', () => {
       expect(result.already_open).toBe(true)
     })
 
-    test('opens a group DM with multiple users', async () => {
+    it('opens a group DM with multiple users', async () => {
       // Given: Multiple user IDs
       ;(mockClient.openConversation as any).mockImplementation(async () => ({
         channel_id: 'G0ABC123',
@@ -269,7 +269,7 @@ describe('Channel Commands', () => {
   })
 
   describe('channel users', () => {
-    test('lists members of a channel via listChannelMembers and getUser', async () => {
+    it('lists members of a channel via listChannelMembers and getUser', async () => {
       // Given: Channel C001 with 3 members
       const memberIds = await mockClient.listChannelMembers('C001')
 
@@ -285,7 +285,7 @@ describe('Channel Commands', () => {
       expect(mockClient.getUser).toHaveBeenCalledTimes(3)
     })
 
-    test('filters out bots by default', async () => {
+    it('filters out bots by default', async () => {
       // Given: Channel with human and bot members
       const memberIds = await mockClient.listChannelMembers('C001')
       const users = await Promise.all(memberIds.map((id) => mockClient.getUser(id)))
@@ -298,7 +298,7 @@ describe('Channel Commands', () => {
       expect(filtered.every((u) => !u.is_bot)).toBe(true)
     })
 
-    test('includes bots with --include-bots flag', async () => {
+    it('includes bots with --include-bots flag', async () => {
       // Given: Channel with human and bot members
       const memberIds = await mockClient.listChannelMembers('C001')
       const users = await Promise.all(memberIds.map((id) => mockClient.getUser(id)))
@@ -309,7 +309,7 @@ describe('Channel Commands', () => {
       expect(users.some((u) => u.is_bot)).toBe(true)
     })
 
-    test('returns user profiles with expected output shape', async () => {
+    it('returns user profiles with expected output shape', async () => {
       // Given: Channel member with profile
       const user = await mockClient.getUser('U001')
 

package/src/platforms/slack/commands/drafts.test.ts

@@ -1,4 +1,4 @@
-import { beforeEach, describe, expect, mock, test } from 'bun:test'
+import { beforeEach, describe, expect, mock, it } from 'bun:test'
 
 import { SlackClient } from '@/platforms/slack/client'
 
@@ -51,7 +51,7 @@ describe('Drafts Commands', () => {
   })
 
   describe('drafts list', () => {
-    test('returns drafts', async () => {
+    it('returns drafts', async () => {
       // Given: Client with drafts
       // When: Getting drafts
       const result = await mockClient.getDrafts()
@@ -62,7 +62,7 @@ describe('Drafts Commands', () => {
       expect(result.drafts[0].message?.text).toBe('Draft message 1')
     })
 
-    test('handles empty message content gracefully', async () => {
+    it('handles empty message content gracefully', async () => {
       // Given: Client with drafts including null/empty message
       // When: Getting drafts
       const result = await mockClient.getDrafts()
@@ -74,7 +74,7 @@ describe('Drafts Commands', () => {
       expect(result.drafts[3].message).toEqual({})
     })
 
-    test('pagination with cursor works', async () => {
+    it('pagination with cursor works', async () => {
       // Given: Client with pagination support
       // When: Getting first page
       const firstPage = await mockClient.getDrafts()
@@ -89,7 +89,7 @@ describe('Drafts Commands', () => {
       expect(secondPage.next_cursor).toBeUndefined()
     })
 
-    test('returns next_cursor for pagination', async () => {
+    it('returns next_cursor for pagination', async () => {
       // Given: Client with more drafts available
       // When: Getting drafts
       const result = await mockClient.getDrafts()
@@ -101,7 +101,7 @@ describe('Drafts Commands', () => {
   })
 
   describe('output formatting', () => {
-    test('formats draft output', async () => {
+    it('formats draft output', async () => {
       // Given: Drafts from API
       const result = await mockClient.getDrafts()
 
@@ -119,7 +119,7 @@ describe('Drafts Commands', () => {
       expect(output[0].text).toBe('Draft message 1')
     })
 
-    test('handles null message in output', async () => {
+    it('handles null message in output', async () => {
       // Given: Draft with null message
       const result = await mockClient.getDrafts()