From 870255459131e683c7f47af960e00aebce383520 Mon Sep 17 00:00:00 2001 From: shahddaghash Date: Fri, 7 Mar 2025 07:16:43 -0800 Subject: [PATCH] Remove setVideoEffects_failsWithoutLibEffectsDep test Removed test that attempted to verify behavior when the lib-effect dependency is missing. This is because it fails when adding lib-transformer to test-util dependency. The introduction of lib-transformer brought in lib-effect as a transitive dependency, which conflicted with the test's expectation of the class not being present. PiperOrigin-RevId: 734547629 --- .../java/androidx/media3/exoplayer/ExoPlayerTest.java | 8 -------- 1 file changed, 8 deletions(-) diff --git a/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/ExoPlayerTest.java b/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/ExoPlayerTest.java index 944af0a284..31beddba0f 100644 --- a/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/ExoPlayerTest.java +++ b/libraries/exoplayer/src/test/java/androidx/media3/exoplayer/ExoPlayerTest.java @@ -16231,14 +16231,6 @@ public class ExoPlayerTest { assertThat(metadataAfterTransition.title.toString()).isEqualTo("title"); } - @Test - public void setVideoEffects_failsWithoutLibEffectsDep() { - ExoPlayer player = parameterizeTestExoPlayerBuilder(new TestExoPlayerBuilder(context)).build(); - IllegalStateException expected = - assertThrows(IllegalStateException.class, () -> player.setVideoEffects(ImmutableList.of())); - assertThat(expected).hasMessageThat().contains("lib-effect dependencies"); - } - @Test public void setPriority_blocksOtherLowPriorityTasksInPriorityTaskManager() throws Exception { PriorityTaskManager priorityTaskManager = new PriorityTaskManager();