fix tests: use getDisplay() instead of getErrorOutput() for CLI validation tests

CommandTester with capture_stderr_separately:true doesn't properly route
SymfonyStyle error() to a separate stderr stream in this Symfony version.
Fix tests to check combined output via getDisplay() which captures
stdout+stderr together, matching actual runtime behavior.

Author: detain

tests/phpunit/unit/Cli/Command/CancelCommandTest.php

@@ -39,12 +39,12 @@ public function testRequiresIdOrRepoOrAll(): void
         $command = $application->find('cancel');
         $commandTester = new CommandTester($command);
 
-        $exitCode = $commandTester->execute([], ['capture_stderr_separately' => true]);
+        $exitCode = $commandTester->execute([]);
 
         $this->assertSame(Command::FAILURE, $exitCode);
         $this->assertStringContainsString(
             'Specify --id, --repo, or --all',
-            $commandTester->getErrorOutput()
+            $commandTester->getDisplay()
         );
     }
 
@@ -57,12 +57,12 @@ public function testAllAndRepoAreMutuallyExclusive(): void
         $exitCode = $commandTester->execute([
             '--all' => true,
             '--repo' => 'owner/repo',
-        ], ['capture_stderr_separately' => true]);
+        ]);
 
         $this->assertSame(Command::FAILURE, $exitCode);
         $this->assertStringContainsString(
             'mutually exclusive',
-            $commandTester->getErrorOutput()
+            $commandTester->getDisplay()
         );
     }
 
@@ -77,7 +77,7 @@ public function testReturnsFailureOnRedisConnectionError(): void
 
         $exitCode = $commandTester->execute([
             '--all' => true,
-        ], ['capture_stderr_separately' => true]);
+        ]);
 
         // Restore environment
         putenv('REDIS_HOST');

tests/phpunit/unit/Cli/Command/ConfigCommandTest.php

@@ -54,12 +54,12 @@ public function testInvalidSetKeyFormat(): void
 
         $exitCode = $commandTester->execute([
             '--set-key' => 'invalid-format-without-equals',
-        ], ['capture_stderr_separately' => true]);
+        ]);
 
         $this->assertSame(Command::FAILURE, $exitCode);
         $this->assertStringContainsString(
             'Invalid format',
-            $commandTester->getErrorOutput()
+            $commandTester->getDisplay()
         );
     }
 
@@ -71,12 +71,12 @@ public function testEmptyKeyInSetKey(): void
 
         $exitCode = $commandTester->execute([
             '--set-key' => '=somevalue',
-        ], ['capture_stderr_separately' => true]);
+        ]);
 
         $this->assertSame(Command::FAILURE, $exitCode);
         $this->assertStringContainsString(
             'Key cannot be empty',
-            $commandTester->getErrorOutput()
+            $commandTester->getDisplay()
         );
     }
 
@@ -101,12 +101,12 @@ public function testResetUnknownKey(): void
 
         $exitCode = $commandTester->execute([
             '--reset' => 'nonexistent.key',
-        ], ['capture_stderr_separately' => true]);
+        ]);
 
         $this->assertSame(Command::FAILURE, $exitCode);
         $this->assertStringContainsString(
             'Unknown configuration key',
-            $commandTester->getErrorOutput()
+            $commandTester->getDisplay()
         );
     }
 

tests/phpunit/unit/Cli/Command/StatusCommandTest.php

@@ -41,12 +41,12 @@ public function testWatchModeRequiresId(): void
 
         $exitCode = $commandTester->execute([
             '--watch' => true,
-        ], ['capture_stderr_separately' => true]);
+        ]);
 
         $this->assertSame(Command::FAILURE, $exitCode);
         $this->assertStringContainsString(
             '--watch requires --id',
-            $commandTester->getErrorOutput()
+            $commandTester->getDisplay()
         );
     }
 

tests/phpunit/unit/Cli/Command/SubmitCommandTest.php

@@ -62,12 +62,12 @@ public function testPostDiffsAndPostBranchAreMutuallyExclusive(): void
             '--pr' => '42',
             '--post-diffs' => true,
             '--post-branch' => true,
-        ], ['capture_stderr_separately' => true]);
+        ]);
 
         $this->assertSame(Command::FAILURE, $exitCode);
         $this->assertStringContainsString(
             'mutually exclusive',
-            $commandTester->getErrorOutput()
+            $commandTester->getDisplay()
         );
     }
 
@@ -82,12 +82,12 @@ public function testSplitChangesAndCombineAreMutuallyExclusive(): void
             '--pr' => '42',
             '--split-changes' => true,
             '--combine' => true,
-        ], ['capture_stderr_separately' => true]);
+        ]);
 
         $this->assertSame(Command::FAILURE, $exitCode);
         $this->assertStringContainsString(
             'mutually exclusive',
-            $commandTester->getErrorOutput()
+            $commandTester->getDisplay()
         );
     }