Merge "[Identity] Check return value is None in identity v3 unit tests"

Author: Jenkins

openstackclient/tests/identity/v2_0/test_endpoint.py

@@ -132,11 +132,12 @@ def test_endpoint_delete_no_options(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         self.endpoints_mock.delete.assert_called_with(
             identity_fakes.endpoint_id,
         )
+        self.assertIsNone(result)
 
 
 class TestEndpointList(TestEndpoint):

openstackclient/tests/identity/v2_0/test_project.py

@@ -613,10 +613,9 @@ def test_project_unset_key(self):
         verifylist = [
             ('property', ['fee', 'fo']),
         ]
-
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
         # Set expected values
         kwargs = {
             'description': identity_fakes.project_description,
@@ -631,3 +630,4 @@ def test_project_unset_key(self):
             identity_fakes.project_id,
             **kwargs
         )
+        self.assertIsNone(result)

openstackclient/tests/identity/v2_0/test_role.py

@@ -240,11 +240,12 @@ def test_role_delete_no_options(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         self.roles_mock.delete.assert_called_with(
             identity_fakes.role_id,
         )
+        self.assertIsNone(result)
 
 
 class TestRoleList(TestRole):
@@ -459,14 +460,15 @@ def test_role_remove(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         # RoleManager.remove_user_role(user, role, tenant=None)
         self.roles_mock.remove_user_role.assert_called_with(
             identity_fakes.user_id,
             identity_fakes.role_id,
             identity_fakes.project_id,
         )
+        self.assertIsNone(result)
 
 
 class TestRoleShow(TestRole):

openstackclient/tests/identity/v2_0/test_service.py

@@ -194,11 +194,12 @@ def test_service_delete_no_options(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         self.services_mock.delete.assert_called_with(
             identity_fakes.service_id,
         )
+        self.assertIsNone(result)
 
 
 class TestServiceList(TestService):

openstackclient/tests/identity/v2_0/test_token.py

@@ -99,6 +99,7 @@ def test_token_revoke(self):
         verifylist = [('token', self.TOKEN)]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         self.tokens_mock.delete.assert_called_with(self.TOKEN)
+        self.assertIsNone(result)

openstackclient/tests/identity/v2_0/test_user.py

@@ -414,11 +414,12 @@ def test_user_delete_no_options(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         self.users_mock.delete.assert_called_with(
             identity_fakes.user_id,
         )
+        self.assertIsNone(result)
 
 
 class TestUserList(TestUser):
@@ -578,7 +579,7 @@ def test_user_set_name(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         # Set expected values
         kwargs = {
@@ -590,6 +591,7 @@ def test_user_set_name(self):
             identity_fakes.user_id,
             **kwargs
         )
+        self.assertIsNone(result)
 
     def test_user_set_password(self):
         arglist = [
@@ -608,13 +610,14 @@ def test_user_set_password(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         # UserManager.update_password(user, password)
         self.users_mock.update_password.assert_called_with(
             identity_fakes.user_id,
             'secret',
         )
+        self.assertIsNone(result)
 
     def test_user_set_password_prompt(self):
         arglist = [
@@ -636,13 +639,14 @@ def test_user_set_password_prompt(self):
         mocker = mock.Mock()
         mocker.return_value = 'abc123'
         with mock.patch("openstackclient.common.utils.get_password", mocker):
-            self.cmd.take_action(parsed_args)
+            result = self.cmd.take_action(parsed_args)
 
         # UserManager.update_password(user, password)
         self.users_mock.update_password.assert_called_with(
             identity_fakes.user_id,
             'abc123',
         )
+        self.assertIsNone(result)
 
     def test_user_set_email(self):
         arglist = [
@@ -660,7 +664,7 @@ def test_user_set_email(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         # Set expected values
         kwargs = {
@@ -672,6 +676,7 @@ def test_user_set_email(self):
             identity_fakes.user_id,
             **kwargs
         )
+        self.assertIsNone(result)
 
     def test_user_set_project(self):
         arglist = [
@@ -689,13 +694,14 @@ def test_user_set_project(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         # UserManager.update_tenant(user, tenant)
         self.users_mock.update_tenant.assert_called_with(
             identity_fakes.user_id,
             identity_fakes.project_id,
         )
+        self.assertIsNone(result)
 
     def test_user_set_enable(self):
         arglist = [
@@ -713,7 +719,7 @@ def test_user_set_enable(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         # Set expected values
         kwargs = {
@@ -724,6 +730,7 @@ def test_user_set_enable(self):
             identity_fakes.user_id,
             **kwargs
         )
+        self.assertIsNone(result)
 
     def test_user_set_disable(self):
         arglist = [
@@ -741,7 +748,7 @@ def test_user_set_disable(self):
         ]
         parsed_args = self.check_parser(self.cmd, arglist, verifylist)
 
-        self.cmd.take_action(parsed_args)
+        result = self.cmd.take_action(parsed_args)
 
         # Set expected values
         kwargs = {
@@ -752,6 +759,7 @@ def test_user_set_disable(self):
             identity_fakes.user_id,
             **kwargs
         )
+        self.assertIsNone(result)
 
 
 class TestUserShow(TestUser):