diff --git a/Octokit.Tests/Models/SearchIssuesRequestExclusionsTests.cs b/Octokit.Tests/Models/SearchIssuesRequestExclusionsTests.cs index 989b794e..0cace60e 100644 --- a/Octokit.Tests/Models/SearchIssuesRequestExclusionsTests.cs +++ b/Octokit.Tests/Models/SearchIssuesRequestExclusionsTests.cs @@ -12,15 +12,15 @@ public class SearchIssuesRequestExclusionsTests [Fact] public void HandlesStringAttributesCorrectly() { - var stringProperties = new List>>() + var stringProperties = new Dictionary>() { - new Tuple>("author:", (x,value) => x.Author = value), - new Tuple>("assignee:", (x,value) => x.Assignee = value), - new Tuple>("mentions:", (x,value) => x.Mentions = value), - new Tuple>("commenter:", (x,value) => x.Commenter = value), - new Tuple>("involves:", (x,value) => x.Involves = value), - new Tuple>("head:", (x,value) => x.Head = value), - new Tuple>("base:", (x,value) => x.Base = value) + { "author:", (x,value) => x.Author = value }, + { "assignee:", (x,value) => x.Assignee = value }, + { "mentions:", (x,value) => x.Mentions = value }, + { "commenter:", (x,value) => x.Commenter = value }, + { "involves:", (x,value) => x.Involves = value }, + { "head:", (x,value) => x.Head = value }, + { "base:", (x,value) => x.Base = value } }; foreach (var property in stringProperties) @@ -28,13 +28,13 @@ public class SearchIssuesRequestExclusionsTests var request = new SearchIssuesRequestExclusions(); // Ensure the specified parameter does not exist when not set - Assert.False(request.MergedQualifiers().Any(x => x.Contains(property.Item1))); + Assert.False(request.MergedQualifiers().Any(x => x.Contains(property.Key))); // Set the parameter - property.Item2(request, "blah"); + property.Value(request, "blah"); // Ensure the specified parameter now exists - Assert.True(request.MergedQualifiers().Count(x => x.Contains(property.Item1)) == 1); + Assert.True(request.MergedQualifiers().Count(x => x.Contains(property.Key)) == 1); } } diff --git a/Octokit.Tests/Models/SearchIssuesRequestTests.cs b/Octokit.Tests/Models/SearchIssuesRequestTests.cs index dc2ac66e..a075237e 100644 --- a/Octokit.Tests/Models/SearchIssuesRequestTests.cs +++ b/Octokit.Tests/Models/SearchIssuesRequestTests.cs @@ -31,17 +31,17 @@ public class SearchIssuesRequestTests [Fact] public void HandlesStringAttributesCorrectly() { - var stringProperties = new List>>() + var stringProperties = new Dictionary> { - new Tuple>("author:", (x,value) => x.Author = value), - new Tuple>("assignee:", (x,value) => x.Assignee = value), - new Tuple>("mentions:", (x,value) => x.Mentions = value), - new Tuple>("commenter:", (x,value) => x.Commenter = value), - new Tuple>("involves:", (x,value) => x.Involves = value), - new Tuple>("team:", (x,value) => x.Team = value), - new Tuple>("head:", (x,value) => x.Head = value), - new Tuple>("base:", (x,value) => x.Base = value), - new Tuple>("user:", (x,value) => x.User = value) + { "author:", (x,value) => x.Author = value }, + { "assignee:", (x,value) => x.Assignee = value }, + { "mentions:", (x,value) => x.Mentions = value }, + { "commenter:", (x,value) => x.Commenter = value }, + { "involves:", (x,value) => x.Involves = value }, + { "team:", (x,value) => x.Team = value }, + { "head:", (x,value) => x.Head = value }, + { "base:", (x,value) => x.Base = value }, + { "user:", (x,value) => x.User = value } }; foreach (var property in stringProperties) @@ -49,25 +49,25 @@ public class SearchIssuesRequestTests var request = new SearchIssuesRequest("query"); // Ensure the specified parameter does not exist when not set - Assert.False(request.MergedQualifiers().Any(x => x.Contains(property.Item1))); + Assert.False(request.MergedQualifiers().Any(x => x.Contains(property.Key))); // Set the parameter - property.Item2(request, "blah"); + property.Value(request, "blah"); // Ensure the specified parameter now exists - Assert.True(request.MergedQualifiers().Count(x => x.Contains(property.Item1)) == 1); + Assert.True(request.MergedQualifiers().Count(x => x.Contains(property.Key)) == 1); } } [Fact] public void HandlesDateRangeAttributesCorrectly() { - var dateProperties = new List>>() + var dateProperties = new Dictionary> { - new Tuple>("created:", (x,value) => x.Created = value), - new Tuple>("updated:", (x,value) => x.Updated = value), - new Tuple>("merged:", (x,value) => x.Merged = value), - new Tuple>("closed:", (x,value) => x.Closed = value) + { "created:", (x,value) => x.Created = value }, + { "updated:", (x,value) => x.Updated = value }, + { "merged:", (x,value) => x.Merged = value }, + { "closed:", (x,value) => x.Closed = value } }; foreach (var property in dateProperties) @@ -75,13 +75,13 @@ public class SearchIssuesRequestTests var request = new SearchIssuesRequest("query"); // Ensure the specified parameter does not exist when not set - Assert.False(request.MergedQualifiers().Any(x => x.Contains(property.Item1))); + Assert.False(request.MergedQualifiers().Any(x => x.Contains(property.Key))); // Set the parameter - property.Item2(request, DateRange.GreaterThan(DateTime.Today.AddDays(-7))); + property.Value(request, DateRange.GreaterThan(DateTime.Today.AddDays(-7))); // Ensure the specified parameter now exists - Assert.True(request.MergedQualifiers().Count(x => x.Contains(property.Item1)) == 1); + Assert.True(request.MergedQualifiers().Count(x => x.Contains(property.Key)) == 1); } }