Merge pull request #286 from newfold-labs/FIX/PRESS7-515-remove-action-link

removed missing object cache url

Author: arunshenoy99

includes/HealthChecks/PersistentObjectCacheHealthCheck.php

@@ -2,7 +2,6 @@
 
 namespace NewfoldLabs\WP\Module\Performance\HealthChecks;
 
-use function NewfoldLabs\WP\Module\LinkTracker\Functions\build_link as buildLink;
 /**
  * Health check for persistent object cache.
  */
@@ -16,12 +15,6 @@ public function __construct() {
 		$this->passing_text = esc_html__( 'Object caching is enabled', 'wp-module-performance' );
 		$this->failing_text = esc_html__( 'Object caching is disabled', 'wp-module-performance' );
 		$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' ),
-			esc_html__( 'Learn more about object caching', 'wp-module-performance' ),
-			__( 'opens in a new tab', 'wp-module-performance' )
-		);
 	}
 
 	/**

tests/phpunit/includes/HealthChecksTest.php

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