Merge pull request #1 from PMudra/master

Update to remote

Author: Roel84

ApiCheck.Test/Comparer/ParameterComparerTest.cs

@@ -30,6 +30,16 @@ public void When_default_flag_changed_should_report()
       sut.Verify(result => result.AddChangedProperty("Default Value", "1", "", Severity.Error), Times.Once);
     }
 
+    [Test]
+    public void When_default_flag_null_changed_should_report()
+    {
+      Assembly assembly1 = ApiBuilder.CreateApi().Class().Method().DefaultParameter(typeof(object), null).Build().Build().Build();
+      Assembly assembly2 = ApiBuilder.CreateApi().Class().Method().Parameter(typeof(object)).Build().Build().Build();
+      Mock<IComparerResult> sut = new Builder(assembly1, assembly2).ComparerResultMock;
+
+      sut.Verify(result => result.AddChangedProperty("Default Value", "null", "", Severity.Error), Times.Once);
+    }
+
     [Test]
     public void When_name_changed_should_report()
     {

ApiCheck/Comparer/ParameterComparer.cs

@@ -25,7 +25,7 @@ private void CompareDefaultValue()
     {
       if (!Equals(ReferenceType.RawDefaultValue, NewType.RawDefaultValue))
       {
-        ComparerResult.AddChangedProperty("Default Value", ReferenceType.RawDefaultValue.ToString(), NewType.RawDefaultValue.ToString(), Severity.Error);
+        ComparerResult.AddChangedProperty("Default Value", (ReferenceType.RawDefaultValue ?? "null").ToString(), (NewType.RawDefaultValue ?? "null").ToString(), Severity.Error);
       }
     }