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

Search Screen Robot testing #327

Open
takahirom opened this issue Sep 9, 2022 · 5 comments · May be fixed by #825
Open

Search Screen Robot testing #327

takahirom opened this issue Sep 9, 2022 · 5 comments · May be fixed by #825
Assignees
@yaoandy107
Labels
welcome contribute

Comments

@takahirom
Copy link
Member

takahirom commented Sep 9, 2022 •
edited
Loading

Idea Description
It can be exciting.
You can refer SessionScreenTest.
I would like to create a test like this.

searchRobot() {
  typeQuery("aaa")
  checkSessionExists()
}

Reference images and links
https://github.com/DroidKaigi/conference-app-2022/blob/main/feature-sessions/src/test/java/io/github/droidkaigi/confsched2022/feature/sessions/SessionsScreenTest.kt
@yaoandy107
Copy link

I'd like to do this task 🙋

@takahirom
Copy link
Member Author

Thanks! I think this is fun issue! If you need any help let us know👍

@yaoandy107
Copy link

yaoandy107 commented Oct 1, 2022 •
edited
Loading

@takahirom Sorry for the lateness. I worked on this a while ago, and it worked fine at the time. But yesterday, when I pulled changes from the main branch. I found out that there's an issue with the new implementation of the SearchTextField in the espresso test. Whenever I type anything on the SearchTextField and all the espresso commands after it will throw an exception

java.lang.IndexOutOfBoundsException: offset(x) should be less than line limit(0)

I have taken some time to search for this issue but have no idea how to solve it. The only issue I found a bit related is this issue.

@takahirom
Copy link
Member Author

takahirom commented Oct 1, 2022 •
edited
Loading

@yaoandy107 You don't have to fix it by yourself. Could you create a pull request with the current status?

@yaoandy107 yaoandy107 linked a pull request Oct 1, 2022 that will close this issue
Open
1 task
@yaoandy107
Copy link

@takahirom I have created a PR for my current status~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yaoandy107 yaoandy107

Labels
welcome contribute
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[WIP] Add search screen testing yaoandy107/conference-app-2022
2 participants
@takahirom @yaoandy107