字符串使用单引号、使用 Promise

Author: jysperm

src/cloudfunction.js

@@ -31,7 +31,7 @@ module.exports = function(AV) {
      * of the function.
      */
     run: function(name, data, options) {
-      var request = AV._request("functions", name, null, 'POST',
+      var request = AV._request('functions', name, null, 'POST',
                                    AV._encode(data, null, true), options && options.sessionToken);
 
       return request.then(function(resp) {

src/object.js

@@ -791,7 +791,7 @@ module.exports = function(AV) {
       }
 
       var self = this;
-      var request = AV._request("classes", this.className, this.id, 'GET',
+      var request = AV._request('classes', this.className, this.id, 'GET',
                                 fetchOptions, options.sessionToken);
       return request.then(function(response) {
         self._finishFetch(self.parse(response), true);
@@ -977,7 +977,7 @@ module.exports = function(AV) {
       }
 
       var request =
-          AV._request("classes", this.className, this.id, 'DELETE', null, options.sessionToken);
+          AV._request('classes', this.className, this.id, 'DELETE', null, options.sessionToken);
       return request.then(function() {
         if (options.wait) {
           triggerDestroy();
@@ -1247,7 +1247,7 @@ module.exports = function(AV) {
           }
       });
       var request =
-          AV._request("classes", className, id, 'DELETE', null, options.sessionToken);
+          AV._request('classes', className, id, 'DELETE', null, options.sessionToken);
       return request._thenRunCallbacks(options);
    };
 

src/query.js

@@ -155,7 +155,7 @@ module.exports = function(AV) {
       options = pvalues;
     }
 
-    var request = AV._request("cloudQuery", null, null, 'GET', params, options && options.sessionToken);
+    var request = AV._request('cloudQuery', null, null, 'GET', params, options && options.sessionToken);
     return request.then(function(response) {
       //query to process results.
       var query = new AV.Query(response.className);
@@ -252,7 +252,7 @@ module.exports = function(AV) {
       return obj;
     },
     _createRequest: function(params, options){
-      return AV._request("classes", this.className, null, "GET",
+      return AV._request('classes', this.className, null, "GET",
                                    params || this.toJSON(), options && options.sessionToken);
     },
 

src/search.js

@@ -125,7 +125,7 @@ module.exports = function(AV) {
      _highlights: null,
      _sortBuilder: null,
     _createRequest: function(params, options){
-      return AV._request("search/select", null, null, "GET",
+      return AV._request('search/select', null, null, 'GET',
                                    params || this.toJSON(), options && options.sessionToken);
     },
 

src/status.js

@@ -58,7 +58,7 @@ module.exports = function(AV) {
     destroy: function(options){
       if(!this.id)
         return AV.Promise.error('The status id is not exists.')._thenRunCallbacks(options);
-      var request = AV._request("statuses", null, this.id, 'DELETE', options && options.sessionToken);
+      var request = AV._request('statuses', null, this.id, 'DELETE', options && options.sessionToken);
       return request._thenRunCallbacks(options);
     },
     /**
@@ -294,7 +294,7 @@ module.exports = function(AV) {
       return new AV.Status();
     },
     _createRequest: function(params, options){
-      return AV._request("subscribe/statuses", null, null, "GET",
+      return AV._request('subscribe/statuses', null, null, 'GET',
                                    params || this.toJSON(), options && options.sessionToken);
     },
 

test/user.js

@@ -299,16 +299,14 @@ describe("User", function() {
     it('User#signUp', function() {
       user = new AV.User();
 
-      user.set("username", username);
-      user.set("password", password);
-      user.set("email", email);
-
-      return user.signUp(null, {
-        success: function(user) {
-          expect(user._isCurrentUser).to.be.equal(false);
-          expect(AV.User._currentUser).to.be.equal(null);
-          expect(user._sessionToken).to.be.ok();
-        }
+      user.set('username', username);
+      user.set('password', password);
+      user.set('email', email);
+
+      return user.signUp().then(function(user) {
+        expect(user._isCurrentUser).to.be.equal(false);
+        expect(AV.User._currentUser).to.be.equal(null);
+        expect(user._sessionToken).to.be.ok();
       });
     });
 
@@ -326,27 +324,21 @@ describe("User", function() {
       });
     });
 
-    it('User#save without token', function(done) {
-      user.save({username: username + 'changed'}, {
-        success: function() {
-          done(new Error('Should not success'));
-        },
-        error: function(err) {
-          expect(err.code).to.be.equal(206);
-          done();
-        }
+    it('User#save without token', function() {
+      return user.save({username: username + 'changed'}).then(function() {
+        throw new Error('Should not success');
+      }, function(err) {
+        expect(err.code).to.be.equal(206);
       });
     });
 
-    it('User#save with token', function(done) {
-      user.save({username: username + 'changed'}, {
-        sessionToken: user.getSessionToken(),
-        success: function() {
-          done();
-        },
-        error: function(err) {
-          done(err);
-        }
+    it('User#save with token', function() {
+      return user.save({
+        username: username + 'changed'
+      }, {sessionToken: user.getSessionToken()}).then(function() {
+        user.fetch().then(function() {
+          expect(user.username).to.be.equal(username + 'changed');
+        });
       });
     });