github/lidarr
1
0
Fork 0
mirror of https://github.com/lidarr/lidarr.git synced 2025-08-19 21:13:28 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Fixed: Make ProcessProviderTests less flaky

Browse source
This commit is contained in:
ta264 2019-08-22 21:16:15 +01:00 • committed by Qstick
commit 1a0b95de03
1 changed files with 35 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

37
src/NzbDrone.Common.Test/ProcessProviderTests.cs
View file

@ -3,6 +3,7 @@ using System.ComponentModel;
using System.Diagnostics;
using System.IO;
using System.Linq;
using System.Threading;
using FluentAssertions;
using NUnit.Framework;
using NzbDrone.Common.Model;
@ -12,6 +13,8 @@ using NzbDrone.Test.Dummy;
namespace NzbDrone.Common.Test
{
// We don't want one tests setup killing processes used in another
[NonParallelizable]
[TestFixture]
public class ProcessProviderTests : TestBase<ProcessProvider>
{
@ -63,9 +66,25 @@ namespace NzbDrone.Common.Test
ExceptionVerification.ExpectedWarns(1);
}
[Test]
public void should_be_able_to_start_process()
{
var process = StartDummyProcess();
var check = Subject.GetProcessById(process.Id);
check.Should().NotBeNull();
process.Refresh();
process.HasExited.Should().BeFalse();
process.Kill();
process.WaitForExit();
process.HasExited.Should().BeTrue();
}
[Test]
[Platform(Exclude="MacOsX")]
public void Should_be_able_to_start_process()
public void exists_should_find_running_process()
{
var process = StartDummyProcess();
@ -94,8 +113,22 @@ namespace NzbDrone.Common.Test
private Process StartDummyProcess()
{
var processStarted = new ManualResetEventSlim();
var path = Path.Combine(TestContext.CurrentContext.TestDirectory, DummyApp.DUMMY_PROCCESS_NAME + ".exe");
return Subject.Start(path);
var process = Subject.Start(path, onOutputDataReceived: (string data) => {
if (data.StartsWith("Dummy process. ID:"))
{
processStarted.Set();
}
});
if (!processStarted.Wait(2000))
{
Assert.Fail("Failed to start process within 2 sec");
}
return process;
}
[Test]