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

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

Comments

@takahirom
Copy link
Member

takahirom commented Sep 9, 2022

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

@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

@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
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants