Write lint check for TestLifecycleOwner use in coroutines [297880630]

Fixed

Bug

Status Update

No update yet.

Description

sa...@google.com

created issue #1

Aug 28, 2023 04:57PM

This is a followup to

https://b.corp.google.com/issues/259344129

We want to add a lint check to guide users to using the new `TestLifecycleOwner` API `setCurrentState` from the context of a coroutine (i.e. `runTest`), rather than accessing the `currentState` property directly.

Comments

il...@google.com <il...@google.com> Sep 28, 2023 10:14PM

Reassigned to jb...@google.com.

ap...@google.com <ap...@google.com> #2Oct 11, 2023 07:41PM

Yigit, do you have time to fix it?

reemission of the same liveData is racy

il...@google.com <il...@google.com> #3Oct 11, 2023 07:56PM

Marked as fixed.

yea i'll take it.

pr...@google.com <pr...@google.com> #4Oct 18, 2023 08:42PM

Thanks for the detailed analysis. This may not be an issue anymore since we've started using Main.immediate there but I' not sure; I'll try to create a test case.

Issue 297880630

Description

Issue summary

Comments

il...@google.com <il...@google.com> Sep 28, 2023 10:14PM

ap...@google.com <ap...@google.com> #2Oct 11, 2023 07:41PM

il...@google.com <il...@google.com> #3Oct 11, 2023 07:56PM

pr...@google.com <pr...@google.com> #4Oct 18, 2023 08:42PM

Add comment

Issue metadata