fix: apply CodeRabbit auto-fixes

Fixed 3 file(s) based on 4 unresolved review comments.

Co-authored-by: CodeRabbit <noreply@coderabbit.ai>

Author: coderabbitai[bot]

tests/i18n.test.ts

@@ -3,8 +3,8 @@ import i18next from 'i18next';
 import { t } from '../src/utils/i18n';
 
 describe('i18n t()', () => {
-  test('returns a non-empty translated string for a known key in English', () => {
-    i18next.changeLanguage('en');
+  test('returns a non-empty translated string for a known key in English', async () => {
+    await i18next.changeLanguage('en');
     const result = t('cancelled');
     expect(typeof result).toBe('string');
     expect(result.length).toBeGreaterThan(0);
@@ -45,12 +45,11 @@ describe('i18n t()', () => {
     expect(result).toContain('abc');
   });
 
-  test('returns the key itself or a fallback for an unknown key', () => {
-    i18next.changeLanguage('en');
+  test('returns the key itself or a fallback for an unknown key', async () => {
+    await i18next.changeLanguage('en');
     const result = t('this_key_does_not_exist_at_all');
     // i18next returns the key string when a key is missing
-    expect(typeof result).toBe('string');
-    expect(result.length).toBeGreaterThan(0);
+    expect(result).toBe('this_key_does_not_exist_at_all');
   });
 
   test('returns different strings for en and zh for the same key', () => {

tests/plugin-config.test.ts

@@ -16,14 +16,14 @@ describe('plugin-config - sentry plugin', () => {
     await fs.remove(tmpDir);
   });
 
-  const sentryPlugin = plugins.find((p) => p.name === 'sentry')!;
+  const sentryPlugin = plugins.find((p) => p.name === 'sentry');
 
   test('sentry plugin exists in the plugins array', () => {
     expect(sentryPlugin).toBeDefined();
   });
 
   test('sentry bundleParams are { sentry: true, sourcemap: true }', () => {
-    expect(sentryPlugin.bundleParams).toEqual({
+    expect(sentryPlugin?.bundleParams).toEqual({
       sentry: true,
       sourcemap: true,
     });
@@ -34,7 +34,7 @@ describe('plugin-config - sentry plugin', () => {
       const origCwd = process.cwd();
       process.chdir(tmpDir);
       try {
-        const result = await sentryPlugin.detect();
+        const result = await sentryPlugin?.detect();
         expect(result).toBe(false);
       } finally {
         process.chdir(origCwd);
@@ -51,7 +51,7 @@ describe('plugin-config - sentry plugin', () => {
       const origCwd = process.cwd();
       process.chdir(tmpDir);
       try {
-        const result = await sentryPlugin.detect();
+        const result = await sentryPlugin?.detect();
         expect(result).toBe(true);
       } finally {
         process.chdir(origCwd);
@@ -68,7 +68,7 @@ describe('plugin-config - sentry plugin', () => {
       const origCwd = process.cwd();
       process.chdir(tmpDir);
       try {
-        const result = await sentryPlugin.detect();
+        const result = await sentryPlugin?.detect();
         expect(result).toBe(true);
       } finally {
         process.chdir(origCwd);
@@ -90,7 +90,7 @@ describe('plugin-config - sentry plugin', () => {
       const origCwd = process.cwd();
       process.chdir(tmpDir);
       try {
-        const result = await sentryPlugin.detect();
+        const result = await sentryPlugin?.detect();
         expect(result).toBe(true);
       } finally {
         process.chdir(origCwd);

tests/user.test.ts

@@ -76,6 +76,12 @@ describe('userCommands.login', () => {
     });
 
     expect(saveSessionSpy).toHaveBeenCalled();
+    expect(replaceSessionSpy).toHaveBeenCalled();
+
+    // Verify call order: replaceSession should be called before saveSession
+    const replaceOrder = replaceSessionSpy.mock.invocationCallOrder[0];
+    const saveOrder = saveSessionSpy.mock.invocationCallOrder[0];
+    expect(replaceOrder).toBeLessThan(saveOrder);
   });
 
   test('prompts for email and password when args are missing', async () => {