Address PR lchrusciel#209 review comments: improve diff output formatting

- Remove redundant matcher error message from assertResponseContent
  The detailed "Value X does not match pattern Y" message was duplicating
  information already visible in the contextual diff

- Move expected file path to end of diff output in absolute format
  The file path now appears at the bottom as "Expected file: /absolute/path"
  making it easier to reference after reviewing the diff

- Update all tests to reflect the new file path format

Note: Pattern matcher normalization (avoiding display of passing pattern
matches in diff) remains a complex issue requiring deeper refactoring of
how expected responses are compared with actual responses.

https://claude.ai/code/session_014m1n4X3T8cCXz2HguPjWFH

Author: claude

src/ApiTestCase.php

@@ -192,7 +192,7 @@ protected function assertResponseContent(string $actualResponse, string $filenam
                 'expectedFilePath' => $expectedFilePath,
             ]);
 
-            self::fail($matcher->getError() . \PHP_EOL . $diff->render($renderer));
+            self::fail($diff->render($renderer));
         }
     }
 

src/Renderer/ContextualDiffRenderer.php

@@ -49,12 +49,6 @@ public function render(): string
     {
         $output = '';
 
-        // Add expected file path if provided
-        if ($this->expectedFilePath !== null) {
-            $output .= sprintf("--- Expected: %s\n", $this->expectedFilePath);
-            $output .= "+++ Actual\n";
-        }
-
         $opCodes = $this->diff->getGroupedOpcodes();
 
         foreach ($opCodes as $group) {
@@ -100,6 +94,12 @@ public function render(): string
             }
         }
 
+        // Add expected file path at the end if provided
+        if ($this->expectedFilePath !== null) {
+            $absolutePath = realpath($this->expectedFilePath) ?: $this->expectedFilePath;
+            $output .= sprintf("\nExpected file: %s\n", $absolutePath);
+        }
+
         return $output;
     }
 

test/src/Tests/Controller/SampleControllerJsonTest.php

@@ -168,9 +168,8 @@ public function testDiffVisualizationShowsContextAndColors(): void
         } catch (AssertionFailedError $e) {
             $message = $e->getMessage();
 
-            $this->assertStringContainsString('--- Expected:', $message);
+            $this->assertStringContainsString('Expected file:', $message);
             $this->assertStringContainsString('incorrect_hello_world.json', $message);
-            $this->assertStringContainsString('+++ Actual', $message);
             $this->assertStringContainsString('-', $message);
             $this->assertStringContainsString('+', $message);
             $this->assertStringContainsString('message', $message);

test/src/Tests/Controller/SampleControllerXmlTest.php

@@ -102,9 +102,8 @@ public function testDiffVisualizationShowsContextAndColors(): void
         } catch (AssertionFailedError $e) {
             $message = $e->getMessage();
 
-            $this->assertStringContainsString('--- Expected:', $message);
+            $this->assertStringContainsString('Expected file:', $message);
             $this->assertStringContainsString('incorrect_hello_world.xml', $message);
-            $this->assertStringContainsString('+++ Actual', $message);
             $this->assertStringContainsString('-', $message);
             $this->assertStringContainsString('+', $message);
         }

test/src/Tests/Renderer/ContextualDiffRendererTest.php

@@ -70,8 +70,7 @@ public function testRenderIncludesExpectedFilePath(): void
         ]);
         $output = $diff->render($renderer);
 
-        $this->assertStringContainsString('--- Expected: /path/to/expected.json', $output);
-        $this->assertStringContainsString('+++ Actual', $output);
+        $this->assertStringContainsString('Expected file: /path/to/expected.json', $output);
     }
 
     public function testRenderHandlesInsertions(): void