Added missing tests to the plugin manager.

Author: colinmurphy

plugins/wpgraphql-logging/src/Plugin.php

@@ -47,7 +47,7 @@ public static function init(): self {
 		 *
 		 * @param \WPGraphQL\Logging\Plugin $instance the instance of the plugin class.
 		 */
-		do_action( 'wpgraphql_logging_init', self::$instance );
+		do_action( 'wpgraphql_logging_plugin_init', self::$instance );
 
 		return self::$instance;
 	}
@@ -56,13 +56,13 @@ public static function init(): self {
 	 * Initialize the plugin admin, frontend & api functionality.
 	 */
 	public function setup(): void {
-		// Initialize configuration caching hooks.
 		ConfigurationHelper::init_cache_hooks();
-
 		SettingsPage::init();
 		ViewLogsPage::init();
 		QueryEventLifecycle::init();
 		DataDeletionScheduler::init();
+
+		do_action( 'wpgraphql_logging_plugin_setup', self::$instance );
 	}
 
 	/**

plugins/wpgraphql-logging/tests/wpunit/Core/PluginTest.php

@@ -8,6 +8,7 @@
 use lucatume\WPBrowser\TestCase\WPTestCase;
 use ReflectionClass;
 use WPGraphQL\Logging\Logger\Database\DatabaseEntity;
+use WPGraphQL\Logging\Events\EventManager;
 
 
 /**
@@ -54,4 +55,96 @@ public function test_clone_method_throws_error() {
 		// Verify the clone exists to ensure the operation completed
 		$this->assertInstanceOf( Plugin::class, $clone );
 	}
+
+	public function test_wakeup_method_throws_error() {
+		$reflection = new ReflectionClass( Plugin::class );
+		$plugin     = $reflection->newInstanceWithoutConstructor();
+
+		$this->setExpectedIncorrectUsage( 'WPGraphQL\Logging\Plugin::__wakeup' );
+		$plugin->__wakeup();
+
+		$this->assertInstanceOf( Plugin::class, $plugin );
+	}
+
+	public function test_clone_is_forbidden_and_triggers_doing_it_wrong() {
+		$reflection = new ReflectionClass( Plugin::class );
+		$plugin     = $reflection->newInstanceWithoutConstructor();
+
+		$this->setExpectedIncorrectUsage( 'WPGraphQL\Logging\Plugin::__clone' );
+		$cloned = null;
+		try {
+			$cloned = clone $plugin;
+		} catch ( \Exception $e ) {
+			// Ignore, as __clone should not throw, just trigger doing_it_wrong.
+		}
+		$this->assertInstanceOf( Plugin::class, $plugin );
+	}
+
+	public function test_wakeup_is_forbidden_and_triggers_doing_it_wrong() {
+		$reflection = new ReflectionClass( Plugin::class );
+		$plugin     = $reflection->newInstanceWithoutConstructor();
+
+		$this->setExpectedIncorrectUsage( 'WPGraphQL\Logging\Plugin::__wakeup' );
+		$plugin->__wakeup();
+
+		$this->assertInstanceOf( Plugin::class, $plugin );
+	}
+
+
+	public function test_can_subscribe_and_emit_custom_event() {
+		$event_name = 'custom_event_test_' . uniqid();
+		$received_payload = null;
+
+		// Subscribe to the custom event.
+		Plugin::on( $event_name, function( $payload ) use ( &$received_payload ) {
+			$received_payload = $payload;
+		} );
+
+		$payload = [ 'foo' => 'bar', 'baz' => 123 ];
+
+		// Emit the event.
+		Plugin::emit( $event_name, $payload );
+
+		// The listener should have received the payload.
+		$this->assertSame( $payload, $received_payload );
+	}
+
+	public function test_transform_event_payload() {
+		$event_name = 'transform_event_test_' . uniqid();
+		$received_payload = null;
+
+		// Register an event listener.
+		Plugin::on( $event_name, function( $payload ) use ( &$received_payload ) {
+			$received_payload = $payload;
+		} );
+
+		// Subscribe to transform the payload.
+		Plugin::transform( $event_name, function( $payload ) {
+			$payload['context']['error'] = true;
+			return $payload;
+		}, 5 );
+
+
+		// Simulate emitting the event with initial payload.
+		$level = 200;
+		$context = [
+			'query'          => 'query { test }',
+			'variables'      => [ 'var1' => 'value' ],
+			'operation_name' => 'TestOperation',
+		];
+		$payload = EventManager::transform( $event_name, [
+			'context' => $context,
+			'level'   => $level,
+		] );
+
+		// Publish the event.
+		Plugin::emit( $event_name, $payload );
+
+
+		// Check the listener received the transformed payload.
+		$this->assertSame( $received_payload,
+			array_merge( $payload, [ 'context' => array_merge( $context, [ 'error' => true ] ) ] ),
+			'The listener should receive the transformed payload'
+		);
+	}
 }