Formatting, empty() check to satisfy PHP_CodeSniffer and PHPStan

Author: stevegrunwell

src/MarkupAssertionsTrait.php

@@ -249,7 +249,7 @@ private function flattenAttributeArray(array $attributes)
 
         array_walk($attributes, function (&$value, $key) {
             // Boolean attributes.
-            if (null === $value) {
+            if (empty($value)) {
                 $value = sprintf('[%s]', $key);
             } else {
                 $value = sprintf('[%s="%s"]', $key, htmlspecialchars($value));

tests/MarkupAssertionsTraitTest.php

@@ -44,8 +44,9 @@ public function assertContainsSelector_should_pick_up_multiple_instances(): void
      * @testdox assertNotContainsSelector() should verify that the given selector does not exist
      * @dataProvider provideSelectorVariants
      */
-    public function assertNotContainsSelector_should_verify_that_the_given_selector_does_not_exist(string $selector): void
-    {
+    public function assertNotContainsSelector_should_verify_that_the_given_selector_does_not_exist(
+        string $selector
+    ): void {
         $this->assertNotContainsSelector(
             $selector,
             '<h1 id="page-title" class="foo bar">This element has little to do with the link.</h1>'
@@ -270,8 +271,11 @@ public function flattenAttributeArray_should_throw_a_RiskyTestError_if_given_an_
      * @testdox getInnerHtmlOfMatchedElements() should retrieve the inner HTML
      * @dataProvider provideInnerHtml
      */
-    public function getInnerHtmlOfMatchedElements_should_retrieve_the_inner_HTML(string $markup, string $selector, string $expected): void
-    {
+    public function getInnerHtmlOfMatchedElements_should_retrieve_the_inner_HTML(
+        string $markup,
+        string $selector,
+        string $expected
+    ): void {
         $method = new \ReflectionMethod($this, 'getInnerHtmlOfMatchedElements');
         $method->setAccessible(true);