Add test assertions for register_hooks.

google · Jun 14, 2024 · bc6f924 · bc6f924
1 parent 8e5416b
commit bc6f924
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/...ludes/Core/Conversion_Tracking/Conversion_Event_Providers/FakeConversionEventProvider.php b/...ludes/Core/Conversion_Tracking/Conversion_Event_Providers/FakeConversionEventProvider.php
@@ -50,4 +50,14 @@ public function register_script() {
 		return $script_asset;
 	}
 
+	/**
+	 * Registers any actions/hooks for this provider.
+	 *
+	 * @since 1.129.0
+	 */
+	public function register_hooks() {
+		// Register a fake action.
+		add_action( 'fake_provider_action', '__return_true' );
+	}
+
 }
diff --git a/tests/phpunit/integration/Core/Conversion_Tracking/Conversion_TrackingTest.php b/tests/phpunit/integration/Core/Conversion_Tracking/Conversion_TrackingTest.php
@@ -82,13 +82,17 @@ public function test_register__not_enqueued_when_tracking_disabled() {
 	 * @dataProvider data_modules
 	 */
 	public function test_register__enqueued_when_snippet_inserted( $module_slug ) {
+		$this->assertFalse( has_action( 'fake_provider_action' ) );
+
 		$this->conversion_tracking->register();
 
 		do_action( "googlesitekit_{$module_slug}_init_tag" );
 		do_action( 'wp_enqueue_scripts' );
 
 		$this->assertTrue( wp_script_is( 'gsk-cep-' . FakeConversionEventProvider_Active::CONVERSION_EVENT_PROVIDER_SLUG ) );
 		$this->assertFalse( wp_script_is( 'gsk-cep-' . FakeConversionEventProvider::CONVERSION_EVENT_PROVIDER_SLUG ) );
+
+		$this->assertTrue( has_action( 'fake_provider_action' ) );
 	}
 
 	public function data_modules() {