commit 601bd57087cd2a943f9533cbd37e488946e460a9
author Clara Fok <clarafok@google.com> Fri Aug 06 17:10:00 2021 -0700
committer Clara Fok <clarafok@google.com> Fri Aug 06 17:10:00 2021 -0700
tree cfa7996af2cdf7af51e9e297637f10a97aa4b3d2
parent 35301616708f420bdb4eca8b9d0579010d5ec8dd

Fix flaky test in LimitOffsetPagingSourceTest

Fixes test load_initialEmptyLoad

Bug: 195794422
Fixes: 195794422
Test: ./gradlew room:room-paging:cC
Change-Id: I8091d6290818402ca68efa06758ea1f3622122ea

room/room-paging/src/androidTest/kotlin/androidx/room/paging/LimitOffsetPagingSourceTest.kt

diff --git a/room/room-paging/src/androidTest/kotlin/androidx/room/paging/LimitOffsetPagingSourceTest.kt b/room/room-paging/src/androidTest/kotlin/androidx/room/paging/LimitOffsetPagingSourceTest.kt
index abcf013..0b2d513 100644
--- a/room/room-paging/src/androidTest/kotlin/androidx/room/paging/LimitOffsetPagingSourceTest.kt
+++ b/room/room-paging/src/androidTest/kotlin/androidx/room/paging/LimitOffsetPagingSourceTest.kt
@@ -172,11 +172,15 @@
             // now add items
             dao.addAllItems(itemsList)
 
-            // the db write should cause pagingSource to realize it is invalid
+            // invalidate pagingSource to imitate invalidation from running refreshVersionSync
+            pagingSource.invalidate()
+            assertTrue(pagingSource.invalid)
+
+            // this refresh should check pagingSource's invalid status, realize it is invalid, and
+            // return a LoadResult.Invalid
             assertThat(pagingSource.refresh()).isInstanceOf(
                 LoadResult.Invalid::class.java
             )
-            assertTrue(pagingSource.invalid)
         }
     }