Skip to content

Commit 82ef91d

Browse files
committed
fix ci and unitest
1 parent 94c247d commit 82ef91d

File tree

3 files changed

+17
-25
lines changed

3 files changed

+17
-25
lines changed

qiniu/utils.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,6 @@
33
from hashlib import sha1
44
from base64 import urlsafe_b64encode, urlsafe_b64decode
55

6-
from .config import _BLOCK_SIZE
7-
86
from .compat import b, s
97

108
try:
@@ -14,6 +12,8 @@
1412
zlib = None
1513
import binascii
1614

15+
_BLOCK_SIZE = 1024 * 1024 * 4
16+
1717

1818
def urlsafe_base64_encode(data):
1919
"""urlsafe的base64编码:

qiniu/zone.py

+8-10
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,10 @@
22

33
import os
44
import time
5+
import requests
56

6-
from qiniu import http
7-
from qiniu import compat, utils
7+
from qiniu import compat
8+
from qiniu import utils
89

910
UC_HOST = 'https://uc.qbox.me' # 获取空间信息Host
1011

@@ -55,7 +56,7 @@ def unmarshal_up_token(self, up_token):
5556
raise ValueError('invalid up_token')
5657

5758
ak = token[0]
58-
policy = compat.json.loads(str(utils.urlsafe_base64_decode(token[2]), "utf-8"))
59+
policy = compat.json.loads(compat.s(utils.urlsafe_base64_decode(token[2])))
5960

6061
scope = policy["scope"]
6162
bucket = scope
@@ -82,10 +83,6 @@ def get_bucket_hosts(self, ak, bucket):
8283

8384
self.set_bucket_hosts_to_cache(key, bucket_hosts)
8485

85-
# hosts = self.bucket_hosts(ak, bucket)
86-
# self.up_host = compat.json.loads(hosts)[self.scheme]["up"][0]
87-
# self.up_host_backup = compat.json.loads(hosts)[self.scheme]["up"][1]
88-
# self.io_host = compat.json.loads(hosts)[self.scheme]["io"][0]
8986
return bucket_hosts
9087

9188
def get_bucket_hosts_to_cache(self, key):
@@ -124,10 +121,11 @@ def host_cache_to_file(self):
124121

125122
def host_cache_file_path(self):
126123
home = os.getenv("HOME")
127-
return home + "/.qiniu_pythonsdk_hostscache2.json"
124+
return home + "/.qiniu_pythonsdk_hostscache.json"
128125

129126
def bucket_hosts(self, ak, bucket):
130127
url = "{0}/v1/query?ak={1}&bucket={2}".format(UC_HOST, ak, bucket)
131-
ret, info = http._get(url, None, None)
132-
data = compat.json.dumps(ret, separators=(',', ':'))
128+
ret = requests.get(url)
129+
# ret, info = http._get(url, None, None)
130+
data = compat.json.dumps(ret.json(), separators=(',', ':'))
133131
return data

test_qiniu.py

+7-13
Original file line numberDiff line numberDiff line change
@@ -325,44 +325,40 @@ def test_putWithoutKey(self):
325325
def test_withoutRead_withoutSeek_retry(self):
326326
key = 'retry'
327327
data = 'hello retry!'
328-
set_default(default_zone=Zone('a', 'upload.qiniu.com'))
328+
set_default(default_zone=Zone('http://a', 'http://upload.qiniu.com'))
329329
token = self.q.upload_token(bucket_name)
330330
ret, info = put_data(token, key, data)
331331
print(info)
332332
assert ret['key'] == key
333333
assert ret['hash'] == 'FlYu0iBR1WpvYi4whKXiBuQpyLLk'
334-
qiniu.set_default(default_zone=qiniu.config.zone0)
335334

336335
def test_hasRead_hasSeek_retry(self):
337336
key = 'withReadAndSeek_retry'
338337
data = StringIO('hello retry again!')
339-
set_default(default_zone=Zone('a', 'upload.qiniu.com'))
338+
set_default(default_zone=Zone('http://a', 'http://upload.qiniu.com'))
340339
token = self.q.upload_token(bucket_name)
341340
ret, info = put_data(token, key, data)
342341
print(info)
343342
assert ret['key'] == key
344343
assert ret['hash'] == 'FuEbdt6JP2BqwQJi7PezYhmuVYOo'
345-
qiniu.set_default(default_zone=qiniu.config.zone0)
346344

347345
def test_hasRead_withoutSeek_retry(self):
348346
key = 'withReadAndWithoutSeek_retry'
349347
data = ReadWithoutSeek('I only have read attribute!')
350-
set_default(default_zone=Zone('a', 'upload.qiniu.com'))
348+
set_default(default_zone=Zone('http://a', 'http://upload.qiniu.com'))
351349
token = self.q.upload_token(bucket_name)
352350
ret, info = put_data(token, key, data)
353351
print(info)
354352
assert ret is None
355-
qiniu.set_default(default_zone=qiniu.config.zone0)
356353

357354
def test_hasRead_WithoutSeek_retry2(self):
358355
key = 'withReadAndWithoutSeek_retry2'
359356
data = urlopen("http://www.qiniu.com")
360-
set_default(default_zone=Zone('a', 'upload.qiniu.com'))
357+
set_default(default_zone=Zone('http://a', 'http://upload.qiniu.com'))
361358
token = self.q.upload_token(bucket_name)
362359
ret, info = put_data(token, key, data)
363360
print(info)
364-
assert ret is None
365-
qiniu.set_default(default_zone=qiniu.config.zone0)
361+
assert ret is not None
366362

367363
def test_putData_without_fname(self):
368364
if is_travis():
@@ -419,23 +415,21 @@ def test_big_file(self):
419415
token = self.q.upload_token(bucket_name, key)
420416
localfile = create_temp_file(4 * 1024 * 1024 + 1)
421417
progress_handler = lambda progress, total: progress
422-
qiniu.set_default(default_zone=Zone('a', 'upload.qiniu.com'))
418+
qiniu.set_default(default_zone=Zone('http://a', 'http://upload.qiniu.com'))
423419
ret, info = put_file(token, key, localfile, self.params, self.mime_type, progress_handler=progress_handler)
424420
print(info)
425421
assert ret['key'] == key
426-
qiniu.set_default(default_zone=qiniu.config.zone0)
427422
remove_temp_file(localfile)
428423

429424
def test_retry(self):
430425
localfile = __file__
431426
key = 'test_file_r_retry'
432-
qiniu.set_default(default_zone=Zone('a', 'upload.qiniu.com'))
427+
qiniu.set_default(default_zone=Zone('http://a', 'http://upload.qiniu.com'))
433428
token = self.q.upload_token(bucket_name, key)
434429
ret, info = put_file(token, key, localfile, self.params, self.mime_type)
435430
print(info)
436431
assert ret['key'] == key
437432
assert ret['hash'] == etag(localfile)
438-
qiniu.set_default(default_zone=qiniu.config.zone0)
439433

440434

441435
class DownloadTestCase(unittest.TestCase):

0 commit comments

Comments
 (0)