From f079b5b9c4c030293b4ebfdf84d8b768b3aa3515 Mon Sep 17 00:00:00 2001
From: Dean Troyer <dtroyer@gmail.com>
Date: Wed, 22 Oct 2014 11:12:47 -0500
Subject: Change --os-auth-plugin to --os-auth-type

User's don't know what a plugin is.

* Internally, os_auth_type and/or auth_type represents what the
  user supplied.
* auth_plugin_name is the name of the selected plugin
* auth_plugin is the actual plugin object

Plugin selection process:
* if --os-auth-type is supplied:
  * if it matches against an available plugin, done
  * (if it can map to an availble plugin type, done; TODO in a followup)
* if --os-auth-type is not supplied:
  * if --os-url and --os-token are supplied, select 'token_endpoint'
  * if --os-username supplied, select identity_api_version + 'password'
  * if --os-token supplied, select identity_api_version + 'token'

Change-Id: Ice4535214e311ebf924087cf77f6d84d76f5f3ee
---
 openstackclient/tests/test_shell.py | 36 ++++++++++++++++++------------------
 1 file changed, 18 insertions(+), 18 deletions(-)

(limited to 'openstackclient/tests/test_shell.py')

diff --git a/openstackclient/tests/test_shell.py b/openstackclient/tests/test_shell.py
index b0c1452e..837a48af 100644
--- a/openstackclient/tests/test_shell.py
+++ b/openstackclient/tests/test_shell.py
@@ -108,8 +108,8 @@ class TestShell(utils.TestCase):
                              default_args["region_name"])
             self.assertEqual(_shell.options.os_trust_id,
                              default_args["trust_id"])
-            self.assertEqual(_shell.options.os_auth_plugin,
-                             default_args['auth_plugin'])
+            self.assertEqual(_shell.options.os_auth_type,
+                             default_args['auth_type'])
 
     def _assert_token_auth(self, cmd_options, default_args):
         with mock.patch("openstackclient.shell.OpenStackShell.initialize_app",
@@ -190,7 +190,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -210,7 +210,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -230,7 +230,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -250,7 +250,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -270,7 +270,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -290,7 +290,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -310,7 +310,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -330,7 +330,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -350,7 +350,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -370,7 +370,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -390,7 +390,7 @@ class TestShellPasswordAuth(TestShell):
             "password": DEFAULT_PASSWORD,
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -410,7 +410,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": DEFAULT_REGION_NAME,
             "trust_id": "",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
@@ -430,12 +430,12 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "1234",
-            "auth_plugin": "",
+            "auth_type": "",
         }
         self._assert_password_auth(flag, kwargs)
 
-    def test_only_auth_plugin_flow(self):
-        flag = "--os-auth-plugin " + "v2password"
+    def test_only_auth_type_flow(self):
+        flag = "--os-auth-type " + "v2password"
         kwargs = {
             "auth_url": "",
             "project_id": "",
@@ -450,7 +450,7 @@ class TestShellPasswordAuth(TestShell):
             "password": "",
             "region_name": "",
             "trust_id": "",
-            "auth_plugin": DEFAULT_AUTH_PLUGIN
+            "auth_type": DEFAULT_AUTH_PLUGIN
         }
         self._assert_password_auth(flag, kwargs)
 
-- 
cgit v1.2.1