Revert insertedCount: 1 requirement for insertOne unified test operation

jyemin · jyemin · commit 939dab27985c · 2021-06-30T11:26:06.000-04:00
JAVA-4211
diff --git a/driver-core/src/test/resources/unified-test-format/crud/insertOne-dots_and_dollars.json b/driver-core/src/test/resources/unified-test-format/crud/insertOne-dots_and_dollars.json
@@ -63,7 +63,6 @@
             }
           },
           "expectResult": {
-            "insertedCount": 1,
             "insertedId": {
               "$$unsetOrMatches": 1
             }
@@ -166,7 +165,6 @@
             }
           },
           "expectResult": {
-            "insertedCount": 1,
             "insertedId": {
               "$$unsetOrMatches": 1
             }
@@ -221,7 +219,6 @@
             }
           },
           "expectResult": {
-            "insertedCount": 1,
             "insertedId": {
               "$$unsetOrMatches": 1
             }
@@ -280,7 +277,6 @@
             }
           },
           "expectResult": {
-            "insertedCount": 1,
             "insertedId": {
               "$$unsetOrMatches": 1
             }
@@ -390,7 +386,6 @@
             }
           },
           "expectResult": {
-            "insertedCount": 1,
             "insertedId": {
               "$$unsetOrMatches": {
                 "a.b": 1
@@ -501,7 +496,6 @@
             }
           },
           "expectResult": {
-            "insertedCount": 1,
             "insertedId": {
               "$$unsetOrMatches": 1
             }
diff --git a/driver-sync/src/test/functional/com/mongodb/client/unified/UnifiedCrudHelper.java b/driver-sync/src/test/functional/com/mongodb/client/unified/UnifiedCrudHelper.java
@@ -529,8 +529,7 @@ OperationResult executeInsertOne(final BsonDocument operation) {
 
     private BsonDocument toExpected(final InsertOneResult result) {
         if (result.wasAcknowledged()) {
-            return new BsonDocument("insertedId", result.getInsertedId())
-                    .append("insertedCount", new BsonInt32(1));
+            return new BsonDocument("insertedId", result.getInsertedId());
         } else {
             return new BsonDocument();
         }

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,6 @@`
`63`	`63`	`}`
`64`	`64`	`},`
`65`	`65`	`"expectResult": {`
`66`		`- "insertedCount": 1,`
`67`	`66`	`"insertedId": {`
`68`	`67`	`"$$unsetOrMatches": 1`
`69`	`68`	`}`
`@@ -166,7 +165,6 @@`
`166`	`165`	`}`
`167`	`166`	`},`
`168`	`167`	`"expectResult": {`
`169`		`- "insertedCount": 1,`
`170`	`168`	`"insertedId": {`
`171`	`169`	`"$$unsetOrMatches": 1`
`172`	`170`	`}`
`@@ -221,7 +219,6 @@`
`221`	`219`	`}`
`222`	`220`	`},`
`223`	`221`	`"expectResult": {`
`224`		`- "insertedCount": 1,`
`225`	`222`	`"insertedId": {`
`226`	`223`	`"$$unsetOrMatches": 1`
`227`	`224`	`}`
`@@ -280,7 +277,6 @@`
`280`	`277`	`}`
`281`	`278`	`},`
`282`	`279`	`"expectResult": {`
`283`		`- "insertedCount": 1,`
`284`	`280`	`"insertedId": {`
`285`	`281`	`"$$unsetOrMatches": 1`
`286`	`282`	`}`
`@@ -390,7 +386,6 @@`
`390`	`386`	`}`
`391`	`387`	`},`
`392`	`388`	`"expectResult": {`
`393`		`- "insertedCount": 1,`
`394`	`389`	`"insertedId": {`
`395`	`390`	`"$$unsetOrMatches": {`
`396`	`391`	`"a.b": 1`
`@@ -501,7 +496,6 @@`
`501`	`496`	`}`
`502`	`497`	`},`
`503`	`498`	`"expectResult": {`
`504`		`- "insertedCount": 1,`
`505`	`499`	`"insertedId": {`
`506`	`500`	`"$$unsetOrMatches": 1`
`507`	`501`	`}`
Original file line number	Diff line number	Diff line change
`@@ -529,8 +529,7 @@ OperationResult executeInsertOne(final BsonDocument operation) {`
`529`	`529`
`530`	`530`	`private BsonDocument toExpected(final InsertOneResult result) {`
`531`	`531`	`if (result.wasAcknowledged()) {`
`532`		`- return new BsonDocument("insertedId", result.getInsertedId())`
`533`		`- .append("insertedCount", new BsonInt32(1));`
	`532`	`+ return new BsonDocument("insertedId", result.getInsertedId());`
`534`	`533`	`} else {`
`535`	`534`	`return new BsonDocument();`
`536`	`535`	`}`