CSHARP-2274: Add tests for passing null options to ListCollectionNames and ListCollections.

Author: rstam

tests/MongoDB.Driver.Tests/MongoDatabaseImplTests.cs

@@ -488,15 +488,20 @@ public void DropCollection_should_execute_a_DropCollectionOperation(
         [ParameterAttributeData]
         public void ListCollectionNames_should_execute_a_ListCollectionsOperation(
             [Values(false, true)] bool usingSession,
+            [Values(false, true)] bool usingOptions,
             [Values(false, true)] bool async)
         {
             var session = CreateSession(usingSession);
             var filterDocument = BsonDocument.Parse("{ name : \"awesome\" }");
             var filterDefinition = (FilterDefinition<BsonDocument>)filterDocument;
-            var options = new ListCollectionNamesOptions
+            ListCollectionNamesOptions options = null;
+            if (usingOptions)
             {
-                Filter = filterDefinition
-            };
+                options = new ListCollectionNamesOptions
+                {
+                    Filter = filterDefinition
+                };
+            }
             var cancellationToken = new CancellationTokenSource().Token;
 
             var mockCursor = new Mock<IAsyncCursor<BsonDocument>>();
@@ -530,8 +535,15 @@ public void ListCollectionNames_should_execute_a_ListCollectionsOperation(
 
             var op = call.Operation.Should().BeOfType<ListCollectionsOperation>().Subject;
             op.DatabaseNamespace.Should().Be(_subject.DatabaseNamespace);
-            op.Filter.Should().Be(filterDocument);
             op.NameOnly.Should().BeTrue();
+            if (usingOptions)
+            {
+                op.Filter.Should().Be(filterDocument);
+            }
+            else
+            {
+                op.Filter.Should().BeNull();
+            }
         }
 
         [SkippableTheory]
@@ -588,15 +600,20 @@ public void ListCollectionNames_should_return_expected_result(
         [ParameterAttributeData]
         public void ListCollections_should_execute_a_ListCollectionsOperation(
             [Values(false, true)] bool usingSession,
+            [Values(false, true)] bool usingOptions,
             [Values(false, true)] bool async)
         {
             var session = CreateSession(usingSession);
             var filterDocument = BsonDocument.Parse("{ name : \"awesome\" }");
             var filterDefinition = (FilterDefinition<BsonDocument>)filterDocument;
-            var options = new ListCollectionsOptions
+            ListCollectionsOptions options = null;
+            if (usingOptions)
             {
-                Filter = filterDefinition
-            };
+                options = new ListCollectionsOptions
+                {
+                    Filter = filterDefinition
+                };
+            }
             var cancellationToken = new CancellationTokenSource().Token;
 
             var mockCursor = new Mock<IAsyncCursor<BsonDocument>>();
@@ -630,8 +647,15 @@ public void ListCollections_should_execute_a_ListCollectionsOperation(
 
             var op = call.Operation.Should().BeOfType<ListCollectionsOperation>().Subject;
             op.DatabaseNamespace.Should().Be(_subject.DatabaseNamespace);
-            op.Filter.Should().Be(filterDocument);
             op.NameOnly.Should().NotHaveValue();
+            if (usingOptions)
+            {
+                op.Filter.Should().Be(filterDocument);
+            }
+            else
+            {
+                op.Filter.Should().BeNull();
+            }
         }
 
         [Theory]