From 79f29680fc29574191d99425565eb8fdbebe38af Mon Sep 17 00:00:00 2001 From: kimvde Date: Tue, 18 Mar 2025 06:33:03 -0700 Subject: [PATCH] Ignore multi image sequence test This test is failing in when the following happens in PlaybackVideoGraphWrapper: - Both InputVideoSinks call onInputStreamChanged with startPosition 1000000000000 - Before the first VideoGraph output frame is available, both InputVideoSinks call onInputStreamChanged with startPosition 1000000200000. This incorrectly erases startPosition 1000000000000 from TimedValueQueue streamStartPositionsUs. Indeed, when 1000000200000 is added for the second time, the whole queue is cleared because this new value is not strictly greater than the previous one. This start position logic should only be used for single-sequence rendering. Disable the test until this is fixed. PiperOrigin-RevId: 737982504 --- .../transformer/CompositionMultipleSequencePlaybackTest.java | 1 + 1 file changed, 1 insertion(+) diff --git a/libraries/transformer/src/androidTest/java/androidx/media3/transformer/CompositionMultipleSequencePlaybackTest.java b/libraries/transformer/src/androidTest/java/androidx/media3/transformer/CompositionMultipleSequencePlaybackTest.java index 853057ce41..1d6ea5a589 100644 --- a/libraries/transformer/src/androidTest/java/androidx/media3/transformer/CompositionMultipleSequencePlaybackTest.java +++ b/libraries/transformer/src/androidTest/java/androidx/media3/transformer/CompositionMultipleSequencePlaybackTest.java @@ -165,6 +165,7 @@ public class CompositionMultipleSequencePlaybackTest { } @Test + @Ignore("TODO: b/391349011 - Re-enable after adapting rendering logic for multiple sequences") public void playback_sequencesOfImages_effectsReceiveCorrectTimestamps() throws Exception { Composition composition = new Composition.Builder(