Skip to content

test: Update Unsupported Block tests for Appium 2#6281

Merged
dcalhoun merged 11 commits into
version-toolkit/gutenberg/rnmobile/appium-2from
test/appium-update-unsupported-block
Oct 16, 2023
Merged

test: Update Unsupported Block tests for Appium 2#6281
dcalhoun merged 11 commits into
version-toolkit/gutenberg/rnmobile/appium-2from
test/appium-update-unsupported-block

Conversation

@dcalhoun

@dcalhoun dcalhoun commented Oct 12, 2023

Copy link
Copy Markdown
Member

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
Comment thread __device-tests__/utils.js
@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

@geriux geriux left a comment

Copy link
Copy Markdown
Contributor

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