Merge pull request #486 from weakish/sms-test

tests(sms): improve request_sms_code & change_phone_number

Author: weakish

tests/test_engine.py

@@ -531,22 +531,26 @@ def test_client():  # type: () -> None
 
 
 def test_request_sms_code():  # type: () -> None
-    if leancloud.client.REGION == "US":
-        return
     try:
         # numbers come from http://www.z-sms.com/
-        cloud.request_sms_code("+8617180654515")
-        cloud.request_sms_code("17180654515")
-        cloud.request_sms_code("17180654515", idd="+86")
-        cloud.request_sms_code("+8617180654515", idd="+86")
-        cloud.request_sms_code("+8617180654515", idd="+44")  # +8617180654515
+        cloud.request_sms_code("+8617180655340")
+        time.sleep(60)
         cloud.request_sms_code("+447365753569")
+        time.sleep(60)
+        cloud.request_sms_code("17180655340")
+        time.sleep(60)
         cloud.request_sms_code("7365753569", idd="+44")
+        time.sleep(60)
+        cloud.request_sms_code("17180655340", idd="+86")
+        time.sleep(60)
         cloud.request_sms_code("+447365753569", idd="+44")
+        time.sleep(60)
+        cloud.request_sms_code("+8617180655340", idd="+44")  # +8617180655340
+        time.sleep(60)
         cloud.request_sms_code("+447365753569", idd="+86")  # +447365753569
     except LeanCloudError as e:
         # 短信发送过于频繁或者欠费或者关闭短信功能
-        if e.code in (601, 160, 119):
+        if e.code in (601, 605, 160, 119):
             pass
         else:
             raise e

tests/test_user.py

@@ -274,28 +274,37 @@ def test_request_mobile_phone_verify():  # type: () -> None
             raise e
 
 
-@with_setup(get_setup_func())
+@with_setup(only_init)
 def test_request_change_phone_number():  # type: () -> None
     user1 = User()
-    user1.set("username", "user1_name")
+    user1.set("username", "py_test_change_phone")
     user1.set("password", "password")
-    user1.login()
+    user1.sign_up()
     try:
-        User.request_change_phone_number("18611111112")
+        # phone number is from http://www.z-sms.com
+        User.request_change_phone_number("+8617180655340")
     except LeanCloudError as e:
         if e.code not in (119, 213, 601):
             raise e
     finally:
         user1.logout()
 
 
-@with_setup(get_setup_func())
+@with_setup(only_init)
 def test_change_phone_number():  # type: () -> None
     try:
-        User.change_phone_number("123456", "18611111112")
+        # phone number is from http://www.z-sms.com
+        User.change_phone_number("196784", "+8617180655340")
     except LeanCloudError as e:
         if e.code != 603:
             raise e
+    else:
+        user1 = User()
+        user1.set("username", "py_test_change_phone")
+        user1.set("password", "password")
+        user1.login()
+        assert user1.get_mobile_phone_number() == "+8617180655340"
+        user1.destroy()
 
 
 @with_setup(get_setup_func())