(GH-79) Correctly handle issues to exclude

Author: gep13

Source/GitReleaseManager.Tests/ReleaseNotesBuilderTests.CorrectlyExcludeIssues.approved.txt

@@ -0,0 +1,7 @@
+As part of this release we had [5 commits](https://github.com/TestUser/FakeRepository/commits/1.2.3) which resulted in [2 issues](https://github.com/FakeRepository/issues/issues?milestone=0&state=closed) being closed.
+
+
+__Bug__
+
+- [__#2__](http://example.com/2) Issue 2
+

Source/GitReleaseManager.Tests/ReleaseNotesBuilderTests.cs

@@ -116,6 +116,12 @@ public void SomeCommitsWrongIssueLabel()
             Assert.Throws<AggregateException>(() => AcceptTest(5, CreateIssue(1, "Test")));
         }
 
+        [Test]
+        public void CorrectlyExcludeIssues()
+        {
+            AcceptTest(5, CreateIssue(1, "Internal Refactoring"), CreateIssue(2, "Bug"));
+        }
+
         private static void AcceptTest(int commits, params Issue[] issues)
         {
             AcceptTest(commits, null, issues);

Source/GitReleaseManager/ReleaseNotesBuilder.cs

@@ -110,7 +110,15 @@ private string GetLabel(string label, Func<LabelAlias, string> func)
 
         private void CheckForValidLabels(Issue issue)
         {
-            var count = this.configuration.IssueLabelsInclude.Sum(issueLabel => issue.Labels.Count(l => l.Name.ToUpperInvariant() == issueLabel.ToUpperInvariant()));
+            var count = 0;
+
+
+            foreach (var issueLabel in issue.Labels)
+            {
+                count += this.configuration.IssueLabelsInclude.Count(issueToInclude => issueLabel.Name.ToUpperInvariant() == issueToInclude.ToUpperInvariant());
+
+                count += this.configuration.IssueLabelsExclude.Count(issueToExclude => issueLabel.Name.ToUpperInvariant() == issueToExclude.ToUpperInvariant());
+            }
 
             if (count != 1)
             {