zoriya/octokit.net
1
0
Fork 0
mirror of https://github.com/zoriya/octokit.net.git synced 2026-06-05 03:30:34 +00:00
Code Issues Packages Projects Releases Wiki Activity

Tidy up asserts

Browse Source
This commit is contained in:
Ryan Gribble
2016-08-15 21:13:35 +10:00
parent 6ebf1939b6
commit 0b1280e8e5
1 changed files with 2 additions and 39 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Octokit.Tests.Integration/Clients/RepositoryBranchesClientTests.cs
+2 -39
View File
@@ -20,14 +20,12 @@ public class RepositoryBranchesClientTests
Assert.NotEmpty(branches);
// Ensure Protection attribute is deserialized
foreach (var branch in branches)
{
Assert.NotNull(branch.Protection);
}
// Ensure Protected attribute is deserialized (at least master branch should be true)
Assert.True(branches.Any(x => x.Protected));
Assert.True(branches.First(x => x.Name == "master").Protected);
}
[IntegrationTest]
@@ -39,14 +37,12 @@ public class RepositoryBranchesClientTests
Assert.NotEmpty(branches);
// Ensure Protection attribute is deserialized
foreach (var branch in branches)
{
Assert.NotNull(branch.Protection);
}
// Ensure Protected attribute is deserialized (at least master branch should be true)
Assert.True(branches.Any(x => x.Protected));
Assert.True(branches.First(x => x.Name == "master").Protected);
}
[IntegrationTest]
@@ -193,11 +189,7 @@ public class RepositoryBranchesClientTests
Assert.NotNull(branch);
Assert.Equal("master", branch.Name);
// Ensure Protection attribute is deserialized
Assert.NotNull(branch.Protection);
// Ensure Protected attribute is deserialized
Assert.True(branch.Protected);
}
@@ -211,10 +203,7 @@ public class RepositoryBranchesClientTests
Assert.NotNull(branch);
Assert.Equal("master", branch.Name);
// Ensure Protection attribute is deserialized
Assert.NotNull(branch.Protection);
// Ensure Protected attribute is deserialized
Assert.True(branch.Protected);
}
}
@@ -339,9 +328,6 @@ public class RepositoryBranchesClientTests
var repoName = _userRepoContext.RepositoryName;
var protection = await _client.GetBranchProtection(repoOwner, repoName, "master");
Assert.NotNull(protection);
Assert.NotNull(protection.RequiredStatusChecks);
Assert.NotNull(protection.RequiredStatusChecks.Contexts);
Assert.True(protection.RequiredStatusChecks.IncludeAdmins);
Assert.True(protection.RequiredStatusChecks.Strict);
Assert.Equal(2, protection.RequiredStatusChecks.Contexts.Count);
@@ -355,9 +341,6 @@ public class RepositoryBranchesClientTests
var repoId = _userRepoContext.RepositoryId;
var protection = await _client.GetBranchProtection(repoId, "master");
Assert.NotNull(protection);
Assert.NotNull(protection.RequiredStatusChecks);
Assert.NotNull(protection.RequiredStatusChecks.Contexts);
Assert.True(protection.RequiredStatusChecks.IncludeAdmins);
Assert.True(protection.RequiredStatusChecks.Strict);
Assert.Equal(2, protection.RequiredStatusChecks.Contexts.Count);
@@ -372,14 +355,10 @@ public class RepositoryBranchesClientTests
var repoName = _orgRepoContext.RepositoryContext.RepositoryName;
var protection = await _client.GetBranchProtection(repoOwner, repoName, "master");
Assert.NotNull(protection);
Assert.NotNull(protection.RequiredStatusChecks);
Assert.NotNull(protection.RequiredStatusChecks.Contexts);
Assert.True(protection.RequiredStatusChecks.IncludeAdmins);
Assert.True(protection.RequiredStatusChecks.Strict);
Assert.Equal(2, protection.RequiredStatusChecks.Contexts.Count);
Assert.NotNull(protection.Restrictions);
Assert.Equal(1, protection.Restrictions.Teams.Count);
Assert.Equal(0, protection.Restrictions.Users.Count);
}
@@ -390,14 +369,10 @@ public class RepositoryBranchesClientTests
var repoId = _orgRepoContext.RepositoryContext.RepositoryId;
var protection = await _client.GetBranchProtection(repoId, "master");
Assert.NotNull(protection);
Assert.NotNull(protection.RequiredStatusChecks);
Assert.NotNull(protection.RequiredStatusChecks.Contexts);
Assert.True(protection.RequiredStatusChecks.IncludeAdmins);
Assert.True(protection.RequiredStatusChecks.Strict);
Assert.Equal(2, protection.RequiredStatusChecks.Contexts.Count);
Assert.NotNull(protection.Restrictions);
Assert.Equal(1, protection.Restrictions.Teams.Count);
Assert.Equal(0, protection.Restrictions.Users.Count);
}
@@ -437,9 +412,6 @@ public class RepositoryBranchesClientTests
var protection = await _client.UpdateBranchProtection(repoOwner, repoName, "master", update);
Assert.NotNull(protection);
Assert.NotNull(protection.RequiredStatusChecks);
Assert.NotNull(protection.RequiredStatusChecks.Contexts);
Assert.False(protection.RequiredStatusChecks.IncludeAdmins);
Assert.False(protection.RequiredStatusChecks.Strict);
Assert.Equal(1, protection.RequiredStatusChecks.Contexts.Count);
@@ -456,9 +428,6 @@ public class RepositoryBranchesClientTests
var protection = await _client.UpdateBranchProtection(repoId, "master", update);
Assert.NotNull(protection);
Assert.NotNull(protection.RequiredStatusChecks);
Assert.NotNull(protection.RequiredStatusChecks.Contexts);
Assert.False(protection.RequiredStatusChecks.IncludeAdmins);
Assert.False(protection.RequiredStatusChecks.Strict);
Assert.Equal(1, protection.RequiredStatusChecks.Contexts.Count);
@@ -477,9 +446,6 @@ public class RepositoryBranchesClientTests
var protection = await _client.UpdateBranchProtection(repoOwner, repoName, "master", update);
Assert.NotNull(protection);
Assert.NotNull(protection.RequiredStatusChecks);
Assert.NotNull(protection.RequiredStatusChecks.Contexts);
Assert.False(protection.RequiredStatusChecks.IncludeAdmins);
Assert.False(protection.RequiredStatusChecks.Strict);
Assert.Equal(1, protection.RequiredStatusChecks.Contexts.Count);
@@ -497,9 +463,6 @@ public class RepositoryBranchesClientTests
var protection = await _client.UpdateBranchProtection(repoId, "master", update);
Assert.NotNull(protection);
Assert.NotNull(protection.RequiredStatusChecks);
Assert.NotNull(protection.RequiredStatusChecks.Contexts);
Assert.False(protection.RequiredStatusChecks.IncludeAdmins);
Assert.False(protection.RequiredStatusChecks.Strict);
Assert.Equal(1, protection.RequiredStatusChecks.Contexts.Count);