From 5964fe5d16defade43086886912f23ceb716ae07 Mon Sep 17 00:00:00 2001 From: Sujal-Gupta-SG Date: Mon, 3 Feb 2025 19:51:22 +0530 Subject: [PATCH] correction --- .../commons/contributions/ContributionsLocalDataSource.kt | 5 +++-- .../nrw/commons/contributions/ContributionsRepository.kt | 2 +- .../contributions/ContributionsListFragmentUnitTests.kt | 2 +- .../free/nrw/commons/contributions/MainActivityUnitTests.kt | 4 ++-- 4 files changed, 7 insertions(+), 6 deletions(-) diff --git a/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsLocalDataSource.kt b/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsLocalDataSource.kt index a64581eb8..2f6b72992 100644 --- a/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsLocalDataSource.kt +++ b/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsLocalDataSource.kt @@ -64,8 +64,9 @@ class ContributionsLocalDataSource @Inject constructor( return contributionDao.deleteContributionsWithStates(states) } - val contributions: DataSource.Factory - get() = contributionDao.fetchContributions() + fun getContributions(): DataSource.Factory { + return contributionDao.fetchContributions() + } /** * Fetches contributions with specific states. diff --git a/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsRepository.kt b/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsRepository.kt index a9714bf62..a78aff582 100644 --- a/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsRepository.kt +++ b/app/src/main/java/fr/free/nrw/commons/contributions/ContributionsRepository.kt @@ -47,7 +47,7 @@ class ContributionsRepository @Inject constructor(private val localDataSource: C } fun fetchContributions(): DataSource.Factory { - return localDataSource.contributions + return localDataSource.getContributions() } /** diff --git a/app/src/test/kotlin/fr/free/nrw/commons/contributions/ContributionsListFragmentUnitTests.kt b/app/src/test/kotlin/fr/free/nrw/commons/contributions/ContributionsListFragmentUnitTests.kt index db2475f63..54228bc13 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/contributions/ContributionsListFragmentUnitTests.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/contributions/ContributionsListFragmentUnitTests.kt @@ -89,7 +89,7 @@ class ContributionsListFragmentUnitTests { Shadows.shadowOf(Looper.getMainLooper()).idle() fragment.rvContributionsList = mock() fragment.scrollToTop() - verify(fragment.rvContributionsList).smoothScrollToPosition(0) + verify(fragment.rvContributionsList)?.smoothScrollToPosition(0) } @Test diff --git a/app/src/test/kotlin/fr/free/nrw/commons/contributions/MainActivityUnitTests.kt b/app/src/test/kotlin/fr/free/nrw/commons/contributions/MainActivityUnitTests.kt index 780322603..b3750c5f3 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/contributions/MainActivityUnitTests.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/contributions/MainActivityUnitTests.kt @@ -448,7 +448,7 @@ class MainActivityUnitTests { fun testOnSetUpPagerNearBy() { val item = Mockito.mock(MenuItem::class.java) `when`(item.title).thenReturn(activity.getString(R.string.nearby_fragment)) - activity.navListener.onNavigationItemSelected(item) + activity.navListener?.onNavigationItemSelected(item) verify(item, Mockito.times(3)).title verify(applicationKvStore, Mockito.times(1)) .putBoolean("last_opened_nearby", true) @@ -459,7 +459,7 @@ class MainActivityUnitTests { fun testOnSetUpPagerOtherThanNearBy() { val item = Mockito.mock(MenuItem::class.java) `when`(item.title).thenReturn(activity.getString(R.string.bookmarks)) - activity.navListener.onNavigationItemSelected(item) + activity.navListener?.onNavigationItemSelected(item) verify(item, Mockito.times(3)).title verify(applicationKvStore, Mockito.times(1)) .putBoolean("last_opened_nearby", false)