Skip to content
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 Unsupported Block tests for Appium 2 #6281

Conversation

dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Oct 12, 2023

Related PRs

What?

Relates to #6276. Update the Unsupported Block tests for Appium 2 and
WebdriverIO.

Why?

Improve the stability and quality of the automated tests.

How?

  • 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: Replace sleep with pause utility

Testing Instructions

Local tests should pass for both platforms when running the following commands:

TEST_RN_PLATFORM=ios npm run device-tests:local -- gutenberg-editor-unsupported-block-visual
TEST_RN_PLATFORM=android npm run device-tests:local -- gutenberg-editor-unsupported-block-visual

PR submission checklist:

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes more info and have added them to RELEASE-NOTES.txt if necessary.

@dcalhoun dcalhoun added Testing Anything related to automated tests [Type] Task labels Oct 12, 2023
@dcalhoun dcalhoun self-assigned this Oct 12, 2023
Update syntax for WebdriverIO, which is a part of the Appium 2 upgrade.
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.
There appears to be a small delay before the selection border displays.
It is unclear why this image needs updating. Its sibling iOS image was
updated in:

4346add
@dcalhoun dcalhoun force-pushed the test/appium-update-unsupported-block branch from f5fa9ff to b2f9ce4 Compare October 12, 2023 20:48
@dcalhoun dcalhoun marked this pull request as ready for review October 13, 2023 17:27
@dcalhoun dcalhoun requested a review from geriux October 13, 2023 17:27
Copy link
Contributor

@geriux geriux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Tests succeeded as expected 🚀

@dcalhoun dcalhoun merged commit 266c613 into version-toolkit/gutenberg/rnmobile/appium-2 Oct 16, 2023
@dcalhoun dcalhoun deleted the test/appium-update-unsupported-block branch October 16, 2023 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing Anything related to automated tests [Type] Task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants