From 42425334ae17606a75ef697a5d0c20a0479d493b Mon Sep 17 00:00:00 2001 From: samrobinson Date: Mon, 7 Aug 2023 10:24:51 +0000 Subject: [PATCH] Mark always passing test as @Ignore. Added a TODO inline linking to the created bug to address this. PiperOrigin-RevId: 554422990 --- .../java/androidx/media3/transformer/MediaItemExportTest.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libraries/transformer/src/test/java/androidx/media3/transformer/MediaItemExportTest.java b/libraries/transformer/src/test/java/androidx/media3/transformer/MediaItemExportTest.java index d4bee63421..10ddc51005 100644 --- a/libraries/transformer/src/test/java/androidx/media3/transformer/MediaItemExportTest.java +++ b/libraries/transformer/src/test/java/androidx/media3/transformer/MediaItemExportTest.java @@ -90,6 +90,7 @@ import java.util.concurrent.atomic.AtomicInteger; import java.util.concurrent.atomic.AtomicReference; import org.junit.After; import org.junit.Before; +import org.junit.Ignore; import org.junit.Test; import org.junit.runner.RunWith; import org.mockito.ArgumentCaptor; @@ -919,6 +920,7 @@ public final class MediaItemExportTest { } @Test + @Ignore("TODO: b/294389961 - Add valid assertion for whether exporter expects decoded/encoded.") public void start_withAssetLoaderAlwaysDecoding_exporterExpectsDecoded() throws Exception { AtomicReference sampleConsumerRef = new AtomicReference<>(); Transformer transformer = @@ -931,6 +933,7 @@ public final class MediaItemExportTest { transformer.start(mediaItem, outputPath); runLooperUntil(transformer.getApplicationLooper(), () -> sampleConsumerRef.get() != null); + // Can never be false. assertThat(sampleConsumerRef.get()).isNotInstanceOf(EncodedSampleExporter.class); }