Skip to content

Commit 967a4d3

Browse files
committed
Add TestRandom.ProcessResultWithRandomExitCode()
1 parent 436a97c commit 967a4d3

File tree

4 files changed

+14
-17
lines changed

4 files changed

+14
-17
lines changed

QuickVsix/CSharpUtils/Process/ProcessResult.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
using System;
2-
using System.Diagnostics;
1+
using System.Diagnostics;
32

43
namespace CSharpUtils
54
{

QuickVsixTests/CSharpUtils/Process/ProcessResultTests.cs

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
using CSharpUtils;
22
using NUnit.Framework;
3-
using System;
4-
using System.Diagnostics.CodeAnalysis;
53

64
[TestFixture]
75
public static class ProcessResultTests
@@ -35,8 +33,8 @@ public static void Duration_ReturnsEndTimeMinusStartTime()
3533
[Test]
3634
public static void Equals_ThrowsIfAnyFieldNotEqual_OtherwiseReturnsTrue()
3735
{
38-
ProcessResult expected = TestRandom.ProcessResultWithExitCode(TestRandom.Int());
39-
ProcessResult actual = TestRandom.ProcessResultWithExitCode(TestRandom.Int());
36+
ProcessResult expected = TestRandom.ProcessResultWithRandomExitCode();
37+
ProcessResult actual = TestRandom.ProcessResultWithRandomExitCode();
4038
Assert2.EqualsThrowsInvalidCastExceptionWhenComparedWithANewObject(expected);
4139

4240
NUnitAsserter nunitAsserterMock = Mock.Strict<NUnitAsserter>();

QuickVsixTests/CSharpUtils/UnitTesting/TestRandom.cs

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
1-
using System;
2-
using System.Collections.Generic;
3-
using System.Collections.ObjectModel;
1+
using System.Collections.ObjectModel;
42
using System.Diagnostics;
53
using System.Diagnostics.CodeAnalysis;
6-
using System.Linq;
74

85
namespace CSharpUtils
96
{
@@ -192,6 +189,12 @@ public static string[] StringArray()
192189
return randomStringArray;
193190
}
194191

192+
public static ProcessResult ProcessResultWithRandomExitCode()
193+
{
194+
int randomExitCode = TestRandom.Int();
195+
return TestableProcessResultWithExitCode(randomExitCode, RandomGenerator.Instance);
196+
}
197+
195198
public static ProcessResult ProcessResultWithExitCode(int exitCode)
196199
{
197200
return TestableProcessResultWithExitCode(exitCode, RandomGenerator.Instance);

QuickVsixTests/CSharpUtils/UnitTesting/TestRandomTests.cs

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,9 @@
1-
using CSharpUtils;
2-
using NUnit.Framework;
3-
using System;
4-
using System.Collections.Generic;
5-
using System.Collections.ObjectModel;
1+
using System.Collections.ObjectModel;
62
using System.Diagnostics;
73
using System.Diagnostics.CodeAnalysis;
8-
using System.Linq;
94
using System.Text.RegularExpressions;
5+
using CSharpUtils;
6+
using NUnit.Framework;
107

118
struct UserType
129
{
@@ -300,7 +297,7 @@ public static void StringArray_ReturnsAStringArrayWithLengthBetween0And2Populate
300297
}
301298

302299
[Test]
303-
public static void ProcessResultWithExitCode_ReturnsNewProcessResultWithAllRandomFields()
300+
public static void TestableProcessResultWithExitCode_DoesSo()
304301
{
305302
var randomGeneratorMock = Mock.Strict<RandomGenerator>();
306303
string fileName = TestRandom.String();

0 commit comments

Comments
 (0)