这是indexloc提供的服务,不要输入任何密码
Skip to content

test: Flush ui thread runnables before testing its result #2628

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

utzcoz
Copy link

@utzcoz utzcoz commented Jul 18, 2025

It prepares for next Robolectric release.

It prepares for next Robolectric release.

Signed-off-by: utzcoz <utzcoz@outlook.com>
@icbaker icbaker self-assigned this Jul 21, 2025
@icbaker
Copy link
Collaborator

icbaker commented Jul 21, 2025

Please can you give more details on what this is fixing?

I've read this comment thread already: #2590 (review)

I tried bumping robolectric to 4.15.1 locally and running `DashPlaybackTest, and it passed without this change, so I'm afraid I don't understand why it's needed.

@utzcoz
Copy link
Author

utzcoz commented Jul 21, 2025

@icbaker Could you try ./gradlew test after upgrading to 4.15.1 locally?

@icbaker
Copy link
Collaborator

icbaker commented Jul 21, 2025

Could you try ./gradlew test after upgrading to 4.15.1 locally?

I tried this and all the tests pass. Please can you give more details on the error this is fixing?

@utzcoz
Copy link
Author

utzcoz commented Jul 22, 2025

I will give another check this weekend.

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.

2 participants