Merge branch 'ignore-unknown-property'

src/ScmBackup.Tests.Integration/ConfigReaderTests.cs

@@ -51,5 +51,14 @@ public void ThrowsExceptionWhenConfigFileIsNotVaild()
 
             Assert.ThrowsAny<YamlException>(() => sut.ReadConfig());
         }
+
+        [Fact]
+        public void IgnoresUnknownProperties_Issue76()
+        {
+            var sut = new ConfigReader();
+            sut.ConfigFileName = "settings-unknown-property.yml";
+
+            var config = sut.ReadConfig(); // should just ignore the unknown property and NOT throw an exception
+        }
     }
 }

src/ScmBackup.Tests.Integration/ScmBackup.Tests.Integration.csproj

@@ -29,6 +29,9 @@
     <None Update="Scm\FakeCommandLineScmTools\FakeCommandLineScm-Command-Windows.bat">
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
+    <None Update="settings-unknown-property.yml">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
     <None Update="testsettings.yml">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>

src/ScmBackup.Tests.Integration/settings-unknown-property.yml

@@ -0,0 +1,5 @@
+localFolder: 'c:\scm-backup'
+
+# this property doesn't exist in the config class:
+unknownProperty:
+    foo: bar

src/ScmBackup/Configuration/ConfigReader.cs

@@ -25,7 +25,10 @@ public Config ReadConfig()
                 this.config = new Config();
 
                 var input = File.ReadAllText(this.ConfigFileName);
-                var deserializer = new DeserializerBuilder().WithNamingConvention(CamelCaseNamingConvention.Instance).Build();
+                var deserializer = new DeserializerBuilder()
+                                    .WithNamingConvention(CamelCaseNamingConvention.Instance)
+                                    .IgnoreUnmatchedProperties()
+                                    .Build();
                 this.config = deserializer.Deserialize<Config>(input);
             }