(#99) Fix unit tests

Author: gep13

src/GitReleaseManager.Core.Tests/Commands/ExportCommandTests.cs

@@ -39,7 +39,7 @@ public async Task Should_Execute_Command()
 
             var releaseText = "releaseText";
 
-            _vcsService.ExportReleasesAsync(options.RepositoryOwner, options.RepositoryName, options.TagName)
+            _vcsService.ExportReleasesAsync(options.RepositoryOwner, options.RepositoryName, options.TagName, options.SkipPrereleases)
                 .Returns(releaseText);
 
             var result = await _command.Execute(options).ConfigureAwait(false);
@@ -51,7 +51,7 @@ public async Task Should_Execute_Command()
             var exportFileContent = File.ReadAllText(_fileOutputPath);
             exportFileContent.ShouldBe(releaseText);
 
-            await _vcsService.Received(1).ExportReleasesAsync(options.RepositoryOwner, options.RepositoryName, options.TagName).ConfigureAwait(false);
+            await _vcsService.Received(1).ExportReleasesAsync(options.RepositoryOwner, options.RepositoryName, options.TagName, options.SkipPrereleases).ConfigureAwait(false);
             _logger.Received(1).Information(Arg.Any<string>(), options.TagName);
 
             if (exportFileExists)

src/GitReleaseManager.Core.Tests/Provider/GitHubProviderTests.cs

@@ -41,6 +41,7 @@ public class GitHubProviderTests
         private const int RELEASE_ID = 1;
         private const int ASSET_ID = 1;
         private const string NOT_FOUND_MESSAGE = "NotFound";
+        private const bool SKIP_PRERELEASES = false;
 
         private readonly Release _release = new Release();
         private readonly ReleaseAssetUpload _releaseAssetUpload = new ReleaseAssetUpload();
@@ -667,7 +668,7 @@ public async Task Should_Get_Releases()
             _mapper.Map<IEnumerable<Release>>(Arg.Any<object>())
                 .Returns(releases);
 
-            var result = await _gitHubProvider.GetReleasesAsync(OWNER, REPOSITORY).ConfigureAwait(false);
+            var result = await _gitHubProvider.GetReleasesAsync(OWNER, REPOSITORY, SKIP_PRERELEASES).ConfigureAwait(false);
             result.ShouldBeSameAs(releases);
 
             await _gitHubClient.Repository.Release.Received(1).GetAll(OWNER, REPOSITORY, Arg.Any<ApiOptions>()).ConfigureAwait(false);
@@ -680,7 +681,7 @@ public async Task Should_Throw_An_Exception_On_Getting_Releases()
             _gitHubClient.Repository.Release.GetAll(OWNER, REPOSITORY, Arg.Any<ApiOptions>())
                 .Returns(Task.FromException<IReadOnlyList<Octokit.Release>>(_exception));
 
-            var ex = await Should.ThrowAsync<ApiException>(() => _gitHubProvider.GetReleasesAsync(OWNER, REPOSITORY)).ConfigureAwait(false);
+            var ex = await Should.ThrowAsync<ApiException>(() => _gitHubProvider.GetReleasesAsync(OWNER, REPOSITORY, SKIP_PRERELEASES)).ConfigureAwait(false);
             ex.Message.ShouldBe(_exception.Message);
             ex.InnerException.ShouldBe(_exception);
         }

src/GitReleaseManager.Core.Tests/VcsServiceTests.cs

@@ -31,6 +31,7 @@ public class VcsServiceTests
         private const string TAG_NAME = "0.1.0";
         private const string RELEASE_NOTES = "Release Notes";
         private const string ASSET_CONTENT = "Asset Content";
+        private const bool SKIP_PRERELEASES = false;
 
         private const string UNABLE_TO_FOUND_MILESTONE_MESSAGE = "Unable to find a {State} milestone with title '{Title}' on '{Owner}/{Repository}'";
         private const string UNABLE_TO_FOUND_RELEASE_MESSAGE = "Unable to find a release with tag '{TagName}' on '{Owner}/{Repository}'";
@@ -591,17 +592,17 @@ public async Task Should_Get_Release_Notes()
             var releases = Enumerable.Empty<Release>();
             var releaseNotes = "Release Notes";
 
-            _vcsProvider.GetReleasesAsync(OWNER, REPOSITORY)
+            _vcsProvider.GetReleasesAsync(OWNER, REPOSITORY, SKIP_PRERELEASES)
                 .Returns(Task.FromResult(releases));
 
             _releaseNotesExporter.ExportReleaseNotes(Arg.Any<IEnumerable<Release>>())
                 .Returns(releaseNotes);
 
-            var result = await _vcsService.ExportReleasesAsync(OWNER, REPOSITORY, null).ConfigureAwait(false);
+            var result = await _vcsService.ExportReleasesAsync(OWNER, REPOSITORY, null, SKIP_PRERELEASES).ConfigureAwait(false);
             result.ShouldBeSameAs(releaseNotes);
 
             await _vcsProvider.DidNotReceive().GetReleaseAsync(Arg.Any<string>(), Arg.Any<string>(), Arg.Any<string>()).ConfigureAwait(false);
