For #10342 - Part 7: Fix TopSiteViewHolderTest and TopSiteItemViewHolderTest

pull/35/head
Gabriel Luong 4 years ago committed by Jonathan Almeida
parent 4f76699ab7
commit 0cc1d32e56

@ -23,7 +23,8 @@ class TestComponents(private val context: Context) : Components(context) {
core.sessionManager, core.sessionManager,
core.store, core.store,
search.searchEngineManager, search.searchEngineManager,
core.webAppShortcutManager core.webAppShortcutManager,
core.topSiteStorage
) )
} }
override val intentProcessors by lazy { mockk<IntentProcessors>(relaxed = true) } override val intentProcessors by lazy { mockk<IntentProcessors>(relaxed = true) }

@ -14,6 +14,7 @@ import mozilla.components.concept.engine.Engine
import mozilla.components.concept.engine.Settings import mozilla.components.concept.engine.Settings
import mozilla.components.concept.fetch.Client import mozilla.components.concept.fetch.Client
import mozilla.components.feature.pwa.WebAppShortcutManager import mozilla.components.feature.pwa.WebAppShortcutManager
import mozilla.components.feature.top.sites.DefaultTopSitesStorage
import mozilla.components.support.base.crash.CrashReporting import mozilla.components.support.base.crash.CrashReporting
class TestCore(context: Context, crashReporter: CrashReporting) : Core(context, crashReporter) { class TestCore(context: Context, crashReporter: CrashReporting) : Core(context, crashReporter) {
@ -26,4 +27,5 @@ class TestCore(context: Context, crashReporter: CrashReporting) : Core(context,
override val client = mockk<Client>() override val client = mockk<Client>()
override val webAppShortcutManager = mockk<WebAppShortcutManager>() override val webAppShortcutManager = mockk<WebAppShortcutManager>()
override val thumbnailStorage = mockk<ThumbnailStorage>() override val thumbnailStorage = mockk<ThumbnailStorage>()
override val topSiteStorage = mockk<DefaultTopSitesStorage>()
} }

@ -32,14 +32,17 @@ class TopSiteViewHolderTest {
@Test @Test
fun `binds list of top sites`() { fun `binds list of top sites`() {
TopSiteViewHolder(view, interactor).bind(listOf( TopSiteViewHolder(view, interactor).bind(
object : TopSite { listOf(
override val id = 1L TopSite(
override val isDefault = true id = 1L,
override val title = "Pocket" title = "Pocket",
override val url = "https://getpocket.com" url = "https://getpocket.com",
} createdAt = 0,
)) type = TopSite.Type.DEFAULT
)
)
)
assertEquals(1, view.top_sites_list.adapter!!.itemCount) assertEquals(1, view.top_sites_list.adapter!!.itemCount)
} }

@ -22,12 +22,13 @@ class TopSiteItemViewHolderTest {
private lateinit var view: View private lateinit var view: View
private lateinit var interactor: TopSiteInteractor private lateinit var interactor: TopSiteInteractor
private val pocket = object : TopSite { private val pocket = TopSite(
override val id = 1L id = 1L,
override val isDefault = true title = "Pocket",
override val title = "Pocket" url = "https://getpocket.com",
override val url = "https://getpocket.com" createdAt = 0,
} type = TopSite.Type.DEFAULT
)
@Before @Before
fun setup() { fun setup() {

Loading…
Cancel
Save