Merge pull request #280 from jysperm/fix-fetch

fix(Object.fetch): fetchOptions.include.join is not a function

Author: wangxiao

src/object.js

@@ -804,7 +804,7 @@ module.exports = function(AV) {
         options = arguments[1] || {};
       }
 
-      if (fetchOptions && fetchOptions.include && fetchOptions.include.length > 0) {
+      if (fetchOptions && fetchOptions.include && _.isArray(fetchOptions.include)) {
         fetchOptions.include = fetchOptions.include.join(',');
       }
 

test/object.js

@@ -363,6 +363,19 @@ describe('Objects', function(){
       });
     });
 
+    it('should compatible with previous include option of fetch', 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){