Commit 123b64c2 authored by Will Meek's avatar Will Meek

Add a fix for license_compliance_spec

Intermittently on master the automated
test would open an empty model for
pipeline license compliance.

This was never observed when running
through the test steps manually.

This commit wraps the click and check for
the element around the wait_until with a
reload true if not found
parent 20c4464c
...@@ -51,16 +51,16 @@ module QA ...@@ -51,16 +51,16 @@ module QA
end end
def approve_license(name) def approve_license(name)
wait_until(reload: true) do
click_license(name) click_license(name)
wait_until(reload: false) do
has_element?(:approve_license_button, wait: 1) has_element?(:approve_license_button, wait: 1)
end end
click_element(:approve_license_button) click_element(:approve_license_button)
end end
def deny_license(name) def deny_license(name)
wait_until(reload: true) do
click_license(name) click_license(name)
wait_until(reload: false) do
has_element?(:deny_license_button, wait: 1) has_element?(:deny_license_button, wait: 1)
end end
click_element(:deny_license_button) click_element(:deny_license_button)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment