Fix unit tests

Author: adamziel

components/DataLiberation/Tests/BlockMarkupUrlProcessorTest.php

@@ -413,7 +413,7 @@ public static function provider_test_css_url_replacement() {
 			'Replace single-quoted URL'          => array(
 				'<div style="background: url(\'https://old.com/image.png\');"></div>',
 				'https://new.com/image.png',
-				'<div style="background: url(\'https://new.com/image.png\');"></div>',
+				'<div style="background: url(&quot;https://new.com/image.png&quot;);"></div>',
 			),
 			'Replace relative URL'               => array(
 				'<div style="background: url(&quot;/old/path.png&quot;);"></div>',

components/DataLiberation/Tests/CSSUrlProcessorTest.php

@@ -344,7 +344,7 @@ public static function provider_test_url_replacement() {
 			'Replace single-quoted URL'             => array(
 				'input'    => "background: url('https://old.com/image.png')",
 				'new_url'  => 'https://new.com/image.png',
-				'expected' => "background: url('https://new.com/image.png')",
+				'expected' => "background: url(\u{22}https://new.com/image.png\u{22})",
 			),
 			'Replace unquoted URL (outputs quoted)' => array(
 				'input'    => 'background: url(https://old.com/image.png)',
@@ -360,7 +360,7 @@ public static function provider_test_url_replacement() {
 			'Sets new URL with single quotes in single-quoted string' => array(
 				'input'    => "background: url('https://old.com/old.png')",
 				'new_url'  => "https://example.com/path'with'quotes.png",
-				'expected' => "background: url('https://example.com/path'with'quotes.png')",
+				'expected' => "background: url(\u{22}https://example.com/path'with'quotes.png\u{22})",
 				// Single quotes not escaped in single-quoted context
 			),
 			'Sets new URL with backslashes in path'                   => array(
@@ -522,7 +522,7 @@ public function test_comprehensive_url_replacement_in_complex_css() {
 .card {
 	/* Multiple URLs in a single property */
 	background: linear-gradient(rgba(0,0,0,0.5), rgba(0,0,0,0.5)),
-	            url('https://replaced.test/url-2'),
+	            url("https://replaced.test/url-2"),
 	            url("https://replaced.test/url-3");
 }
 
@@ -537,7 +537,7 @@ public function test_comprehensive_url_replacement_in_complex_css() {
 }
 
 .cursor {
-	cursor: url('https://replaced.test/url-5'), auto;
+	cursor: url("https://replaced.test/url-5"), auto;
 }
 
 .content::before {

components/DataLiberation/URL/class-cssprocessor.php

@@ -838,27 +838,26 @@ public function get_token_value_length(): ?int {
 	 * @return bool Whether the value was successfully updated.
 	 */
 	public function set_token_value( string $new_value ): bool {
-		// Only URL tokens are currently supported.
-		if ( self::TOKEN_URL !== $this->token_type ) {
-			return false;
-		}
-
-		// Ensure we have valid token value boundaries.
-		if ( null === $this->token_value_starts_at || null === $this->token_value_length ) {
-			return false;
+		// Only URL and string tokens are currently supported.
+		switch ($this->token_type) {
+			case self::TOKEN_URL:
+				$this->lexical_updates[] = array(
+					'start'  => $this->token_value_starts_at,
+					'length' => $this->token_value_length,
+					'text'   => $this->escape_url_value( $new_value ),
+				);
+				return true;
+			case self::TOKEN_STRING:
+				$this->lexical_updates[] = array(
+					'start'  => $this->token_starts_at,
+					'length' => $this->token_length,
+					'text'   => $this->escape_url_value( $new_value ),
+				);
+				return true;
+			default:
+				_doing_it_wrong( __METHOD__, 'set_token_value() only supports URL and string tokens. Got token type: ' . $this->token_type, '1.0.0' );
+				return false;
 		}
-
-		// Escape the URL value for unquoted URL syntax.
-		$escaped_value = $this->escape_url_value( $new_value );
-
-		// Queue the lexical update.
-		$this->lexical_updates[] = array(
-			'start'  => $this->token_value_starts_at,
-			'length' => $this->token_value_length,
-			'text'   => $escaped_value,
-		);
-
-		return true;
 	}
 
 	/**