Fix a bug where adding a new LinkedCommand required a restart

LicenseHeaderManager.Test/LicenseHeaderManager.Test.csproj

@@ -94,6 +94,7 @@
     <Compile Include="CommentParserTests.cs" />
     <Compile Include="LinkedFileFilterTest.cs" />
     <Compile Include="LinkedFileHandlerTest.cs" />
+    <Compile Include="OptionsPageTest.cs" />
     <Compile Include="ProjectItemInspectionTest.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
   </ItemGroup>

LicenseHeaderManager.Test/OptionsPageTest.cs

@@ -0,0 +1,36 @@
+using System;
+using LicenseHeaderManager.Options;
+using NUnit.Framework;
+
+namespace LicenseHeaderManager.Test
+{
+  [TestFixture]
+  public class OptionsPageTest
+  {
+    [Test]
+    public void TestLinkedCommandsChangedCalledWhenNewLinkedCommandSavedWithExistingLinkedCommands ()
+    {
+      var optionsPage = new OptionsPage();
+      var wasCalled = false;
+      optionsPage.LinkedCommandsChanged += (sender, args) => wasCalled = true;
+      const string emptySerializedLinkedCommands = "1*System.String*<LinkedCommands/>";
+      optionsPage.LinkedCommandsSerialized = emptySerializedLinkedCommands;
+
+      optionsPage.LinkedCommands.Add (new LinkedCommand());
+
+      Assert.That (wasCalled, Is.True);
+    }
+
+    [Test]
+    public void TestLinkedCommandsChangedCalledWhenNewLinkedCommandSavedWithDefaultLinkedCommands ()
+    {
+      var optionsPage = new OptionsPage();
+      var wasCalled = false;
+      optionsPage.LinkedCommandsChanged += (sender, args) => wasCalled = true;
+
+      optionsPage.LinkedCommands.Add (new LinkedCommand());
+
+      Assert.That (wasCalled, Is.True);
+    }
+  }
+}

LicenseHeaderManager/Options/OptionsPage.cs

@@ -83,8 +83,8 @@ public ObservableCollection<LinkedCommand> LinkedCommands
     // ReSharper disable once UnusedMember.Global
     public string LinkedCommandsSerialized
     {
-      get { return _linkedCommandConverter.ToXml (_linkedCommands); }
-      set { _linkedCommands = new ObservableCollection<LinkedCommand> (_linkedCommandConverter.FromXml (value)); }
+      get { return _linkedCommandConverter.ToXml (LinkedCommands); }
+      set { LinkedCommands = new ObservableCollection<LinkedCommand> (_linkedCommandConverter.FromXml (value)); }
     }
 
     private void OnLinkedCommandsChanged (object sender, NotifyCollectionChangedEventArgs e)