fix: improve code formatting and consistency in InlineCss middleware and related tests

Author: renatomarinho

src/Middleware/InlineCss.php

@@ -22,7 +22,7 @@ public function apply($buffer)
 
         $this->class = collect($matches[1])->mapWithKeys(function ($item) {
 
-            return [ 'page_speed_'.rand() => $item[0] ];
+            return ['page_speed_' . rand() => $item[0]];
         })->unique();
 
         return $this->injectStyle()->injectClass()->fixHTML()->html;
@@ -36,7 +36,8 @@ private function injectStyle()
 
             $this->style[] = [
                 'class' => $class,
-                'attributes' => preg_quote($attributes, '/')];
+                'attributes' => preg_quote($attributes, '/')
+            ];
         });
 
         $injectStyle = implode(' ', $this->inline);
@@ -54,7 +55,7 @@ private function injectClass()
     {
         collect($this->style)->each(function ($item) {
             $replace = [
-                '/style="'.$item['attributes'].'"/' => "class=\"{$item['class']}\"",
+                '/style="' . $item['attributes'] . '"/' => "class=\"{$item['class']}\"",
             ];
 
             $this->html = $this->replace($replace, $this->html);
@@ -77,7 +78,7 @@ private function fixHTML()
 
             if (count($matches[1]) > 1) {
                 $replace = [
-                    '/>/' => "class=\"".implode(' ', $matches[1])."\">",
+                    '/>/' => "class=\"" . implode(' ', $matches[1]) . "\">",
                 ];
 
                 $newHTML[] = str_replace(

tests/Middleware/InlineCssJavaScriptFrameworksTest.php

@@ -37,11 +37,11 @@ public function test_angularjs_ng_class_preserved(): void
         // ng-class attributes must be preserved exactly
         $this->assertStringContainsString('ng-class="{active: isActive, disabled: isDisabled}"', $result);
         $this->assertStringContainsString('ng-class="dynamicClass"', $result);
-        
+
         // Should have style tag with converted inline styles
         $this->assertStringContainsString('<style>', $result);
         $this->assertStringContainsString('page_speed_', $result);
-        
+
         // Inline styles should be converted to classes
         $this->assertStringNotContainsString('style="padding: 10px;"', $result);
         $this->assertStringNotContainsString('style="font-weight: bold;"', $result);
@@ -71,12 +71,12 @@ public function test_alpinejs_class_shorthand_preserved(): void
         // Alpine.js :class must be preserved
         $this->assertStringContainsString(':class="{ \'bg-blue\': open, \'bg-gray\': !open }"', $result);
         $this->assertStringContainsString(':class="open ? \'visible\' : \'hidden\'"', $result);
-        
+
         // Inline styles should be converted to classes
         $this->assertStringNotContainsString('style="padding: 5px;"', $result);
         $this->assertStringNotContainsString('style="margin: 10px;"', $result);
         $this->assertStringNotContainsString('style="color: red;"', $result);
-        
+
         // Should have page_speed classes
         $this->assertStringContainsString('page_speed_', $result);
     }
@@ -107,7 +107,7 @@ public function test_vuejs_vbind_class_preserved(): void
         $this->assertStringContainsString('v-bind:class="{ active: isActive, \'text-danger\': hasError }"', $result);
         $this->assertStringContainsString('v-bind:class="classObject"', $result);
         $this->assertStringContainsString('v-bind:class="[activeClass, errorClass]"', $result);
-        
+
         // Inline styles should be converted
         $this->assertStringNotContainsString('style="padding: 20px;"', $result);
         $this->assertStringNotContainsString('style="font-size: 14px;"', $result);
@@ -144,13 +144,13 @@ public function test_mixed_class_and_framework_attributes(): void
         $this->assertStringContainsString('ng-class="{expanded: isExpanded}"', $result);
         $this->assertStringContainsString('v-bind:class="vueClasses"', $result);
         $this->assertStringContainsString(':class="alpineClasses"', $result);
-        
+
         // Inline styles should be converted to page_speed classes
         $this->assertStringNotContainsString('style="border: 1px solid;"', $result);
         $this->assertStringNotContainsString('style="padding: 10px;"', $result);
         $this->assertStringNotContainsString('style="margin: 5px;"', $result);
         $this->assertStringNotContainsString('style="display: flex;"', $result);
-        
+
         // Should have style tag with converted classes
         $this->assertStringContainsString('<style>', $result);
     }
@@ -179,7 +179,7 @@ public function test_normal_class_still_works(): void
         $this->assertStringNotContainsString('style="width: 100%;"', $result);
         $this->assertStringNotContainsString('style="font-weight: bold;"', $result);
         $this->assertStringNotContainsString('style="color: yellow;"', $result);
-        
+
         // Should have page_speed classes instead
         $this->assertStringContainsString('class="page_speed_', $result);
     }
@@ -204,7 +204,7 @@ public function test_attribute_ending_with_class(): void
 
         // Attribute ending with "class" should not be affected
         $this->assertStringContainsString('data-css-class="some-value"', $result);
-        
+
         // Inline style should be converted
         $this->assertStringNotContainsString('style="background: blue;"', $result);
     }
@@ -246,13 +246,13 @@ public function test_complex_multi_framework_scenario(): void
         $this->assertStringContainsString(':class="open ? \'show\' : \'hide\'"', $result);
         $this->assertStringContainsString('v-bind:class="contentClasses"', $result);
         $this->assertStringContainsString('ng-class="{loading: isLoading}"', $result);
-        
+
         // Inline styles should be converted
         $this->assertStringNotContainsString('style="background: white;"', $result);
         $this->assertStringNotContainsString('style="padding: 10px;"', $result);
         $this->assertStringNotContainsString('style="border: 1px solid;"', $result);
         $this->assertStringNotContainsString('style="font-size: 24px;"', $result);
-        
+
         // Should have style tag
         $this->assertStringContainsString('<style>', $result);
     }