commit e4b129f304a28db71a538af35afae0af91b02129
author Nikolai Vavilov <vvnicholas@gmail.com> Tue Jul 19 10:34:58 2016 +0300
committer Nikolai Vavilov <vvnicholas@gmail.com> Tue Jul 19 10:34:58 2016 +0300
tree 6e8b2cfcf82f18a70bc874c327fc33b711c84680
parent db1b2a06e3999ffd6e52719ec0216849aa515ba1

restore old behavior for toObject

js/message_test.js

diff --git a/js/message_test.js b/js/message_test.js
index c9eee17..b779143 100644
--- a/js/message_test.js
+++ b/js/message_test.js
@@ -145,7 +145,11 @@
         undefined, undefined, undefined, undefined]);
     var result = foo.toObject();
     assertObjectEquals({
-      aNestedMessage: {},
+      aString: undefined,
+      anOutOfOrderBool: undefined,
+      aNestedMessage: {
+        anInt: undefined
+      },
       // Note: JsPb converts undefined repeated fields to empty arrays.
       aRepeatedMessageList: [],
       aRepeatedStringList: []
@@ -180,7 +184,14 @@
     var response = new proto.jspb.test.DefaultValues();
 
     // Test toObject
-    var expectedObject = {};
+    var expectedObject = {
+      stringField: defaultString,
+      boolField: true,
+      intField: 11,
+      enumField: 13,
+      emptyField: '',
+      bytesField: 'bW9v'
+    };
     assertObjectEquals(expectedObject, response.toObject());
 
 
@@ -748,7 +759,12 @@
     assertObjectEquals({id: 'g1', someBoolList: [true, false]},
         groups[0].toObject());
     assertObjectEquals({
-      repeatedGroupList: [{id: 'g1', someBoolList: [true, false]}]
+      repeatedGroupList: [{id: 'g1', someBoolList: [true, false]}],
+      requiredGroup: {id: undefined},
+      optionalGroup: undefined,
+      requiredSimple: {aRepeatedStringList: [], aString: undefined},
+      optionalSimple: undefined,
+      id: undefined
     }, group.toObject());
     var group1 = new proto.jspb.test.TestGroup1();
     group1.setGroup(someGroup);