diff --git a/app/src/main/java/org/mozilla/fenix/home/topsites/TopSiteItemViewHolder.kt b/app/src/main/java/org/mozilla/fenix/home/topsites/TopSiteItemViewHolder.kt index ab13c2abba..5af87487f1 100644 --- a/app/src/main/java/org/mozilla/fenix/home/topsites/TopSiteItemViewHolder.kt +++ b/app/src/main/java/org/mozilla/fenix/home/topsites/TopSiteItemViewHolder.kt @@ -39,7 +39,7 @@ class TopSiteItemViewHolder( private val binding = TopSiteItemBinding.bind(view) init { - binding.topSiteItem.setOnLongClickListener { + itemView.setOnLongClickListener { interactor.onTopSiteMenuOpened() TopSites.longPress.record(TopSites.LongPressExtra(topSite.name())) @@ -72,7 +72,7 @@ class TopSiteItemViewHolder( } fun bind(topSite: TopSite, position: Int) { - binding.topSiteItem.setOnClickListener { + itemView.setOnClickListener { interactor.onSelectTopSite(topSite, position) } diff --git a/app/src/test/java/org/mozilla/fenix/home/topsites/TopSiteItemViewHolderTest.kt b/app/src/test/java/org/mozilla/fenix/home/topsites/TopSiteItemViewHolderTest.kt index 97aef255b7..cd41e14d8d 100644 --- a/app/src/test/java/org/mozilla/fenix/home/topsites/TopSiteItemViewHolderTest.kt +++ b/app/src/test/java/org/mozilla/fenix/home/topsites/TopSiteItemViewHolderTest.kt @@ -58,7 +58,7 @@ class TopSiteItemViewHolderTest { fun `calls interactor on click`() { TopSiteItemViewHolder(binding.root, lifecycleOwner, interactor).bind(pocket, position = 0) - binding.topSiteItem.performClick() + binding.root.performClick() verify { interactor.onSelectTopSite(pocket, position = 0) } } @@ -67,7 +67,7 @@ class TopSiteItemViewHolderTest { every { testContext.components.analytics } returns mockk(relaxed = true) TopSiteItemViewHolder(binding.root, lifecycleOwner, interactor).bind(pocket, position = 0) - binding.topSiteItem.performLongClick() + binding.root.performLongClick() verify { interactor.onTopSiteMenuOpened() } }