diff --git a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailProvider.java b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailProvider.java deleted file mode 100644 index 358db00e2..000000000 --- a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailProvider.java +++ /dev/null @@ -1,15 +0,0 @@ -package fr.free.nrw.commons.media; - -import fr.free.nrw.commons.Media; - -public interface MediaDetailProvider { - - Media getMediaAtPosition(int i); - - int getTotalMediaCount(); - - Integer getContributionStateAt(int position); - - // Reload media detail fragment once media is nominated - void refreshNominatedMedia(int index); -} diff --git a/app/src/main/java/fr/free/nrw/commons/media/MediaDetailProvider.kt b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailProvider.kt new file mode 100644 index 000000000..591adfe75 --- /dev/null +++ b/app/src/main/java/fr/free/nrw/commons/media/MediaDetailProvider.kt @@ -0,0 +1,14 @@ +package fr.free.nrw.commons.media + +import fr.free.nrw.commons.Media + +interface MediaDetailProvider { + fun getMediaAtPosition(i: Int): Media? + + fun getTotalMediaCount(): Int + + fun getContributionStateAt(position: Int): Int? + + // Reload media detail fragment once media is nominated + fun refreshNominatedMedia(index: Int) +} diff --git a/app/src/test/kotlin/fr/free/nrw/commons/bookmarks/BookmarkListRootFragmentUnitTest.kt b/app/src/test/kotlin/fr/free/nrw/commons/bookmarks/BookmarkListRootFragmentUnitTest.kt index 5098bd0c1..d168fe6e4 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/bookmarks/BookmarkListRootFragmentUnitTest.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/bookmarks/BookmarkListRootFragmentUnitTest.kt @@ -235,7 +235,7 @@ class BookmarkListRootFragmentUnitTest { @Throws(Exception::class) fun testGetTotalMediaCountCaseNull() { whenever(bookmarksPagerAdapter.mediaAdapter).thenReturn(null) - Assert.assertEquals(fragment.totalMediaCount, 0) + Assert.assertEquals(fragment.getTotalMediaCount(), 0) } @Test @@ -244,7 +244,7 @@ class BookmarkListRootFragmentUnitTest { val listAdapter = mock(ListAdapter::class.java) whenever(bookmarksPagerAdapter.mediaAdapter).thenReturn(listAdapter) whenever(listAdapter.count).thenReturn(1) - Assert.assertEquals(fragment.totalMediaCount, 1) + Assert.assertEquals(fragment.getTotalMediaCount(), 1) } @Test diff --git a/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryDetailsActivityUnitTests.kt b/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryDetailsActivityUnitTests.kt index fd6bc7976..b1f40a08f 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryDetailsActivityUnitTests.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/category/CategoryDetailsActivityUnitTests.kt @@ -76,7 +76,7 @@ class CategoryDetailsActivityUnitTests { @Test @Throws(Exception::class) fun testGetTotalMediaCount() { - activity.totalMediaCount + activity.getTotalMediaCount() } @Test diff --git a/app/src/test/kotlin/fr/free/nrw/commons/contributions/ContributionsFragmentUnitTests.kt b/app/src/test/kotlin/fr/free/nrw/commons/contributions/ContributionsFragmentUnitTests.kt index e3f1c86cc..16848f7dd 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/contributions/ContributionsFragmentUnitTests.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/contributions/ContributionsFragmentUnitTests.kt @@ -334,7 +334,7 @@ class ContributionsFragmentUnitTests { @Throws(Exception::class) fun testGetTotalMediaCount() { Shadows.shadowOf(Looper.getMainLooper()).idle() - fragment.totalMediaCount + fragment.getTotalMediaCount() } @Test diff --git a/app/src/test/kotlin/fr/free/nrw/commons/explore/ExploreListRootFragmentUnitTest.kt b/app/src/test/kotlin/fr/free/nrw/commons/explore/ExploreListRootFragmentUnitTest.kt index 9dc94293f..62068f4f9 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/explore/ExploreListRootFragmentUnitTest.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/explore/ExploreListRootFragmentUnitTest.kt @@ -189,8 +189,8 @@ class ExploreListRootFragmentUnitTest { @Test @Throws(Exception::class) fun testGetTotalMediaCount() { - `when`(listFragment.totalMediaCount).thenReturn(1) - Assert.assertEquals(fragment.totalMediaCount, 1) + `when`(listFragment.getTotalMediaCount()).thenReturn(1) + Assert.assertEquals(fragment.getTotalMediaCount(), 1) } @Test @@ -199,7 +199,7 @@ class ExploreListRootFragmentUnitTest { val field: Field = ExploreListRootFragment::class.java.getDeclaredField("listFragment") field.isAccessible = true field.set(fragment, null) - Assert.assertEquals(fragment.totalMediaCount, 0) + Assert.assertEquals(fragment.getTotalMediaCount(), 0) } @Test diff --git a/app/src/test/kotlin/fr/free/nrw/commons/explore/depictions/WikidataItemDetailsActivityUnitTests.kt b/app/src/test/kotlin/fr/free/nrw/commons/explore/depictions/WikidataItemDetailsActivityUnitTests.kt index bf5aca6e2..8ba2e86a8 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/explore/depictions/WikidataItemDetailsActivityUnitTests.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/explore/depictions/WikidataItemDetailsActivityUnitTests.kt @@ -108,7 +108,7 @@ class WikidataItemDetailsActivityUnitTests { @Test @Throws(Exception::class) fun testGetTotalMediaCount() { - activity.totalMediaCount + activity.getTotalMediaCount() } @Test diff --git a/app/src/test/kotlin/fr/free/nrw/commons/explore/search/SearchActivityUnitTests.kt b/app/src/test/kotlin/fr/free/nrw/commons/explore/search/SearchActivityUnitTests.kt index 751046e7f..00b9c0fd9 100644 --- a/app/src/test/kotlin/fr/free/nrw/commons/explore/search/SearchActivityUnitTests.kt +++ b/app/src/test/kotlin/fr/free/nrw/commons/explore/search/SearchActivityUnitTests.kt @@ -160,8 +160,8 @@ class SearchActivityUnitTests { fun testGetTotalMediaCount() { val num = 1 Whitebox.setInternalState(activity, "searchMediaFragment", searchMediaFragment) - `when`(searchMediaFragment.totalMediaCount).thenReturn(num) - assertEquals(activity.totalMediaCount, num) + `when`(searchMediaFragment.getTotalMediaCount()).thenReturn(num) + assertEquals(activity.getTotalMediaCount(), num) } @Test