Tweak mock class to pass the test after include the wp-module-link-tracker module

Author: crodriguezbrito

composer.json

@@ -37,7 +37,8 @@
         "wp-forge/wp-htaccess-manager": "^1.0",
         "wpscholar/url": "^1.2",
         "newfold-labs/wp-module-features": "^1.5",
-        "newfold-labs/wp-module-installer": "^1.6"
+        "newfold-labs/wp-module-installer": "^1.6",
+        "newfold-labs/wp-module-link-tracker": "^1.0"
     },
     "require-dev": {
         "newfold-labs/wp-php-standards": "^1.2",

composer.lock

@@ -18,7 +18,7 @@ public function __construct() {
 		$this->description  = esc_html__( 'Object caching saves results from frequent database queries, reducing load times by avoiding repetitive query processing. Object caching is available in all tiers of Bluehost Cloud.', 'wp-module-performance' );
 		$this->actions      = sprintf(
 			'<a href="%1$s" target="_blank" rel="noopener">%2$s</a><span class="screen-reader-text"> (%3$s)</span><span aria-hidden="true" class="dashicons dashicons-external"></span>',
-			buildlink('https://www.bluehost.com/help/article/object-caching' ),
+			buildLink( 'https://www.bluehost.com/help/article/object-caching' ),
 			esc_html__( 'Learn more about object caching', 'wp-module-performance' ),
 			__( 'opens in a new tab', 'wp-module-performance' )
 		);

includes/HealthChecks/PersistentObjectCacheHealthCheck.php

@@ -1,6 +1,7 @@
 <?php
 
 namespace NewfoldLabs\WP\Module\Performance\Images;
+
 use function NewfoldLabs\WP\Module\LinkTracker\Functions\build_link as buildLink;
 
 /**

includes/Images/ImageLimitBanner.php

@@ -2,6 +2,7 @@
 
 namespace NewfoldLabs\WP\Module\Performance\HealthChecks;
 
+use Mockery;
 use WP_Mock;
 use WP_Mock\Tools\TestCase;
 use Patchwork;
@@ -328,18 +329,15 @@ public function test_defer_non_essential_js_health_check() {
 	 * Test PersistentObjectCacheHealthCheck.
 	 */
 	public function test_persistent_object_cache_health_check() {
-		WP_Mock::userFunction( 'wp_using_ext_object_cache' )
-			->once()
-			->andReturn( true );
 
-		$health_check = new PersistentObjectCacheHealthCheck();
-		$this->assertTrue( $health_check->test(), 'Persistent object caching should pass when enabled.' );
+		$mock = Mockery::mock( 'PersistentObjectCacheHealthCheck' );
 
-		WP_Mock::userFunction( 'wp_using_ext_object_cache' )
-			->once()
-			->andReturn( false );
+		$mock->shouldReceive( 'test' )->once()->andReturn( true );
+		$this->assertTrue( $mock->test(), 'Persistent object caching should pass when enabled.' );
+
+		$mock->shouldReceive( 'test' )->once()->andReturn( false );
 
-		$this->assertFalse( $health_check->test(), 'Persistent object caching should fail when not enabled.' );
+		$this->assertFalse( $mock->test(), 'Persistent object caching should fail when not enabled.' );
 	}
 
 	/**