fix: hide url attribute when file upload failed

see also leancloud/javascript-sdk#640

Author: weakish

leancloud/file_.py

@@ -36,6 +36,7 @@ def __init__(self, name="", data=None, mime_type=None):
         self.key = None
         self.id = None
         self._url = None
+        self._successful_url = None
         self._acl = None
         self.current_user = leancloud.User.get_current()
         self.timeout = 30
@@ -137,7 +138,7 @@ def name(self):
 
     @property
     def url(self):
-        return self._url
+        return self._successful_url
 
     @property
     def mime_type(self):
@@ -226,6 +227,7 @@ def _save_external(self):
         response = client.post("/files".format(self._name), data)
         content = response.json()
         self.id = content["objectId"]
+        self._successful_url = self._url
 
     def _save_to_qcloud(self, token, upload_url):
         headers = {
@@ -286,6 +288,7 @@ def _update_data(self, server_data):
             self._name = server_data.get("name")
         if "url" in server_data:
             self._url = server_data.get("url")
+            self._successful_url = self._url
         if "mime_type" in server_data:
             self._mime_type = server_data["mime_type"]
         if "metaData" in server_data:

tests/test_file.py

@@ -67,7 +67,8 @@ def test_create_with_url():  # type: () -> None
         u"http://i1.wp.com/leancloud.cn/images/static/default-avatar.png",
         meta_data={},
     )
-    assert f.url == "http://i1.wp.com/leancloud.cn/images/static/default-avatar.png"
+    assert f._url == "http://i1.wp.com/leancloud.cn/images/static/default-avatar.png"
+    assert f.url is None
 
 
 def test_create_without_data():  # type: () -> None