ensure end of body placement

gwleuverink · Aug 18, 2024 · edf49c2 · edf49c2
1 parent 2fb3468
commit edf49c2
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/src/InjectAssets.php b/src/InjectAssets.php
@@ -56,12 +56,12 @@ protected function inject(string $html, string $assets): string
 
         if ($html->test('/<\s*\/\s*head\s*>/i')) {
             return $html
-                ->replaceMatches('/(<\s*\\s*head\s*>)/i', '$1' . $assets)
+                ->replaceMatches('/(<\s*\/\s*head\s*>)/i', $assets . '$1')
                 ->toString();
         }
 
         return $html
-            ->replaceMatches('/(<\s*html(?:\s[^>])*>)/i', '$1' . $assets)
+            ->replaceMatches('/(<\s*\/\s*html\s*>)/i', $assets . '$1')
             ->toString();
     }
 }
diff --git a/tests/Feature/InjectsAssetsTest.php b/tests/Feature/InjectsAssetsTest.php
@@ -42,6 +42,22 @@
         ->assertSee($expected, false);
 });
 
+it('injects assets into the end of the html body when no head tag is present', function () {
+    Route::get('test-inject-in-response', fn () => '<html><p>Hello World</p></html>');
+
+    $expected = <<< 'HTML'
+    <html><p>Hello World</p>
+    <!--[TEST_PACKAGE]-->
+    TEST_PACKAGE_ASSETS_INJECTED
+    <!--[ENDTEST_PACKAGE]-->
+    </html>
+    HTML;
+
+    $this->get('test-inject-in-response')
+        ->assertOk()
+        ->assertSee($expected, false);
+});
+
 it('doesnt inject assets into responses without a closing html tag', function () {
     Route::get('test-inject-in-response', fn () => 'OK');