Changeset 59987 for trunk/tests/phpunit/tests/media.php

Timestamp:

03/16/2025 06:49:39 PM (3 months ago)

Author:

joedolson

Message:

Media: Add 'muted' attribute and normalize HTML attributes.

Add the 'muted' attribute to the audio shortcode. Fix boolean attributes to meet HTML5 standards. Replaces instances like attr="1" with attr for loop, autoplay, and muted, and improves handling of the preload attribute to only output valid values.

Props shub07, dmsnell, debarghyabanerjee, audrasjb, narenin, apermo, joedolson.
Fixes #61515.

File:

• trunk/tests/phpunit/tests/media.php (modified) (3 diffs)

trunk/tests/phpunit/tests/media.php

@@ -942 +942 @@
         $this->assertStringNotContainsString( 'loop', $actual );
         $this->assertStringNotContainsString( 'autoplay', $actual );
+        $this->assertStringNotContainsString( 'muted', $actual );
         $this->assertStringContainsString( 'preload="none"', $actual );
         $this->assertStringContainsString( 'class="wp-audio-shortcode"', $actual );
@@ -951 +952 @@
                 'loop'     => true,
                 'autoplay' => true,
-                'preload'  => true,
+                'muted'    => true,
+                'preload'  => 'none',
                 'class'    => 'foobar',
                 'style'    => 'padding:0;',
@@ -958 +960 @@
 
         $this->assertStringContainsString( 'src="https://example.com/foo.mp3', $actual );
-        $this->assertStringContainsString( 'loop="1"', $actual );
-        $this->assertStringContainsString( 'autoplay="1"', $actual );
-        $this->assertStringContainsString( 'preload="1"', $actual );
+        $this->assertStringContainsString( 'loop', $actual );
+        $this->assertStringContainsString( 'autoplay', $actual );
+        $this->assertStringContainsString( 'muted', $actual );
+        $this->assertStringContainsString( 'preload="none"', $actual );
         $this->assertStringContainsString( 'class="foobar"', $actual );
         $this->assertStringContainsString( 'style="padding:0;"', $actual );