Remove label checks

Author: andreasohlund

src/Compiler/ReleaseNotesBuilder.cs

@@ -1,6 +1,5 @@
 using System;
 using System.Collections.Generic;
-using System.Globalization;
 using System.IO;
 using System.Linq;
 using System.Text;
@@ -36,7 +35,7 @@ public async Task<string> BuildReleaseNotes()
  milestones = await gitHubClient.GetMilestones();
 
  GetTargetMilestone();
- var issues = await GetIssues(targetMilestone);
+ var issues = await gitHubClient.GetIssues(targetMilestone);
  var stringBuilder = new StringBuilder();
  var previousMilestone = GetPreviousMilestone();
  var numberOfCommits = await gitHubClient.GetNumberOfCommitsBetween(previousMilestone, targetMilestone);
@@ -122,31 +121,6 @@ static async Task AddFooter(StringBuilder stringBuilder)
  }
  }
 
- 
- async Task<List<Issue>> GetIssues(Milestone milestone)
- {
- var issues = await gitHubClient.GetIssues(milestone);
- foreach (var issue in issues)
- {
- CheckForValidLabels(issue);
- }
- return issues;
- }
-
- static void CheckForValidLabels(Issue issue)
- {
- if (issue.Labels.Count(label => label.Name.StartsWith(LabelPrefix)) != 1)
- {
- var message = string.Format(
- CultureInfo.CurrentCulture,
- "Bad issue {0}. Expected to find a single label starting with '{1}'.",
- issue.HtmlUrl,
- LabelPrefix);
-
- throw new InvalidOperationException(message);
- }
- }
-
  void Append(IEnumerable<Issue> issues, string labelName, StringBuilder builder)
  {
  var relevantIssues = issues

src/Tests/ReleaseNotesBuilderIntegrationTests.cs

@@ -1,55 +1,25 @@
 namespace ReleaseNotesCompiler.Tests
 {
- using System.Diagnostics;
+ using System;
  using System.Threading.Tasks;
  using NUnit.Framework;
  using ReleaseNotesCompiler;
 
  [TestFixture]
  public class ReleaseNotesBuilderIntegrationTests
  {
- [Test]
  [Explicit]
- public async Task SingleMilestone()
+ [TestCase("NServiceBus", "5.1.0")]
+ [TestCase("ServiceControl", "1.0.0-Beta4")]
+ [TestCase("NServiceBus", "6.0.0")]
+ public async Task CoreVersion6(string repo, string version)
  {
  var gitHubClient = ClientBuilder.Build();
 
- var releaseNotesBuilder = new ReleaseNotesBuilder(new DefaultGitHubClient(gitHubClient, "Particular", "NServiceBus"), "Particular", "NServiceBus", "5.1.0");
+ var releaseNotesBuilder = new ReleaseNotesBuilder(new DefaultGitHubClient(gitHubClient, "Particular", repo), "Particular", repo, version);
  var result = await releaseNotesBuilder.BuildReleaseNotes();
- Debug.WriteLine(result);
+ Console.WriteLine(result);
  ClipBoardHelper.SetClipboard(result);
  }
-
- [Test]
- [Explicit]
- public async Task SingleMilestone3()
- {
- var gitHubClient = ClientBuilder.Build();
-
- var releaseNotesBuilder = new ReleaseNotesBuilder(new DefaultGitHubClient(gitHubClient, "Particular", "ServiceControl"), "Particular", "ServiceControl", "1.0.0-Beta4");
- var result = await releaseNotesBuilder.BuildReleaseNotes();
- Debug.WriteLine(result);
- ClipBoardHelper.SetClipboard(result);
- }
-
-
- [Test]
- [Explicit]
- public async Task CoreVersion6()
- {
- var gitHubClient = ClientBuilder.Build();
-
- var releaseNotesBuilder = new ReleaseNotesBuilder(new DefaultGitHubClient(gitHubClient, "Particular", "NServiceBus"), "Particular", "NServiceBus", "6.0.0");
- var result = await releaseNotesBuilder.BuildReleaseNotes();
- Debug.WriteLine(result);
- ClipBoardHelper.SetClipboard(result);
- }
-
- [Test]
- [Explicit]
- public void OctokitTests()
- {
- ClientBuilder.Build();
- }
  }
 }