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

Fix emptyResultIsReturned_withNotMatchingQuery unit test #1040

Merged
merged 3 commits into from
Feb 20, 2024

Conversation

SimonMarquis
Copy link
Contributor

searchResultUiState transitively relied on getSearchContentsCount updates and on userDataRepository to emit something.

@SimonMarquis
Copy link
Contributor Author

@thagikura gentle ping since you introduced this TODO in #685 🙂

// TODO: Figure out to get the latest emitted ui State? The result is emitted as EmptyQuery

@thagikura
Copy link
Contributor

LGTM. Thanks for doing this! But since I no longer work at Google, please have someone from Google look at it.

@dturner FYI

@dfavaro
Copy link

dfavaro commented Jan 31, 2024

Hi @SimonMarquis and @dturner, can you please review this PR and merge it? Would be nice to have these changes available into main

`searchResultUiState` transitively relied on `getSearchContentsCount` updates and on `userDataRepository` to emit something.
@dturner dturner merged commit 1d2029a into android:main Feb 20, 2024
4 checks passed
@SimonMarquis SimonMarquis deleted the fix-unit-test branch February 20, 2024 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants