Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(947)

Unified Diff: telemetry/telemetry/internal/backends/form_based_credentials_backend_unittest_base.py

Issue 2692763002: Revert of [Telemetry] Switch clients to new JavaScript API (batch 5) (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: telemetry/telemetry/internal/backends/form_based_credentials_backend_unittest_base.py
diff --git a/telemetry/telemetry/internal/backends/form_based_credentials_backend_unittest_base.py b/telemetry/telemetry/internal/backends/form_based_credentials_backend_unittest_base.py
index 6e53be33f5b6d83d7ed22a49f6a9c01881b0818a..437689b516a5576b2ccd91bffb7559774cd24907 100644
--- a/telemetry/telemetry/internal/backends/form_based_credentials_backend_unittest_base.py
+++ b/telemetry/telemetry/internal/backends/form_based_credentials_backend_unittest_base.py
@@ -20,8 +20,6 @@
password_element_id, form_element_id,
already_logged_in_js): # pylint: disable=no-self-use
del form_element_id # Unused.
- del email_element_id # Unused.
- del password_element_id # Unused.
tab = simple_mock.MockObject()
ar = simple_mock.MockObject()
@@ -29,30 +27,29 @@
'password': 'blargh'}
tab.ExpectCall('Navigate', login_page_url)
- tab.ExpectCall(
- 'EvaluateJavaScript2', already_logged_in_js).WillReturn(False)
+ tab.ExpectCall('EvaluateJavaScript', already_logged_in_js).WillReturn(False)
tab.ExpectCall('WaitForDocumentReadyStateToBeInteractiveOrBetter')
ar.ExpectCall(
- 'WaitForJavaScriptCondition2',
+ 'WaitForJavaScriptCondition',
'(document.querySelector({{ form_id }}) !== null) || ({{ @code }})')
ar.ExpectCall('WaitForNavigate')
def VerifyEmail(js):
- assert '{{ selector }}' in js
- assert '{{ username }}' in js
- tab.ExpectCall('ExecuteJavaScript2', _).WhenCalled(VerifyEmail)
+ assert email_element_id in js
+ assert 'blah' in js
+ tab.ExpectCall('ExecuteJavaScript', _).WhenCalled(VerifyEmail)
def VerifyPw(js):
- assert '{{ selector }}' in js
- assert '{{ password }}' in js
- tab.ExpectCall('ExecuteJavaScript2', _).WhenCalled(VerifyPw)
+ assert password_element_id in js
+ assert 'largh' in js
+ tab.ExpectCall('ExecuteJavaScript', _).WhenCalled(VerifyPw)
def VerifySubmit(js):
assert '.submit' in js or '.click' in js
- tab.ExpectCall('ExecuteJavaScript2', _).WhenCalled(VerifySubmit)
+ tab.ExpectCall('ExecuteJavaScript', _).WhenCalled(VerifySubmit)
# Checking for form still up.
- tab.ExpectCall('EvaluateJavaScript2', _).WillReturn(False)
+ tab.ExpectCall('EvaluateJavaScript', _).WillReturn(False)
backend.LoginNeeded(tab, ar, config)

Powered by Google App Engine
This is Rietveld 408576698