x/python-glareclient
Code Issues Proposed changes

Merge "Fix blob upload output"

Browse Source
This commit is contained in:
Jenkins 2016-09-27 16:05:12 +00:00 committed by Gerrit Code Review
commit 5cf1030ca8
3 changed files with 55 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
glareclient/osc/v1/blobs.py
View File

@ -90,13 +90,7 @@ class UploadBlob(command.ShowOne):
data = client.artifacts.get(parsed_args.id, data = client.artifacts.get(parsed_args.id,
type_name=parsed_args.type_name) type_name=parsed_args.type_name)
size = data[parsed_args.blob_property].pop('size', None) data_to_display = {'blob_property': parsed_args.blob_property}
data_to_display = {'blob_property': parsed_args.blob_property,
'id': parsed_args.id,
'name': data['name'],
'version': data['version'],
'status': data['status'],
'size': utils.make_size_human_readable(size)}
data_to_display.update(data[parsed_args.blob_property]) data_to_display.update(data[parsed_args.blob_property])
return self.dict2columns(data_to_display) return self.dict2columns(data_to_display)

21
glareclient/tests/unit/osc/v1/fakes.py
View File

@ -20,6 +20,17 @@ from glareclient.common import utils as g_utils
import mock import mock
from osc_lib.tests import utils from osc_lib.tests import utils
blob_fixture = {
"status": "active",
"url": "fake_url",
"md5": "35d83e8eedfbdb87ff97d1f2761f8ebf",
"sha1": "942854360eeec1335537702399c5aed940401602",
"sha256": "d8a7834fc6652f316322d80196f6dcf2"
"94417030e37c15412e4deb7a67a367dd",
"external": False,
"content_type": "application/octet-stream",
"size": 594}
def mock_list(*args, **kwargs): def mock_list(*args, **kwargs):
return [{'id': 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', return [{'id': 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba',
@ -43,11 +54,11 @@ def mock_get(*args, **kwargs):
'owner': 'f649c77999e449e89627024f71b76603', 'owner': 'f649c77999e449e89627024f71b76603',
'visibility': 'private', 'visibility': 'private',
'status': 'active', 'status': 'active',
'blob': {'size': 1}, 'blob': blob_fixture,
'image': {'size': 1}, 'image': blob_fixture,
'package': {'size': 1}, 'package': blob_fixture,
'template': {'size': 1}, 'template': blob_fixture,
'environment': {'size': 1}} 'environment': blob_fixture}
def mock_g_servs(*args, **kwargs): def mock_g_servs(*args, **kwargs):

52
glareclient/tests/unit/osc/v1/test_blob.py
View File

@ -37,12 +37,16 @@ class TestUploadBlob(TestBlobs):
# Command to test # Command to test
self.cmd = osc_blob.UploadBlob(self.app, None) self.cmd = osc_blob.UploadBlob(self.app, None)
self.COLUMNS = ('blob_property', 'id', 'name', self.COLUMNS = ('blob_property', 'content_type', 'external',
'size', 'status', 'version') 'md5', 'sha1', 'sha256', 'size', 'status', 'url')
def test_upload_images(self): def test_upload_images(self):
exp_data = ('image', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', exp_data = ('image', 'application/octet-stream', False,
'art1', '1B', 'active', '0.0.0') '35d83e8eedfbdb87ff97d1f2761f8ebf',
'942854360eeec1335537702399c5aed940401602',
'd8a7834fc6652f316322d80196f6dcf2'
'94417030e37c15412e4deb7a67a367dd',
594, 'active', 'fake_url')
arglist = ['images', arglist = ['images',
'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba',
'--file', '/path/to/file'] '--file', '/path/to/file']
@ -54,8 +58,12 @@ class TestUploadBlob(TestBlobs):
self.assertEqual(exp_data, data) self.assertEqual(exp_data, data)
def test_upload_tosca_template(self): def test_upload_tosca_template(self):
exp_data = ('template', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', exp_data = ('template', 'application/octet-stream', False,
'art1', '1B', 'active', '0.0.0') '35d83e8eedfbdb87ff97d1f2761f8ebf',
'942854360eeec1335537702399c5aed940401602',
'd8a7834fc6652f316322d80196f6dcf2'
'94417030e37c15412e4deb7a67a367dd',
594, 'active', 'fake_url')
arglist = ['tosca_templates', arglist = ['tosca_templates',
'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba',
'--file', '/path/to/file'] '--file', '/path/to/file']
@ -67,8 +75,12 @@ class TestUploadBlob(TestBlobs):
self.assertEqual(exp_data, data) self.assertEqual(exp_data, data)
def test_upload_heat_template(self): def test_upload_heat_template(self):
exp_data = ('template', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', exp_data = ('template', 'application/octet-stream', False,
'art1', '1B', 'active', '0.0.0') '35d83e8eedfbdb87ff97d1f2761f8ebf',
'942854360eeec1335537702399c5aed940401602',
'd8a7834fc6652f316322d80196f6dcf2'
'94417030e37c15412e4deb7a67a367dd',
594, 'active', 'fake_url')
arglist = ['heat_templates', arglist = ['heat_templates',
'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba',
'--file', '/path/to/file'] '--file', '/path/to/file']
@ -80,8 +92,12 @@ class TestUploadBlob(TestBlobs):
self.assertEqual(exp_data, data) self.assertEqual(exp_data, data)
def test_upload_environment(self): def test_upload_environment(self):
exp_data = ('environment', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', exp_data = ('environment', 'application/octet-stream', False,
'art1', '1B', 'active', '0.0.0') '35d83e8eedfbdb87ff97d1f2761f8ebf',
'942854360eeec1335537702399c5aed940401602',
'd8a7834fc6652f316322d80196f6dcf2'
'94417030e37c15412e4deb7a67a367dd',
594, 'active', 'fake_url')
arglist = ['heat_environments', arglist = ['heat_environments',
'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba',
'--file', '/path/to/file'] '--file', '/path/to/file']
@ -93,8 +109,12 @@ class TestUploadBlob(TestBlobs):
self.assertEqual(exp_data, data) self.assertEqual(exp_data, data)
def test_upload_package(self): def test_upload_package(self):
exp_data = ('package', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', exp_data = ('package', 'application/octet-stream', False,
'art1', '1B', 'active', '0.0.0') '35d83e8eedfbdb87ff97d1f2761f8ebf',
'942854360eeec1335537702399c5aed940401602',
'd8a7834fc6652f316322d80196f6dcf2'
'94417030e37c15412e4deb7a67a367dd',
594, 'active', 'fake_url')
arglist = ['murano_packages', arglist = ['murano_packages',
'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba',
'--file', '/path/to/file'] '--file', '/path/to/file']
@ -116,8 +136,12 @@ class TestUploadBlob(TestBlobs):
self.cmd.take_action(parsed_args) self.cmd.take_action(parsed_args)
def test_upload_blob_with_blob_prop(self): def test_upload_blob_with_blob_prop(self):
exp_data = ('blob', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', exp_data = ('blob', 'application/octet-stream', False,
'art1', '1B', 'active', '0.0.0') '35d83e8eedfbdb87ff97d1f2761f8ebf',
'942854360eeec1335537702399c5aed940401602',
'd8a7834fc6652f316322d80196f6dcf2'
'94417030e37c15412e4deb7a67a367dd',
594, 'active', 'fake_url')
arglist = ['sample_artifact', arglist = ['sample_artifact',
'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba', 'fc15c365-d4f9-4b8b-a090-d9e230f1f6ba',
'--file', '/path/to/file', '--file', '/path/to/file',