Skip to content

CPLAT-13506 Add user-events JS Interop #6

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

Merged
merged 110 commits into from
May 14, 2021
Merged

Conversation

sydneyjodon-wk
Copy link
Contributor

@sydneyjodon-wk sydneyjodon-wk commented May 7, 2021

This PR wraps all the userEvent utilities and exposes them via UserEvent:

  • UserEvent.click
  • UserEvent.dblClick
  • UserEvent.type
  • UserEvent.typeWithDelay (needs to be a separate method to return a Future)
  • UserEvent.keyboard
  • UserEvent.keyboardWithDelay (needs to be a separate method to return a Future)
  • UserEvent.upload
  • UserEvent.clear
  • UserEvent.selectOptions
  • UserEvent.deselectOptions
  • UserEvent.tab
  • UserEvent.hover
  • UserEvent.unhover
  • UserEvent.paste

Note: The early commit history got messed up because I was basing off Aaron's branch while he was still working on it, but the diff for this PR is correct.

QA Instructions

  • CI passes.
  • Verify test coverage.
  • Download dartdoc artifact from CI and serve it via dhttpd <path to dartdoc directory> and verify the docs look as expected.

Copy link
Contributor

@aaronlademann-wf aaronlademann-wf left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Contributor

@greglittlefield-wf greglittlefield-wf left a comment

Choose a reason for hiding this comment

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

Some comments, but nothing major.

I'm super excited to use these APIs to write tests!!

Copy link
Contributor

@greglittlefield-wf greglittlefield-wf left a comment

Choose a reason for hiding this comment

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

+10

@rmconsole-wf
Copy link

@greglittlefield-wf your review was submitted after a new commit (0b71921) was pushed, but is being applied to 875435c. Please review the latest commit and resubmit your approval.

Copy link
Contributor

@greglittlefield-wf greglittlefield-wf left a comment

Choose a reason for hiding this comment

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

+10

Copy link
Contributor

@greglittlefield-wf greglittlefield-wf left a comment

Choose a reason for hiding this comment

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

Thanks for addressing that last bit even though you didn't have to!

+10

Copy link
Contributor

@aaronlademann-wf aaronlademann-wf left a comment

Choose a reason for hiding this comment

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

+1

@aaronlademann-wf
Copy link
Contributor

@Workiva/release-management-pp

Copy link

@rmconsole-wf rmconsole-wf left a comment

Choose a reason for hiding this comment

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

+1 from RM

@rm-astro-wf rm-astro-wf deleted the CPLAT-13506-user-events branch May 14, 2021 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants