Improve PlayerRepository and fix MediaItemPosition

media/api/current.api

@@ -89,6 +89,7 @@ package com.google.android.horologist.media.repository {
     method public boolean hasPreviousMediaItem();
     method public void pause();
     method public void play();
+    method public void play(int mediaItemIndex);
     method public void prepare();
     method public void release();
     method public void removeMediaItem(int index);

media/build.gradle

@@ -22,15 +22,16 @@ plugins {
 }
 
 java {
-    sourceCompatibility = JavaVersion.VERSION_1_7
-    targetCompatibility = JavaVersion.VERSION_1_7
+    sourceCompatibility = JavaVersion.VERSION_1_8
+    targetCompatibility = JavaVersion.VERSION_1_8
 }
 
 dependencies {
 
     implementation libs.kotlinx.coroutines.core
 
     testImplementation libs.junit
+    testImplementation libs.truth
 }
 
 apply plugin: "com.vanniktech.maven.publish"

media/src/main/java/com/google/android/horologist/media/model/MediaItemPosition.kt

@@ -39,10 +39,10 @@ public sealed class MediaItemPosition(
             current: Duration,
             duration: Duration
         ): KnownDuration {
-            check(current.isNegative()) {
+            check(!current.isNegative()) {
                 "Current position can't be a negative value [current: $current] [duration: $duration]."
             }
-            check(!duration.isPositive()) {
+            check(duration.isPositive()) {
                 "Duration has to be greater than zero [current: $current] [duration: $duration]."
             }
             check(current <= duration) {

media/src/main/java/com/google/android/horologist/media/repository/PlayerRepository.kt

@@ -22,6 +22,7 @@ import com.google.android.horologist.media.model.MediaItem
 import com.google.android.horologist.media.model.MediaItemPosition
 import com.google.android.horologist.media.model.PlayerState
 import kotlinx.coroutines.flow.StateFlow
+import kotlin.time.Duration
 
 @ExperimentalHorologistMediaApi
 public interface PlayerRepository {
@@ -61,6 +62,11 @@ public interface PlayerRepository {
      */
     public fun play()
 
+    /**
+     * Play [media item][MediaItem] at given index as soon as player is ready.
+     */
+    public fun play(mediaItemIndex: Int)
+
     /**
      * Pauses playback.
      */
@@ -88,10 +94,8 @@ public interface PlayerRepository {
 
     /**
      * Returns the [seekBack] increment.
-     *
-     * @return The seek back increment, in milliseconds.
      */
-    public fun getSeekBackIncrement(): Long
+    public fun getSeekBackIncrement(): Duration
 
     /**
      * Seeks back in the [current media item][currentMediaItem] by [seek back increment][getSeekBackIncrement].
@@ -100,10 +104,8 @@ public interface PlayerRepository {
 
     /**
      * Returns the [seekForward] increment.
-     *
-     * @return The seek forward increment, in milliseconds.
      */
-    public fun getSeekForwardIncrement(): Long
+    public fun getSeekForwardIncrement(): Duration
 
     /**
      * Seek forward in the [current media item][currentMediaItem] by [seek forward increment][getSeekForwardIncrement].

media/src/test/java/com/google/android/horologist/media/model/MediaItemPositionTest.kt

@@ -16,13 +16,29 @@
 
 package com.google.android.horologist.media.model
 
+import com.google.common.truth.Truth.assertThat
 import org.junit.Assert.assertThrows
 import org.junit.Test
 import kotlin.time.Duration
 import kotlin.time.Duration.Companion.seconds
 
 class MediaItemPositionTest {
 
+    @Test
+    fun givenValidValues_whenCreateKnownPosition_thenCreateCorrectly() {
+        // given
+        val current = 1.seconds
+        val duration = 2.seconds
+
+        // when
+        val result = MediaItemPosition.create(current = current, duration = duration)
+
+        // then
+        assertThat(result.current).isEqualTo(current)
+        assertThat(result.duration).isEqualTo(duration)
+        assertThat(result.percent).isEqualTo(0.5f)
+    }
+
     @Test
     fun givenCurrentPositionIsNegative_whenCreateKnownPosition_thenExceptionIsThrown() {
         // given