fix: Discussion Test Cases

Fixes: LEARNER-10372
edx · Jan 12, 2025 · caa9acc · caa9acc
1 parent 1cb0ef1
commit caa9acc
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 3 deletions.
diff --git a/...test/java/org/openedx/discussion/presentation/comments/DiscussionCommentsViewModelTest.kt b/...test/java/org/openedx/discussion/presentation/comments/DiscussionCommentsViewModelTest.kt
@@ -137,6 +137,7 @@ class DiscussionCommentsViewModelTest {
     @Before
     fun setUp() {
         Dispatchers.setMain(dispatcher)
+        every { analytics.logScreenEvent(any(), any()) } returns Unit
         every { resourceManager.getString(R.string.core_error_no_connection) } returns noInternet
         every { resourceManager.getString(R.string.core_error_unknown_error) } returns somethingWrong
         every { resourceManager.getString(org.openedx.discussion.R.string.discussion_comment_added) } returns commentAddedSuccessfully
@@ -1109,4 +1110,4 @@ class DiscussionCommentsViewModelTest {
         assert(viewModel.uiState.value is DiscussionCommentsUIState.Success)
     }
 
-}
+}
diff --git a/...st/java/org/openedx/discussion/presentation/responses/DiscussionResponsesViewModelTest.kt b/...st/java/org/openedx/discussion/presentation/responses/DiscussionResponsesViewModelTest.kt
@@ -129,6 +129,7 @@ class DiscussionResponsesViewModelTest {
     @Before
     fun setUp() {
         Dispatchers.setMain(dispatcher)
+        every { analytics.logScreenEvent(any(), any()) } returns Unit
         every { resourceManager.getString(R.string.core_error_no_connection) } returns noInternet
         every { resourceManager.getString(R.string.core_error_unknown_error) } returns somethingWrong
         every { resourceManager.getString(org.openedx.discussion.R.string.discussion_comment_added) } returns commentAddedSuccessfully
@@ -631,4 +632,4 @@ class DiscussionResponsesViewModelTest {
         assert(viewModel.uiState.value is DiscussionResponsesUIState.Success)
     }
 
-}
+}
diff --git a/...c/test/java/org/openedx/discussion/presentation/threads/DiscussionThreadsViewModelTest.kt b/...c/test/java/org/openedx/discussion/presentation/threads/DiscussionThreadsViewModelTest.kt
@@ -32,6 +32,7 @@ import org.openedx.core.system.ResourceManager
 import org.openedx.discussion.domain.interactor.DiscussionInteractor
 import org.openedx.discussion.domain.model.DiscussionType
 import org.openedx.discussion.domain.model.ThreadsData
+import org.openedx.discussion.presentation.DiscussionAnalytics
 import org.openedx.discussion.presentation.topics.DiscussionTopicsViewModel
 import org.openedx.discussion.system.notifier.DiscussionNotifier
 import org.openedx.discussion.system.notifier.DiscussionThreadAdded
@@ -49,6 +50,7 @@ class DiscussionThreadsViewModelTest {
     private val resourceManager = mockk<ResourceManager>()
     private val interactor = mockk<DiscussionInteractor>()
     private val notifier = mockk<DiscussionNotifier>()
+    private val analytics = mockk<DiscussionAnalytics>()
 
     private val noInternet = "Slow or no internet connection"
     private val somethingWrong = "Something went wrong"
@@ -128,6 +130,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         advanceUntilIdle()
 
@@ -148,6 +151,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         coEvery { interactor.getAllThreads(any(), any(), any(), any()) } throws Exception()
         advanceUntilIdle()
@@ -179,6 +183,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         advanceUntilIdle()
 
@@ -207,6 +212,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         advanceUntilIdle()
 
@@ -227,6 +233,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         coEvery {
             interactor.getFollowingThreads(
@@ -282,6 +289,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         advanceUntilIdle()
 
@@ -310,6 +318,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         advanceUntilIdle()
 
@@ -330,6 +339,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         coEvery { interactor.getThreads(any(), any(), any(), any(), any()) } throws Exception()
         advanceUntilIdle()
@@ -361,6 +371,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         advanceUntilIdle()
 
@@ -380,6 +391,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         coEvery { interactor.getThreads(any(), any(), any(), any(), any()) } returns ThreadsData(
             threads,
@@ -400,6 +412,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         coEvery { interactor.getThreads(any(), any(), any(), any(), any()) } returns ThreadsData(
             threads,
@@ -420,6 +433,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         coEvery { interactor.getThreads(any(), any(), any(), any(), any()) } returns ThreadsData(
             threads,
@@ -450,6 +464,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
         viewModel.updateThread("")
         advanceUntilIdle()
@@ -486,6 +501,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
 
 
@@ -525,6 +541,7 @@ class DiscussionThreadsViewModelTest {
             interactor,
             resourceManager,
             notifier,
+            analytics,
         )
 
         val mockLifeCycleOwner: LifecycleOwner = mockk()
@@ -539,4 +556,4 @@ class DiscussionThreadsViewModelTest {
     }
 
 
-}
+}