Skip to content

Commit 9cf7836

Browse files
Apply suggestions from code review
Co-authored-by: Gabriel Luong <gabriel.luong@gmail.com> Signed-off-by: Devota Aabel <14295757+devotaaabel@users.noreply.github.com>
1 parent f12911b commit 9cf7836

3 files changed

Lines changed: 5 additions & 5 deletions

File tree

mobile/android/fenix/app/src/main/java/org/mozilla/fenix/home/pocket/controller/PocketStoriesController.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ interface PocketStoriesController {
5858
* Callback to decide what should happen as an effect of a new list of stories being shown.
5959
*
6060
* @param storiesShown the new list of [PocketStory]es shown to the user.
61-
* @param source the surface where the stories were shown.
61+
* @param source The surface where the stories were shown.
6262
*/
6363
fun handleStoriesShown(storiesShown: List<PocketStory>, source: StoriesImpressionSource)
6464

@@ -75,7 +75,7 @@ interface PocketStoriesController {
7575
* @param storyClicked The just clicked [PocketStory].
7676
* @param storyPosition `row x column x index` matrix representing the grid and index position
7777
* of the clicked story.
78-
* @param source the surface where the clicked story was shown.
78+
* @param source The surface where the clicked story was shown.
7979
*/
8080
fun handleStoryClicked(
8181
storyClicked: PocketStory,

mobile/android/fenix/app/src/main/java/org/mozilla/fenix/home/pocket/interactor/PocketStoriesInteractor.kt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ interface PocketStoriesInteractor {
2727
* Callback for then new stories are shown to the user.
2828
*
2929
* @param storiesShown The new list of [PocketRecommendedStory]es shown to the user.
30-
* @param source the surface where the stories were shown.
30+
* @param source The surface where the stories were shown.
3131
*/
3232
fun onStoriesShown(storiesShown: List<PocketStory>, source: StoriesImpressionSource)
3333

@@ -44,7 +44,7 @@ interface PocketStoriesInteractor {
4444
* @param storyClicked The just clicked [PocketStory].
4545
* @param storyPosition `row x column x index` matrix representing the grid and index position
4646
* of the clicked story.
47-
* @param source the surface where the clicked story was shown.
47+
* @param source The surface where the clicked story was shown.
4848
*/
4949
fun onStoryClicked(
5050
storyClicked: PocketStory,

mobile/android/fenix/app/src/test/java/org/mozilla/fenix/home/middleware/HomeTelemetryMiddlewareTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ class HomeTelemetryMiddlewareTest {
5656
assertEquals(recommendation.topic, extraValues["topic"])
5757
assertEquals(position.toString(), extraValues["position"])
5858
assertEquals("false", extraValues["is_sponsored"])
59-
assertEquals("homepage", extraValues["source"])
59+
assertEquals(StoriesImpressionSource.HOMEPAGE.sourceName, extraValues["source"])
6060

6161
pingReceived = true
6262
}

0 commit comments

Comments
 (0)