Fix deprecated _admin_bar_bump_cb; Fix failing PHP unit tests

.github/workflows/build-test-measure.yml

@@ -548,6 +548,20 @@ jobs:
         run: |
           echo "WP_ENVIRONMENT_TYPE=local" >> $GITHUB_ENV
 
+      - name: Override default PHPUnit configuration
+        if: ${{ matrix.experimental == true && needs.pre-run.outputs.changed-php-count > 0 }}
+        run: |
+          cp phpunit.xml.dist phpunit.xml
+
+          # Avoid converting deprecations, errors, notices, and warnings to exceptions in experimental mode.
+          sed -i 's/convertDeprecationsToExceptions="true"/convertDeprecationsToExceptions="false"/g' phpunit.xml
+          sed -i 's/convertErrorsToExceptions="true"/convertErrorsToExceptions="false"/g' phpunit.xml
+          sed -i 's/convertNoticesToExceptions="true"/convertNoticesToExceptions="false"/g' phpunit.xml
+          sed -i 's/convertWarningsToExceptions="true"/convertWarningsToExceptions="false"/g' phpunit.xml
+        working-directory: ${{ env.WP_CORE_DIR }}/src/wp-content/plugins/amp
+        env:
+          WP_TESTS_DIR: /tmp/wordpress-tests-lib
+
       - name: Run tests
         if: ${{ matrix.coverage == false && needs.pre-run.outputs.changed-php-count > 0 }}
         run: |

includes/class-amp-theme-support.php

@@ -84,6 +84,13 @@
 	 */
 	const READER_MODE_TEMPLATE_DIRECTORY = 'amp';
 
