CSHARP-1688: Remove WriteConcern from CreateIndexesUsingInsertOperation.

Author: rstam

src/MongoDB.Driver.Core/Core/Operations/CreateIndexesOperation.cs

@@ -141,10 +141,7 @@ internal IWriteOperation<BsonDocument> CreateOperation(SemanticVersion serverVer
             }
             else
             {
-                return new CreateIndexesUsingInsertOperation(_collectionNamespace, _requests, _messageEncoderSettings)
-                {
-                    WriteConcern = _writeConcern
-                };
+                return new CreateIndexesUsingInsertOperation(_collectionNamespace, _requests, _messageEncoderSettings);
             }
         }
    }

src/MongoDB.Driver.Core/Core/Operations/CreateIndexesUsingInsertOperation.cs

@@ -37,7 +37,6 @@ public class CreateIndexesUsingInsertOperation : IWriteOperation<BsonDocument>
         private readonly CollectionNamespace _collectionNamespace;
         private readonly MessageEncoderSettings _messageEncoderSettings;
         private readonly IEnumerable<CreateIndexRequest> _requests;
-        private WriteConcern _writeConcern = WriteConcern.Acknowledged;
 
         // constructors
         /// <summary>
@@ -90,18 +89,6 @@ public IEnumerable<CreateIndexRequest> Requests
             get { return _requests; }
         }
 
-        /// <summary>
-        /// Gets or sets the write concern.
-        /// </summary>
-        /// <value>
-        /// The write concern.
-        /// </value>
-        public WriteConcern WriteConcern
-        {
-            get { return _writeConcern; }
-            set { _writeConcern = Ensure.IsNotNull(value, nameof(value)); }
-        }
-
         // public methods
         /// <inheritdoc/>
         public BsonDocument Execute(IWriteBinding binding, CancellationToken cancellationToken)
@@ -150,10 +137,7 @@ internal InsertOpcodeOperation<BsonDocument> CreateOperation(SemanticVersion ser
                 systemIndexesCollection,
                 documentSource,
                 BsonDocumentSerializer.Instance,
-                _messageEncoderSettings)
-            {
-                WriteConcern = _writeConcern
-            };
+                _messageEncoderSettings);
         }
     }
 }

tests/MongoDB.Driver.Core.Tests/Core/Operations/CreateIndexesOperationTests.cs

@@ -99,7 +99,6 @@ public void CreateOperation_should_return_expected_result(
                 operation.CollectionNamespace.Should().BeSameAs(_collectionNamespace);
                 operation.MessageEncoderSettings.Should().BeSameAs(_messageEncoderSettings);
                 operation.Requests.Should().Equal(requests);
-                operation.WriteConcern.Should().BeSameAs(writeConcern);
             }
         }
 

tests/MongoDB.Driver.Core.Tests/Core/Operations/CreateIndexesUsingInsertOperationTests.cs

@@ -38,8 +38,6 @@ public void constructor_should_initialize_subject()
             subject.CollectionNamespace.Should().BeSameAs(_collectionNamespace);
             subject.Requests.Should().Equal(requests);
             subject.MessageEncoderSettings.Should().BeSameAs(_messageEncoderSettings);
-
-            subject.WriteConcern.Should().BeSameAs(WriteConcern.Acknowledged);
         }
 
         [Fact]
@@ -75,10 +73,7 @@ public void CreateOperation_should_return_expected_result()
             var request = new CreateIndexRequest(new BsonDocument("x", 1));
             var requests = new[] { request };
             var writeConcern = new WriteConcern(1);
-            var subject = new CreateIndexesUsingInsertOperation(_collectionNamespace, requests, _messageEncoderSettings)
-            {
-                WriteConcern = writeConcern
-            };
+            var subject = new CreateIndexesUsingInsertOperation(_collectionNamespace, requests, _messageEncoderSettings);
 
             var result = subject.CreateOperation(null, request);
 
@@ -91,7 +86,6 @@ public void CreateOperation_should_return_expected_result()
             result.MaxMessageSize.Should().NotHaveValue();
             result.MessageEncoderSettings.Should().BeSameAs(_messageEncoderSettings);
             result.Serializer.Should().BeSameAs(BsonDocumentSerializer.Instance);
-            result.WriteConcern.Should().BeSameAs(writeConcern);
         }
 
         [SkippableTheory]
@@ -205,51 +199,6 @@ public void Execute_should_work_when_unique_is_true(
             index["unique"].ToBoolean().Should().BeTrue();
         }
 
-        [SkippableTheory]
-        [ParameterAttributeData]
-        public void Execute_should_throw_when_a_write_concern_error_occurs(
-            [Values(false, true)]
-            bool async)
-        {
-            RequireServer.Check().Supports(Feature.CreateIndexesCommand, Feature.CommandsThatWriteAcceptWriteConcern).ClusterType(ClusterType.ReplicaSet);
-            DropCollection();
-            var requests = new[] { new CreateIndexRequest(new BsonDocument("x", 1)) };
-            var subject = new CreateIndexesUsingInsertOperation(_collectionNamespace, requests, _messageEncoderSettings)
-            {
-                WriteConcern = new WriteConcern(9)
-            };
-
-            var exception = Record.Exception(() => ExecuteOperation(subject, async));
-
-            exception.Should().BeOfType<MongoWriteConcernException>();
-        }
-
-        [Theory]
-        [ParameterAttributeData]
-        public void WriteConcern_get_and_set_should_work(
-            [Values(1, 2)]
-            int w)
-        {
-            var subject = new CreateIndexesUsingInsertOperation(_collectionNamespace, Enumerable.Empty<CreateIndexRequest>(), _messageEncoderSettings);
-            var value = new WriteConcern(w);
-
-            subject.WriteConcern = value;
-            var result = subject.WriteConcern;
-
-            result.Should().BeSameAs(value);
-        }
-
-        [Fact]
-        public void WriteConcern_set_should_throw_when_value_is_null()
-        {
-            var subject = new CreateIndexesUsingInsertOperation(_collectionNamespace, Enumerable.Empty<CreateIndexRequest>(), _messageEncoderSettings);
-
-            var exception = Record.Exception(() => { subject.WriteConcern = null; });
-
-            var argumentNullException = exception.Should().BeOfType<ArgumentNullException>().Subject;
-            argumentNullException.ParamName.Should().Be("value");
-        }
-
         private List<BsonDocument> ListIndexes()
         {
             var listIndexesOperation = new ListIndexesOperation(_collectionNamespace, _messageEncoderSettings);