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: Add Sanity Test Suite 1 #5548

Merged
merged 29 commits into from
Mar 27, 2023
Merged

test: Add Sanity Test Suite 1 #5548

merged 29 commits into from
Mar 27, 2023

Conversation

dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Mar 9, 2023

Related PRs

Description

  • Add Columns block placeholder test
  • Add Columns block display test
  • Add Columns block mover buttons test
  • Add Columns block dark mode test
  • Add Columns proportionate fill previews test

To test: n/a

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 [Type] Enhancement Improves a current area of the editor Testing Anything related to automated tests labels Mar 9, 2023
@dcalhoun dcalhoun self-assigned this Mar 9, 2023
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Mar 10, 2023

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@dcalhoun dcalhoun marked this pull request as ready for review March 10, 2023 01:46
@dcalhoun dcalhoun requested a review from geriux March 10, 2023 01:48
@dcalhoun
Copy link
Member Author

dcalhoun commented Mar 10, 2023

👋🏻 @geriux. I refrained from creating new helpers so far, as I wanted to avoid abstracing prematurely. That said, I am open to any you believe are worthwhile. I'll likely re-review the tests myself seeking low-level abstractions that may make sense, but wanted to request your review now regardless.

Let me know what you think. Thanks!

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.

Hey @dcalhoun 👋 I've left some initial comments, so exciting to see more tests added!

dcalhoun and others added 9 commits March 13, 2023 14:53
Co-authored-by: Gerardo Pacheco <[email protected]>
The available elements within this context are unexpected, with certain
elements unavailable or covered by non-visual elements, e.g. resize
observer. I consider this change a workaround. There is likely a better
way of achieving this, possibly an existing helper that could be used or
modified to support this context.
Increase the stability and adaptability of the slider interaction logic.
@dcalhoun dcalhoun requested a review from geriux March 17, 2023 21:28
@dcalhoun
Copy link
Member Author

@geriux this PR and the related PR are both ready for another review. Thanks!

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! I left one comment related to the name of the test but overall looks good to me! Thank you for making the changes, so exciting to have more tests being added! ❤️

I also approved the Gutenberg PR via WordPress/gutenberg#48978 (review)

@dcalhoun dcalhoun enabled auto-merge March 27, 2023 14:16
@dcalhoun dcalhoun merged commit 5b65e25 into trunk Mar 27, 2023
@dcalhoun dcalhoun deleted the test/add-sanity-test-suite-1 branch March 27, 2023 17:18
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] Enhancement Improves a current area of the editor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants