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.

Sign up for GitHub

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

Jump to bottom

#2311 fix appium combined locator in reports #2315

Merged
merged 1 commit into from May 28, 2023
Merged

#2311 fix appium combined locator in reports #2315

merged 1 commit into from May 28, 2023

Conversation

asolntsev
Copy link
Member

@asolntsev asolntsev commented May 28, 2023 •
edited

fix for #2311

now the report will look better:

| [android:by [text=Graphi], ios:By.xpath:/]    | should have(text "Graphics") | PASS      | 1098       |
| [name*=ComputeSum]                            | click()                      | PASS      | 918        |
| [name*=Answ]                                  | should have(text "6")        | PASS      | 361        |
| XCUIElementTypeStaticText[name*=Answ]         | should have(text "6")        | PASS      | 306        |
| XCUIElementTypeStaticText[name=Answer]        | should have(text "6")        | PASS      | 313        |

@asolntsev asolntsev added this to the 6.15.0 milestone May 28, 2023
@asolntsev asolntsev linked an issue May 28, 2023 that may be closed by this pull request
Selenide-appium: [unknow locator] is shown in profiler when CombinedBy is used #2311
Closed
@sonarcloud
Copy link

sonarcloud bot commented May 28, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@asolntsev asolntsev self-assigned this May 28, 2023
@asolntsev asolntsev merged commit 036cbc0 into main May 28, 2023
14 checks passed
@asolntsev asolntsev deleted the 2311-fix-appium-combined-locator-in-reports branch May 28, 2023 16:59
@asolntsev asolntsev mentioned this pull request May 28, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@asolntsev asolntsev

Labels
appium bug 🐛 reports
Projects
None yet
Milestone
6.15.0
Development

Successfully merging this pull request may close these issues.

Selenide-appium: [unknow locator] is shown in profiler when CombinedBy is used
1 participant
@asolntsev