Fix test

Author: Hosch250

RetailCoder.VBE/Settings/ExperimentalFeatures.cs

@@ -35,5 +35,20 @@ public override string ToString()
         {
             return Key;
         }
+
+        public override bool Equals(object obj)
+        {
+            return obj is ExperimentalFeatures value &&
+                   value.IsEnabled == IsEnabled &&
+                   value.Key == Key;
+        }
+
+        public override int GetHashCode()
+        {
+            unchecked
+            {
+                return (IsEnabled.GetHashCode() * 397) ^ (Key != null ? Key.GetHashCode() : 0);
+            }
+        }
     }
 }

RetailCoder.VBE/UI/Settings/GeneralSettingsViewModel.cs

@@ -159,20 +159,6 @@ public MinimumLogLevel SelectedLogLevel
             }
         }
 
-        private bool _sourceControlEnabled;
-        public bool SourceControlEnabled
-        {
-            get => _sourceControlEnabled;
-            set
-            {
-                if (_sourceControlEnabled != value)
-                {
-                    _sourceControlEnabled = value;
-                    OnPropertyChanged();
-                }
-            }
-        }
-
         public CommandBase ShowLogFolderCommand { get; }
 
         private void ShowLogFolder()

Retailcoder.VBE/UI/RubberduckUI.resx

@@ -2052,14 +2052,14 @@ Would you like to import them to Rubberduck?</value>
     <value>Method '{0}' is an implementation of event '{1}.{2}'.  Rename event '{2}' instead?</value>
     <comment>0: Selected target Identifier; 1: Event Parent; 2: Event name</comment>
   </data>
-  <data name="GeneralSettings_EnableExperimentalFeatures" xml:space="preserve">
-    <value>Enable experimental features. Requires a restart to take effect.</value>
+  <data name="GeneralSettings_EnableSourceControl" xml:space="preserve">
+    <value>Enable Source Control.</value>
   </data>
   <data name="GeneralSettings_ExperimentalFeaturesWarning" xml:space="preserve">
     <value>Only enable these if you know what you're doing. Enabling and/or using features from this section may result in things breaking unexpectedly and irrevocable data loss.</value>
   </data>
   <data name="GeneralSettings_ExperimentalFeatures" xml:space="preserve">
-    <value>Experimental Features:</value>
+    <value>Experimental Features (Requires a restart to take effect):</value>
   </data>
   <data name="ToolsMenu_ExportProject" xml:space="preserve">
     <value>Export Active Project...</value>

RubberduckTests/Settings/GeneralSettingsTests.cs

@@ -1,10 +1,12 @@
+using System.Collections.Generic;
 using System.Linq;
 using Microsoft.VisualStudio.TestTools.UnitTesting;
 using Rubberduck.Settings;
 using Rubberduck.UI.Settings;
 using GeneralSettings = Rubberduck.Settings.GeneralSettings;
 using Rubberduck.Common;
 using Moq;
+using Rubberduck.UI;
 
 namespace RubberduckTests.Settings
 {
@@ -23,6 +25,14 @@ private Configuration GetDefaultConfig()
                 Language = new DisplayLanguageSetting("en-US"),
                 IsAutoSaveEnabled = false,
                 AutoSavePeriod = 10,
+                EnableExperimentalFeatures = new List<ExperimentalFeatures>
+                {
+                    new ExperimentalFeatures
+                    {
+                        Key = nameof(RubberduckUI.GeneralSettings_EnableSourceControl),
+                        IsEnabled = true
+                    }
+                }
                 //Delimiter = '.'
             };
 
@@ -142,7 +152,7 @@ public void SourceControlEnabledIsSetInCtor()
             var defaultConfig = GetDefaultConfig();
             var viewModel = new GeneralSettingsViewModel(defaultConfig, GetOperatingSystemMock().Object);
 
-            Assert.AreEqual(defaultConfig.UserSettings.GeneralSettings.EnableExperimentalFeatures, viewModel.SourceControlEnabled);
+            Assert.IsTrue(defaultConfig.UserSettings.GeneralSettings.EnableExperimentalFeatures.SequenceEqual(viewModel.ExperimentalFeatures));
         }
 
         //[TestCategory("Settings")]