Added functions to translate between uuid and uri
This change is a subset of patch 294559 to help the reviews and decouple code. Added _is_uuid_valid to check is a uuid is valid. Added get_uri_from_uuid to translate a given resource UUID to a resource URI. Change-Id: Ida83b476e722f0e3c6f987cd5b893f8a8127c3b4
This commit is contained in:
parent
3d4bf19585
commit
6954368c94
49
oneview_client/tests/unit/test_utils.py
Normal file
49
oneview_client/tests/unit/test_utils.py
Normal file
@ -0,0 +1,49 @@
|
|||||||
|
# -*- encoding: utf-8 -*-
|
||||||
|
#
|
||||||
|
# (c) Copyright 2016 Hewlett Packard Enterprise Development LP
|
||||||
|
# Copyright 2016 Universidade Federal de Campina Grande
|
||||||
|
#
|
||||||
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
|
# you may not use this file except in compliance with the License.
|
||||||
|
# You may obtain a copy of the License at
|
||||||
|
#
|
||||||
|
# http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
#
|
||||||
|
# Unless required by applicable law or agreed to in writing, software
|
||||||
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
||||||
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
||||||
|
# See the License for the specific language governing permissions and
|
||||||
|
# limitations under the License.
|
||||||
|
|
||||||
|
import unittest
|
||||||
|
|
||||||
|
from oneview_client import utils
|
||||||
|
|
||||||
|
|
||||||
|
class UtilsTestCase(unittest.TestCase):
|
||||||
|
def test__is_uuid_valid_with_valid_uuid(self):
|
||||||
|
uuid = 'abcdef12-3456-789f-edcb-aabcdef12345'
|
||||||
|
self.assertTrue(utils._is_uuid_valid(uuid))
|
||||||
|
|
||||||
|
def test__is_uuid_valid_with_invalid_uuid(self):
|
||||||
|
uuid = 'abcdef12-3456-789f-edcb-aabcdef12345?'
|
||||||
|
self.assertFalse(utils._is_uuid_valid(uuid))
|
||||||
|
|
||||||
|
def test_get_uuid_from_uri(self):
|
||||||
|
expected_uuid = 'aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa'
|
||||||
|
uri = '/rest/resource/' + expected_uuid
|
||||||
|
self.assertEqual(expected_uuid, utils.get_uuid_from_uri(uri))
|
||||||
|
|
||||||
|
def test_get_uuid_from_uri_as_none(self):
|
||||||
|
self.assertEqual(None, utils.get_uuid_from_uri(None))
|
||||||
|
|
||||||
|
def test_get_uri_from_uuid_with_valid_uuid(self):
|
||||||
|
prefix = '/rest/resource/'
|
||||||
|
uuid = 'aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa'
|
||||||
|
expected_uri = prefix + uuid
|
||||||
|
self.assertEqual(expected_uri, utils.get_uri_from_uuid(prefix, uuid))
|
||||||
|
|
||||||
|
def test_get_uri_from_uuid_with_invalid_uuid(self):
|
||||||
|
prefix = '/rest/resource/'
|
||||||
|
uuid = 'aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa?'
|
||||||
|
self.assertEqual(None, utils.get_uri_from_uuid(prefix, uuid))
|
@ -13,7 +13,23 @@
|
|||||||
# See the License for the specific language governing permissions and
|
# See the License for the specific language governing permissions and
|
||||||
# limitations under the License.
|
# limitations under the License.
|
||||||
|
|
||||||
|
import re
|
||||||
|
|
||||||
|
UUID_PATTERN = "^[0-9a-fA-F]{8}-[0-9a-fA-F]{4}-[0-9a-fA-F]{4}-" +\
|
||||||
|
"[0-9a-fA-F]{4}-[0-9a-fA-F]{12}$"
|
||||||
|
|
||||||
|
|
||||||
|
def _is_uuid_valid(uuid):
|
||||||
|
if re.match(UUID_PATTERN, uuid):
|
||||||
|
return True
|
||||||
|
return False
|
||||||
|
|
||||||
|
|
||||||
def get_uuid_from_uri(uri):
|
def get_uuid_from_uri(uri):
|
||||||
if uri:
|
if uri:
|
||||||
return uri.split("/")[-1]
|
return uri.split("/")[-1]
|
||||||
|
|
||||||
|
|
||||||
|
def get_uri_from_uuid(resource_prefix, uuid):
|
||||||
|
if uuid and _is_uuid_valid(uuid):
|
||||||
|
return str(resource_prefix) + str(uuid)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user