chore: replace test() with it() in all vitest test files (#268)

Author: felipefreitag

tests/commands/api-keys/create.test.ts

@@ -5,7 +5,6 @@ import {
   expect,
   it,
   type MockInstance,
-  test,
   vi,
 } from 'vitest';
 import {
@@ -57,7 +56,7 @@ describe('api-keys create command', () => {
     }
   });
 
-  test('creates API key with --name flag', async () => {
+  it('creates API key with --name flag', async () => {
     spies = setupOutputSpies();
 
     const { createApiKeyCommand } = await import(
@@ -72,7 +71,7 @@ describe('api-keys create command', () => {
     expect(args.name).toBe('Production');
   });
 
-  test('passes permission flag to SDK', async () => {
+  it('passes permission flag to SDK', async () => {
     spies = setupOutputSpies();
 
     const { createApiKeyCommand } = await import(
@@ -87,7 +86,7 @@ describe('api-keys create command', () => {
     expect(args.permission).toBe('sending_access');
   });
 
-  test('passes domain_id (snake_case) to SDK when --domain-id is provided', async () => {
+  it('passes domain_id (snake_case) to SDK when --domain-id is provided', async () => {
     spies = setupOutputSpies();
 
     const { createApiKeyCommand } = await import(
@@ -109,7 +108,7 @@ describe('api-keys create command', () => {
     expect(args.domain_id).toBe('domain-123');
   });
 
-  test('outputs JSON result when non-interactive', async () => {
+  it('outputs JSON result when non-interactive', async () => {
     spies = setupOutputSpies();
 
     const { createApiKeyCommand } = await import(
@@ -125,7 +124,7 @@ describe('api-keys create command', () => {
     expect(parsed.token).toBe('re_testtoken1234567890');
   });
 
-  test('errors with missing_name when --name absent in non-interactive mode', async () => {
+  it('errors with missing_name when --name absent in non-interactive mode', async () => {
     setNonInteractive();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();
@@ -141,7 +140,7 @@ describe('api-keys create command', () => {
     expect(output).toContain('missing_name');
   });
 
-  test('errors with missing_name when --json is set even in TTY', async () => {
+  it('errors with missing_name when --json is set even in TTY', async () => {
     Object.defineProperty(process.stdin, 'isTTY', {
       value: true,
       writable: true,
@@ -174,7 +173,7 @@ describe('api-keys create command', () => {
     expect(output).toContain('missing_name');
   });
 
-  test('does not call SDK when missing_name error is raised', async () => {
+  it('does not call SDK when missing_name error is raised', async () => {
     setNonInteractive();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();
@@ -189,7 +188,7 @@ describe('api-keys create command', () => {
     expect(mockCreate).not.toHaveBeenCalled();
   });
 
-  test('errors with auth_error when no API key', async () => {
+  it('errors with auth_error when no API key', async () => {
     setNonInteractive();
     delete process.env.RESEND_API_KEY;
     process.env.XDG_CONFIG_HOME = '/tmp/nonexistent-resend';
@@ -209,7 +208,7 @@ describe('api-keys create command', () => {
     expect(output).toContain('auth_error');
   });
 
-  test('errors with create_error when SDK returns an error', async () => {
+  it('errors with create_error when SDK returns an error', async () => {
     setNonInteractive();
     mockCreate.mockResolvedValueOnce(
       mockSdkError('Name already taken', 'validation_error'),

tests/commands/api-keys/delete.test.ts

@@ -3,8 +3,8 @@ import {
   beforeEach,
   describe,
   expect,
+  it,
   type MockInstance,
-  test,
   vi,
 } from 'vitest';
 import {
@@ -51,7 +51,7 @@ describe('api-keys delete command', () => {
     exitSpy = undefined;
   });
 
-  test('deletes API key with --yes flag', async () => {
+  it('deletes API key with --yes flag', async () => {
     spies = setupOutputSpies();
 
     const { deleteApiKeyCommand } = await import(
@@ -64,7 +64,7 @@ describe('api-keys delete command', () => {
     expect(mockRemove).toHaveBeenCalledWith('test-key-id');
   });
 
-  test('outputs synthesized deleted JSON on success', async () => {
+  it('outputs synthesized deleted JSON on success', async () => {
     spies = setupOutputSpies();
 
     const { deleteApiKeyCommand } = await import(
@@ -80,7 +80,7 @@ describe('api-keys delete command', () => {
     expect(parsed.id).toBe('test-key-id');
   });
 
-  test('errors with confirmation_required when --yes absent in non-interactive mode', async () => {
+  it('errors with confirmation_required when --yes absent in non-interactive mode', async () => {
     setNonInteractive();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();
@@ -96,7 +96,7 @@ describe('api-keys delete command', () => {
     expect(output).toContain('confirmation_required');
   });
 
-  test('does not call SDK when confirmation is required but not given', async () => {
+  it('does not call SDK when confirmation is required but not given', async () => {
     setNonInteractive();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();
@@ -111,7 +111,7 @@ describe('api-keys delete command', () => {
     expect(mockRemove).not.toHaveBeenCalled();
   });
 
-  test('errors with auth_error when no API key', async () => {
+  it('errors with auth_error when no API key', async () => {
     setNonInteractive();
     delete process.env.RESEND_API_KEY;
     process.env.XDG_CONFIG_HOME = '/tmp/nonexistent-resend';
@@ -131,7 +131,7 @@ describe('api-keys delete command', () => {
     expect(output).toContain('auth_error');
   });
 
-  test('errors with delete_error when SDK returns an error', async () => {
+  it('errors with delete_error when SDK returns an error', async () => {
     setNonInteractive();
     mockRemove.mockResolvedValueOnce(
       mockSdkError('API key not found', 'not_found'),

tests/commands/auth/list.test.ts

@@ -2,7 +2,7 @@ import { mkdirSync, rmSync } from 'node:fs';
 import { tmpdir } from 'node:os';
 import { join } from 'node:path';
 import { Command } from '@commander-js/extra-typings';
-import { afterEach, beforeEach, describe, expect, test } from 'vitest';
+import { afterEach, beforeEach, describe, expect, it } from 'vitest';
 import { storeApiKey } from '../../../src/lib/config';
 import { captureTestEnv, setupOutputSpies } from '../../helpers';
 
@@ -35,7 +35,7 @@ describe('auth list command', () => {
     rmSync(tmpDir, { recursive: true, force: true });
   });
 
-  test('lists profiles in JSON mode', async () => {
+  it('lists profiles in JSON mode', async () => {
     spies = setupOutputSpies();
     storeApiKey('re_default', 'default');
     storeApiKey('re_staging', 'staging');
@@ -50,7 +50,7 @@ describe('auth list command', () => {
     ]);
   });
 
-  test('shows message when no profiles configured', async () => {
+  it('shows message when no profiles configured', async () => {
     spies = setupOutputSpies();
 
     const program = await createProgram();

tests/commands/auth/login.test.ts

@@ -12,8 +12,8 @@ import {
   beforeEach,
   describe,
   expect,
+  it,
   type MockInstance,
-  test,
   vi,
 } from 'vitest';
 import {
@@ -72,7 +72,7 @@ describe('login command', () => {
     rmSync(tmpDir, { recursive: true, force: true });
   });
 
-  test('rejects key that fails API validation', async () => {
+  it('rejects key that fails API validation', async () => {
     mockDomainListResult = {
       data: null,
       error: {
@@ -101,7 +101,7 @@ describe('login command', () => {
     expect(existsSync(configPath)).toBe(false);
   });
 
-  test('rejects key not starting with re_', async () => {
+  it('rejects key not starting with re_', async () => {
     setNonInteractive();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();
@@ -115,7 +115,7 @@ describe('login command', () => {
     expect(output).toContain('invalid_key_format');
   });
 
-  test('rejects empty or whitespace-only key with missing_key in non-interactive', async () => {
+  it('rejects empty or whitespace-only key with missing_key in non-interactive', async () => {
     setNonInteractive();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();
@@ -129,7 +129,7 @@ describe('login command', () => {
     expect(output).toContain('missing_key');
   });
 
-  test('trims API key before storing', async () => {
+  it('trims API key before storing', async () => {
     setNonInteractive();
 
     const { loginCommand } = await import('../../../src/commands/auth/login');
@@ -142,7 +142,7 @@ describe('login command', () => {
     expect(data.profiles.default.api_key).toBe('re_trimmed_key_456');
   });
 
-  test('stores valid key to credentials.json and sets active_profile', async () => {
+  it('stores valid key to credentials.json and sets active_profile', async () => {
     setupOutputSpies();
 
     const { loginCommand } = await import('../../../src/commands/auth/login');
@@ -156,7 +156,7 @@ describe('login command', () => {
     expect(data.active_profile).toBe('default');
   });
 
-  test('requires --key in non-interactive mode', async () => {
+  it('requires --key in non-interactive mode', async () => {
     setupOutputSpies();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();
@@ -169,7 +169,7 @@ describe('login command', () => {
     expect(output).toContain('missing_key');
   });
 
-  test('errors with missing_key when --json is set but --key is omitted even in TTY', async () => {
+  it('errors with missing_key when --json is set but --key is omitted even in TTY', async () => {
     Object.defineProperty(process.stdin, 'isTTY', {
       value: true,
       writable: true,
@@ -201,7 +201,7 @@ describe('login command', () => {
     loginCommand.parent = null;
   });
 
-  test('non-interactive login stores as default when profiles exist', async () => {
+  it('non-interactive login stores as default when profiles exist', async () => {
     // Pre-populate credentials with an existing profile
     const configDir = join(tmpDir, 'resend');
     mkdirSync(configDir, { recursive: true });
@@ -228,7 +228,7 @@ describe('login command', () => {
     expect(data.active_profile).toBe('production');
   });
 
-  test('auto-switches to profile specified via --profile flag', async () => {
+  it('auto-switches to profile specified via --profile flag', async () => {
     setupOutputSpies();
 
     const { Command } = await import('@commander-js/extra-typings');
@@ -263,7 +263,7 @@ describe('login command', () => {
     expect(data.profiles.staging.api_key).toBe('re_staging_key_123');
   });
 
-  test('deprecated --team alias works like --profile', async () => {
+  it('deprecated --team alias works like --profile', async () => {
     setupOutputSpies();
 
     const { Command } = await import('@commander-js/extra-typings');
@@ -287,7 +287,7 @@ describe('login command', () => {
     expect(data.profiles.legacy.api_key).toBe('re_team_alias_key_123');
   });
 
-  test('rejects invalid profile name with invalid_profile_name', async () => {
+  it('rejects invalid profile name with invalid_profile_name', async () => {
     setNonInteractive();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();
@@ -312,7 +312,7 @@ describe('login command', () => {
     expect(output).toContain('invalid_profile_name');
   });
 
-  test('trims --profile before storing', async () => {
+  it('trims --profile before storing', async () => {
     setupOutputSpies();
 
     const { Command } = await import('@commander-js/extra-typings');
@@ -337,7 +337,7 @@ describe('login command', () => {
     expect(data.active_profile).toBe('myprofile');
   });
 
-  test('--json output includes success, config_path, and profile', async () => {
+  it('--json output includes success, config_path, and profile', async () => {
     setupOutputSpies();
     const logSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
 
@@ -363,7 +363,7 @@ describe('login command', () => {
     expect(parsed.profile).toBe('prod');
   });
 
-  test('accepts sending-only key and stores permission', async () => {
+  it('accepts sending-only key and stores permission', async () => {
     mockDomainListResult = {
       data: null,
       error: {
@@ -386,7 +386,7 @@ describe('login command', () => {
     expect(data.profiles.default.permission).toBe('sending_access');
   });
 
-  test('stores full_access permission for valid full access key', async () => {
+  it('stores full_access permission for valid full access key', async () => {
     setupOutputSpies();
 
     const { loginCommand } = await import('../../../src/commands/auth/login');
@@ -400,7 +400,7 @@ describe('login command', () => {
     expect(data.profiles.default.permission).toBe('full_access');
   });
 
-  test('--json output includes permission for sending-only key', async () => {
+  it('--json output includes permission for sending-only key', async () => {
     mockDomainListResult = {
       data: null,
       error: {

tests/commands/auth/logout.test.ts

@@ -6,8 +6,8 @@ import {
   beforeEach,
   describe,
   expect,
+  it,
   type MockInstance,
-  test,
   vi,
 } from 'vitest';
 import {
@@ -60,7 +60,7 @@ describe('logout command', () => {
     );
   }
 
-  test('removes credentials file when it exists (non-interactive)', async () => {
+  it('removes credentials file when it exists (non-interactive)', async () => {
     spies = setupOutputSpies();
     writeCredentials();
 
@@ -75,7 +75,7 @@ describe('logout command', () => {
     expect(output.config_path).toContain('credentials.json');
   });
 
-  test('exits cleanly when no credentials file exists (non-interactive)', async () => {
+  it('exits cleanly when no credentials file exists (non-interactive)', async () => {
     spies = setupOutputSpies();
 
     const { logoutCommand } = await import('../../../src/commands/auth/logout');
@@ -86,7 +86,7 @@ describe('logout command', () => {
     expect(output.already_logged_out).toBe(true);
   });
 
-  test('logout without --profile removes all profiles', async () => {
+  it('logout without --profile removes all profiles', async () => {
     spies = setupOutputSpies();
     writeCredentials({ staging: 're_staging_key', production: 're_prod_key' });
 
@@ -101,7 +101,7 @@ describe('logout command', () => {
     expect(output.profile).toBe('all');
   });
 
-  test('logout with --profile removes only that profile', async () => {
+  it('logout with --profile removes only that profile', async () => {
     spies = setupOutputSpies();
     writeCredentials({ staging: 're_staging_key', production: 're_prod_key' });
 
@@ -133,7 +133,7 @@ describe('logout command', () => {
     expect(output.profile).toBe('staging');
   });
 
-  test('exits with error when file removal fails', async () => {
+  it('exits with error when file removal fails', async () => {
     spies = setupOutputSpies();
     errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
     exitSpy = mockExitThrow();