Updated plugin tests for activation to ensure config is only set when there is no existing config.

Author: colinmurphy

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

@@ -60,6 +60,17 @@ public function test_clone_method_throws_error() {
 	}
 
 	public function test_plugin_activate() {
+
+		// Delet configuration option
+		$configuration = ConfigurationHelper::get_instance();
+		$option_key = $configuration->get_option_key();
+		delete_option( $option_key );
+
+		// Verify that the configuration option has been deleted
+		$option_value = get_option( $option_key );
+		$this->assertEmpty( $option_value );
+
+
 		$plugin = Plugin::init();
 		$plugin::activate();
 
@@ -71,7 +82,6 @@ public function test_plugin_activate() {
 		$this->assertEquals( $table_exists, $table_name );
 
 		// Verify that the default configuration has been set
-		$configuration = ConfigurationHelper::get_instance();
 		$option_value = $configuration->get_option_value( WPGRAPHQL_LOGGING_SETTINGS_KEY );
 		$this->assertNotEmpty( $option_value );
 		$default_configuration = [
@@ -99,6 +109,26 @@ public function test_plugin_activate() {
 		$this->assertEquals( $option_value, $default_configuration );
 	}
 
+	public function test_plugin_activate_when_configuration_already_exists() {
+		$configuration = ConfigurationHelper::get_instance();
+		$option_key = $configuration->get_option_key();
+		$default_configuration = [
+			BasicConfigurationTab::get_name() => [
+				BasicConfigurationTab::ENABLED => true,
+			],
+		];
+		update_option( $option_key, $default_configuration );
+
+
+		$plugin = Plugin::init();
+		$plugin::activate();
+
+		// Verify that the default configuration has not been set
+		$configuration = ConfigurationHelper::get_instance();
+		$option_value = $configuration->get_option_value( WPGRAPHQL_LOGGING_SETTINGS_KEY );
+		$this->assertEquals( $option_value, $default_configuration );
+	}
+
 	public function test_wakeup_method_throws_error() {
 		$reflection = new ReflectionClass( Plugin::class );
 		$plugin     = $reflection->newInstanceWithoutConstructor();