Merge pull request #33 from raylanlin/test/coverage-improvements

test: add missing tests for file/upload, search/query, vision/describe

Author: MiniMax-AI-Dev

test/commands/auth/login.test.ts

@@ -18,7 +18,7 @@ describe('auth login command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -31,7 +31,7 @@ describe('auth login command', () => {
         yes: false,
         dryRun: false,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       }),
     ).rejects.toThrow('--api-key is required');

test/commands/auth/logout.test.ts

@@ -17,7 +17,7 @@ describe('auth logout command', () => {
       noColor: true,
       yes: false,
       dryRun: true,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -33,7 +33,7 @@ describe('auth logout command', () => {
         yes: false,
         dryRun: true,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       });
 

test/commands/auth/refresh.test.ts

@@ -17,7 +17,7 @@ describe('auth refresh command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -29,7 +29,7 @@ describe('auth refresh command', () => {
         yes: false,
         dryRun: false,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       }),
     ).rejects.toThrow('not authenticated via OAuth');

test/commands/auth/status.test.ts

@@ -17,7 +17,7 @@ describe('auth status command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -33,7 +33,7 @@ describe('auth status command', () => {
         yes: false,
         dryRun: false,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       });
 

test/commands/config/set.test.ts

@@ -17,7 +17,7 @@ describe('config set command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -29,7 +29,7 @@ describe('config set command', () => {
         yes: false,
         dryRun: false,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       }),
     ).rejects.toThrow('--key and --value are required');
@@ -46,7 +46,7 @@ describe('config set command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -60,7 +60,7 @@ describe('config set command', () => {
         yes: false,
         dryRun: false,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       }),
     ).rejects.toThrow('Invalid config key');

test/commands/config/show.test.ts

@@ -18,7 +18,7 @@ describe('config show command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -34,7 +34,7 @@ describe('config show command', () => {
         yes: false,
         dryRun: false,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       });
 

test/commands/file/upload.test.ts

@@ -0,0 +1,38 @@
+import { describe, it, expect } from 'bun:test';
+import { default as uploadCommand } from '../../../src/commands/file/upload';
+
+describe('file upload command', () => {
+  it('has correct name', () => {
+    expect(uploadCommand.name).toBe('file upload');
+  });
+
+  it('requires file argument', async () => {
+    const config = {
+      apiKey: 'test-key',
+      region: 'global' as const,
+      baseUrl: 'https://api.minimax.io',
+      output: 'text' as const,
+      timeout: 10,
+      verbose: false,
+      quiet: false,
+      noColor: true,
+      yes: false,
+      dryRun: true,
+      nonInteractive: true,
+      async: false,
+    };
+
+    await expect(
+      uploadCommand.execute(config, {
+        quiet: false,
+        verbose: false,
+        noColor: true,
+        yes: false,
+        dryRun: true,
+        help: false,
+        nonInteractive: true,
+        async: false,
+      }),
+    ).rejects.toThrow('Missing required argument: --file');
+  });
+});

test/commands/image/generate.test.ts

@@ -18,7 +18,7 @@ describe('image generate command', () => {
       noColor: true,
       yes: false,
       dryRun: false,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -30,7 +30,7 @@ describe('image generate command', () => {
         yes: false,
         dryRun: false,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       }),
     ).rejects.toThrow('Missing required argument: --prompt');

test/commands/quota/show.test.ts

@@ -18,7 +18,7 @@ describe('quota show command', () => {
       noColor: true,
       yes: false,
       dryRun: true,
-      nonInteractive: false,
+      nonInteractive: true,
       async: false,
     };
 
@@ -34,7 +34,7 @@ describe('quota show command', () => {
         yes: false,
         dryRun: true,
         help: false,
-        nonInteractive: false,
+        nonInteractive: true,
         async: false,
       });
 

test/commands/search/query.test.ts

@@ -0,0 +1,38 @@
+import { describe, it, expect } from 'bun:test';
+import { default as queryCommand } from '../../../src/commands/search/query';
+
+describe('search query command', () => {
+  it('has correct name', () => {
+    expect(queryCommand.name).toBe('search query');
+  });
+
+  it('requires q argument', async () => {
+    const config = {
+      apiKey: 'test-key',
+      region: 'global' as const,
+      baseUrl: 'https://api.minimax.io',
+      output: 'text' as const,
+      timeout: 10,
+      verbose: false,
+      quiet: false,
+      noColor: true,
+      yes: false,
+      dryRun: true,
+      nonInteractive: true,
+      async: false,
+    };
+
+    await expect(
+      queryCommand.execute(config, {
+        quiet: false,
+        verbose: false,
+        noColor: true,
+        yes: false,
+        dryRun: true,
+        help: false,
+        nonInteractive: true,
+        async: false,
+      }),
+    ).rejects.toThrow('--q is required');
+  });
+});