Merge pull request #276 from jysperm/fix-fetch

fix(Object.fetch): options 默认为空对象、允许 include 选项为数组

Author: wangxiao

src/object.js

@@ -795,13 +795,17 @@ module.exports = function(AV) {
      *     completes.
      */
     fetch: function() {
-      var options = null;
+      var options = {};
       var fetchOptions = {};
       if(arguments.length === 1) {
         options = arguments[0];
       } else if(arguments.length === 2) {
         fetchOptions = arguments[0];
-        options = arguments[1];
+        options = arguments[1] || {};
+      }
+
+      if (fetchOptions && fetchOptions.include && fetchOptions.include.length > 0) {
+        fetchOptions.include = fetchOptions.include.join(',');
       }
 
       var self = this;

test/object.js

@@ -350,6 +350,19 @@ describe('Objects', function(){
       });
     });
 
+    it('should fetch include authors successfully', function() {
+      var myPost = new Post();
+      myPost.set('author1', new Person({name: '1'}));
+      myPost.set('author2', new Person({name: '2'}));
+      return myPost.save().then(function() {
+        myPost = AV.Object.createWithoutData('Post', myPost.id);
+        return myPost.fetch({include: ['author1', 'author2']}, {}).then(function() {
+          expect(myPost.get('author1').get('name')).to.be('1');
+          expect(myPost.get('author2').get('name')).to.be('2');
+        });
+      });
+    });
+
     /*
 
        it("it should fetch relation post",function(done){