diff options
| author | Tang Chen <chen.tang@easystack.cn> | 2016-03-05 16:29:18 +0800 |
|---|---|---|
| committer | Tang Chen <chen.tang@easystack.cn> | 2016-03-08 15:58:25 +0800 |
| commit | 762c4c9bdf66995198fa03751b861a859b9d44a1 (patch) | |
| tree | bd9db0488ec60bc93b4b564fd0677047e601d24d /openstackclient/tests/identity/v3/test_identity_provider.py | |
| parent | 752705ae307e0bf53d4282120009bf3d155da137 (diff) | |
| download | python-openstackclient-762c4c9bdf66995198fa03751b861a859b9d44a1.tar.gz | |
[Identity] Check return value is None in identity v3 unit tests
take_action() in commands inheriting from Command returns nothing.
So we should assert the return is None in the unit tests of these
commands.
Change-Id: I02af06b3d476aac2d93a23ef2111cdc7fa0892ec
Partial-Bug: #1550636
Diffstat (limited to 'openstackclient/tests/identity/v3/test_identity_provider.py')
| -rw-r--r-- | openstackclient/tests/identity/v3/test_identity_provider.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/openstackclient/tests/identity/v3/test_identity_provider.py b/openstackclient/tests/identity/v3/test_identity_provider.py index ddad6ffb..1ee7aade 100644 --- a/openstackclient/tests/identity/v3/test_identity_provider.py +++ b/openstackclient/tests/identity/v3/test_identity_provider.py @@ -258,10 +258,13 @@ class TestIdentityProviderDelete(TestIdentityProvider): ('identity_provider', identity_fakes.idp_id), ] parsed_args = self.check_parser(self.cmd, arglist, verifylist) - self.cmd.take_action(parsed_args) + + result = self.cmd.take_action(parsed_args) + self.identity_providers_mock.delete.assert_called_with( identity_fakes.idp_id, ) + self.assertIsNone(result) class TestIdentityProviderList(TestIdentityProvider): |