[components] For mozilla-mobile/android-components#12184 - Stop refre…

…shing sponsored stories if profile is deleted

This also allows to easily start a "new" Worker to download stories again when
`startPeriodicStoriesRefresh` whereas we'd just enqueue the existing one which
may be set to only refresh the stories after a few hours still.

android-components/components/service/pocket/src/main/java/mozilla/components/service/pocket/PocketStoriesService.kt

@@ -128,6 +128,7 @@ class PocketStoriesService(
      * Delete all stored user data used for downloading personalized sponsored stories.
      */
     suspend fun deleteProfile(): Boolean {
+        stopPeriodicSponsoredStoriesRefresh()
         return spocsUseCases?.deleteProfile?.invoke() ?: false
     }
 

android-components/components/service/pocket/src/test/java/mozilla/components/service/pocket/PocketStoriesServiceTest.kt

@@ -31,6 +31,8 @@ import org.junit.Test
 import org.junit.runner.RunWith
 import org.mockito.Mockito.doReturn
 import org.mockito.Mockito.never
+import org.mockito.Mockito.spy
+import org.mockito.Mockito.times
 import org.mockito.Mockito.verify
 import java.util.UUID
 import kotlin.reflect.KVisibility
@@ -167,14 +169,16 @@ class PocketStoriesServiceTest {
 
     @Test
     fun `GIVEN PocketStoriesService WHEN deleteProfile THEN delegate to spocs useCases`() = runTest {
-        val noProfileResponse = service.deleteProfile()
+        val mockedService = spy(service)
+        val noProfileResponse = mockedService.deleteProfile()
         assertFalse(noProfileResponse)
 
         val deleteProfileUseCase: DeleteProfile = mock()
         doReturn(deleteProfileUseCase).`when`(spocsUseCases).deleteProfile
         doReturn(true).`when`(deleteProfileUseCase).invoke()
-        val existingProfileResponse = service.deleteProfile()
+        val existingProfileResponse = mockedService.deleteProfile()
         assertTrue(existingProfileResponse)
+        verify(mockedService, times(2)).stopPeriodicSponsoredStoriesRefresh()
     }
 
     @Test