index on unique-indexes: c454180 Revert "Log objects rather than JSON stringified objects (parse-community#1922)"

Author: drew-gross

spec/ParseAPI.spec.js

@@ -81,44 +81,51 @@ describe('miscellaneous', function() {
     .catch(done);
   });
 
-  it('ensure that email is uniquely indexed', done => {
-    let numCreated = 0;
-    let numFailed = 0;
-
-    let user1 = new Parse.User();
-    user1.setPassword('asdf');
-    user1.setUsername('u1');
-    user1.setEmail('[email protected]');
-    let p1 = user1.signUp();
-    p1.then(user => {
-      numCreated++;
-      expect(numCreated).toEqual(1);
-    }, error => {
-      numFailed++;
-      expect(numFailed).toEqual(1);
-      expect(error.code).toEqual(Parse.Error.EMAIL_TAKEN);
-    });
+  fit('ensure that email is uniquely indexed', done => {
+    DatabaseAdapter._indexBuildsCompleted('test')
+    .then(() => {
+      let numCreated = 0;
+      let numFailed = 0;
+
+      let user1 = new Parse.User();
+      user1.setPassword('asdf');
+      user1.setUsername('u1');
+      user1.setEmail('[email protected]');
+      let p1 = user1.signUp();
+      p1.then(user => {
+        numCreated++;
+        expect(numCreated).toEqual(1);
+      }, error => {
+        numFailed++;
+        console.log(error);
+        expect(numFailed).toEqual(1);
+        expect(error.code).toEqual(Parse.Error.EMAIL_TAKEN);
+      });
 
-    let user2 = new Parse.User();
-    user2.setPassword('asdf');
-    user2.setUsername('u2');
-    user2.setEmail('[email protected]');
-    let p2 = user2.signUp();
-    p2.then(user => {
-      numCreated++;
-      expect(numCreated).toEqual(1);
-    }, error => {
-      numFailed++;
-      expect(numFailed).toEqual(1);
-      expect(error.code).toEqual(Parse.Error.EMAIL_TAKEN);
-    });
+      let user2 = new Parse.User();
+      user2.setPassword('asdf');
+      user2.setUsername('u2');
+      user2.setEmail('[email protected]');
+      let p2 = user2.signUp();
+      p2.then(user => {
+        numCreated++;
+        expect(numCreated).toEqual(1);
+      }, error => {
+        numFailed++;
+        console.log(error);
+        expect(numFailed).toEqual(1);
+        expect(error.code).toEqual(Parse.Error.EMAIL_TAKEN);
+      });
 
-    Parse.Promise.when([p1, p2])
-    .then(() => {
-      fail('one of the users should not have been created');
-      done();
-    })
-    .catch(done);
+      Parse.Promise.when([p1, p2])
+      .then(() => {
+        fail('one of the users should not have been created');
+        done();
+      })
+      .catch(error => {
+        fail('index build failed')
+        done();
+      });
   });
 
   it('ensure that if people already have duplicate users, they can still sign up new users', done => {

src/RestWrite.js

@@ -388,11 +388,15 @@ RestWrite.prototype.transformUser = function() {
       if (results.length > 0) {
         throw new Parse.Error(Parse.Error.EMAIL_TAKEN, 'Account already exists for this email address.');
       }
-      // We updated the email, send a new validation
-      this.storage['sendVerificationEmail'] = true;
-      this.config.userController.setEmailVerifyToken(this.data);
+      return;
     });
   })
+  .then(() => {
+    // We updated the email, send a new validation
+    this.storage['sendVerificationEmail'] = true;
+    this.config.userController.setEmailVerifyToken(this.data);
+    return;
+  })
 };
 
 RestWrite.prototype.createSessionTokenIfNeeded = function() {