add second newline in GuidelineWriterTests

Author: ChipNeedham

tests/Unit/Install/GuidelineWriterTest.php

@@ -64,7 +64,7 @@
     $writer->write('test guidelines content');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("<laravel-boost-guidelines>\ntest guidelines content\n</laravel-boost-guidelines>\n");
+    expect($content)->toBe("<laravel-boost-guidelines>\ntest guidelines content\n\n</laravel-boost-guidelines>\n");
 
     unlink($tempFile);
 });
@@ -82,7 +82,7 @@
     $writer->write('new guidelines');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("# Existing content\n\nSome text here.\n\n===\n\n<laravel-boost-guidelines>\nnew guidelines\n</laravel-boost-guidelines>\n");
+    expect($content)->toBe("# Existing content\n\nSome text here.\n\n===\n\n<laravel-boost-guidelines>\nnew guidelines\n\n</laravel-boost-guidelines>\n");
 
     unlink($tempFile);
 });
@@ -101,7 +101,7 @@
     $writer->write('updated guidelines');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("# Header\n\n<laravel-boost-guidelines>\nupdated guidelines\n</laravel-boost-guidelines>\n\n# Footer\n");
+    expect($content)->toBe("# Header\n\n<laravel-boost-guidelines>\nupdated guidelines\n\n</laravel-boost-guidelines>\n\n# Footer\n");
 
     unlink($tempFile);
 });
@@ -120,7 +120,7 @@
     $writer->write('updated guidelines');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("# Header\n\n<laravel-boost-guidelines>\nupdated guidelines\n</laravel-boost-guidelines>\n\n# Footer\n");
+    expect($content)->toBe("# Header\n\n<laravel-boost-guidelines>\nupdated guidelines\n\n</laravel-boost-guidelines>\n\n# Footer\n");
 
     // Assert no double newline at the end
     expect(substr($content, -2))->not->toBe("\n\n");
@@ -145,7 +145,7 @@
 
     $content = file_get_contents($tempFile);
     // Should replace in-place, preserving structure
-    expect($content)->toBe("Start\n<laravel-boost-guidelines>\nsingle line\n</laravel-boost-guidelines>\nEnd\n");
+    expect($content)->toBe("Start\n<laravel-boost-guidelines>\nsingle line\n\n</laravel-boost-guidelines>\nEnd\n");
 
     unlink($tempFile);
 });
@@ -165,7 +165,7 @@
 
     $content = file_get_contents($tempFile);
     // Should replace first occurrence, second block remains untouched due to non-greedy matching
-    expect($content)->toBe("Start\n<laravel-boost-guidelines>\nreplacement\n</laravel-boost-guidelines>\nMiddle\n<laravel-boost-guidelines>\nsecond\n</laravel-boost-guidelines>\nEnd\n");
+    expect($content)->toBe("Start\n<laravel-boost-guidelines>\nreplacement\n\n</laravel-boost-guidelines>\nMiddle\n<laravel-boost-guidelines>\nsecond\n</laravel-boost-guidelines>\nEnd\n");
 
     unlink($tempFile);
 });
@@ -199,7 +199,7 @@
     $writer->write('my guidelines');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("# Title\n\nParagraph 1\n\nParagraph 2\n\n===\n\n<laravel-boost-guidelines>\nmy guidelines\n</laravel-boost-guidelines>\n");
+    expect($content)->toBe("# Title\n\nParagraph 1\n\nParagraph 2\n\n===\n\n<laravel-boost-guidelines>\nmy guidelines\n\n</laravel-boost-guidelines>\n");
 
     unlink($tempFile);
 });
@@ -217,7 +217,7 @@
     $writer->write('first guidelines');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("<laravel-boost-guidelines>\nfirst guidelines\n</laravel-boost-guidelines>\n");
+    expect($content)->toBe("<laravel-boost-guidelines>\nfirst guidelines\n\n</laravel-boost-guidelines>\n");
 
     unlink($tempFile);
 });
@@ -235,7 +235,7 @@
     $writer->write('clean guidelines');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("<laravel-boost-guidelines>\nclean guidelines\n</laravel-boost-guidelines>\n");
+    expect($content)->toBe("<laravel-boost-guidelines>\nclean guidelines\n\n</laravel-boost-guidelines>\n");
 
     unlink($tempFile);
 });
@@ -255,7 +255,7 @@
 
     expect($result)->toBe(GuidelineWriter::REPLACED);
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("# Title\n\n<other-rules>\nShould not be touched\n</other-rules>\n\n<laravel-boost-guidelines>\nnew guidelines\n</laravel-boost-guidelines>\n\n<custom-config>\nAlso untouched\n</custom-config>\n");
+    expect($content)->toBe("# Title\n\n<other-rules>\nShould not be touched\n</other-rules>\n\n<laravel-boost-guidelines>\nnew guidelines\n\n</laravel-boost-guidelines>\n\n<custom-config>\nAlso untouched\n</custom-config>\n");
 
     unlink($tempFile);
 });
@@ -288,7 +288,7 @@
     );
 
     // Verify exact structure
-    expect($content)->toBe("# My Project\n\n<laravel-boost-guidelines>\nupdated guidelines from boost\n</laravel-boost-guidelines>\n\n# User Added Section\nThis content was added by the user after the guidelines.\n\n## Another user section\nMore content here.\n");
+    expect($content)->toBe("# My Project\n\n<laravel-boost-guidelines>\nupdated guidelines from boost\n\n</laravel-boost-guidelines>\n\n# User Added Section\nThis content was added by the user after the guidelines.\n\n## Another user section\nMore content here.\n");
 
     unlink($tempFile);
 });
@@ -310,7 +310,7 @@
     $writer->write('new guidelines');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("---\nalwaysApply: true\n---\n# Existing content\n\nSome text here.\n\n===\n\n<laravel-boost-guidelines>\nnew guidelines\n</laravel-boost-guidelines>\n");
+    expect($content)->toBe("---\nalwaysApply: true\n---\n# Existing content\n\nSome text here.\n\n===\n\n<laravel-boost-guidelines>\nnew guidelines\n\n</laravel-boost-guidelines>\n");
 
     unlink($tempFile);
 });
@@ -328,7 +328,7 @@
     $writer->write('new guidelines');
 
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("---\ncustomOption: true\n---\n# Existing content\n\nSome text here.\n\n===\n\n<laravel-boost-guidelines>\nnew guidelines\n</laravel-boost-guidelines>\n");
+    expect($content)->toBe("---\ncustomOption: true\n---\n# Existing content\n\nSome text here.\n\n===\n\n<laravel-boost-guidelines>\nnew guidelines\n\n</laravel-boost-guidelines>\n");
 
     unlink($tempFile);
 });
@@ -347,7 +347,7 @@
 
     expect($result)->toBe(GuidelineWriter::NEW);
     $content = file_get_contents($tempFile);
-    expect($content)->toBe("# Existing content\n\nSome text here.\n\n===\n\n<laravel-boost-guidelines>\nnew guidelines\n</laravel-boost-guidelines>\n");
+    expect($content)->toBe("# Existing content\n\nSome text here.\n\n===\n\n<laravel-boost-guidelines>\nnew guidelines\n\n</laravel-boost-guidelines>\n");
 
     unlink($tempFile);
 });