reseting mail address

[hocnguyen]

概要(Summary)

• Fixed #161

動作確認手順(Step for Confirmation)

• Run the unit test.

• Confirm operation: ①ダッシュボードでユーザー名が"YamadaTarou"とパスワードが"password"で会員を登録する。 ②以下のソースコードを実行する。

  var user1 = new ncmb.User({ userName: "YamadaTarou", password: "password" });
  
  // YamadaTarouとpasswordでログインする
  ncmb.User.login(user1)
  .then(function (data) {
    alert(JSON.stringify(“User1: Login successfully: " + data.toString(), null, 4));
  
    // test1@example.comへメールアドレスを変更する
    user1.set("mailAddress", “test1@example.com")
      .update(function (err, data) {
        if (err) {
          alert(JSON.stringify(“User1: Update mail test1@example.com error: “ + err, null, 4));
        } else {
          alert(JSON.stringify(“User1: Update mail test1@example.com successfully: " + data.toString(), null, 4));
  
          // 同様ユーザー名とパスワードで別変数で作成してログインする
          var user2 = new ncmb.User({ userName: "YamadaTarou", password: "password" });
          ncmb.User.login(user2)
            .then(function (data) {
              alert(JSON.stringify(“User2: Login successfully: " + data.toString(), null, 4));
  
              // test2@example.comへまたメールアドレスを変更する
              user2.set("mailAddress", "test2@example.com"")
                .update(function (err, data) {
                  if (err) {
                    alert(JSON.stringify(“User2: Update mail test2@example.com error: “ + err, null, 4));
                  } else {
                    alert(JSON.stringify(“User2: Update mail test2@example.com successfully: " + data.toString(), null, 4));
  
                    // user1はメールアドレス以外別フィールドを変更する
                    user1.set("phoneNumber", "00000")
                      .update(function (err, data) {
                        if (err) {
                          alert(JSON.stringify(“User1: Update phone 00000 error: “ + err, null, 4));
                        } else {
                          alert(JSON.stringify(“User1: Update phone 00000 successfully: " + data.toString(), null, 4));
  
                          // メールアドレスが再度変更されたか（上書きされたか）確認する
                          var user3 = new ncmb.User({ userName: "YamadaTarou", password: "password" });
                          ncmb.User.login(user3)
                            .then(function (data) {
                              alert(JSON.stringify(“User3:  User login successfully. MailAddress=“ + data.get("mailAddress"), null, 4));
                            })
                            .catch(function (err) {
                              alert(JSON.stringify(“User3: Login error" , null, 4));
                            });
                        }
                      });
                  }
                });
            })
            .catch(function (err) {
              alert(JSON.stringify(“User2: Login error" , null, 4));
            });
        }
      });
  })
  .catch(function (err) {
    alert(JSON.stringify(“User1: Login error" , null, 4));
  });

[hocnguyen]

@ykokw すみません、 I separate from fix/logout_expired_token branch to this branch for issue 161. Please help me review. Thanks

[ykokw]

@hocnguyen Thank you for creating p-r ! I think that sdk should reset _mailAddressChanged flag after both singup and update.

And I can't find test code to confirm this fixing. Could you add test code or tell me procedure to confirm.

[ykokw]

自動テスト側は修正がないので、動作確認手順としては不足です。

• テストコードを追加する
• 動作確認手順を追記する

のどちらかを対応お願いいたします。