test: simplify user addition test

This commit is contained in:
Ankush Menat 2022-08-24 14:24:10 +05:30
parent 40f54d04b7
commit 6ec5c05e4e

View file

@ -694,18 +694,12 @@ class TestSiteMigration(BaseTestCommands):
class TestAddNewUser(BaseTestCommands):
def test_create_user(self):
self.execute(
f"bench --site {TEST_SITE} add-user test@gmail.com --first-name test --last-name test --password 123 --user-type 'System User' --add-role 'Accounts User' --add-role 'Sales User'"
"bench --site {site} add-user test@gmail.com --first-name test --last-name test --password 123 --user-type 'System User' --add-role 'Accounts User' --add-role 'Sales User'"
)
self.assertEqual(self.returncode, 0)
roles = []
user = frappe.get_doc("User", "test@gmail.com")
for i in user.roles:
role = frappe.get_doc("Has Role", i.name)
roles.append(role.role)
self.assertEqual(user.name, "test@gmail.com")
self.assertIn("Accounts User", roles)
self.assertIn("Sales User", roles)
self.assertTrue(len(roles) == 2)
roles = {r.role for r in user.roles}
self.assertEqual({"Accounts User", "Sales User"}, roles)
class TestBenchBuild(BaseTestCommands):