Fixed namespaces for test classes. Added missing tests for the database entity.

colinmurphy · colinmurphy · commit 716d53493de9 · 2025-09-26T19:41:11.000+01:00
diff --git a/plugins/wpgraphql-logging/src/Logger/Database/LogsRepository.php b/plugins/wpgraphql-logging/src/Logger/Database/LogsRepository.php
@@ -22,7 +22,6 @@ class LogsRepository {
 	 * @return array<\WPGraphQL\Logging\Logger\Database\DatabaseEntity>
 	 */
 	public function get_logs(array $args = []): array {
-		global $wpdb;
 		$defaults = [
 			'number'  => 100,
 			'offset'  => 0,
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/ConfigurationHelperTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/ConfigurationHelperTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Tests\WPUnit\Admin\Settings;
+namespace WPGraphQL\Logging\Tests\Admin\Settings;
 
 use WPGraphQL\Logging\Admin\Settings\ConfigurationHelper;
 use Codeception\TestCase\WPTestCase;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/Field/CheckboxFieldTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/Field/CheckboxFieldTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace WPGraphQL\Logging\wpunit\Admin\Settings\Fields\Field;
+namespace WPGraphQL\Logging\Tests\Admin\Settings\Fields\Field;
 
 use WPGraphQL\Logging\Admin\Settings\Fields\Field\CheckboxField;
 use WPGraphQL\Logging\Admin\Settings\Fields\SettingsFieldInterface;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/Field/SelectFieldTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/Field/SelectFieldTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace WPGraphQL\Logging\wpunit\Admin\Settings\Fields\Field;
+namespace WPGraphQL\Logging\Tests\Admin\Settings\Fields\Field;
 
 use WPGraphQL\Logging\Admin\Settings\Fields\Field\SelectField;
 use WPGraphQL\Logging\Admin\Settings\Fields\SettingsFieldInterface;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/Field/TextInputFieldTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/Field/TextInputFieldTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace WPGraphQL\Logging\wpunit\Admin\Settings\Fields\Field;
+namespace WPGraphQL\Logging\Tests\Admin\Settings\Fields;
 
 use WPGraphQL\Logging\Admin\Settings\Fields\Field\TextInputField;
 use WPGraphQL\Logging\Admin\Settings\Fields\SettingsFieldInterface;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/Field/TextIntegerFieldTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/Field/TextIntegerFieldTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace WPGraphQL\Logging\wpunit\Admin\Settings\Fields\Field;
+namespace WPGraphQL\Logging\Tests\Admin\Settings\Fields;
 
 use WPGraphQL\Logging\Admin\Settings\Fields\Field\TextIntegerField;
 use WPGraphQL\Logging\Admin\Settings\Fields\SettingsFieldInterface;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/SettingsFieldCollectionTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Fields/SettingsFieldCollectionTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Tests\WPUnit\Admin\Settings\Fields;
+namespace WPGraphQL\Logging\Tests\Admin\Settings\Fields;
 
 use WPGraphQL\Logging\Admin\Settings\Fields\SettingsFieldCollection;
 use WPGraphQL\Logging\Admin\Settings\Fields\SettingsFieldInterface;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Menu/MenuPageTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/Menu/MenuPageTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Tests\WPUnit\Admin\Settings\Menu;
+namespace WPGraphQL\Logging\Tests\Admin\Settings\Menu;
 
 use WPGraphQL\Logging\Admin\SettingsPage;
 use WPGraphQL\Logging\Admin\Settings\Menu\MenuPage;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/SettingsFormManagerTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/Settings/SettingsFormManagerTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace WPGraphQL\Logging\Tests\WPUnit\Admin\Settings;
+namespace WPGraphQL\Logging\Tests\Admin\Settings;
 
 use Codeception\TestCase\WPTestCase;
 use WPGraphQL\Logging\Admin\Settings\ConfigurationHelper;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/SettingsPageTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/SettingsPageTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace WPGraphQL\Logging\wpunit\Admin;
+namespace WPGraphQL\Logging\Tests\Admin;
 
 use lucatume\WPBrowser\TestCase\WPTestCase;
 use WPGraphQL\Logging\Admin\SettingsPage;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/View/Download/DownloadLogServiceTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/View/Download/DownloadLogServiceTest.php
@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace WPGraphQL\Logging\wpunit\Admin\View\Download;
+namespace WPGraphQL\Logging\Tests\Admin\View\Download;
 
 use lucatume\WPBrowser\TestCase\WPTestCase;
 use WPGraphQL\Logging\Admin\View\Download\DownloadLogService;
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/View/ViewLogsPageTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/View/ViewLogsPageTest.php
@@ -2,14 +2,20 @@
 
 declare(strict_types=1);
 
-namespace Tests\WPUnit\Admin\View;
+namespace WPGraphQL\Logging\Tests\Admin\View;
 
 
 use WPGraphQL\Logging\Admin\ViewLogsPage;
 use WPGraphQL\Logging\Logger\Database\LogsRepository;
 use Codeception\TestCase\WPTestCase;
 use Brain\Monkey;
 
+/**
+ * Test for the ViewLogsPage
+ *
+ * @package WPGraphQL\Logging
+ * @since 0.0.1
+ */
 class ViewLogsPageTest extends WPTestCase {
 
 
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Core/ActivationTest.php b/plugins/wpgraphql-logging/tests/wpunit/Core/ActivationTest.php
@@ -8,8 +8,12 @@
 use lucatume\WPBrowser\TestCase\WPTestCase;
 use WPGraphQL\Logging\Logger\Database\DatabaseEntity;
 
+
 /**
- * Test class for the activation callback.
+ * Test for the activation callback
+ *
+ * @package WPGraphQL\Logging
+ * @since 0.0.1
  */
 class ActivationTest extends WPTestCase {
 
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Core/AutoloaderTest.php b/plugins/wpgraphql-logging/tests/wpunit/Core/AutoloaderTest.php
@@ -8,8 +8,12 @@
 use lucatume\WPBrowser\TestCase\WPTestCase;
 use ReflectionClass;
 
+
 /**
- * Test class for the Autoloader.
+ * Test for the Autoloader
+ *
+ * @package WPGraphQL\Logging
+ * @since 0.0.1
  */
 class Autoloader_Test extends WPTestCase {
 
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Core/DeactivationTest.php b/plugins/wpgraphql-logging/tests/wpunit/Core/DeactivationTest.php
@@ -7,9 +7,12 @@
 
 use lucatume\WPBrowser\TestCase\WPTestCase;
 
+
 /**
- * Test class for the de-activation callback.
+ * Test for the deactivation callback
  *
+ * @package WPGraphQL\Logging
+ * @since 0.0.1
  */
 class DeactivationTest extends WPTestCase {
 	protected function setUp(): void {
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Core/PluginTest.php b/plugins/wpgraphql-logging/tests/wpunit/Core/PluginTest.php
@@ -9,11 +9,12 @@
 use ReflectionClass;
 use WPGraphQL\Logging\Logger\Database\DatabaseEntity;
 
+
 /**
- * Class PluginTest
- *
- * Tests for the Plugin class
+ * Test for the Plugin
  *
+ * @package WPGraphQL\Logging
+ * @since 0.0.1
  */
 class PluginTest extends WPTestCase {
 
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Events/EventManagerTest.php b/plugins/wpgraphql-logging/tests/wpunit/Events/EventManagerTest.php
@@ -9,10 +9,12 @@
 use WPGraphQL\Logging\Events\EventManager;
 use WPGraphQL\Logging\Events\Events;
 
+
 /**
- * Class EventManagerTest
+ * Test for the EventManager
  *
- * Tests for the EventManager class.
+ * @package WPGraphQL\Logging
+ * @since 0.0.1
  */
 class EventManagerTest extends WPTestCase {
 
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Logger/Database/DatabaseEntityTest.php b/plugins/wpgraphql-logging/tests/wpunit/Logger/Database/DatabaseEntityTest.php
@@ -2,17 +2,16 @@
 
 declare(strict_types=1);
 
-namespace WPGraphQL\Logging\Tests\Database;
+namespace WPGraphQL\Logging\Tests\Logger\Database;
 
 use lucatume\WPBrowser\TestCase\WPTestCase;
 use DateTimeImmutable;
 use ReflectionClass;
 use WPGraphQL\Logging\Logger\Database\DatabaseEntity;
+use Mockery;
 
 /**
- * Class DatabaseEntityTest
- *
- * Tests for the DatabaseEntity class.
+ * Test for the DatabaseEntity
  *
  * @package WPGraphQL\Logging
  * @since 0.0.1
@@ -178,4 +177,123 @@ public function test_save_returns_zero_on_database_failure(): void
         // 6. Assert that the save method returned 0, indicating failure.
         $this->assertSame(0, $result_id, 'The save() method should return 0 when the database insert fails.');
     }
+
+	public function test_get_query() : void
+	{
+		$mockEntity = Mockery::mock(DatabaseEntity::class)->makePartial();
+		$mockEntity->shouldReceive('get_context')->andReturn([
+			'query' => 'query GetAllPosts { posts { nodes { title content } } }'
+		]);
+
+		$this->assertEquals(
+			'query GetAllPosts { posts { nodes { title content } } }',
+			$mockEntity->get_query()
+		);
+	}
+
+	public function test_get_query_in_request() : void
+	{
+		$mockEntity = Mockery::mock(DatabaseEntity::class)->makePartial();
+		$mockEntity->shouldReceive('get_context')->andReturn([
+			'request' => [
+				'params' => [
+					'query' => 'query GetAllPosts { posts { nodes { title content } } }'
+				]
+			]
+		]);
+
+		$this->assertEquals(
+			'query GetAllPosts { posts { nodes { title content } } }',
+			$mockEntity->get_query()
+		);
+	}
+
+	public function test_get_invalid_query() : void
+	{
+		$mockEntity = Mockery::mock(DatabaseEntity::class)->makePartial();
+		$mockEntity->shouldReceive('get_context')->andReturn([
+			'request' => 'query GetAllPosts { posts { nodes { title content } } }'
+		]);
+
+		$this->assertNull(
+			$mockEntity->get_query()
+		);
+
+		$mockEntity = Mockery::mock(DatabaseEntity::class)->makePartial();
+		$mockEntity->shouldReceive('get_context')->andReturn([
+			'request' => [
+				'query GetAllPosts { posts { nodes { title content } } }'
+			]
+		]);
+
+		$this->assertNull(
+			$mockEntity->get_query()
+		);
+
+		$mockEntity = Mockery::mock(DatabaseEntity::class)->makePartial();
+		$mockEntity->shouldReceive('get_context')->andReturn([]);
+
+		$this->assertNull(
+			$mockEntity->get_query()
+		);
+
+
+		$mockEntity = Mockery::mock(DatabaseEntity::class)->makePartial();
+		$mockEntity->shouldReceive('get_context')->andReturn([
+			'request' => [
+				'params' => [
+					'invalid_key' => 'query GetAllPosts { posts { nodes { title content } } }'
+				]
+			]
+		]);
+
+		$this->assertNull(
+			$mockEntity->get_query()
+		);
+	}
+
+	public function test_find_logs() : void {
+
+		$log_data = [
+			'channel'    => 'wpgraphql_logging',
+			'level'      => 200,
+			'level_name' => 'INFO',
+			'message'    => 'WPGraphQL Outgoing Response',
+			'context'    => [
+				'site_url'      => 'http://test.local',
+				'wp_version'    => '6.8.2',
+				'wp_debug_mode' => true,
+				'plugin_version'=> '0.0.1'
+			],
+			'extra'      => [
+				'ip' => '127.0.0.1',
+				'url' => '/index.php?graphql',
+				'server' => 'test.local',
+				'referrer' => 'http://test.local/wp-admin/admin.php?page=graphiql-ide',
+				'process_id' => 5819,
+				'http_method' => 'POST',
+				'memory_usage' => '14 MB',
+				'wpgraphql_query' => 'query GetPost($uri: ID!) { post(id: $uri, idType: URI) { title content } }',
+				'memory_peak_usage' => '14 MB',
+				'wpgraphql_variables' => [
+					'uri' => 'hello-world'
+				],
+				'wpgraphql_operation_name' => 'GetPost'
+			]
+		];
+
+        // Create and save the entity
+        $entity = DatabaseEntity::create(...array_values($log_data));
+        $insert_id = $entity->save();
+
+		$where_clauses = [
+			'level' => 200,
+			'id' => $insert_id
+		];
+		$logs = DatabaseEntity::find_logs(10, 0, $where_clauses, 'id', 'DESC');
+		$this->assertIsArray($logs);
+		$this->assertCount(1, $logs);
+		$this->assertInstanceOf(DatabaseEntity::class, $logs[0]);
+
+	}
 }
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Logger/LoggerServiceTest.php b/plugins/wpgraphql-logging/tests/wpunit/Logger/LoggerServiceTest.php
@@ -12,10 +12,12 @@
 use WPGraphQL\Logging\Logger\Database\DatabaseEntity;
 use Monolog\LogRecord;
 
+
 /**
- * Class LoggerServiceTest
+ * Test for the LoggerService
  *
- * Tests for the LoggerService class.
+ * @package WPGraphQL\Logging
+ * @since 0.0.1
  */
 class LoggerServiceTest extends WPTestCase
 {
