SERVER-8046 Cleanup Element::writeArrayTo and add a test for the new implementation

1 files changed, 14 insertions, 3 deletions

diff --git a/src/mongo/bson/mutable/mutable_bson_test.cpp b/src/mongo/bson/mutable/mutable_bson_test.cpp
index 57a8eeb64ce..af8f3224fc1 100644
--- a/src/mongo/bson/mutable/mutable_bson_test.cpp
+++ b/src/mongo/bson/mutable/mutable_bson_test.cpp
@@ -1407,14 +1407,25 @@ namespace {
     TEST(Document, toStringEphemeralObject) {
         mmb::Document doc;
         mmb::Element e = doc.makeElementObject("foo");
-        doc.root().pushBack(e);
-        e.appendDouble("d", 1.0);
-        e.appendString("s", "str");
+        ASSERT_OK(doc.root().pushBack(e));
+        ASSERT_OK(e.appendDouble("d", 1.0));
+        ASSERT_OK(e.appendString("s", "str"));
         ASSERT_EQUALS(
             mongo::fromjson("{ foo: { d : 1.0, s : 'str' } }").firstElement().toString(),
             e.toString());
     }
 
+    TEST(Document, toStringEphemeralArray) {
+        mmb::Document doc;
+        mmb::Element e = doc.makeElementArray("foo");
+        ASSERT_OK(doc.root().pushBack(e));
+        ASSERT_OK(e.appendDouble(mongo::StringData(), 1.0));
+        ASSERT_OK(e.appendString(mongo::StringData(), "str"));
+        ASSERT_EQUALS(
+            mongo::fromjson("{ foo: [ 1.0, 'str' ] }").firstElement().toString(),
+            e.toString());
+    }
+
     TEST(Document, ElementCloningToDifferentDocument) {
 
         const char initial[] = "{ a : 1, b : [ 1, 2, 3 ], c : { 'c' : 'c' }, d : [ 4, 5, 6 ] }";