Avoid send null current chapter when not needed (#587)

Author: StaehliJ

pillarbox-player/src/main/java/ch/srgssr/pillarbox/player/tracker/TimeRangeTracker.kt

@@ -137,7 +137,10 @@ private class ChapterCreditsTracker<T : TimeRange>(
             val timeRange = message as? T ?: return@Target
             when (messageType) {
                 TYPE_ENTER -> currentTimeRange = timeRange
-                TYPE_EXIT -> currentTimeRange = null
+                TYPE_EXIT -> {
+                    val nextTimeRange = timeRanges.firstOrNullAtPosition(player.currentPosition)
+                    if (nextTimeRange == null) currentTimeRange = null
+                }
             }
         }
         val playerMessages = mutableListOf<PlayerMessage>()

pillarbox-player/src/test/java/ch/srgssr/pillarbox/player/tracker/ChapterTrackerTest.kt

@@ -68,12 +68,11 @@ class ChapterTrackerTest {
         player.addMediaItem(ChapterAssetLoader.MEDIA_ITEM)
         TestPlayerRunHelper.runUntilPlaybackState(player, Player.STATE_ENDED)
 
-        val expectedChapters = listOf(ChapterAssetLoader.CHAPTER_1, null, ChapterAssetLoader.CHAPTER_2, null)
+        val expectedChapters = listOf(ChapterAssetLoader.CHAPTER_1, ChapterAssetLoader.CHAPTER_2, null)
         val receivedChapters = mutableListOf<Chapter?>()
         verify {
             listener.onChapterChanged(captureNullable(receivedChapters))
         }
-        assertEquals(expectedChapters.size, receivedChapters.size)
         assertEquals(expectedChapters, receivedChapters)
     }