Release player in e2e playback tests.

Not releasing the player means the playback thread keeps running and also keeps its entire allocated playback buffer. PiperOrigin-RevId: 330958821

Release player in e2e playback tests.
Not releasing the player means the playback thread keeps running and also keeps its entire allocated playback buffer. PiperOrigin-RevId: 330958821
267cc537 · tonihei · Oliver Woodman · 03323278 · 267cc537 · 267cc537
Commit 267cc537 authored Sep 10, 2020 by tonihei Committed by Oliver Woodman Sep 11, 2020
Showing with 2 additions and 0 deletions
library/core/src/test/java/com/google/android/exoplayer2/e2etest/Mp4PlaybackTest.java
library/core/src/test/java/com/google/android/exoplayer2/e2etest/TsPlaybackTest.java
--- a/library/core/src/test/java/com/google/android/exoplayer2/e2etest/Mp4PlaybackTest.java
+++ b/library/core/src/test/java/com/google/android/exoplayer2/e2etest/Mp4PlaybackTest.java
@@ -55,6 +55,7 @@ public class Mp4PlaybackTest {
    player.prepare();
    player.play();
    TestExoPlayer.runUntilPlaybackState(player, Player.STATE_ENDED);
+    player.release();
    DumpFileAsserts.assertOutput(
        ApplicationProvider.getApplicationContext(),

--- a/library/core/src/test/java/com/google/android/exoplayer2/e2etest/TsPlaybackTest.java
+++ b/library/core/src/test/java/com/google/android/exoplayer2/e2etest/TsPlaybackTest.java
@@ -55,6 +55,7 @@ public class TsPlaybackTest {
    player.prepare();
    player.play();
    TestExoPlayer.runUntilPlaybackState(player, Player.STATE_ENDED);
+    player.release();
    DumpFileAsserts.assertOutput(
        ApplicationProvider.getApplicationContext(),