From cd38e4d3521389c3457aeb67dea6b5e2ce4158ce Mon Sep 17 00:00:00 2001 From: Joseph Nuthalapati Date: Fri, 27 Jul 2018 12:50:53 +0530 Subject: [PATCH] functional tests: Fix first test failing on a pristine VM When a user creates an admin account on a pristine VM, the user is already logged in. The tests wrongly assume that another login is required. Signed-off-by: Joseph Nuthalapati --- functional_tests/support/interface.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/functional_tests/support/interface.py b/functional_tests/support/interface.py index c27198224..ce33f3db6 100644 --- a/functional_tests/support/interface.py +++ b/functional_tests/support/interface.py @@ -45,7 +45,6 @@ def login(browser, url, username, password): browser.visit(default_url + '/plinth/firstboot/welcome') submit(browser) # click the "Start Setup" button create_admin_account(browser, username, password) - login(browser, url, username, password) def is_login_prompt(browser): @@ -79,16 +78,16 @@ def create_user(browser, name, password): def rename_user(browser, old_name, new_name): nav_to_module(browser, 'users') with wait_for_page_update(browser): - browser.find_link_by_href( - '/plinth/sys/users/' + old_name + '/edit/').first.click() + browser.find_link_by_href('/plinth/sys/users/' + old_name + + '/edit/').first.click() browser.find_by_id('id_username').fill(new_name) submit(browser) def delete_user(browser, name): nav_to_module(browser, 'users') - delete_link = browser.find_link_by_href( - '/plinth/sys/users/' + name + '/delete/') + delete_link = browser.find_link_by_href('/plinth/sys/users/' + name + + '/delete/') if delete_link: with wait_for_page_update(browser): delete_link.first.click()