Merge branch 'release/0.4.2'

Author: pascalberger

nuspec/nuget/Cake.Issues.Reporting.Generic.nuspec

@@ -21,7 +21,7 @@ See the Project Site for an overview of the whole ecosystem of addins for workin
     <requireLicenseAcceptance>false</requireLicenseAcceptance>
     <copyright>Copyright © Pascal Berger</copyright>
     <tags>Cake Script Cake-Issues Cake-ReportFormat Issues Reporting Html Markdown</tags>
-    <releaseNotes>https://github.com/cake-contrib/Cake.Issues.Reporting.Generic/releases/tag/0.4.1</releaseNotes>
+    <releaseNotes>https://github.com/cake-contrib/Cake.Issues.Reporting.Generic/releases/tag/0.4.2</releaseNotes>
   </metadata>
   <files>
     <file src="Cake.Issues.Reporting.Generic.dll" target="lib\net46" />

src/Cake.Issues.Reporting.Generic.Tests/Cake.Issues.Reporting.Generic.Tests.csproj

@@ -85,6 +85,8 @@
   <ItemGroup>
     <Compile Include="ColumnSortOrderExtensionsTests.cs" />
     <Compile Include="DevExtremeThemeExtensionsTests.cs" />
+    <Compile Include="ExceptionAssertExtensions.cs" />
+    <Compile Include="HtmlDxDataGridColumnDescriptionTests.cs" />
     <Compile Include="IIssueExtensionTests.cs" />
     <Compile Include="UriExtensionsTests.cs" />
     <Compile Include="FileLinkSettingsTests.cs" />

src/Cake.Issues.Reporting.Generic.Tests/ExceptionAssertExtensions.cs

@@ -0,0 +1,23 @@
+namespace Cake.Issues.Reporting.Generic.Tests
+{
+    using System;
+    using Microsoft.CSharp.RuntimeBinder;
+    using Xunit;
+
+    /// <summary>
+    /// Extensions for asserting exceptions.
+    /// </summary>
+    internal static class ExceptionAssertExtensions
+    {
+        /// <summary>
+        /// Checks if an execption is of type <see cref="T:Microsoft.CSharp.RuntimeBinder.RuntimeBinderException" />.
+        /// </summary>
+        /// <param name="exception">Exception to check.</param>
+        /// <param name="message">Expected exception message.</param>
+        public static void IsRuntimeBinderException(this Exception exception, string message)
+        {
+            Assert.IsType<RuntimeBinderException>(exception);
+            Assert.Equal(message, exception.Message);
+        }
+    }
+}

src/Cake.Issues.Reporting.Generic.Tests/GenericIssueReportFixture.cs

@@ -1,9 +1,11 @@
 namespace Cake.Issues.Reporting.Generic.Tests
 {
+    using System;
     using System.Collections.Generic;
     using System.IO;
     using Cake.Testing;
     using Core.Diagnostics;
+    using Shouldly;
 
     internal class GenericIssueReportFixture
     {
@@ -62,5 +64,28 @@ public string CreateReport(IEnumerable<IIssue> issues)
                 }
             }
         }
+
+        public void TestReportCreation(Action<GenericIssueReportFormatSettings> settings)
+        {
+            // Given
+            settings(this.GenericIssueReportFormatSettings);
+
+            // When
+            var result =
+                this.CreateReport(
+                    new List<IIssue>
+                    {
+                            IssueBuilder
+                                .NewIssue("Message Foo", "ProviderType Foo", "ProviderName Foo")
+                                .InFile(@"src\Cake.Issues.Reporting.Generic.Tests\Foo.cs", 10)
+                                .OfRule("Rule Foo")
+                                .WithPriority(IssuePriority.Warning)
+                                .Create(),
+                    });
+
+            // Then
+            // Currently only checks if genertions failed or not without checking actual output.
+            result.ShouldNotBeNull();
+        }
     }
 }

src/Cake.Issues.Reporting.Generic.Tests/HtmlDxDataGridColumnDescriptionTests.cs

@@ -0,0 +1,112 @@
+namespace Cake.Issues.Reporting.Generic.Tests
+{
+    using System;
+    using Cake.Issues.Testing;
+    using Shouldly;
+    using Xunit;
+
+    public sealed class HtmlDxDataGridColumnDescriptionTests
+    {
+        public sealed class TheCtor
+        {
+            [Fact]
+            public void Should_Throw_If_Id_Is_Null()
+            {
+                // Given
+                string id = null;
+                object valueRetriever(IIssue issue)
+                {
+                    return true;
+                }
+
+                // When
+                var result = Record.Exception(() => new HtmlDxDataGridColumnDescription(id, valueRetriever));
+
+                // Then
+                result.IsArgumentNullException("id");
+            }
+
+            [Fact]
+            public void Should_Throw_If_Id_Is_Empty()
+            {
+                // Given
+                var id = string.Empty;
+                object valueRetriever(IIssue issue)
+                {
+                    return true;
+                }
+
+                // When
+                var result = Record.Exception(() => new HtmlDxDataGridColumnDescription(id, valueRetriever));
+
+                // Then
+                result.IsArgumentOutOfRangeException("id");
+            }
+
+            [Fact]
+            public void Should_Throw_If_Id_Is_Whitespace()
+            {
+                // Given
+                var id = " ";
+                object valueRetriever(IIssue issue)
+                {
+                    return true;
+                }
+
+                // When
+                var result = Record.Exception(() => new HtmlDxDataGridColumnDescription(id, valueRetriever));
+
+                // Then
+                result.IsArgumentOutOfRangeException("id");
+            }
+
+            [Fact]
+            public void Should_Throw_If_ValueRetriever_Is_Null()
+            {
+                // Given
+                var id = "foo";
+                Func<IIssue, object> valueRetriever = null;
+
+                // When
+                var result = Record.Exception(() => new HtmlDxDataGridColumnDescription(id, valueRetriever));
+
+                // Then
+                result.IsArgumentNullException("valueRetriever");
+            }
+
+            [Fact]
+            public void Should_Assign_Id()
+            {
+                // Given
+                var id = "foo";
+                object valueRetriever(IIssue issue)
+                {
+                    return true;
+                }
+
+                // When
+                var result = new HtmlDxDataGridColumnDescription(id, valueRetriever);
+
+                // Then
+                result.Id.ShouldBe(id);
+            }
+
+            [Fact]
+            public void Should_Assign_ValueRetriever()
+            {
+                // Given
+                var id = "foo";
+                object valueRetriever(IIssue issue)
+                {
+                    return true;
+                }
+
+                // When
+                var result = new HtmlDxDataGridColumnDescription(id, valueRetriever);
+
+                // Then
+                result.ValueRetriever.ShouldBe(valueRetriever);
+            }
+        }
+    }
+}