From c5a8773b175e29ec5b7da003596488180ec52a7d Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Mon, 25 Apr 2016 16:24:00 +0200 Subject: selenium skeleton and README --- selenium/README | 7 +++++++ selenium/wallet-test.py | 32 ++++++++++++++++++++++++++++++++ 2 files changed, 39 insertions(+) create mode 100644 selenium/README create mode 100644 selenium/wallet-test.py (limited to 'selenium') diff --git a/selenium/README b/selenium/README new file mode 100644 index 000000000..de99e6610 --- /dev/null +++ b/selenium/README @@ -0,0 +1,7 @@ +Directory containing testscases for testing the wallet with Selenium ChromeDriver. + +[1] Contains the ChromeDriver Pythonic documentation +[2] Tells which fields (and which values) the 'loggingPrefs' capability expects + +[1] http://seleniumhq.github.io/selenium/docs/api/py/index.html +[2] https://github.com/SeleniumHQ/selenium/wiki/DesiredCapabilities#loggingpreferences-json-object diff --git a/selenium/wallet-test.py b/selenium/wallet-test.py new file mode 100644 index 000000000..0725539a1 --- /dev/null +++ b/selenium/wallet-test.py @@ -0,0 +1,32 @@ +from selenium import webdriver +import time + +co = webdriver.ChromeOptions() +co.add_argument("load-extension=/home/marcello/Taler/wallet-webex") +cap = webdriver.DesiredCapabilities.CHROME.copy() +cap['loggingPrefs'] = {'driver': 'INFO', 'browser': 'INFO'} +client = webdriver.Chrome(chrome_options=co, desired_capabilities=cap) +client.get('https://taler.net') +listener = """\ + document.addEventListener('taler-id', function(evt){ + window['extId'] = evt.detail.id; + }); + evt = new CustomEvent('taler-query-id'); + document.dispatchEvent(evt); + """ +client.execute_script(listener) +poll = """\ + if(window.extId) + return window.extId; + else return false; + """ +# TODO intelligent poller needed +time.sleep(1) +ext_id = client.execute_script(poll) + +labels = ['balance'] +for l in labels: + client.get('chrome-extensio://' + ext_id + '/popup/popup.html#/' + l) +for log_type in ['browser']: + for log in client.get_log(log_type): + print(log['level'] + ': ' + log['message']) -- cgit v1.2.3 From 3777d850ebfb730fc309813c3effff08bda69910 Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Mon, 25 Apr 2016 17:58:01 +0200 Subject: injecting bank registration script --- selenium/wallet-test.py | 34 +++++++++++++++++++++++++++++----- 1 file changed, 29 insertions(+), 5 deletions(-) (limited to 'selenium') diff --git a/selenium/wallet-test.py b/selenium/wallet-test.py index 0725539a1..718fb5ccf 100644 --- a/selenium/wallet-test.py +++ b/selenium/wallet-test.py @@ -11,7 +11,7 @@ listener = """\ document.addEventListener('taler-id', function(evt){ window['extId'] = evt.detail.id; }); - evt = new CustomEvent('taler-query-id'); + var evt = new CustomEvent('taler-query-id'); document.dispatchEvent(evt); """ client.execute_script(listener) @@ -24,9 +24,33 @@ poll = """\ time.sleep(1) ext_id = client.execute_script(poll) +# if client has error from its activity, ptints it and returns True +def is_error(client): + for log_type in ['browser']: + for log in client.get_log(log_type): + if log['level'] is 'error': + print(log['level'] + ': ' + log['message']) + return True + + labels = ['balance'] +# labels = ['balance', 'history', 'debug'] for l in labels: - client.get('chrome-extensio://' + ext_id + '/popup/popup.html#/' + l) -for log_type in ['browser']: - for log in client.get_log(log_type): - print(log['level'] + ': ' + log['message']) + client.get('chrome-extension://' + ext_id + '/popup/popup.html#/' + l) + +# TODO assert here +is_error(client) + +# visit bank and trigger withdrawal +client.get('https://bank.test.taler.net') +client.get('https://bank.test.taler.net/accounts/register') + +register = """\ + var form = document.getElementsByTagName('form'); + form.username.value = 'test'; + form.password.value = 'test'; + form.submit(); + """ + +# TODO assert here +client.execute_script(register) -- cgit v1.2.3 From 67b37dd31ebb2ffa53c5ea169fb7c711e56785e3 Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Tue, 26 Apr 2016 10:29:05 +0200 Subject: putting tests inside "unittest" classes --- selenium/wallet-test.py | 112 ++++++++++++++++++++++++++++++------------------ 1 file changed, 71 insertions(+), 41 deletions(-) (limited to 'selenium') diff --git a/selenium/wallet-test.py b/selenium/wallet-test.py index 718fb5ccf..6027d8e1d 100644 --- a/selenium/wallet-test.py +++ b/selenium/wallet-test.py @@ -1,31 +1,36 @@ from selenium import webdriver import time +import unittest + + +def client_setup(): + """Return a Chrome browser the extension's id""" + co = webdriver.ChromeOptions() + co.add_argument("load-extension=/home/marcello/wallet-webex") + cap = webdriver.DesiredCapabilities.CHROME.copy() + cap['loggingPrefs'] = {'driver': 'INFO', 'browser': 'INFO'} + client = webdriver.Chrome(chrome_options=co, desired_capabilities=cap) + client.get('https://taler.net') + listener = """\ + document.addEventListener('taler-id', function(evt){ + window['extId'] = evt.detail.id; + }); + var evt = new CustomEvent('taler-query-id'); + document.dispatchEvent(evt); + """ + client.execute_script(listener) + poll = """\ + if(window.extId) + return window.extId; + else return false; + """ + # Todo: put some delay in polling + ext_id = client.execute_script(poll) + return {'client': client, 'ext_id': ext_id} + -co = webdriver.ChromeOptions() -co.add_argument("load-extension=/home/marcello/Taler/wallet-webex") -cap = webdriver.DesiredCapabilities.CHROME.copy() -cap['loggingPrefs'] = {'driver': 'INFO', 'browser': 'INFO'} -client = webdriver.Chrome(chrome_options=co, desired_capabilities=cap) -client.get('https://taler.net') -listener = """\ - document.addEventListener('taler-id', function(evt){ - window['extId'] = evt.detail.id; - }); - var evt = new CustomEvent('taler-query-id'); - document.dispatchEvent(evt); - """ -client.execute_script(listener) -poll = """\ - if(window.extId) - return window.extId; - else return false; - """ -# TODO intelligent poller needed -time.sleep(1) -ext_id = client.execute_script(poll) - -# if client has error from its activity, ptints it and returns True def is_error(client): + """In case of errors in the browser, print them and return True""" for log_type in ['browser']: for log in client.get_log(log_type): if log['level'] is 'error': @@ -33,24 +38,49 @@ def is_error(client): return True -labels = ['balance'] -# labels = ['balance', 'history', 'debug'] -for l in labels: - client.get('chrome-extension://' + ext_id + '/popup/popup.html#/' + l) -# TODO assert here -is_error(client) +class PopupTestCase(unittest.TestCase): + """Test wallet's popups""" + def setUp(self): + ret = client_setup() + self.client = ret['client'] + self.ext_id = ret['ext_id'] + + def tearDown(self): + self.client.close() + + def test_popup(self): + # keeping only 'balance' to get tests faster. To be + # extended with 'history' and 'debug' + labels = ['balance'] + for l in labels: + self.client.get('chrome-extension://' + self.ext_id + '/popup/popup.html#/' + l) + self.assertNotEqual(True, is_error(self.client)) + +class BankTestCase(unittest.TestCase): + """Test withdrawal (after registering a new user)""" + def setUp(self): + ret = client_setup() + self.client = ret['client'] + self.ext_id = ret['ext_id'] + + def tearDown(self): + self.client.close() + + + def test_withdrawal(self): + bank_url = 'http://127.0.0.1:9898' + self.client.get(bank_url + '/accounts/register') -# visit bank and trigger withdrawal -client.get('https://bank.test.taler.net') -client.get('https://bank.test.taler.net/accounts/register') + register = """\ + var form = document.getElementsByTagName('form')[0]; + form.username.value = '%s'; + form.password.value = 'test'; + form.submit(); + """ % str(int(time.time())) # need fresh username -register = """\ - var form = document.getElementsByTagName('form'); - form.username.value = 'test'; - form.password.value = 'test'; - form.submit(); - """ + self.client.execute_script(register) + self.assertNotEqual(True, is_error(self.client)) -# TODO assert here -client.execute_script(register) +if __name__ == '__main__': + unittest.main() -- cgit v1.2.3 From 1f70c33d25450b08a7d135fdd50e068b81fa616b Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Tue, 26 Apr 2016 15:46:08 +0200 Subject: using webdriver-provided wait facility --- selenium/wallet-test.py | 81 ++++++++++++++++++++++++++++++++----------------- 1 file changed, 53 insertions(+), 28 deletions(-) (limited to 'selenium') diff --git a/selenium/wallet-test.py b/selenium/wallet-test.py index 6027d8e1d..4ee396073 100644 --- a/selenium/wallet-test.py +++ b/selenium/wallet-test.py @@ -1,8 +1,15 @@ from selenium import webdriver +from selenium.webdriver.common.by import By +from selenium.webdriver.support import expected_conditions as EC +from selenium.webdriver.support.ui import WebDriverWait import time +import logging import unittest +import json +logger = logging.getLogger(__name__) + def client_setup(): """Return a Chrome browser the extension's id""" co = webdriver.ChromeOptions() @@ -13,20 +20,17 @@ def client_setup(): client.get('https://taler.net') listener = """\ document.addEventListener('taler-id', function(evt){ - window['extId'] = evt.detail.id; + var html = document.getElementsByTagName('html')[0]; + html.setAttribute('data-taler-wallet-id', evt.detail.id); }); + var evt = new CustomEvent('taler-query-id'); document.dispatchEvent(evt); """ client.execute_script(listener) - poll = """\ - if(window.extId) - return window.extId; - else return false; - """ - # Todo: put some delay in polling - ext_id = client.execute_script(poll) - return {'client': client, 'ext_id': ext_id} + client.implicitly_wait(5) + html = client.find_element(By.TAG_NAME, "html") + return {'client': client, 'ext_id': html.get_attribute('data-taler-wallet-id')} def is_error(client): @@ -38,24 +42,23 @@ def is_error(client): return True - -class PopupTestCase(unittest.TestCase): - """Test wallet's popups""" - def setUp(self): - ret = client_setup() - self.client = ret['client'] - self.ext_id = ret['ext_id'] - - def tearDown(self): - self.client.close() - - def test_popup(self): - # keeping only 'balance' to get tests faster. To be - # extended with 'history' and 'debug' - labels = ['balance'] - for l in labels: - self.client.get('chrome-extension://' + self.ext_id + '/popup/popup.html#/' + l) - self.assertNotEqual(True, is_error(self.client)) +# class PopupTestCase(unittest.TestCase): +# """Test wallet's popups""" +# def setUp(self): +# ret = client_setup() +# self.client = ret['client'] +# self.ext_id = ret['ext_id'] +# +# def tearDown(self): +# self.client.close() +# +# def test_popup(self): +# # keeping only 'balance' to get tests faster. To be +# # extended with 'history' and 'debug' +# labels = ['balance'] +# for l in labels: +# self.client.get('chrome-extension://' + self.ext_id + '/popup/popup.html#/' + l) +# self.assertNotEqual(True, is_error(self.client)) class BankTestCase(unittest.TestCase): """Test withdrawal (after registering a new user)""" @@ -65,7 +68,8 @@ class BankTestCase(unittest.TestCase): self.ext_id = ret['ext_id'] def tearDown(self): - self.client.close() + pass + # self.client.close() def test_withdrawal(self): @@ -82,5 +86,26 @@ class BankTestCase(unittest.TestCase): self.client.execute_script(register) self.assertNotEqual(True, is_error(self.client)) + button = self.client.execute_script("return document.getElementById('select-exchange')") + button.click() + # Note: this further 'get()' seems needed to get the in-wallet page + location = self.client.execute_script("return document.location.href") + self.client.get(location) + # wallet needs time to check the exchange: thus wait until button is clickable + wait = WebDriverWait(self.client, 10) + button = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[1]"))) + button.click() + # check if captcha is in gotten page + # Note, a wait for getting the inputElem below could be needed + inputElem = self.client.find_element(By.XPATH, "//input[@name='pin_0']") + self.assertNotEqual(None, inputElem) + # get the question + question = self.client.find_element(By.XPATH, "//span[@class='captcha-question']/div") + questionTok = question.text.split() + op1 = int(questionTok[2]) + op2 = int(questionTok[4]) + res = {'+': op1 + op2, '-': op1 - op2, u'\u00d7': op1 * op2} + inputElem.send_keys(res[questionTok[3]]) + if __name__ == '__main__': unittest.main() -- cgit v1.2.3 From ba0ef6b6e8a51ba10ceaba02835bd46004110ccf Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Tue, 26 Apr 2016 17:24:10 +0200 Subject: still on native delayers --- selenium/wallet-test.py | 27 +++++++++++++++++---------- 1 file changed, 17 insertions(+), 10 deletions(-) (limited to 'selenium') diff --git a/selenium/wallet-test.py b/selenium/wallet-test.py index 4ee396073..9cd05fe1a 100644 --- a/selenium/wallet-test.py +++ b/selenium/wallet-test.py @@ -40,6 +40,7 @@ def is_error(client): if log['level'] is 'error': print(log['level'] + ': ' + log['message']) return True + return False # class PopupTestCase(unittest.TestCase): @@ -58,7 +59,7 @@ def is_error(client): # labels = ['balance'] # for l in labels: # self.client.get('chrome-extension://' + self.ext_id + '/popup/popup.html#/' + l) -# self.assertNotEqual(True, is_error(self.client)) +# self.assertFalse(is_error(self.client)) class BankTestCase(unittest.TestCase): """Test withdrawal (after registering a new user)""" @@ -84,21 +85,22 @@ class BankTestCase(unittest.TestCase): """ % str(int(time.time())) # need fresh username self.client.execute_script(register) - self.assertNotEqual(True, is_error(self.client)) - - button = self.client.execute_script("return document.getElementById('select-exchange')") + self.assertFalse(is_error(self.client)) + wait = WebDriverWait(self.client, 10) + # WARNING, 'button' below *gets* clicked but the an error about ExpiredTimeout + # gets thrown and printed on the console + button = wait.until(EC.element_to_be_clickable((By.ID, "select-exchange"))) + # click to confirm the amount to withdraw button.click() - # Note: this further 'get()' seems needed to get the in-wallet page + # Note: this further 'get()' seems needed to get the in-wallet page loaded location = self.client.execute_script("return document.location.href") self.client.get(location) - # wallet needs time to check the exchange: thus wait until button is clickable - wait = WebDriverWait(self.client, 10) button = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[1]"))) + # This click returns the captcha page (put wait?) button.click() - # check if captcha is in gotten page - # Note, a wait for getting the inputElem below could be needed + # Note: a wait for getting the inputElem below could be needed inputElem = self.client.find_element(By.XPATH, "//input[@name='pin_0']") - self.assertNotEqual(None, inputElem) + self.assertIsNotNone(inputElem) # get the question question = self.client.find_element(By.XPATH, "//span[@class='captcha-question']/div") questionTok = question.text.split() @@ -106,6 +108,11 @@ class BankTestCase(unittest.TestCase): op2 = int(questionTok[4]) res = {'+': op1 + op2, '-': op1 - op2, u'\u00d7': op1 * op2} inputElem.send_keys(res[questionTok[3]]) + form = self.client.find_element(By.TAG_NAME, "form") + form.submit() + # check if successful message exists + msg_succ = self.client.find_element(By.CLASS_NAME, "informational-ok") + self.assertNotNone(msg_succ) if __name__ == '__main__': unittest.main() -- cgit v1.2.3 From 90c67fc7178ad2219d37ae36d0a0b7c842de75fe Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Wed, 27 Apr 2016 10:47:01 +0200 Subject: selenium withdrawal minor fixes --- selenium/wallet-test.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'selenium') diff --git a/selenium/wallet-test.py b/selenium/wallet-test.py index 9cd05fe1a..1e4410c11 100644 --- a/selenium/wallet-test.py +++ b/selenium/wallet-test.py @@ -76,7 +76,7 @@ class BankTestCase(unittest.TestCase): def test_withdrawal(self): bank_url = 'http://127.0.0.1:9898' self.client.get(bank_url + '/accounts/register') - + self.client.find_element(By.TAG_NAME, "form") register = """\ var form = document.getElementsByTagName('form')[0]; form.username.value = '%s'; @@ -87,14 +87,14 @@ class BankTestCase(unittest.TestCase): self.client.execute_script(register) self.assertFalse(is_error(self.client)) wait = WebDriverWait(self.client, 10) - # WARNING, 'button' below *gets* clicked but the an error about ExpiredTimeout - # gets thrown and printed on the console + button = self.client.find_element(By.ID, "select-exchange") button = wait.until(EC.element_to_be_clickable((By.ID, "select-exchange"))) # click to confirm the amount to withdraw button.click() # Note: this further 'get()' seems needed to get the in-wallet page loaded location = self.client.execute_script("return document.location.href") self.client.get(location) + # Confirm xchg button = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[1]"))) # This click returns the captcha page (put wait?) button.click() -- cgit v1.2.3 From 320eac1675fd3c4b2de4d5dd404ecc413b9ea505 Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Wed, 27 Apr 2016 13:54:35 +0200 Subject: selenium test not in unittest --- selenium/test.py | 100 ++++++++++++++++++++++++++++++++++++++++ selenium/wallet-test.py | 118 ------------------------------------------------ 2 files changed, 100 insertions(+), 118 deletions(-) create mode 100644 selenium/test.py delete mode 100644 selenium/wallet-test.py (limited to 'selenium') diff --git a/selenium/test.py b/selenium/test.py new file mode 100644 index 000000000..8720c79d7 --- /dev/null +++ b/selenium/test.py @@ -0,0 +1,100 @@ +from selenium import webdriver +from selenium.webdriver.common.by import By +from selenium.webdriver.support import expected_conditions as EC +from selenium.webdriver.support.ui import WebDriverWait +import time +import logging + +logger = logging.getLogger(__name__) +bank_url = 'http://127.0.0.1:9898' + +def client_setup(): + """Return a dict containing the driver and the extension's id""" + co = webdriver.ChromeOptions() + co.add_argument("load-extension=/home/marcello/wallet-webex") + cap = webdriver.DesiredCapabilities.CHROME.copy() + cap['loggingPrefs'] = {'driver': 'INFO', 'browser': 'INFO'} + client = webdriver.Chrome(chrome_options=co, desired_capabilities=cap) + client.get('https://taler.net') + listener = """\ + document.addEventListener('taler-id', function(evt){ + var html = document.getElementsByTagName('html')[0]; + html.setAttribute('data-taler-wallet-id', evt.detail.id); + }); + + var evt = new CustomEvent('taler-query-id'); + document.dispatchEvent(evt); + """ + client.execute_script(listener) + client.implicitly_wait(5) + html = client.find_element(By.TAG_NAME, "html") + return {'client': client, 'ext_id': html.get_attribute('data-taler-wallet-id')} + + +def is_error(client): + """Return True in case of errors in the browser, False otherwise""" + for log_type in ['browser']: + for log in client.get_log(log_type): + if log['level'] is 'error': + print(log['level'] + ': ' + log['message']) + return True + return False + + +def register(client): + """Register a new user to the bank delaying its execution until the + profile page is shown""" + client.get(bank_url + '/accounts/register') + client.find_element(By.TAG_NAME, "form") + register = """\ + var form = document.getElementsByTagName('form')[0]; + form.username.value = '%s'; + form.password.value = 'test'; + form.submit(); + """ % str(int(time.time())) # need fresh username + + client.execute_script(register) + # need implicit wait to be set up + button = client.find_element(By.ID, "select-exchange") + # when button is gotten, the browser is in the profile page + # so the function can return + if not is_error(client): + logger.info('correctly registered at bank') + else: + logger.error('User not registered at bank') + + +def withdraw(client): + """Register and withdraw (1) KUDOS for a fresh user""" + register(client) + # trigger withdrawal button + button = client.find_element(By.ID, "select-exchange") + button.click() + location = client.execute_script("return document.location.href") + client.get(location) + # Confirm xchg + wait = WebDriverWait(client, 10) + button = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[1]"))) + # This click returns the captcha page (put wait?) + button.click() + answer = client.find_element(By.XPATH, "//input[@name='pin_0']") + question = client.find_element(By.XPATH, "//span[@class='captcha-question']/div") + questionTok = question.text.split() + op1 = int(questionTok[2]) + op2 = int(questionTok[4]) + res = {'+': op1 + op2, '-': op1 - op2, u'\u00d7': op1 * op2} + answer.send_keys(res[questionTok[3]]) + form = client.find_element(By.TAG_NAME, "form") + form.submit() + # check outcome + msg_succ = client.find_element(By.CLASS_NAME, "informational-ok") + if not msg_succ: + logger.error('Error in withdrawing') + else: logger.info('Withdrawal successful') + + + +ret = client_setup() +client = ret['client'] +withdraw(client) +client.close() diff --git a/selenium/wallet-test.py b/selenium/wallet-test.py deleted file mode 100644 index 1e4410c11..000000000 --- a/selenium/wallet-test.py +++ /dev/null @@ -1,118 +0,0 @@ -from selenium import webdriver -from selenium.webdriver.common.by import By -from selenium.webdriver.support import expected_conditions as EC -from selenium.webdriver.support.ui import WebDriverWait -import time -import logging -import unittest -import json - - -logger = logging.getLogger(__name__) - -def client_setup(): - """Return a Chrome browser the extension's id""" - co = webdriver.ChromeOptions() - co.add_argument("load-extension=/home/marcello/wallet-webex") - cap = webdriver.DesiredCapabilities.CHROME.copy() - cap['loggingPrefs'] = {'driver': 'INFO', 'browser': 'INFO'} - client = webdriver.Chrome(chrome_options=co, desired_capabilities=cap) - client.get('https://taler.net') - listener = """\ - document.addEventListener('taler-id', function(evt){ - var html = document.getElementsByTagName('html')[0]; - html.setAttribute('data-taler-wallet-id', evt.detail.id); - }); - - var evt = new CustomEvent('taler-query-id'); - document.dispatchEvent(evt); - """ - client.execute_script(listener) - client.implicitly_wait(5) - html = client.find_element(By.TAG_NAME, "html") - return {'client': client, 'ext_id': html.get_attribute('data-taler-wallet-id')} - - -def is_error(client): - """In case of errors in the browser, print them and return True""" - for log_type in ['browser']: - for log in client.get_log(log_type): - if log['level'] is 'error': - print(log['level'] + ': ' + log['message']) - return True - return False - - -# class PopupTestCase(unittest.TestCase): -# """Test wallet's popups""" -# def setUp(self): -# ret = client_setup() -# self.client = ret['client'] -# self.ext_id = ret['ext_id'] -# -# def tearDown(self): -# self.client.close() -# -# def test_popup(self): -# # keeping only 'balance' to get tests faster. To be -# # extended with 'history' and 'debug' -# labels = ['balance'] -# for l in labels: -# self.client.get('chrome-extension://' + self.ext_id + '/popup/popup.html#/' + l) -# self.assertFalse(is_error(self.client)) - -class BankTestCase(unittest.TestCase): - """Test withdrawal (after registering a new user)""" - def setUp(self): - ret = client_setup() - self.client = ret['client'] - self.ext_id = ret['ext_id'] - - def tearDown(self): - pass - # self.client.close() - - - def test_withdrawal(self): - bank_url = 'http://127.0.0.1:9898' - self.client.get(bank_url + '/accounts/register') - self.client.find_element(By.TAG_NAME, "form") - register = """\ - var form = document.getElementsByTagName('form')[0]; - form.username.value = '%s'; - form.password.value = 'test'; - form.submit(); - """ % str(int(time.time())) # need fresh username - - self.client.execute_script(register) - self.assertFalse(is_error(self.client)) - wait = WebDriverWait(self.client, 10) - button = self.client.find_element(By.ID, "select-exchange") - button = wait.until(EC.element_to_be_clickable((By.ID, "select-exchange"))) - # click to confirm the amount to withdraw - button.click() - # Note: this further 'get()' seems needed to get the in-wallet page loaded - location = self.client.execute_script("return document.location.href") - self.client.get(location) - # Confirm xchg - button = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[1]"))) - # This click returns the captcha page (put wait?) - button.click() - # Note: a wait for getting the inputElem below could be needed - inputElem = self.client.find_element(By.XPATH, "//input[@name='pin_0']") - self.assertIsNotNone(inputElem) - # get the question - question = self.client.find_element(By.XPATH, "//span[@class='captcha-question']/div") - questionTok = question.text.split() - op1 = int(questionTok[2]) - op2 = int(questionTok[4]) - res = {'+': op1 + op2, '-': op1 - op2, u'\u00d7': op1 * op2} - inputElem.send_keys(res[questionTok[3]]) - form = self.client.find_element(By.TAG_NAME, "form") - form.submit() - # check if successful message exists - msg_succ = self.client.find_element(By.CLASS_NAME, "informational-ok") - self.assertNotNone(msg_succ) - -if __name__ == '__main__': - unittest.main() -- cgit v1.2.3 From 07715206e54cab23788e3167e5a77509ffd3e369 Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Wed, 27 Apr 2016 14:39:39 +0200 Subject: make donations via selenium --- selenium/test.py | 20 ++++++++++++++++---- 1 file changed, 16 insertions(+), 4 deletions(-) (limited to 'selenium') diff --git a/selenium/test.py b/selenium/test.py index 8720c79d7..c0a5de742 100644 --- a/selenium/test.py +++ b/selenium/test.py @@ -6,7 +6,8 @@ import time import logging logger = logging.getLogger(__name__) -bank_url = 'http://127.0.0.1:9898' +bank = 'http://bank.test.taler.net' +donations = 'http://shop.test.taler.net' def client_setup(): """Return a dict containing the driver and the extension's id""" @@ -41,10 +42,21 @@ def is_error(client): return False +def make_donation(client): + """Make donation at shop.test.taler.net. Assume the wallet has coins""" + client.get(donations) + form = client.find_element(By.TAG_NAME, "form") + form.submit() # amount and receiver chosen + confirm_taler = client.find_element(By.XPATH, "//form//input[@type='button']") + confirm_taler.click() # Taler as payment option chosen + wait = WebDriverWait(client, 10) + confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) + confirm_pay.click() + def register(client): """Register a new user to the bank delaying its execution until the profile page is shown""" - client.get(bank_url + '/accounts/register') + client.get(bank + '/accounts/register') client.find_element(By.TAG_NAME, "form") register = """\ var form = document.getElementsByTagName('form')[0]; @@ -93,8 +105,8 @@ def withdraw(client): else: logger.info('Withdrawal successful') - ret = client_setup() client = ret['client'] withdraw(client) -client.close() +make_donation(client) +# client.close() -- cgit v1.2.3 From 5ea6f8dd00266e1b5cf76826f54364119a295948 Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Wed, 27 Apr 2016 16:58:06 +0200 Subject: selenium tests in "/component" form + fetching baseurl from env --- selenium/test.py | 45 +++++++++++++++++++++++++++++++++++++++------ 1 file changed, 39 insertions(+), 6 deletions(-) (limited to 'selenium') diff --git a/selenium/test.py b/selenium/test.py index c0a5de742..a6155321e 100644 --- a/selenium/test.py +++ b/selenium/test.py @@ -1,13 +1,22 @@ +#!/usr/bin/env python3 + +""" +Tests for the wallet. It looks for an env variable called TALER_BASEURL +where it appends "/banks" etc. in order to find bank and shops. If not +found, it defaults to https://test.taler.net/ +""" + from selenium import webdriver from selenium.webdriver.common.by import By from selenium.webdriver.support import expected_conditions as EC from selenium.webdriver.support.ui import WebDriverWait +from urllib import parse import time import logging +import os logger = logging.getLogger(__name__) -bank = 'http://bank.test.taler.net' -donations = 'http://shop.test.taler.net' +taler_baseurl = os.environ['TALER_BASEURL'] if 'TALER_BASEURL' in os.environ else 'https://test.taler.net/' def client_setup(): """Return a dict containing the driver and the extension's id""" @@ -27,10 +36,19 @@ def client_setup(): document.dispatchEvent(evt); """ client.execute_script(listener) - client.implicitly_wait(5) html = client.find_element(By.TAG_NAME, "html") return {'client': client, 'ext_id': html.get_attribute('data-taler-wallet-id')} +# Note that the db appears to be reset automatically by the driver +def destroy_db(client, ext_id): + url = 'chrome-extension://' + ext_id + '/popup/popup.html#/debug' + client.get(url) + button = client.find_element(By.XPATH, "//div[@id='content']/button[3]") + button.click() + time.sleep(4) + alert = client.switch_to.alert + alert.accept() + def is_error(client): """Return True in case of errors in the browser, False otherwise""" @@ -44,7 +62,7 @@ def is_error(client): def make_donation(client): """Make donation at shop.test.taler.net. Assume the wallet has coins""" - client.get(donations) + client.get(parse.urljoin(taler_baseurl, "shop")) form = client.find_element(By.TAG_NAME, "form") form.submit() # amount and receiver chosen confirm_taler = client.find_element(By.XPATH, "//form//input[@type='button']") @@ -53,10 +71,23 @@ def make_donation(client): confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) confirm_pay.click() + +def buy_article(client): + """Buy article at blog.test.taler.net. Assume the wallet has coins""" + client.get(parse.urljoin(taler_baseurl, "blog")) + teaser = client.find_element(By.XPATH, "//ul/h3/a[1]") # Pick 'Foreword' chapter + teaser.click() + wait = WebDriverWait(client, 10) + confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) + confirm_pay.click() + + def register(client): """Register a new user to the bank delaying its execution until the profile page is shown""" - client.get(bank + '/accounts/register') + client.get(parse.urljoin(taler_baseurl, "bank")) + register_link = client.find_element(By.XPATH, "//a[@href='/accounts/register/']") + register_link.click() client.find_element(By.TAG_NAME, "form") register = """\ var form = document.getElementsByTagName('form')[0]; @@ -107,6 +138,8 @@ def withdraw(client): ret = client_setup() client = ret['client'] +client.implicitly_wait(10) withdraw(client) make_donation(client) -# client.close() +buy_article(client) +client.close() -- cgit v1.2.3 From ce87fc0b90262cb44dda780b128c1ab0afd3b609 Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Thu, 28 Apr 2016 15:08:34 +0200 Subject: deploying try-catch(es) in selenium code --- selenium/test.py | 105 +++++++++++++++++++++++++++++++++++++++++++------------ 1 file changed, 83 insertions(+), 22 deletions(-) (limited to 'selenium') diff --git a/selenium/test.py b/selenium/test.py index a6155321e..d697f73c8 100644 --- a/selenium/test.py +++ b/selenium/test.py @@ -10,18 +10,21 @@ from selenium import webdriver from selenium.webdriver.common.by import By from selenium.webdriver.support import expected_conditions as EC from selenium.webdriver.support.ui import WebDriverWait +from selenium.common.exceptions import NoSuchElementException, TimeoutException from urllib import parse +import argparse import time import logging +import sys import os logger = logging.getLogger(__name__) taler_baseurl = os.environ['TALER_BASEURL'] if 'TALER_BASEURL' in os.environ else 'https://test.taler.net/' -def client_setup(): +def client_setup(args): """Return a dict containing the driver and the extension's id""" co = webdriver.ChromeOptions() - co.add_argument("load-extension=/home/marcello/wallet-webex") + co.add_argument("load-extension=" + args.extdir) cap = webdriver.DesiredCapabilities.CHROME.copy() cap['loggingPrefs'] = {'driver': 'INFO', 'browser': 'INFO'} client = webdriver.Chrome(chrome_options=co, desired_capabilities=cap) @@ -63,22 +66,44 @@ def is_error(client): def make_donation(client): """Make donation at shop.test.taler.net. Assume the wallet has coins""" client.get(parse.urljoin(taler_baseurl, "shop")) - form = client.find_element(By.TAG_NAME, "form") + try: + form = client.find_element(By.TAG_NAME, "form") + except NoSuchElementException: + logger.error('No donation form found') + sys.exit(1) form.submit() # amount and receiver chosen - confirm_taler = client.find_element(By.XPATH, "//form//input[@type='button']") + try: + confirm_taler = client.find_element(By.XPATH, "//form//input[@type='button']") + except NoSuchElementException: + logger.error('Could not trigger contract on donation shop') + sys.exit(1) confirm_taler.click() # Taler as payment option chosen + # HUNG once, an explicit get() may be needed wait = WebDriverWait(client, 10) - confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) + try: + confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) + except TimeoutException: + logger.error('Could not confirm payment on donation shop') + sys.exit(1) confirm_pay.click() def buy_article(client): """Buy article at blog.test.taler.net. Assume the wallet has coins""" client.get(parse.urljoin(taler_baseurl, "blog")) - teaser = client.find_element(By.XPATH, "//ul/h3/a[1]") # Pick 'Foreword' chapter + try: + teaser = client.find_element(By.XPATH, "//ul/h3/a[1]") # Pick 'Foreword' chapter + except NoSuchElementException: + logger.error('Could not choose "Foreword" chapter on blog') + sys.exit(1) teaser.click() + # HUNG once, an explicit get() may be needed wait = WebDriverWait(client, 10) - confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) + try: + confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) + except TimeoutException: + logger.error('Could not confirm payment on blog') + sys.exit(1) confirm_pay.click() @@ -86,9 +111,18 @@ def register(client): """Register a new user to the bank delaying its execution until the profile page is shown""" client.get(parse.urljoin(taler_baseurl, "bank")) - register_link = client.find_element(By.XPATH, "//a[@href='/accounts/register/']") + try: + register_link = client.find_element(By.XPATH, "//a[@href='/accounts/register/']") + except NoSuchElementException: + logger.error("Could not find register link on bank's homepage") + sys.exit(1) register_link.click() - client.find_element(By.TAG_NAME, "form") + try: + client.find_element(By.TAG_NAME, "form") + except NoSuchElementException: + logger.error("Register form not found") + sys.exit(1) + register = """\ var form = document.getElementsByTagName('form')[0]; form.username.value = '%s'; @@ -98,7 +132,11 @@ def register(client): client.execute_script(register) # need implicit wait to be set up - button = client.find_element(By.ID, "select-exchange") + try: + button = client.find_element(By.ID, "select-exchange") + except NoSuchElementException: + logger.error("Selecting exchange impossible") + sys.exit(1) # when button is gotten, the browser is in the profile page # so the function can return if not is_error(client): @@ -111,35 +149,58 @@ def withdraw(client): """Register and withdraw (1) KUDOS for a fresh user""" register(client) # trigger withdrawal button - button = client.find_element(By.ID, "select-exchange") + try: + button = client.find_element(By.ID, "select-exchange") + except NoSuchElementException: + logger.error("Selecting exchange impossible") + sys.exit(1) button.click() location = client.execute_script("return document.location.href") client.get(location) # Confirm xchg wait = WebDriverWait(client, 10) - button = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[1]"))) + try: + button = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[1]"))) + except TimeoutException: + logger.error("Could not confirm exchange (therefore provide withdrawal needed data)") + sys.exit(1) # This click returns the captcha page (put wait?) button.click() - answer = client.find_element(By.XPATH, "//input[@name='pin_0']") - question = client.find_element(By.XPATH, "//span[@class='captcha-question']/div") + try: + answer = client.find_element(By.XPATH, "//input[@name='pin_0']") + question = client.find_element(By.XPATH, "//span[@class='captcha-question']/div") + except NoSuchElementException: + logger.error("Captcha page not gotten or malformed") + sys.exit(1) questionTok = question.text.split() op1 = int(questionTok[2]) op2 = int(questionTok[4]) res = {'+': op1 + op2, '-': op1 - op2, u'\u00d7': op1 * op2} answer.send_keys(res[questionTok[3]]) - form = client.find_element(By.TAG_NAME, "form") + try: + form = client.find_element(By.TAG_NAME, "form") + except NoSuchElementException: + logger.error("Could not submit captcha answer (therefore trigger withdrawal)") + sys.exit(1) form.submit() # check outcome - msg_succ = client.find_element(By.CLASS_NAME, "informational-ok") - if not msg_succ: - logger.error('Error in withdrawing') - else: logger.info('Withdrawal successful') - - -ret = client_setup() + try: + client.find_element(By.CLASS_NAME, "informational-ok") + except NoSuchElementException: + logger.error("Withdrawal not completed") + sys.exit(1) + logger.info("Withdrawal completed") + + +parser = argparse.ArgumentParser() +parser.add_argument('--extdir', help="Folder containing the unpacked extension", metavar="EXTDIR", type=str, dest="extdir", required=True) +args = parser.parse_args() +ret = client_setup(args) client = ret['client'] client.implicitly_wait(10) withdraw(client) make_donation(client) buy_article(client) +logger.info("Test passed") client.close() +sys.exit(0) -- cgit v1.2.3 From 08e60506d5df6db78fe29541b751fb7bf13861d0 Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Tue, 3 May 2016 11:12:12 +0200 Subject: selenium: the wallet can select amounts from the bank's dropdown --- selenium/test.py | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) (limited to 'selenium') diff --git a/selenium/test.py b/selenium/test.py index d697f73c8..b2bece4e0 100644 --- a/selenium/test.py +++ b/selenium/test.py @@ -145,7 +145,7 @@ def register(client): logger.error('User not registered at bank') -def withdraw(client): +def withdraw(client, amount_value=None): """Register and withdraw (1) KUDOS for a fresh user""" register(client) # trigger withdrawal button @@ -154,6 +154,13 @@ def withdraw(client): except NoSuchElementException: logger.error("Selecting exchange impossible") sys.exit(1) + if amount_value: + xpath = "//select/option[@value='" + str(amount_value) + "']" + try: + client.find_element(By.XPATH, xpath) + except NoSuchElementException: + logger.error("value '" + str(amount_value) + "' is not offered by this bank to withdraw, please adapt it") + sys.exit(1) button.click() location = client.execute_script("return document.location.href") client.get(location) @@ -198,7 +205,7 @@ args = parser.parse_args() ret = client_setup(args) client = ret['client'] client.implicitly_wait(10) -withdraw(client) +withdraw(client, 11) make_donation(client) buy_article(client) logger.info("Test passed") -- cgit v1.2.3 From 048e4f8dc9cfca81907ff6562125a92f9661882c Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Tue, 3 May 2016 14:16:04 +0200 Subject: nondefault amounts to donate via selenium --- selenium/test.py | 36 +++++++++++++++++++----------------- 1 file changed, 19 insertions(+), 17 deletions(-) (limited to 'selenium') diff --git a/selenium/test.py b/selenium/test.py index b2bece4e0..3d46585bf 100644 --- a/selenium/test.py +++ b/selenium/test.py @@ -42,17 +42,6 @@ def client_setup(args): html = client.find_element(By.TAG_NAME, "html") return {'client': client, 'ext_id': html.get_attribute('data-taler-wallet-id')} -# Note that the db appears to be reset automatically by the driver -def destroy_db(client, ext_id): - url = 'chrome-extension://' + ext_id + '/popup/popup.html#/debug' - client.get(url) - button = client.find_element(By.XPATH, "//div[@id='content']/button[3]") - button.click() - time.sleep(4) - alert = client.switch_to.alert - alert.accept() - - def is_error(client): """Return True in case of errors in the browser, False otherwise""" for log_type in ['browser']: @@ -63,7 +52,7 @@ def is_error(client): return False -def make_donation(client): +def make_donation(client, amount_value=None): """Make donation at shop.test.taler.net. Assume the wallet has coins""" client.get(parse.urljoin(taler_baseurl, "shop")) try: @@ -71,6 +60,14 @@ def make_donation(client): except NoSuchElementException: logger.error('No donation form found') sys.exit(1) + if amount_value: + xpath = "//select[@id='taler-donation']/option[@value='" + str(amount_value) + "']" + try: + desired_amount = client.find_element(By.XPATH, xpath) + desired_amount.click() + except NoSuchElementException: + logger.error("value '" + str(amount_value) + "' is not offered by this shop to donate, please adapt it") + sys.exit(1) form.submit() # amount and receiver chosen try: confirm_taler = client.find_element(By.XPATH, "//form//input[@type='button']") @@ -78,7 +75,9 @@ def make_donation(client): logger.error('Could not trigger contract on donation shop') sys.exit(1) confirm_taler.click() # Taler as payment option chosen - # HUNG once, an explicit get() may be needed + # explicit get() is needed, it hangs (sometimes) otherwise + time.sleep(1) + client.get(client.current_url) wait = WebDriverWait(client, 10) try: confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) @@ -97,7 +96,9 @@ def buy_article(client): logger.error('Could not choose "Foreword" chapter on blog') sys.exit(1) teaser.click() - # HUNG once, an explicit get() may be needed + # explicit get() is needed, it hangs (sometimes) otherwise + time.sleep(1) + client.get(client.current_url) wait = WebDriverWait(client, 10) try: confirm_pay = wait.until(EC.element_to_be_clickable((By.XPATH, "//button[@class='accept']"))) @@ -157,7 +158,8 @@ def withdraw(client, amount_value=None): if amount_value: xpath = "//select/option[@value='" + str(amount_value) + "']" try: - client.find_element(By.XPATH, xpath) + desired_amount = client.find_element(By.XPATH, xpath) + desired_amount.click() except NoSuchElementException: logger.error("value '" + str(amount_value) + "' is not offered by this bank to withdraw, please adapt it") sys.exit(1) @@ -205,8 +207,8 @@ args = parser.parse_args() ret = client_setup(args) client = ret['client'] client.implicitly_wait(10) -withdraw(client, 11) -make_donation(client) +withdraw(client, 10) +make_donation(client, 6.0) buy_article(client) logger.info("Test passed") client.close() -- cgit v1.2.3 From dc83b85e85110eb7ed2e96ecdbcbb90b5a5620aa Mon Sep 17 00:00:00 2001 From: Marcello Stanisci Date: Tue, 3 May 2016 15:05:58 +0200 Subject: selenium: adding function to switch between 'test' and 'demo' base URLs, used to introduce currency mismatches when testing errors management --- selenium/test.py | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) (limited to 'selenium') diff --git a/selenium/test.py b/selenium/test.py index 3d46585bf..14140e953 100644 --- a/selenium/test.py +++ b/selenium/test.py @@ -17,9 +17,10 @@ import time import logging import sys import os +import re logger = logging.getLogger(__name__) -taler_baseurl = os.environ['TALER_BASEURL'] if 'TALER_BASEURL' in os.environ else 'https://test.taler.net/' +taler_baseurl = os.environ.get('TALER_BASEURL', 'https://test.taler.net/') def client_setup(args): """Return a dict containing the driver and the extension's id""" @@ -52,6 +53,17 @@ def is_error(client): return False +def switch_base(): + """If 'test' is in TALER_BASEURL, then make it be 'demo', and viceversa. + Used to trig currency mismatch errors. It assumes that the https://{test,demo}.taler.net + layout is being used""" + global taler_baseurl + url = parse.urlparse(taler_baseurl) + if url[1] == 'test.taler.net': + taler_baseurl = "https://demo.taler.net" + if url[1] == 'demo.taler.net': + taler_baseurl = "https://test.taler.net" + def make_donation(client, amount_value=None): """Make donation at shop.test.taler.net. Assume the wallet has coins""" client.get(parse.urljoin(taler_baseurl, "shop")) @@ -208,6 +220,7 @@ ret = client_setup(args) client = ret['client'] client.implicitly_wait(10) withdraw(client, 10) +switch_base() # inducing error make_donation(client, 6.0) buy_article(client) logger.info("Test passed") -- cgit v1.2.3