-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: Update E2E utilities #55322
test: Update E2E utilities #55322
Conversation
Size Change: 0 B Total Size: 1.65 MB ℹ️ View Unchanged
|
WebdriverIO (the replacement for the deprecated WD driver library) does not have a method for retrieving a device's pixel ratio. Therefore, this sets an explicit value for each of our test devices.
Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade.
Seems this method has changed or been removed for Android.
This utility appeared to fail occasionally due to an empty `addButton` value.
8639bc3
to
6d30ce6
Compare
This updates the iOS variant to mirror the structure of the Android variant in the name of consistency.
Flaky tests detected in 8e7c4ff. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/6538799429
|
@@ -302,7 +301,10 @@ class EditorPage { | |||
async waitForKeyboardToBeHidden() { | |||
const { addButtonLocation } = this.initialValues; | |||
const addButton = await this.getAddBlockButton(); | |||
const location = await addButton.getLocation(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Occasionally, this query would fail with Appium 2 with the following error. Guarding against an empty query return avoids this error and results in the recursive loop of waitForKeyboardToBeHidden
to occur.
TypeError: Cannot read properties of undefined (reading 'getLocation')
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🚀 Nice work!
Mirror Android implementation found in Appium 2.
Debugging why the tests began failing. Both the PHP and Android failures are odd given the only change since the last successful build was iOS-only testing code: 5d684e2. I am unable to reproduce the Android failures locally so far. |
Oh that's weird! Maybe enabling more logging could help to debug, commenting this or changing the level of logging could help. |
Attempt to debug CI failures. ``` FAIL __device-tests__/gutenberg-editor-rendering-media-blocks.test.js (73.496 s) ● Gutenberg Editor Rendering Media Blocks test › should be able to render blocks correctly unknown method: Not implemented at getErrorFromResponseBody (file:/Users/runner/work/gutenberg/gutenberg/node_modules/webdriver/build/utils.js:194:12) at NodeJSRequest._request (file:/Users/runner/work/gutenberg/gutenberg/node_modules/webdriver/build/request/index.js:164:23) at Browser.wrapCommandFn (file:/Users/runner/work/gutenberg/gutenberg/node_modules/@wdio/utils/build/shim.js:81:29) ```
Avoid the deprecated `touchAction` command. > The `touchAction` command is deprecated and will be removed in a future version. We recommend to use the action command instead with pointer type touch, e.g.: > > await browser.action('pointer', { > parameters: { pointerType: 'touch' } > }) https://webdriver.io/docs/api/element/touchAction/ This may also relate to the fact that Mobile JSON Wire Protocol is deprecated as well. > Mobile JSON Wire Protocol > > The Mobile JSON Wire Protocol is a super set of mobile commands on top of the JSON Wire Protocol. Given this one is deprecated the Mobile JSON Wire Protocol also got deprecated. Appium might still support some of its commands but it is not recommended to use them. https://webdriver.io/docs/api/protocols#mobile-json-wire-protocol
This reverts commit 7bb46de.
This prevents `pause` from being called without a parameter, which throws an error.
Still do not know why the PHP tests fail, but given this does not target the main branch, I am going to merge this to unblock other work. @geriux please feel free to share any additional change requests. I'd be happy to follow up as needed. Thanks again for the help. |
* Dependencies update: - Bumps Appium to version 2.0.0 - Removes wd in favor of WebdriverIO * - Adds appium:setup script to install needed drivers for Appium. - Updates test:e2e:build-wda with the new Appium home path for the xcuitest driver. - Updates test:e2e:android:local and test:e2e:ios:local to add the newly added appium:setup script to install drivers if needed. * Update Android Runner to cache the Appium drivers * Update iOS Runner - Bump Xcode to 14.2 * Update iOS Runner with these changes: - Move the Launch of the simulator step to the top so it starts booting up the simulator in the background, so it doesn't block the next steps while it's stating. - Install and cache the Appium drivers. * Updates the E2E capabilities settings: - It removes unneeded values after the migration. - Bumps the iOS platform version to 16.2. - Adds specific SauceLabs configuration that sets the Appium version to 2.0.0. - Adds a prefixKeysWithAppium util to append the "appium:" prefix for the Appium capabilities settings. This is added to simplify the modifications we do for some of the settings in runtime. * Updates the E2E serverConfigs file with the updated SauceLabs settings for Appium 2 and WebdriverIO * E2E Utils: - Remove import of wd and set it as null temporarily while we migrate its usage to WebdriverIO * E2E Utils: Update getIOSPlatformVersions to get the desired platform version from the capabilities settings instead of an inline value * E2E Utils: - Update Remote configuration with the new structure for WebdriverIO and Appium 2 - Uses the prefixKeysWithAppium util for the Appium capabilities, we cannot use "appium:options" since that's only available for real devices and we use simulators instead. - The app capability value was updated to the new storage:filename= prefix this was important to make SauceLabs to work correctly * E2E Utils - Update the stopDriver function with: - Changes sessionID to sessionId - Removes the hash value to append in the Job URL, these URLs are public so it's not needed to append the auth param. - Updates driver.quit() to driver.deleteSession() * E2E Utils - Updates swipeFromTo to WebdriverIO * E2E Utils - Updates isEditorVisible to WebdriverIO It adds a custom timeout to waitForDisplayed, the simulators in CI are slow so we need to give enough time to find the post title locator. * Remove usage of sauceJobStatus, WebdriverIO internally handles this * EditorPage - Remove import of wd and keep a null value while we migrate to WebdriverIO * EditorPage - Migrate initializeEditor to WebdriverIO * Gutenberg Editor Rendering Media Blocks test - Migrate to WebdriverIO * E2E Utils - Revert removing passing the auth token for the jobURL, it also updates the logic to the latest implementation from SauceLabs docs. * Remove idleTimeout desktop-only SauceLabs option * [Mobile] - Update E2E Block Insertion tests (#55167) * E2E Utils, updates the typing helpers: - Removes the platform specfic typing functionality in favor of WebdriverIO's addValue * E2E Utils - Updates getKeycode and pressKeycode to WebdriverIO * E2E Utils - Update tapStatusBariOS to WebdriverIO * E2E Utils - Update toggleHtmlMode to WebdriverIO * E2E Utils - Update waitForVisible to WebdriverIO * E2E Editor Page - Update getBlockAtPosition to WebdriverIO * E2E Editor Page - Update getTitleElement to WebdriverIO * E2E Editor Page - Update getHtmlContent to WebdriverIO * E2E Editor Page - Update swipeToolbarToElement to WebdriverIO * E2E Editor Page - Update addNewBlock to WebdriverIO * E2E Editor Page - Update findBlockButton to WebdriverIO * E2E Editor Page - Update assertSlashInserterPresent to WebdriverIO * E2E EditorPage - Update waitForElementToBeDisplayedById to WebdriverIO * E2E Test - Block Insertion - Update to WebdriverIO * Update typeString scroll delay comment Co-authored-by: David Calhoun <github@davidcalhoun.me> --------- Co-authored-by: David Calhoun <github@davidcalhoun.me> * test: Update Search tests for Appium 2 (#55170) * test: Update getSearchBlockTextElement query Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Update waitForElementToBeDisplayedByXPath query Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Update verifySearchElementText query Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Update getSearchBlockTextElement query Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * E2E Editor Page - Update dismissKeyboard to use WebdiverIO's hideKeyboard (#55211) * test: Update Media Blocks tests for Appium 2 (#55222) * test: Replace sleep with pause utility Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Replace elementsByAccessibilityId usage Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Replace elementByXPath usage Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Repair enterCaptionToSelectedImageBlock utility Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. Additionally, the `clearValue` method did not succeed on the caption Button element. Hence, the utility was expanded to instead type into the TextView. * test: Update Paragraph tests for Appium 2 (#55215) * test: Update clickMiddleOfElement helper Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Update clickBeginningOfElement helper Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Update getTextForParagraphBlockAtPosition helper Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Replace elementsByXPath usage Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * E2E Utils - Migrate dragAndDropAfterElement to WebdriverIO (#55254) * [Mobile] - Move Appium drivers setup to bash script (#55257) * Move Appium drivers setup to bash script * Update drivers' names casing for consistency * Rename appium:setup to test:e2e:setup * test: Update E2E utilities (#55322) * test: Set static device pixel ratio WebdriverIO (the replacement for the deprecated WD driver library) does not have a method for retrieving a device's pixel ratio. Therefore, this sets an explicit value for each of our test devices. * test: Update the `getAddBlockButton` utility Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade. * test: Update toggleDarkMode utility Seems this method has changed or been removed for Android. * test: Increase robustness of waitForKeyboardToBeHidden utility This utility appeared to fail occasionally due to an empty `addButton` value. * test: toggleDarkMode uses consistent approach across platforms This updates the iOS variant to mirror the structure of the Android variant in the name of consistency. * refactor: Rename devicePixelRatio to pixelRatio Mirror Android implementation found in Appium 2. * test: Increase logging levels Attempt to debug CI failures. ``` FAIL __device-tests__/gutenberg-editor-rendering-media-blocks.test.js (73.496 s) ● Gutenberg Editor Rendering Media Blocks test › should be able to render blocks correctly unknown method: Not implemented at getErrorFromResponseBody (file:/Users/runner/work/gutenberg/gutenberg/node_modules/webdriver/build/utils.js:194:12) at NodeJSRequest._request (file:/Users/runner/work/gutenberg/gutenberg/node_modules/webdriver/build/request/index.js:164:23) at Browser.wrapCommandFn (file:/Users/runner/work/gutenberg/gutenberg/node_modules/@wdio/utils/build/shim.js:81:29) ``` * refactor: Prefer `action` to `touchAction` Avoid the deprecated `touchAction` command. > The `touchAction` command is deprecated and will be removed in a future version. We recommend to use the action command instead with pointer type touch, e.g.: > > await browser.action('pointer', { > parameters: { pointerType: 'touch' } > }) https://webdriver.io/docs/api/element/touchAction/ This may also relate to the fact that Mobile JSON Wire Protocol is deprecated as well. > Mobile JSON Wire Protocol > > The Mobile JSON Wire Protocol is a super set of mobile commands on top of the JSON Wire Protocol. Given this one is deprecated the Mobile JSON Wire Protocol also got deprecated. Appium might still support some of its commands but it is not recommended to use them. https://webdriver.io/docs/api/protocols#mobile-json-wire-protocol * Revert "test: Increase logging levels" This reverts commit 7bb46de. * test: Set default `swipeFromTo` delay value This prevents `pause` from being called without a parameter, which throws an error. * test: Replace swipe gesture with simple tap (#55427) It was unclear as to why the swipe gesture failed to close the bottom sheet, but a simple tap towards the middle-top of the display achieves the same result. * [Mobile] - Appium 2 Migration - Updating more utils (#55420) * Updates the longPressMiddleOfElement util by using WebdriverIO's action. It adds tapping on the element first before long-pressing due to an issue with iOS not triggering the long-press event. The default waitTime was reduced as well to 1000. * Update selectTextFromElement to WebdriverIO. It also changes the logic for iOS where it uses the "Select All" option from the context menu instead of using "doubleTap". Timeouts have been reduced as well. * Refactors selectTextFromElement for Android, the previous implementation would only select the word that was located in the middle of the element, with the new changes it will select all text by long-pressing from the start of the element to the end. Unfortunately we can't access the context menu as we do on iOS so it has to be done manually. * Add inline duration value and adds inline comments to explain the functionality * [Mobile] - Update closePicker util and updates Image Block test data (#55484) * Update closePicker util name to closeMediaPicker, it also unifies the logic for both platforms by having the same locator and using WebdriverIO's waitForDisplayed. * Update test data for the image block, by changing the image caption to avoid issues in CI and autocorrect * [Mobile] Improve Screen recording on Android for Appium 2 (#55443) * Improve Screenrecording on Android by making sure of the following: The sdcard folder should exist in the emulator, if not there's a script that will create it. For skipped tests, it will remove empty folders that weren't needed. When having multiple Android emulators opened, it will use the deviceID that Appium is currently using. * Rename getAndroidDeviceID function to getDeviceID since it's not platform specific * Fix comment * Temp: log session * Use adb devices list to get the first active android emulator * Moves getAndroidEmulatorID to its own file and replaces its usage. It also adds specifying the device ID for Android local builds for cases when multiple devices/emulators are connected. * Remove unneeded filtering * Move setting the androidDeviceID value for Android cases only * docs: Update E2E test documentation (#55519) * docs: Rewrite end-to-end testing documentation Update for breaking changes from upgrading Appium 2. Reduce verbosity and duplication to improve the approachability and maintainability of the documentation. * docs: Remove E2E test contributing documentation This was replaced by a section in the directory's README documentation. * docs: Rewrite end-to-end testing Getting Started section Update for breaking changes from upgrading Appium 2. Reduce verbosity and duplication between the `WordPress/gutenberg` repository to improve the approachability and maintainability of the documentation. * docs: Update links in Gutenberg for Mobile Apps code documentation Documents have been updated and consolidated to reduce verbosity and duplication to improve the approachability and maintainability of the documentation. * [Mobile] Remove `wd` usage and removes disabled tests (#55541) * Device tests - Removes skipped tests that haven't been enabled back for a while. These were disabled due to flakiness most likely to the interactions with the context menu to copy/paste text. If needed this can be restored once the flakiness issues are worked on. * Device Tests - Drag & Drop: removes disabled tests. These used utils that were flaky and have been disabled for a long time, they can be added back once stable utils are in place for pasting content. * Editor Page - Removes setHtmlContent since this util is not being used anymore. It can be added back if its needed. * Device Tests Utils - Removes usage of the wd library. It also removes, doubleTap, clickElementOutsideOfTextInput, and related utils to the context menu due to them being flaky and not currenlty in use: tapSelectAllAboveElement, tapCopyAboveElement and tapPasteAboveElement. * test: Improve E2E test consistency (#55554) * test: Mitigate conflicts between development server caches Occasionally, cryptic npm package dependency errors can occur between test runs. They appear to originate from stale development caches, e.g. Metro server. Clearing these caches, which E2E tests should not rely upon mitigates the likelihood of these errors. * test: Prefix informative E2E test setup logs Merely an attempt to add structure to the outputted logs. * test: Avoid setup scripts during test runs There may be clarity and efficiency by having an explicit setup script. It was avoid running unnecessary scripts during test runs. It is also referenced in the current documentation as a script to run intentionally. * build: Ignore generated Android files (#55569) These assets are generated at build time for E2E testing. Seemingly, we do not leverage these generated files for the binary/bundle that we provide to the host apps. Therefore, we can safely ignore these files. * docs: Avoid GMA project name until it is approved Use of the "Gutenberg for Mobile Apps" name is still under review. In the meantime, we should use the existing "Mobile Gutenberg" project name. --------- Co-authored-by: David Calhoun <github@davidcalhoun.me>
Related PRs
What?
Update E2E utilities for Appium 2 and WebdriverIO.
Why?
Improve the stability and quality of the automated tests.
How?
WebdriverIO (the replacement for the deprecated WD driver library) does
not have a method for retrieving a device's pixel ratio. Therefore, this
sets an explicit value for each of our test devices.
getAddBlockButton
utilityTesting Instructions
See wordpress-mobile/gutenberg-mobile#6281.
Testing Instructions for Keyboard
n/a
Screenshots or screencast
n/a