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

feat: Android自动化测试步骤中新增拖拽操作以及拖拽动作拆分的三个独立事件操作 #449

Merged

Conversation

YeungHoiChiu
Copy link
Contributor

@YeungHoiChiu YeungHoiChiu commented Jun 3, 2024

Whether this PR is eventually merged or not, Sonic will thank you very much for your contribution.

无论此PR最终是否合并,Sonic组织都非常感谢您的贡献。

Checklist

  • The title starts with fix, fea, or doc. | 标题为fix、feat或doc开头。
  • I have checked that there are no duplicate pull requests with this request. | 我已检查没有与此请求重复的拉取请求。
  • I have considered and confirmed that this submission is valuable to others. | 我已经考虑过,并确认这份呈件对其他人很有价值。
  • I accept that this submission may not be used. | 我接受此提交可能不会被使用。

Description

整体内容描述:

  1. 对接driver新增uia模式的拖拽(drag)、touchAction
  2. 对原来swipe相关方法中出现拖拽词语的进行纠正为滑动

为什么要增加touchAction?

  • 考虑有些拖拽目标定位是在长按图标之后才会出现的,所以将拖拽3个步骤拆成独立步骤,经过测试长按之后接拖拽事件也是可以的

功能测试

  1. 3个模式下,drag操作均符合预期
  2. motionEvent(即touchAction)中down、up操作基本符合预期
  3. adb模式下,move motionEvent符合预期,uia和sonic模式会出现卡顿,即划线过程中会出现一次停顿
    image
  4. motionEvent,在手机桌面上操作图片拖拽,有可能需要执行两次move操作,图标才会移动,可能是机型差异,而在微信小程序菜单中则不会出现这个问题
对应前端:SonicCloudOrg/sonic-client-web#284

@YeungHoiChiu YeungHoiChiu changed the title Feat android uitest draganddrop feat: Android自动化测试步骤中新增拖拽操作以及拖拽动作拆分的三个独立事件操作 Jun 3, 2024
@prlabeler prlabeler bot added the feature label Jun 3, 2024
@YeungHoiChiu YeungHoiChiu marked this pull request as draft June 20, 2024 15:27
@YeungHoiChiu
Copy link
Contributor Author

等我第二天再自测一下再修改回PR Ready

@YeungHoiChiu YeungHoiChiu marked this pull request as ready for review June 21, 2024 08:13
@ZhouYixun ZhouYixun merged commit 3ec07d9 into SonicCloudOrg:main Aug 8, 2024
1 check failed
@YeungHoiChiu YeungHoiChiu deleted the feat-android-uitest-draganddrop branch August 8, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants