Fix incorrect variable name

valid_versions does not exist here, it is obviously supposed to mean
_VALID_VERSIONS.

Closes-Bug: #1386232
Change-Id: I477baa7642feba72f80d884d6183512185b02cf1

cinderclient/client.py

@@ -66,7 +66,7 @@ def get_volume_api_from_url(url):
             return version[1:]
 
     msg = "Invalid client version '%s'. must be one of: %s" % (
-        (version, ', '.join(valid_versions)))
+        (version, ', '.join(_VALID_VERSIONS)))
     raise exceptions.UnsupportedVersion(msg)
 
 

cinderclient/tests/test_client.py

@@ -61,3 +61,16 @@ class ClientTest(utils.TestCase):
 
         self.assertNotIn("fakePassword", output[1])
         self.assertIn("fakeUser", output[1])
+
+    def test_versions(self):
+        v1_url = 'http://fakeurl/v1/tenants'
+        v2_url = 'http://fakeurl/v2/tenants'
+        unknown_url = 'http://fakeurl/v9/tenants'
+
+        self.assertEqual('1',
+                         cinderclient.client.get_volume_api_from_url(v1_url))
+        self.assertEqual('2',
+                         cinderclient.client.get_volume_api_from_url(v2_url))
+        self.assertRaises(cinderclient.exceptions.UnsupportedVersion,
+                          cinderclient.client.get_volume_api_from_url,
+                          unknown_url)