fix: ensure error-path tests set non-interactive mode before spying

Tests that spy on console.log for error output must first establish
non-interactive mode, otherwise the assertion depends on host TTY state.

Author: felipefreitag

tests/commands/auth/login.test.ts

@@ -20,6 +20,7 @@ import {
   captureTestEnv,
   expectExit1,
   mockExitThrow,
+  setNonInteractive,
   setupOutputSpies,
 } from '../../helpers';
 
@@ -75,6 +76,7 @@ describe('login command', () => {
       },
     };
 
+    setNonInteractive();
     errorSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
     exitSpy = mockExitThrow();
 
@@ -94,6 +96,7 @@ describe('login command', () => {
   });
 
   test('rejects key not starting with re_', async () => {
+    setNonInteractive();
     errorSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
     exitSpy = mockExitThrow();
 

tests/commands/auth/logout.test.ts

@@ -134,9 +134,9 @@ describe('logout command', () => {
   });
 
   test('exits with error when file removal fails', async () => {
+    spies = setupOutputSpies();
     errorSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
     exitSpy = mockExitThrow();
-    spies = setupOutputSpies();
     writeCredentials();
 
     // Make the credentials file a directory so unlinkSync throws