agent-messenger 2.10.0 → 2.10.2

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

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, spyOn, it } from 'bun:test'
 import * as childProcess from 'node:child_process'
 
 import { WebexClient } from '../client'
@@ -42,7 +42,7 @@ describe('auth commands', () => {
   })
 
   describe('loginAction with --token', () => {
-    test('authenticates with provided token (bot token flow)', async () => {
+    it('authenticates with provided token (bot token flow)', async () => {
       protoSpy(WebexClient.prototype, 'login').mockResolvedValue(new WebexClient())
       protoSpy(WebexClient.prototype, 'testAuth').mockResolvedValue(mockPerson)
       protoSpy(WebexCredentialManager.prototype, 'saveConfig').mockResolvedValue(undefined)
@@ -56,7 +56,7 @@ describe('auth commands', () => {
       expect(output.user.displayName).toBe('Test User')
     })
 
-    test('saves tokenType as manual with expiresAt 0', async () => {
+    it('saves tokenType as manual with expiresAt 0', async () => {
       protoSpy(WebexClient.prototype, 'login').mockResolvedValue(new WebexClient())
       protoSpy(WebexClient.prototype, 'testAuth').mockResolvedValue(mockPerson)
       const saveSpy = protoSpy(WebexCredentialManager.prototype, 'saveConfig').mockResolvedValue(undefined)
@@ -71,7 +71,7 @@ describe('auth commands', () => {
   })
 
   describe('loginAction with --client-id and --client-secret', () => {
-    test('uses provided credentials for Device Grant flow', async () => {
+    it('uses provided credentials for Device Grant flow', async () => {
       protoSpy(WebexCredentialManager.prototype, 'requestDeviceCode').mockResolvedValue({
         deviceCode: 'd',
         userCode: 'u',
@@ -101,7 +101,7 @@ describe('auth commands', () => {
       )
     })
 
-    test('saves tokenType as oauth in config', async () => {
+    it('saves tokenType as oauth in config', async () => {
       protoSpy(WebexCredentialManager.prototype, 'requestDeviceCode').mockResolvedValue({
         deviceCode: 'd',
         userCode: 'u',
@@ -125,7 +125,7 @@ describe('auth commands', () => {
       expect(savedConfig.tokenType).toBe('oauth')
     })
 
-    test('saves clientId and clientSecret in config', async () => {
+    it('saves clientId and clientSecret in config', async () => {
       protoSpy(WebexCredentialManager.prototype, 'requestDeviceCode').mockResolvedValue({
         deviceCode: 'd',
         userCode: 'u',
@@ -152,7 +152,7 @@ describe('auth commands', () => {
   })
 
   describe('statusAction', () => {
-    test('shows authenticated status when token is valid', async () => {
+    it('shows authenticated status when token is valid', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue(null)
       protoSpy(WebexCredentialManager.prototype, 'getToken').mockResolvedValue('valid-token')
       protoSpy(WebexClient.prototype, 'login').mockResolvedValue(new WebexClient())
@@ -166,7 +166,7 @@ describe('auth commands', () => {
       expect(output.user.displayName).toBe('Test User')
     })
 
-    test('shows not authenticated when no token', async () => {
+    it('shows not authenticated when no token', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue(null)
       protoSpy(WebexCredentialManager.prototype, 'getToken').mockResolvedValue(null)
       const exitSpy = protoSpy(process, 'exit').mockImplementation(() => undefined as never)
@@ -179,7 +179,7 @@ describe('auth commands', () => {
       expect(exitSpy).toHaveBeenCalledWith(1)
     })
 
-    test('shows not authenticated when token validation fails', async () => {
+    it('shows not authenticated when token validation fails', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue(null)
       protoSpy(WebexCredentialManager.prototype, 'getToken').mockResolvedValue('invalid-token')
       protoSpy(WebexClient.prototype, 'login').mockResolvedValue(new WebexClient())
@@ -192,7 +192,7 @@ describe('auth commands', () => {
       expect(output.authenticated).toBe(false)
     })
 
-    test('loads config for stored client credentials', async () => {
+    it('loads config for stored client credentials', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue({
         accessToken: 'at',
         refreshToken: 'rt',
@@ -211,7 +211,7 @@ describe('auth commands', () => {
   })
 
   describe('extractAction', () => {
-    test('passes deviceUrl and tokenType to client.login', async () => {
+    it('passes deviceUrl and tokenType to client.login', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'extracted-token-at-least-twenty-chars',
         refreshToken: 'refresh-token',
@@ -232,7 +232,7 @@ describe('auth commands', () => {
       })
     })
 
-    test('attempts refresh when token is expired', async () => {
+    it('attempts refresh when token is expired', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'expired-token-at-least-twenty-chars-',
         refreshToken: 'valid-refresh-token',
@@ -257,7 +257,7 @@ describe('auth commands', () => {
       expect(output.refreshed).toBe(true)
     })
 
-    test('reports expired token with actionable hint when refresh fails', async () => {
+    it('reports expired token with actionable hint when refresh fails', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'expired-token-at-least-twenty-chars-',
         refreshToken: 'bad-refresh-token',
@@ -278,7 +278,7 @@ describe('auth commands', () => {
       expect(exitSpy).toHaveBeenCalledWith(1)
     })
 
-    test('rethrows non-auth errors even when token is expired', async () => {
+    it('rethrows non-auth errors even when token is expired', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'expired-token-at-least-twenty-chars-',
         refreshToken: 'bad-refresh-token',
@@ -298,7 +298,7 @@ describe('auth commands', () => {
       }
     })
 
-    test('rethrows non-expiry auth errors', async () => {
+    it('rethrows non-expiry auth errors', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue({
         accessToken: 'valid-token-at-least-twenty-chars-xx',
         expiresAt: Date.now() + 3600000,
@@ -316,7 +316,7 @@ describe('auth commands', () => {
       }
     })
 
-    test('outputs no token found when extract returns null', async () => {
+    it('outputs no token found when extract returns null', async () => {
       protoSpy(WebexTokenExtractor.prototype, 'extract').mockResolvedValue(null)
       const exitSpy = protoSpy(process, 'exit').mockImplementation(() => undefined as never)
 
@@ -330,7 +330,7 @@ describe('auth commands', () => {
   })
 
   describe('logoutAction', () => {
-    test('clears credentials when authenticated', async () => {
+    it('clears credentials when authenticated', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue({
         accessToken: 'token',
         refreshToken: 'refresh',
@@ -346,7 +346,7 @@ describe('auth commands', () => {
       expect(output.success).toBe(true)
     })
 
-    test('shows error when not authenticated', async () => {
+    it('shows error when not authenticated', async () => {
       protoSpy(WebexCredentialManager.prototype, 'loadConfig').mockResolvedValue(null)
       const exitSpy = protoSpy(process, 'exit').mockImplementation(() => undefined as never)
 

package/src/platforms/webex/commands/member.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 import { WebexError } from '../types'
 
@@ -59,7 +59,7 @@ describe('member commands', () => {
     consoleSpy.mockRestore()
   })
 
-  test('listAction calls listMemberships with spaceId and outputs mapped members', async () => {
+  it('calls listMemberships with spaceId and outputs mapped members', async () => {
     await listAction('room-1', {})
 
     expect(mockListMemberships).toHaveBeenCalledWith('room-1', { max: undefined })
@@ -85,13 +85,13 @@ describe('member commands', () => {
     )
   })
 
-  test('listAction passes limit option to listMemberships', async () => {
+  it('passes limit option to listMemberships', async () => {
     await listAction('room-1', { limit: 25 })
 
     expect(mockListMemberships).toHaveBeenCalledWith('room-1', { max: 25 })
   })
 
-  test('listAction handles not-authenticated case', async () => {
+  it('throws when not authenticated', async () => {
     mockLogin.mockImplementation(async () => {
       throw new WebexError('No Webex credentials found.', 'no_credentials')
     })
@@ -101,7 +101,7 @@ describe('member commands', () => {
     expect(mockHandleError).toHaveBeenCalledWith(expect.any(WebexError))
   })
 
-  test('listAction handles API error', async () => {
+  it('throws on API error', async () => {
     mockListMemberships.mockImplementation(async () => {
       throw new Error('API failure')
     })

package/src/platforms/webex/commands/message.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, mock, spyOn, it } from 'bun:test'
 
 import { WebexError } from '../types'
 
@@ -77,7 +77,7 @@ afterEach(() => {
   consoleLogSpy.mockRestore()
 })
 
-test('send: calls sendMessage with correct args and outputs result', async () => {
+it('calls sendMessage with correct args and outputs result', async () => {
   await sendAction('space_456', 'Hello world', { pretty: false })
 
   expect(mockSendMessage).toHaveBeenCalledWith('space_456', 'Hello world', {
@@ -90,13 +90,13 @@ test('send: calls sendMessage with correct args and outputs result', async () =>
   expect(output).toContain('user@example.com')
 })
 
-test('send: with --markdown passes markdown option', async () => {
+it('passes markdown option when --markdown flag is set on send', async () => {
   await sendAction('space_456', '**bold**', { markdown: true, pretty: false })
 
   expect(mockSendMessage).toHaveBeenCalledWith('space_456', '**bold**', { markdown: true })
 })
 
-test('send: not authenticated shows error', async () => {
+it('throws when not authenticated on send', async () => {
   mockLogin.mockImplementation(async () => {
     throw new WebexError('No Webex credentials found.', 'no_credentials')
   })
@@ -106,7 +106,7 @@ test('send: not authenticated shows error', async () => {
   expect(mockHandleError).toHaveBeenCalledWith(expect.any(WebexError))
 })
 
-test('dm: calls sendDirectMessage with email and text', async () => {
+it('calls sendDirectMessage with email and text', async () => {
   await dmAction('alice@example.com', 'Hello!', { pretty: false })
 
   expect(mockSendDirectMessage).toHaveBeenCalledWith('alice@example.com', 'Hello!', {
@@ -117,7 +117,7 @@ test('dm: calls sendDirectMessage with email and text', async () => {
   expect(output).toContain('msg_123')
 })
 
-test('dm: with --markdown passes markdown option', async () => {
+it('passes markdown option to sendDirectMessage when --markdown flag is set', async () => {
   await dmAction('alice@example.com', '**bold**', { markdown: true, pretty: false })
 
   expect(mockSendDirectMessage).toHaveBeenCalledWith('alice@example.com', '**bold**', {
@@ -125,7 +125,7 @@ test('dm: with --markdown passes markdown option', async () => {
   })
 })
 
-test('list: calls listMessages with limit and outputs array', async () => {
+it('calls listMessages with limit and outputs array', async () => {
   await listAction('space_456', { limit: 50, pretty: false })
 
   expect(mockListMessages).toHaveBeenCalledWith('space_456', { max: 50 })
@@ -135,7 +135,7 @@ test('list: calls listMessages with limit and outputs array', async () => {
   expect(output).toContain('msg_124')
 })
 
-test('get: calls getMessage with correct id and outputs result', async () => {
+it('calls getMessage with correct id and outputs result', async () => {
   await getAction('msg_123', { pretty: false })
 
   expect(mockGetMessage).toHaveBeenCalledWith('msg_123')
@@ -145,7 +145,7 @@ test('get: calls getMessage with correct id and outputs result', async () => {
   expect(output).toContain('user@example.com')
 })
 
-test('delete: with --force calls deleteMessage and outputs deleted id', async () => {
+it('calls deleteMessage and outputs deleted id when --force flag is set', async () => {
   await deleteAction('msg_123', { force: true, pretty: false })
 
   expect(mockDeleteMessage).toHaveBeenCalledWith('msg_123')
@@ -155,7 +155,7 @@ test('delete: with --force calls deleteMessage and outputs deleted id', async ()
   expect(output).toContain('msg_123')
 })
 
-test('delete: without --force shows warning and does not delete', async () => {
+it('shows warning and does not delete without --force flag', async () => {
   try {
     await deleteAction('msg_123', { force: false, pretty: false })
   } catch {}
@@ -167,7 +167,7 @@ test('delete: without --force shows warning and does not delete', async () => {
   expect(output).toContain('--force')
 })
 
-test('edit: calls editMessage with roomId in args and outputs result', async () => {
+it('calls editMessage with roomId in args and outputs result', async () => {
   await editAction('msg_123', 'space_456', 'Updated message', { pretty: false })
 
   expect(mockEditMessage).toHaveBeenCalledWith('msg_123', 'space_456', 'Updated message', {
@@ -179,7 +179,7 @@ test('edit: calls editMessage with roomId in args and outputs result', async ()
   expect(output).toContain('Updated message')
 })
 
-test('edit: with --markdown passes markdown option', async () => {
+it('passes markdown option to editMessage when --markdown flag is set', async () => {
   await editAction('msg_123', 'space_456', '**updated**', { markdown: true, pretty: false })
 
   expect(mockEditMessage).toHaveBeenCalledWith('msg_123', 'space_456', '**updated**', {

package/src/platforms/webex/commands/snapshot.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 import { WebexError } from '../types'
 
@@ -79,7 +79,7 @@ describe('snapshot command', () => {
     consoleSpy.mockRestore()
   })
 
-  test('brief snapshot returns spaces with id and title only', async () => {
+  it('returns spaces with id and title only in brief mode', async () => {
     await snapshotAction({})
 
     expect(consoleSpy).toHaveBeenCalled()
@@ -91,7 +91,7 @@ describe('snapshot command', () => {
     expect(output.hint).toBeDefined()
   })
 
-  test('full snapshot returns spaces with id, title, type, lastActivity', async () => {
+  it('returns spaces with id, title, type, and lastActivity in full mode', async () => {
     await snapshotAction({ full: true })
 
     expect(consoleSpy).toHaveBeenCalled()
@@ -104,7 +104,7 @@ describe('snapshot command', () => {
     expect(output.hint).toBeUndefined()
   })
 
-  test('filters spaces to only those in my memberships', async () => {
+  it('filters spaces to only those in my memberships', async () => {
     await snapshotAction({})
 
     const output = JSON.parse(consoleSpy.mock.calls[consoleSpy.mock.calls.length - 1][0])
@@ -112,7 +112,7 @@ describe('snapshot command', () => {
     expect(output.spaces[0].id).toBe('space-1')
   })
 
-  test('not authenticated outputs error', async () => {
+  it('throws when not authenticated', async () => {
     mockLogin.mockImplementation(async () => {
       throw new WebexError('No Webex credentials found.', 'no_credentials')
     })

package/src/platforms/webex/commands/space.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, spyOn, it } from 'bun:test'
 
 import { WebexError } from '../types'
 
@@ -74,7 +74,7 @@ afterEach(() => {
 })
 
 describe('listAction', () => {
-  test('calls listSpaces and outputs mapped array', async () => {
+  it('calls listSpaces and outputs mapped array', async () => {
     await listAction({})
 
     expect(mockListSpaces).toHaveBeenCalled()
@@ -98,19 +98,19 @@ describe('listAction', () => {
     )
   })
 
-  test('passes type and limit options to listSpaces', async () => {
+  it('passes type and limit options to listSpaces', async () => {
     await listAction({ type: 'group', limit: 10 })
 
     expect(mockListSpaces).toHaveBeenCalledWith({ type: 'group', max: 10 })
   })
 
-  test('passes undefined type and limit when not provided', async () => {
+  it('passes undefined type and limit when not provided', async () => {
     await listAction({})
 
     expect(mockListSpaces).toHaveBeenCalledWith({ type: undefined, max: undefined })
   })
 
-  test('outputs pretty-printed JSON when pretty is true', async () => {
+  it('outputs pretty-printed JSON when pretty is true', async () => {
     await listAction({ pretty: true })
 
     expect(consoleLogSpy).toHaveBeenCalledWith(
@@ -137,7 +137,7 @@ describe('listAction', () => {
     )
   })
 
-  test('not authenticated: outputs error and exits', async () => {
+  it('throws when not authenticated', async () => {
     mockLogin.mockImplementation(async () => {
       throw new WebexError('No Webex credentials found.', 'no_credentials')
     })
@@ -150,7 +150,7 @@ describe('listAction', () => {
 })
 
 describe('infoAction', () => {
-  test('calls getSpace with spaceId and outputs space details', async () => {
+  it('calls getSpace with spaceId and outputs space details', async () => {
     await infoAction('space-1', {})
 
     expect(mockGetSpace).toHaveBeenCalledWith('space-1')
@@ -168,7 +168,7 @@ describe('infoAction', () => {
     )
   })
 
-  test('outputs null for teamId when not present', async () => {
+  it('outputs null for teamId when not present', async () => {
     mockGetSpace.mockImplementation(() => Promise.resolve({ ...mockSpace, teamId: undefined }))
 
     await infoAction('space-1', {})
@@ -179,7 +179,7 @@ describe('infoAction', () => {
     expect(output.teamId).toBeNull()
   })
 
-  test('outputs pretty-printed JSON when pretty is true', async () => {
+  it('outputs pretty-printed JSON when pretty is true', async () => {
     await infoAction('space-1', { pretty: true })
 
     expect(consoleLogSpy).toHaveBeenCalledWith(
@@ -200,7 +200,7 @@ describe('infoAction', () => {
     )
   })
 
-  test('not authenticated: outputs error and exits', async () => {
+  it('throws when not authenticated', async () => {
     mockLogin.mockImplementation(async () => {
       throw new WebexError('No Webex credentials found.', 'no_credentials')
     })

package/src/platforms/webex/commands/whoami.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, expect, spyOn, test } from 'bun:test'
+import { afterEach, beforeEach, expect, spyOn, it } from 'bun:test'
 
 import * as clientModule from '../client'
 import { WebexError } from '../types'
@@ -42,19 +42,19 @@ afterEach(() => {
   processExitSpy?.mockRestore()
 })
 
-test('whoami command is defined with correct name and description', () => {
+it('whoami command is defined with correct name and description', () => {
   expect(whoamiCommand).toBeDefined()
   expect(whoamiCommand.name()).toBe('whoami')
   expect(whoamiCommand.description()).toBe('Show current authenticated user')
 })
 
-test('whoami command has --pretty option', () => {
+it('whoami command has --pretty option', () => {
   const options = whoamiCommand.options
   const hasPretty = options.some((opt: { long?: string }) => opt.long === '--pretty')
   expect(hasPretty).toBe(true)
 })
 
-test('whoami calls testAuth and outputs user fields', async () => {
+it('whoami calls testAuth and outputs user fields', async () => {
   // given: authenticated webex user
   // when: running whoami
   await whoamiCommand.parseAsync([], { from: 'user' })
@@ -75,7 +75,7 @@ test('whoami calls testAuth and outputs user fields', async () => {
   )
 })
 
-test('whoami outputs pretty-printed JSON when --pretty flag is passed', async () => {
+it('whoami outputs pretty-printed JSON when --pretty flag is passed', async () => {
   // given: authenticated webex user
   // when: running whoami with --pretty
   await whoamiCommand.parseAsync(['--pretty'], { from: 'user' })
@@ -100,7 +100,7 @@ test('whoami outputs pretty-printed JSON when --pretty flag is passed', async ()
   )
 })
 
-test('whoami exits with code 1 when not authenticated', async () => {
+it('whoami exits with code 1 when not authenticated', async () => {
   // given: no credentials
   webexClientSpy.mockImplementation(
     () =>

package/src/platforms/webex/credential-manager.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, mock, test } from 'bun:test'
+import { afterEach, beforeEach, describe, expect, mock, it } from 'bun:test'
 import { mkdtemp, rm, writeFile } from 'node:fs/promises'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
@@ -18,11 +18,11 @@ describe('WebexCredentialManager', () => {
     await rm(tempDir, { recursive: true, force: true })
   })
 
-  test('loadConfig returns null when no file exists', async () => {
+  it('loadConfig returns null when no file exists', async () => {
     expect(await credManager.loadConfig()).toBeNull()
   })
 
-  test('saveConfig + loadConfig round trip with OAuth tokens', async () => {
+  it('saveConfig and loadConfig round-trip OAuth tokens', async () => {
     const config = {
       accessToken: 'test-access-token',
       refreshToken: 'test-refresh-token',
@@ -33,7 +33,7 @@ describe('WebexCredentialManager', () => {
     expect(loaded).toEqual(config)
   })
 
-  test('getToken returns accessToken when not expired', async () => {
+  it('getToken returns accessToken when not expired', async () => {
     await credManager.saveConfig({
       accessToken: 'valid-token',
       refreshToken: 'refresh',
@@ -43,7 +43,7 @@ describe('WebexCredentialManager', () => {
     expect(token).toBe('valid-token')
   })
 
-  test('getToken returns null when expired and no refresh available', async () => {
+  it('getToken returns null when expired and no refresh available', async () => {
     await credManager.saveConfig({
       accessToken: 'expired-token',
       refreshToken: 'bad-refresh',
@@ -53,7 +53,7 @@ describe('WebexCredentialManager', () => {
     expect(token).toBeNull()
   })
 
-  test('getToken auto-refreshes expired token', async () => {
+  it('getToken auto-refreshes expired token', async () => {
     const originalFetch = globalThis.fetch
     globalThis.fetch = mock(() =>
       Promise.resolve(
@@ -87,7 +87,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('requestDeviceCode calls device authorize endpoint', async () => {
+  it('requestDeviceCode calls device authorize endpoint', async () => {
     const originalFetch = globalThis.fetch
     globalThis.fetch = mock(() =>
       Promise.resolve(
@@ -114,7 +114,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('requestDeviceCode throws on failure', async () => {
+  it('requestDeviceCode throws on failure', async () => {
     const originalFetch = globalThis.fetch
     globalThis.fetch = mock(() =>
       Promise.resolve(new Response('{"error":"invalid_client"}', { status: 400 })),
@@ -125,7 +125,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('pollDeviceToken polls until authorized', async () => {
+  it('pollDeviceToken polls until authorized', async () => {
     const originalFetch = globalThis.fetch
     let callCount = 0
     globalThis.fetch = mock(() => {
@@ -152,7 +152,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('clearCredentials removes the file', async () => {
+  it('clearCredentials removes the file', async () => {
     await credManager.saveConfig({
       accessToken: 'token',
       refreshToken: 'refresh',
@@ -162,11 +162,11 @@ describe('WebexCredentialManager', () => {
     expect(await credManager.loadConfig()).toBeNull()
   })
 
-  test('clearCredentials does nothing when no file', async () => {
+  it('clearCredentials does nothing when no file', async () => {
     await credManager.clearCredentials() // Should not throw
   })
 
-  test('credentials file has 0o600 permissions', async () => {
+  it('saves credentials file with 0o600 permissions', async () => {
     await credManager.saveConfig({
       accessToken: 'token',
       refreshToken: 'refresh',
@@ -179,7 +179,7 @@ describe('WebexCredentialManager', () => {
     expect(mode).toBe(0o600)
   })
 
-  test('pollDeviceToken with undefined clientSecret uses empty Basic auth', async () => {
+  it('pollDeviceToken with undefined clientSecret uses empty Basic auth', async () => {
     const originalFetch = globalThis.fetch
     let capturedAuth: string | null = null
     globalThis.fetch = mock((url: string, init?: RequestInit) => {
@@ -202,7 +202,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('pollDeviceToken does not auto-save config', async () => {
+  it('pollDeviceToken does not auto-save config', async () => {
     const originalFetch = globalThis.fetch
     globalThis.fetch = mock(() =>
       Promise.resolve(
@@ -225,7 +225,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('getToken returns null when expired and no client credentials available', async () => {
+  it('getToken returns null when expired and no client credentials available', async () => {
     await credManager.saveConfig({
       accessToken: 'expired-token',
       refreshToken: 'valid-refresh',
@@ -236,7 +236,7 @@ describe('WebexCredentialManager', () => {
     expect(token).toBeNull()
   })
 
-  test('getToken returns manual token without attempting refresh', async () => {
+  it('getToken returns manual token without attempting refresh', async () => {
     await credManager.saveConfig({
       accessToken: 'my-bot-token',
       refreshToken: '',
@@ -248,7 +248,7 @@ describe('WebexCredentialManager', () => {
     expect(token).toBe('my-bot-token')
   })
 
-  test('getToken uses stored clientId/clientSecret for refresh', async () => {
+  it('getToken uses stored clientId/clientSecret for refresh', async () => {
     const originalFetch = globalThis.fetch
     globalThis.fetch = mock(() =>
       Promise.resolve(
@@ -277,7 +277,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('saveConfig persists clientId and clientSecret', async () => {
+  it('saveConfig persists clientId and clientSecret', async () => {
     await credManager.saveConfig({
       accessToken: 'token',
       refreshToken: 'refresh',
@@ -291,7 +291,7 @@ describe('WebexCredentialManager', () => {
     expect(loaded?.clientSecret).toBe('my-client-secret')
   })
 
-  test('getToken tries refresh for expired extracted tokens', async () => {
+  it('getToken tries refresh for expired extracted tokens', async () => {
     const originalFetch = globalThis.fetch
     globalThis.fetch = mock(() =>
       Promise.resolve(
@@ -323,7 +323,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('getToken returns expired extracted token when refresh fails', async () => {
+  it('getToken returns expired extracted token when refresh fails', async () => {
     const originalFetch = globalThis.fetch
     globalThis.fetch = mock(() =>
       Promise.resolve(new Response('{"error":"invalid_grant"}', { status: 400 })),
@@ -342,7 +342,7 @@ describe('WebexCredentialManager', () => {
     globalThis.fetch = originalFetch
   })
 
-  test('getToken returns non-expired extracted token without refresh', async () => {
+  it('getToken returns non-expired extracted token without refresh', async () => {
     await credManager.saveConfig({
       accessToken: 'valid-extracted-token',
       refreshToken: 'refresh',
@@ -354,7 +354,7 @@ describe('WebexCredentialManager', () => {
     expect(token).toBe('valid-extracted-token')
   })
 
-  test('loadConfig backward compat — old config without clientId/clientSecret', async () => {
+  it('loadConfig handles old config without clientId/clientSecret', async () => {
     // Write raw JSON without clientId/clientSecret fields
     const credPath = join(tempDir, 'webex-credentials.json')
     await writeFile(