+	/**
+	 * `wp-emoji-styles` styles handle.
+	 *
+	 * @var string
+	 */
+	const WP_EMOJI_STYLES = 'wp-emoji-styles';
+
 	/**
 	 * Sanitizers, with keys as class names and values as arguments.
 	 *
@@ -848,7 +855,13 @@
 		// Prevent emoji detection and emoji loading since platforms/browsers now support emoji natively (and Twemoji is not AMP-compatible).
 		add_filter( 'wp_resource_hints', [ __CLASS__, 'filter_resource_hints_to_remove_emoji_dns_prefetch' ], 10, 2 );
 		remove_action( 'wp_head', 'print_emoji_detection_script', 7 );
-		remove_action( 'wp_print_styles', 'print_emoji_styles' );
+
+		if ( function_exists( 'wp_enqueue_emoji_styles' ) ) {
+			// By default, emoji styles are enqueued at priority 10.
+			add_action( 'wp_print_styles', [ __CLASS__, 'dequeue_emoji_styles' ], 11 );
+		} else {
+			remove_action( 'wp_print_styles', 'print_emoji_styles' );
+		}
 
 		// The AMP version of the skip link is implemented by AMP_Accessibility_Sanitizer::add_skip_link().
 		remove_action( 'wp_footer', 'gutenberg_the_skip_link' );
@@ -907,6 +920,15 @@
 		);
 	}
 
+	/**
+	 * Dequeue emoji styles.
+	 *
+	 * @since 2.4.3
+	 */
+	public static function dequeue_emoji_styles() {
+		wp_dequeue_style( self::WP_EMOJI_STYLES );
+	}
+
 	/**
 	 * Filter resource hints to remove the emoji CDN (s.w.org).
 	 *
@@ -1225,12 +1247,21 @@
 		} else {
 			$header_callback = '_admin_bar_bump_cb';
 		}
-		remove_action( 'wp_head', $header_callback );
+
+		// @see <https://core.trac.wordpress.org/ticket/58775>.
+		if ( ! function_exists( 'wp_enqueue_admin_bar_header_styles' ) ) {
+			remove_action( 'wp_head', $header_callback );
+		}
+
 		if ( '__return_false' !== $header_callback ) {
-			ob_start();
-			$header_callback();
-			$style = ob_get_clean();
-			$data  = trim( preg_replace( '#<style[^>]*>(.*)</style>#is', '$1', $style ) ); // See wp_add_inline_style().
+			if ( ! function_exists( 'wp_enqueue_admin_bar_header_styles' ) ) {
+				ob_start();
+				$header_callback();
+				$style = ob_get_clean();
+				$data  = trim( preg_replace( '#<style[^>]*>(.*)</style>#is', '$1', $style ) ); // See wp_add_inline_style().
+			} else {
+				$data = '';
+			}
 
 			// Override AMP's position:relative on the body for the sake of the AMP viewer, which is not relevant an an Admin Bar context.
 			if ( amp_is_dev_mode() ) {

src/Support/SupportData.php

@@ -251,7 +251,10 @@
 			$loopback_status = ( ! empty( $loopback_status->status ) ) ? $loopback_status->status : '';
 		}
 
-		if ( function_exists( 'wp_is_https_supported' ) ) {
+		if ( function_exists( 'wp_get_https_detection_errors' ) ) {
+			$https_errors = wp_get_https_detection_errors();
+			$is_ssl       = empty( $https_errors );
+		} elseif ( function_exists( 'wp_is_https_supported' ) ) {
 			$is_ssl = wp_is_https_supported();
 		} else {
 			$is_ssl = is_ssl();

tests/php/src/DevTools/LikelyCulpritDetectorTest.php

@@ -128,13 +128,13 @@ public function single_step_trace_data() {
 			],
 
 			'parent theme' => [
-				[ get_template_directory() . '/functions.php' ],
+				[ '{{ get_template_directory }}/functions.php' ],
 				'theme',
 				'default',
 			],
 
 			'child theme'  => [
-				[ get_stylesheet_directory() . '/functions.php' ],
+				[ '{{ get_stylesheet_directory }}/functions.php' ],
 				'theme',
 				'default',
 			],
@@ -167,8 +167,8 @@ public function multi_step_trace_data() {
 						__FILE__, // AMP plugin is skipped.
 						WP_PLUGIN_DIR . '/bad-plugin/bad-plugin.php', // <== Likely culprit.
 						WP_CONTENT_DIR . '/mu-plugins/bad-mu-plugin.php',
-						get_template_directory() . '/functions.php',
-						get_stylesheet_directory() . '/functions.php',
+						'{{ get_template_directory }}/functions.php',
+						'{{ get_stylesheet_directory }}/functions.php',
 					],
 					'plugin',
 					'bad-plugin',
@@ -180,8 +180,8 @@ public function multi_step_trace_data() {
 						__FILE__, // AMP plugin is skipped.
 						WP_CONTENT_DIR . '/mu-plugins/bad-mu-plugin.php', // <== Likely culprit.
 						WP_PLUGIN_DIR . '/bad-plugin/bad-plugin.php',
-						get_template_directory() . '/functions.php',
-						get_stylesheet_directory() . '/functions.php',
+						'{{ get_template_directory }}/functions.php',
+						'{{ get_stylesheet_directory }}/functions.php',
 					],
 					'mu-plugin',
 					'bad-mu-plugin.php',
@@ -191,10 +191,10 @@ public function multi_step_trace_data() {
 					[
 						ABSPATH . '/wp-includes/some-file.php', // Core is skipped.
 						__FILE__, // AMP plugin is skipped.
-						get_template_directory() . '/functions.php', // <== Likely culprit.
+						'{{ get_template_directory }}/functions.php', // <== Likely culprit.
 						WP_PLUGIN_DIR . '/bad-plugin/bad-plugin.php',
 						WP_CONTENT_DIR . '/mu-plugins/bad-mu-plugin.php',
-						get_stylesheet_directory() . '/functions.php',
+						'{{ get_stylesheet_directory }}/functions.php',
 					],
 					'theme',
 					'default',
@@ -204,10 +204,10 @@ public function multi_step_trace_data() {
 					[
 						ABSPATH . '/wp-includes/some-file.php', // Core is skipped.
 						__FILE__, // AMP plugin is skipped.
-						get_stylesheet_directory() . '/functions.php', // <== Likely culprit.
+						'{{ get_stylesheet_directory }}/functions.php', // <== Likely culprit.
 						WP_PLUGIN_DIR . '/bad-plugin/bad-plugin.php',
 						WP_CONTENT_DIR . '/mu-plugins/bad-mu-plugin.php',
-						get_template_directory() . '/functions.php',
+						'{{ get_template_directory }}/functions.php',
 					],
 					'theme',
 					'default',
@@ -275,7 +275,12 @@ public function test_analyze_trace( $file_stack, $expected_type, $expected_name
 	private function get_trace_from_file_stack( $file_stack ) {
 		return array_map(
 			static function ( $file ) {
-				return [ 'file' => $file ];
+				$file = str_replace( '{{ get_template_directory }}', get_template_directory(), $file );
+				$file = str_replace( '{{ get_stylesheet_directory }}', get_stylesheet_directory(), $file );
+
+				return [
+					'file' => $file,
+				];
 			},
 			$file_stack
 		);