Skip to content

Commit 4e5ff3e

Browse files
fix xunit compiler warnings for unit tests
1 parent 997ae38 commit 4e5ff3e

File tree

5 files changed

+16
-16
lines changed

5 files changed

+16
-16
lines changed

src/ScmBackup.Tests/ApiRepositoriesTests.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ public void AddAddsItem()
1919
var sut = new ApiRepositories();
2020
sut.AddItem(source, list);
2121

22-
Assert.Equal(1, sut.Dic.Count);
22+
Assert.Single(sut.Dic);
2323
}
2424

2525
[Fact]
@@ -40,8 +40,8 @@ public void GetSourcesReturnsSources()
4040
var result = sut.GetSources();
4141

4242
Assert.Equal(2, result.Count());
43-
Assert.True(result.Contains(source1));
44-
Assert.True(result.Contains(source2));
43+
Assert.Contains(source1, result);
44+
Assert.Contains(source2, result);
4545
}
4646

4747
[Fact]
@@ -103,7 +103,7 @@ public void GetScmTypesReturnsHashSet()
103103

104104
var result = sut.GetScmTypes();
105105

106-
Assert.Equal(1, result.Count);
106+
Assert.Single(result);
107107
Assert.Equal(ScmType.Git, result.First());
108108
}
109109

src/ScmBackup.Tests/Hosters/IConfigSourceValidatorTests.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ public void ReturnsErrorWhenHosterIsWrong()
2525
var result = sut.Validate(config);
2626

2727
Assert.False(result.IsValid);
28-
Assert.Equal(1, result.Messages.Count);
28+
Assert.Single(result.Messages);
2929
Assert.Equal(ErrorLevel.Error, result.Messages[0].Error);
3030
}
3131

@@ -39,7 +39,7 @@ public void ReturnsErrorWhenTypeIsInvalid(string value)
3939
var result = sut.Validate(config);
4040

4141
Assert.False(result.IsValid);
42-
Assert.Equal(1, result.Messages.Count);
42+
Assert.Single(result.Messages);
4343
Assert.Equal(ErrorLevel.Error, result.Messages[0].Error);
4444
}
4545

@@ -53,7 +53,7 @@ public void ValidatesWhenTypeIsValid(string value)
5353
var result = sut.Validate(config);
5454

5555
Assert.True(result.IsValid);
56-
Assert.False(result.Messages.Any());
56+
Assert.Empty(result.Messages);
5757
}
5858

5959
[Theory]
@@ -67,7 +67,7 @@ public void ReturnsErrorWhenNameIsEmpty(string value)
6767
var result = sut.Validate(config);
6868

6969
Assert.False(result.IsValid);
70-
Assert.Equal(1, result.Messages.Count);
70+
Assert.Single(result.Messages);
7171
Assert.Equal(ErrorLevel.Error, result.Messages[0].Error);
7272
}
7373

@@ -84,7 +84,7 @@ public void ReturnsErrorWhenAuthNameORPasswordAreSet(string user, string passwor
8484
var result = sut.Validate(config);
8585

8686
Assert.False(result.IsValid);
87-
Assert.Equal(1, result.Messages.Count);
87+
Assert.Single(result.Messages);
8888
Assert.Equal(ErrorLevel.Error, result.Messages[0].Error);
8989
}
9090

@@ -100,7 +100,7 @@ public void ReturnsWarningWhenAuthNameAndPasswordAreEmpty(string value)
100100
var result = sut.Validate(config);
101101

102102
Assert.True(result.IsValid);
103-
Assert.Equal(1, result.Messages.Count);
103+
Assert.Single(result.Messages);
104104
Assert.Equal(ErrorLevel.Warn, result.Messages[0].Error);
105105
}
106106
}

src/ScmBackup.Tests/IgnoringHosterApiCallerTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public void IgnoresRepo()
3232

3333
var list = this.sut.GetRepositoryList(this.source);
3434

35-
Assert.Equal(1, list.Count);
35+
Assert.Single(list);
3636
Assert.Equal("baz", list.First().ShortName);
3737
}
3838

src/ScmBackup.Tests/MessageLoggerTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ public void LogsMessage()
1515
sut.Log(ErrorLevel.Info, "foo {0}", "bar");
1616

1717
var result = msg.GetMessages().ToList();
18-
Assert.Equal(1, result.Count);
18+
Assert.Single(result);
1919

2020
string text = result.First();
2121

src/ScmBackup.Tests/ValidationResultTests.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ public void AddMessageAddsSingleMessage()
1717
{
1818
sut.AddMessage(ErrorLevel.Info, "i");
1919

20-
Assert.Equal(1, sut.Messages.Count);
20+
Assert.Single(sut.Messages);
2121
}
2222

2323
[Fact]
@@ -30,8 +30,8 @@ public void AddMessageAddsConfigSourceTitle()
3030
sut.AddMessage(ErrorLevel.Info, "message");
3131

3232
var createdText = sut.Messages.First().Message;
33-
Assert.True(createdText.StartsWith("foo"));
34-
Assert.True(createdText.EndsWith("message"));
33+
Assert.StartsWith("foo", createdText);
34+
Assert.EndsWith("message", createdText);
3535
}
3636

3737
[Fact]
@@ -56,7 +56,7 @@ public void IsValidIsFalseWithErrorMessage()
5656
{
5757
sut.AddMessage(ErrorLevel.Error, "e");
5858

59-
Assert.Equal(1, sut.Messages.Count);
59+
Assert.Single(sut.Messages);
6060
Assert.False(sut.IsValid);
6161
}
6262

0 commit comments

Comments
 (0)