MT#62763 Fix pycodestyle issues

- Add pyproject.toml and update tox.ini.
- Fix «== None» into «is None».
- Fix escaping of |.
- Fix misplaced spaces around commas.
- Add explicit exception catching.

Change-Id: I97199c4ed8a24dc870bab4178bf1c103e7f4e7e0
mr26.0
Guillem Jover 5 months ago
parent 45131fd432
commit 8e50b38caa

@ -0,0 +1,7 @@
[tool.black]
# Match line-length expected by pycodestyle.
line-length = 200
# Current version in Debian trixie.
target-version = ['py313']
# Enable preview features, expected by pycodestyle.
preview = true

@ -31,7 +31,7 @@ def login_panel(driver):
driver.implicitly_wait(1)
driver.find_element(By.XPATH, '//*[@id="q-app"]/div/footer/div/button/span[2]/span').click()
driver.implicitly_wait(10)
except:
except Exception:
pass
finally:
driver.implicitly_wait(10)

@ -25,7 +25,7 @@ def click_js(driver, element):
if element[:1] == "/":
webelement = driver.find_element(By.XPATH, element)
else:
webelement = driver.find_element(By.LINK_TEXT ,element)
webelement = driver.find_element(By.LINK_TEXT, element)
driver.execute_script("arguments[0].click();", webelement)

@ -46,7 +46,7 @@ def preparation():
del i
for customer in customers.keys():
create_subscriber(driver, customer, customers[customer])
if os.environ.get('vmtype') == None:
if os.environ.get('vmtype') is None:
os.environ['vmtype'] = "CE"
driver.quit()
except Exception:
@ -625,19 +625,19 @@ class testrun(unittest.TestCase):
wait_for_invisibility(driver, '//*[@id="csc-page-call-forwarding"]//button[contains(., "Add forwarding")]/span/svg')
wait_for_invisibility(driver, '//*[@id="csc-wrapper-call-forwarding"]/div/div[2]/div[4]/svg')
driver.find_element(By.XPATH, '//*[@id="csc-wrapper-call-forwarding"]/div[1]//div/span[contains(., "Number")]').click()
fill_element(driver, "/html/body//label//div//input", "checking <script>alert('test')</script> asdфывфів123!@#$%^&*()_+[]\|}{;'\":,./?><EOL")
fill_element(driver, "/html/body//label//div//input", "checking <script>alert('test')</script> asdфывфів123!@#$%^&*()_+[]\\|}{;'\":,./?><EOL")
driver.find_element(By.XPATH, '/html/body//div/button[contains(., "Set")]').click()
self.assertTrue(
len(driver.find_elements(By.XPATH, '/html/body//div[@role="alert"]//div[contains(., "Failed to create cfdestinationset.")]')) > 0, "Illegal call foward was created")
wait_for_invisibility(driver, '//*[@id="csc-wrapper-call-forwarding"]/div/div[2]/div[4]/svg')
driver.find_element(By.XPATH, '//*[@id="csc-wrapper-call-forwarding"]/div[2]//div/span[contains(., "Number")]').click()
fill_element(driver, "/html/body//label//div//input", "checking <script>alert('test')</script> asdфывфів123!@#$%^&*()_+[]\|}{;'\":,./?><EOL")
fill_element(driver, "/html/body//label//div//input", "checking <script>alert('test')</script> asdфывфів123!@#$%^&*()_+[]\\|}{;'\":,./?><EOL")
driver.find_element(By.XPATH, '/html/body//div/button[contains(., "Set")]').click()
self.assertTrue(
len(driver.find_elements(By.XPATH, '/html/body//div[@role="alert"]//div[contains(., "Failed to create cfdestinationset.")]')) > 0, "Illegal call foward was created")
wait_for_invisibility(driver, '//*[@id="csc-wrapper-call-forwarding"]/div/div[2]/div[4]/svg')
driver.find_element(By.XPATH, '//*[@id="csc-wrapper-call-forwarding"]/div[3]//div/span[contains(., "Number")]').click()
fill_element(driver, "/html/body//label//div//input", "checking <script>alert('test')</script> asdфывфів123!@#$%^&*()_+[]\|}{;'\":,./?><EOL")
fill_element(driver, "/html/body//label//div//input", "checking <script>alert('test')</script> asdфывфів123!@#$%^&*()_+[]\\|}{;'\":,./?><EOL")
driver.find_element(By.XPATH, '/html/body//div/button[contains(., "Set")]').click()
self.assertTrue(
len(driver.find_elements(By.XPATH, '/html/body//div[@role="alert"]//div[contains(., "Failed to create cfdestinationset.")]')) > 0, "Illegal call foward was created")

@ -1,2 +1,2 @@
[pep8]
[pycodestyle]
max-line-length = 200

Loading…
Cancel
Save