Merge "Adds domain specification for SetUser"

1 files changed, 36 insertions, 0 deletions

diff --git a/openstackclient/tests/unit/identity/v3/test_user.py b/openstackclient/tests/unit/identity/v3/test_user.py
index 3c1f49a6..2ce66e94 100644
--- a/openstackclient/tests/unit/identity/v3/test_user.py
+++ b/openstackclient/tests/unit/identity/v3/test_user.py
@@ -684,9 +684,14 @@ class TestUserList(TestUser):
 class TestUserSet(TestUser):
 
     project = identity_fakes.FakeProject.create_one_project()
+    domain = identity_fakes.FakeDomain.create_one_domain()
     user = identity_fakes.FakeUser.create_one_user(
         attrs={'default_project_id': project.id}
     )
+    user2 = identity_fakes.FakeUser.create_one_user(
+        attrs={'default_project_id': project.id,
+               'domain_id': domain.id}
+    )
 
     def setUp(self):
         super(TestUserSet, self).setUp()
@@ -748,6 +753,37 @@ class TestUserSet(TestUser):
         )
         self.assertIsNone(result)
 
+    def test_user_set_specify_domain(self):
+        arglist = [
+            '--name', 'qwerty',
+            '--domain', self.domain.id,
+            self.user2.name
+        ]
+        verifylist = [
+            ('name', 'qwerty'),
+            ('password', None),
+            ('domain', self.domain.id),
+            ('email', None),
+            ('project', None),
+            ('enable', False),
+            ('disable', False),
+            ('user', self.user2.name),
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+
+        result = self.cmd.take_action(parsed_args)
+
+        kwargs = {
+            'enabled': True,
+            'name': 'qwerty'
+        }
+
+        self.users_mock.update.assert_called_with(
+            self.user.id,
+            **kwargs
+        )
+        self.assertIsNone(result)
+
     def test_user_set_password(self):
         arglist = [
             '--password', 'secret',