Add tests for each search parameter

Octokit.Tests/Models/SearchIssuesRequestTests.cs

@@ -1,9 +1,11 @@
 using System;
+using System.Collections.Generic;
+using System.Linq;
 using Octokit;
 using Octokit.Tests.Helpers;
 using Xunit;
 
-internal class SearchIssuesRequestTests
+public class SearchIssuesRequestTests
 {
     public class TheMergedQualifiersMethod
     {
@@ -25,5 +27,167 @@ internal class SearchIssuesRequestTests
             Assert.True(string.IsNullOrWhiteSpace(request.Sort));
             Assert.False(request.Parameters.ContainsKey("sort"));
         }
+
+        [Fact]
+        public void HandlesStringAttributesCorrectly()
+        {
+            var stringProperties = new List<Tuple<string, Action<SearchIssuesRequest, string>>>()
+            {
+                new Tuple<string, Action<SearchIssuesRequest, string>>("author:", (x,value) => x.Author = value),
+                new Tuple<string, Action<SearchIssuesRequest, string>>("assignee:", (x,value) => x.Assignee = value),
+                new Tuple<string, Action<SearchIssuesRequest, string>>("mentions:", (x,value) => x.Mentions = value),
+                new Tuple<string, Action<SearchIssuesRequest, string>>("commenter:", (x,value) => x.Commenter = value),
+                new Tuple<string, Action<SearchIssuesRequest, string>>("involves:", (x,value) => x.Involves = value),
+                new Tuple<string, Action<SearchIssuesRequest, string>>("team:", (x,value) => x.Team = value),
+                new Tuple<string, Action<SearchIssuesRequest, string>>("head:", (x,value) => x.Head = value),
+                new Tuple<string, Action<SearchIssuesRequest, string>>("base:", (x,value) => x.Base = value),
+                new Tuple<string, Action<SearchIssuesRequest, string>>("user:", (x,value) => x.User = value)
+            };
+
+            foreach (var property in stringProperties)
+            {
+                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)));
+
+                // Set the parameter
+                property.Item2(request, "blah");
+
+                // Ensure the specified parameter now exists
+                Assert.True(request.MergedQualifiers().Count(x => x.Contains(property.Item1)) == 1);
+            }
+        }
+
+        [Fact]
+        public void HandlesDateRangeAttributesCorrectly()
+        {
+            var dateProperties = new List<Tuple<string, Action<SearchIssuesRequest, DateRange>>>()
+            {
+                new Tuple<string, Action<SearchIssuesRequest, DateRange>>("created:", (x,value) => x.Created = value),
+                new Tuple<string, Action<SearchIssuesRequest, DateRange>>("updated:", (x,value) => x.Updated = value),
+                new Tuple<string, Action<SearchIssuesRequest, DateRange>>("merged:", (x,value) => x.Merged = value),
+                new Tuple<string, Action<SearchIssuesRequest, DateRange>>("closed:", (x,value) => x.Closed = value)
+            };
+
+            foreach (var property in dateProperties)
+            {
+                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)));
+
+                // Set the parameter
+                property.Item2(request, DateRange.GreaterThan(DateTime.Today.AddDays(-7)));
+
+                // Ensure the specified parameter now exists
+                Assert.True(request.MergedQualifiers().Count(x => x.Contains(property.Item1)) == 1);
+            }
+        }
+
+        [Fact]
+        public void HandlesInAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("in:")));
+
+            request.In = new List<IssueInQualifier> { IssueInQualifier.Body, IssueInQualifier.Comment };
+            Assert.True(request.MergedQualifiers().Contains("in:body,comment"));
+        }
+
+        [Fact]
+        public void HandlesStateAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("state:")));
+
+            request.State = ItemState.Closed;
+            Assert.True(request.MergedQualifiers().Contains("state:closed"));
+        }
+
+        [Fact]
+        public void HandlesLabelsAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("label:")));
+
+            request.Labels = new List<string> { "label1", "label2" };
+            Assert.True(request.MergedQualifiers().Contains("label:label1"));
+            Assert.True(request.MergedQualifiers().Contains("label:label2"));
+        }
+
+        [Fact]
+        public void HandlesNotLabelsAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("-label:")));
+
+            request.NotLabels = new List<string> { "label1", "label2" };
+            Assert.True(request.MergedQualifiers().Contains("-label:label1"));
+            Assert.True(request.MergedQualifiers().Contains("-label:label2"));
+        }
+
+        [Fact]
+        public void HandlesNoMetadataAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("no:")));
+
+            request.No = IssueNoMetadataQualifier.Milestone;
+            Assert.True(request.MergedQualifiers().Contains("no:milestone"));
+        }
+
+        [Fact]
+        public void HandlesLanguageAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("language:")));
+
+            request.Language = Language.CSharp;
+            Assert.True(request.MergedQualifiers().Contains("language:CSharp"));
+        }
+
+        [Fact]
+        public void HandlesIsAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("is:")));
+
+            request.Is = new List<IssueIsQualifier> { IssueIsQualifier.Merged, IssueIsQualifier.PullRequest };
+            Assert.True(request.MergedQualifiers().Contains("is:merged"));
+            Assert.True(request.MergedQualifiers().Contains("is:pr"));
+        }
+
+        [Fact]
+        public void HandlesStatusAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("status:")));
+
+            request.Status = CommitState.Error;
+            Assert.True(request.MergedQualifiers().Contains("status:error"));
+        }
+
+        [Fact]
+        public void HandlesCommentsAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("comments:")));
+
+            request.Comments = Range.GreaterThan(5);
+            Assert.True(request.MergedQualifiers().Contains("comments:>5"));
+        }
+
+        [Fact]
+        public void HandlesRepoAttributeCorrectly()
+        {
+            var request = new SearchIssuesRequest("test");
+            Assert.False(request.MergedQualifiers().Any(x => x.Contains("repo:")));
+
+            request.Repos.Add("myorg", "repo1");
+            request.Repos.Add("myorg", "repo2");
+            Assert.True(request.MergedQualifiers().Contains("repo:myorg/repo1"));
+            Assert.True(request.MergedQualifiers().Contains("repo:myorg/repo2"));
+        }
     }
-}
+}