]> review.fuel-infra Code Review - openstack-build/heat-build.git/commitdiff
Remove is_uuid() from HeatIdentifier class
authorZane Bitter <zbitter@redhat.com>
Fri, 30 Nov 2012 14:59:23 +0000 (15:59 +0100)
committerZane Bitter <zbitter@redhat.com>
Fri, 30 Nov 2012 16:05:13 +0000 (17:05 +0100)
We now use the equivalent functionality from openstack-common.

Change-Id: I915cdb92b89d3548fa03e1820d8b90b2aac4e44b
Signed-off-by: Zane Bitter <zbitter@redhat.com>
heat/engine/identifier.py
heat/tests/test_identifier.py

index 02174101a4b18391ca71b7496c243c5804340829..9b9ae94f54519664e519af53d131c90157db7624 100644 (file)
@@ -26,12 +26,6 @@ class HeatIdentifier(collections.Mapping):
     )
     path_re = re.compile(r'stacks/([^/]+)/([^/]+)(.*)')
 
-    HEX_ELEM = '[0-9A-Fa-f]'
-    UUID_PATTERN = '-'.join([HEX_ELEM + '{8}', HEX_ELEM + '{4}',
-                         HEX_ELEM + '{4}', HEX_ELEM + '{4}',
-                         HEX_ELEM + '{12}'])
-    uuid_re = re.compile(r'^' + UUID_PATTERN + '$')
-
     def __init__(self, tenant, stack_name, stack_id, path=''):
         '''
         Initialise a HeatIdentifier from a Tenant ID, Stack name, Stack ID
@@ -48,10 +42,6 @@ class HeatIdentifier(collections.Mapping):
             self.PATH: path,
         }
 
-    @classmethod
-    def is_uuid(cls, uuid):
-        return HeatIdentifier.uuid_re.match(uuid)
-
     @classmethod
     def from_arn(cls, arn):
         '''
index cf78a0eacf2ab6045959817ebcf3097706f1997d..1327ae9797e830bbe1536cc11a5bea883a59d826 100644 (file)
@@ -20,7 +20,6 @@ import mox
 import json
 
 from heat.engine import identifier
-from heat.common import utils
 
 
 @attr(tag=['unit', 'identifier'])
@@ -226,17 +225,6 @@ class IdentifierTest(unittest.TestCase):
         hi = identifier.HeatIdentifier('t', 's', 'i', 'p1/p2/p3')
         self.assertEqual(hi._path_components(), ['p1', 'p2', 'p3'])
 
-    def test_uuid_match(self):
-        uuid = utils.generate_uuid()
-        self.assertTrue(identifier.HeatIdentifier.is_uuid(uuid))
-        self.assertFalse(identifier.HeatIdentifier.is_uuid('a' + uuid))
-        self.assertFalse(identifier.HeatIdentifier.is_uuid(
-            'zzzzzzzz-zzzz-zzzz-zzzzzzzzzzzz'))
-        self.assertFalse(identifier.HeatIdentifier.is_uuid(uuid + 'a'))
-        for i in xrange(100):
-            self.assertTrue(identifier.HeatIdentifier.is_uuid(
-                utils.generate_uuid()))
-
 
 @attr(tag=['unit', 'identifier'])
 @attr(speed='fast')