Skip to content

Commit 87bd5ec

Browse files
committed
use logMock.Object instead of class field
1 parent be2c1cc commit 87bd5ec

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/OctoshiftCLI.Tests/Octoshift/Services/ArchiveUploadersTests.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ public void Constructor_Should_Use_Default_When_Environment_Variable_Not_Set()
7878
.Returns(() => null);
7979

8080
// Act
81-
var archiveUploader = new ArchiveUploader(_githubClientMock.Object, UPLOADS_URL, _logMock.Object, retryPolicy, environmentVariableProviderMock.Object);
81+
var archiveUploader = new ArchiveUploader(_githubClientMock.Object, UPLOADS_URL, logMock.Object, retryPolicy, environmentVariableProviderMock.Object);
8282

8383
// Assert
8484
archiveUploader._streamSizeLimit.Should().Be(defaultSize);
@@ -98,7 +98,7 @@ public void Constructor_Should_Use_Default_When_Environment_Variable_Is_Invalid(
9898
.Returns("invalid_value");
9999

100100
// Act
101-
var archiveUploader = new ArchiveUploader(_githubClientMock.Object, UPLOADS_URL, _logMock.Object, retryPolicy, environmentVariableProviderMock.Object);
101+
var archiveUploader = new ArchiveUploader(_githubClientMock.Object, UPLOADS_URL, logMock.Object, retryPolicy, environmentVariableProviderMock.Object);
102102

103103
// Assert
104104
archiveUploader._streamSizeLimit.Should().Be(defaultSize);
@@ -118,7 +118,7 @@ public void Constructor_Should_Use_Default_When_Environment_Variable_Is_Zero()
118118
.Returns("0");
119119

120120
// Act
121-
var archiveUploader = new ArchiveUploader(_githubClientMock.Object, UPLOADS_URL, _logMock.Object, retryPolicy, environmentVariableProviderMock.Object);
121+
var archiveUploader = new ArchiveUploader(_githubClientMock.Object, UPLOADS_URL, logMock.Object, retryPolicy, environmentVariableProviderMock.Object);
122122

123123
// Assert
124124
archiveUploader._streamSizeLimit.Should().Be(defaultSize);
@@ -138,7 +138,7 @@ public void Constructor_Should_Use_Default_When_Environment_Variable_Is_Negative
138138
.Returns("-1000");
139139

140140
// Act
141-
var archiveUploader = new ArchiveUploader(_githubClientMock.Object, UPLOADS_URL, _logMock.Object, retryPolicy, environmentVariableProviderMock.Object);
141+
var archiveUploader = new ArchiveUploader(_githubClientMock.Object, UPLOADS_URL, logMock.Object, retryPolicy, environmentVariableProviderMock.Object);
142142

143143
// Assert
144144
archiveUploader._streamSizeLimit.Should().Be(defaultSize);

0 commit comments

Comments
 (0)