diff --git a/extensions/ima/build.gradle b/extensions/ima/build.gradle index 33985b2639..faae0b3679 100644 --- a/extensions/ima/build.gradle +++ b/extensions/ima/build.gradle @@ -35,6 +35,7 @@ dependencies { androidTestImplementation 'androidx.multidex:multidex:' + androidxMultidexVersion androidTestImplementation 'androidx.test:rules:' + androidxTestRulesVersion androidTestImplementation 'androidx.test:runner:' + androidxTestRunnerVersion + androidTestImplementation 'com.google.guava:guava:' + guavaVersion androidTestCompileOnly 'org.checkerframework:checker-qual:' + checkerframeworkVersion testImplementation project(modulePrefix + 'testutils') testImplementation 'com.google.guava:guava:' + guavaVersion diff --git a/extensions/ima/src/androidTest/java/com/google/android/exoplayer2/ext/ima/ImaPlaybackTest.java b/extensions/ima/src/androidTest/java/com/google/android/exoplayer2/ext/ima/ImaPlaybackTest.java index 0e685e55ea..31cd29de94 100644 --- a/extensions/ima/src/androidTest/java/com/google/android/exoplayer2/ext/ima/ImaPlaybackTest.java +++ b/extensions/ima/src/androidTest/java/com/google/android/exoplayer2/ext/ima/ImaPlaybackTest.java @@ -20,7 +20,6 @@ import static com.google.common.truth.Truth.assertThat; import android.content.Context; import android.net.Uri; import android.view.Surface; -import android.view.View; import android.view.ViewGroup; import android.widget.FrameLayout; import androidx.annotation.Nullable; @@ -39,6 +38,7 @@ import com.google.android.exoplayer2.decoder.DecoderCounters; import com.google.android.exoplayer2.drm.DrmSessionManager; import com.google.android.exoplayer2.source.DefaultMediaSourceFactory; import com.google.android.exoplayer2.source.MediaSource; +import com.google.android.exoplayer2.source.ads.AdsLoader; import com.google.android.exoplayer2.source.ads.AdsLoader.AdViewProvider; import com.google.android.exoplayer2.source.ads.AdsMediaSource; import com.google.android.exoplayer2.testutil.ActionSchedule; @@ -50,6 +50,7 @@ import com.google.android.exoplayer2.upstream.DataSource; import com.google.android.exoplayer2.upstream.DefaultDataSourceFactory; import com.google.android.exoplayer2.util.Assertions; import com.google.android.exoplayer2.util.Util; +import com.google.common.collect.ImmutableList; import java.util.ArrayList; import java.util.Arrays; import java.util.List; @@ -249,14 +250,15 @@ public final class ImaPlaybackTest { dataSourceFactory, Assertions.checkNotNull(imaAdsLoader), new AdViewProvider() { + @Override public ViewGroup getAdViewGroup() { return overlayFrameLayout; } @Override - public View[] getAdOverlayViews() { - return new View[0]; + public ImmutableList getAdOverlayInfos() { + return ImmutableList.of(); } }); } diff --git a/extensions/ima/src/test/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoaderTest.java b/extensions/ima/src/test/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoaderTest.java index a9acbad72c..23b7110103 100644 --- a/extensions/ima/src/test/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoaderTest.java +++ b/extensions/ima/src/test/java/com/google/android/exoplayer2/ext/ima/ImaAdsLoaderTest.java @@ -116,7 +116,6 @@ public final class ImaAdsLoaderTest { @Mock private AdEvent mockPostrollFetchErrorAdEvent; private ViewGroup adViewGroup; - private View adOverlayView; private AdsLoader.AdViewProvider adViewProvider; private AdEvent.AdEventListener adEventListener; private ContentProgressProvider contentProgressProvider; @@ -129,7 +128,7 @@ public final class ImaAdsLoaderTest { public void setUp() { setupMocks(); adViewGroup = new FrameLayout(ApplicationProvider.getApplicationContext()); - adOverlayView = new View(ApplicationProvider.getApplicationContext()); + View adOverlayView = new View(ApplicationProvider.getApplicationContext()); adViewProvider = new AdsLoader.AdViewProvider() { @Override @@ -138,8 +137,9 @@ public final class ImaAdsLoaderTest { } @Override - public View[] getAdOverlayViews() { - return new View[] {adOverlayView}; + public ImmutableList getAdOverlayInfos() { + return ImmutableList.of( + new AdsLoader.OverlayInfo(adOverlayView, AdsLoader.OverlayInfo.PURPOSE_CLOSE_AD)); } }; }