-            await _vcsProvider.Received(1).GetReleasesAsync(OWNER, REPOSITORY).ConfigureAwait(false);
+            await _vcsProvider.Received(1).GetReleasesAsync(OWNER, REPOSITORY, SKIP_PRERELEASES).ConfigureAwait(false);
             _logger.Received(1).Verbose(Arg.Any<string>(), OWNER, REPOSITORY);
             _releaseNotesExporter.Received(1).ExportReleaseNotes(Arg.Any<IEnumerable<Release>>());
         }
@@ -617,11 +618,11 @@ public async Task Should_Get_Release_Notes_For_Tag()
             _releaseNotesExporter.ExportReleaseNotes(Arg.Any<IEnumerable<Release>>())
                 .Returns(RELEASE_NOTES);
 
-            var result = await _vcsService.ExportReleasesAsync(OWNER, REPOSITORY, TAG_NAME).ConfigureAwait(false);
+            var result = await _vcsService.ExportReleasesAsync(OWNER, REPOSITORY, TAG_NAME, SKIP_PRERELEASES).ConfigureAwait(false);
             result.ShouldBeSameAs(RELEASE_NOTES);
 
             await _vcsProvider.Received(1).GetReleaseAsync(OWNER, REPOSITORY, TAG_NAME).ConfigureAwait(false);
-            await _vcsProvider.DidNotReceive().GetReleasesAsync(Arg.Any<string>(), Arg.Any<string>()).ConfigureAwait(false);
+            await _vcsProvider.DidNotReceive().GetReleasesAsync(Arg.Any<string>(), Arg.Any<string>(), Arg.Any<bool>()).ConfigureAwait(false);
             _logger.Received(1).Verbose(Arg.Any<string>(), OWNER, REPOSITORY, TAG_NAME);
             _releaseNotesExporter.Received(1).ExportReleaseNotes(Arg.Any<IEnumerable<Release>>());
         }
@@ -635,7 +636,7 @@ public async Task Should_Get_Default_Release_Notes_For_Non_Existent_Tag()
             _releaseNotesExporter.ExportReleaseNotes(Arg.Any<IEnumerable<Release>>())
                 .Returns(RELEASE_NOTES);
 
-            var result = await _vcsService.ExportReleasesAsync(OWNER, REPOSITORY, TAG_NAME).ConfigureAwait(false);
+            var result = await _vcsService.ExportReleasesAsync(OWNER, REPOSITORY, TAG_NAME, SKIP_PRERELEASES).ConfigureAwait(false);
             result.ShouldBeSameAs(RELEASE_NOTES);
 
             await _vcsProvider.Received(1).GetReleaseAsync(OWNER, REPOSITORY, TAG_NAME).ConfigureAwait(false);

src/GitReleaseManager.Core/Commands/ExportCommand.cs

@@ -19,7 +19,7 @@ public ExportCommand(IVcsService vcsService, ILogger logger)
         public async Task<int> Execute(ExportSubOptions options)
         {
             _logger.Information("Exporting release {TagName}", options.TagName);
-            var releasesContent = await _vcsService.ExportReleasesAsync(options.RepositoryOwner, options.RepositoryName, options.TagName).ConfigureAwait(false);
+            var releasesContent = await _vcsService.ExportReleasesAsync(options.RepositoryOwner, options.RepositoryName, options.TagName, options.SkipPrereleases).ConfigureAwait(false);
 
             using (var sw = new StreamWriter(File.Open(options.FileOutputPath, FileMode.OpenOrCreate)))
             {

src/GitReleaseManager.IntegrationTests/GitHubProviderIntegrationTests.cs

@@ -19,6 +19,7 @@ public class GitHubProviderIntegrationTests
     {
         private const string OWNER = "GitTools";
         private const string REPOSITORY = "GitReleaseManager";
+        private const bool SKIP_PRERELEASES = false;
 
         private GitHubProvider _gitHubProvider;
         private IGitHubClient _gitHubClient;
@@ -87,7 +88,7 @@ public async Task Should_Get_Issue_Comments()
         [Order(5)]
         public async Task Should_Get_Releases()
         {
-            var result = await _gitHubProvider.GetReleasesAsync(OWNER, REPOSITORY).ConfigureAwait(false);
+            var result = await _gitHubProvider.GetReleasesAsync(OWNER, REPOSITORY, SKIP_PRERELEASES).ConfigureAwait(false);
             result.Count().ShouldBeGreaterThan(0);
 
             var orderedReleases = result.OrderByDescending(r => r.Id).ToList();

src/GitReleaseManager.Tests/VcsServiceMock.cs

@@ -45,7 +45,7 @@ public Task AddAssetsAsync(string owner, string repository, string tagName, ILis
             throw new System.NotImplementedException();
         }
 
-        public Task<string> ExportReleasesAsync(string owner, string repository, string tagName)
+        public Task<string> ExportReleasesAsync(string owner, string repository, string tagName, bool skipPrereleases)
         {
             throw new System.NotImplementedException();
         }