From a202fd0c9c9886da5923013eb81769337224daf8 Mon Sep 17 00:00:00 2001 From: ibaker Date: Wed, 10 Jul 2024 04:31:53 -0700 Subject: [PATCH] Add parameter comments in `ReorderingSeiMessageQueueTest` PiperOrigin-RevId: 650959556 --- .../ReorderingSeiMessageQueueTest.java | 26 +++++++++++-------- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/libraries/container/src/test/java/androidx/media3/container/ReorderingSeiMessageQueueTest.java b/libraries/container/src/test/java/androidx/media3/container/ReorderingSeiMessageQueueTest.java index 053080068c..1015b54225 100644 --- a/libraries/container/src/test/java/androidx/media3/container/ReorderingSeiMessageQueueTest.java +++ b/libraries/container/src/test/java/androidx/media3/container/ReorderingSeiMessageQueueTest.java @@ -50,17 +50,19 @@ public final class ReorderingSeiMessageQueueTest { ParsableByteArray scratchData = new ParsableByteArray(); byte[] data1 = TestUtil.buildTestData(5); scratchData.reset(data1); - reorderingQueue.add(345, scratchData); + reorderingQueue.add(/* presentationTimeUs= */ 345, scratchData); byte[] data2 = TestUtil.buildTestData(10); scratchData.reset(data2); - reorderingQueue.add(123, scratchData); + reorderingQueue.add(/* presentationTimeUs= */ 123, scratchData); assertThat(emittedMessages).isEmpty(); reorderingQueue.flush(); assertThat(emittedMessages) - .containsExactly(new SeiMessage(123, data2), new SeiMessage(345, data1)) + .containsExactly( + new SeiMessage(/* presentationTimeUs= */ 123, data2), + new SeiMessage(/* presentationTimeUs= */ 345, data1)) .inOrder(); } @@ -74,16 +76,17 @@ public final class ReorderingSeiMessageQueueTest { ParsableByteArray scratchData = new ParsableByteArray(); byte[] data1 = TestUtil.buildTestData(5); scratchData.reset(data1); - reorderingQueue.add(345, scratchData); + reorderingQueue.add(/* presentationTimeUs= */ 345, scratchData); byte[] data2 = TestUtil.buildTestData(10); scratchData.reset(data2); - reorderingQueue.add(123, scratchData); + reorderingQueue.add(/* presentationTimeUs= */ 123, scratchData); assertThat(emittedMessages).isEmpty(); reorderingQueue.setMaxSize(1); - assertThat(emittedMessages).containsExactly(new SeiMessage(123, data2)); + assertThat(emittedMessages) + .containsExactly(new SeiMessage(/* presentationTimeUs= */ 123, data2)); } @Test @@ -100,15 +103,16 @@ public final class ReorderingSeiMessageQueueTest { ParsableByteArray scratchData = new ParsableByteArray(); byte[] data1 = TestUtil.buildTestData(5); scratchData.reset(data1); - reorderingQueue.add(345, scratchData); + reorderingQueue.add(/* presentationTimeUs= */ 345, scratchData); assertThat(emittedMessages).isEmpty(); byte[] data2 = TestUtil.buildTestData(10); scratchData.reset(data2); - reorderingQueue.add(123, scratchData); + reorderingQueue.add(/* presentationTimeUs= */ 123, scratchData); - assertThat(emittedMessages).containsExactly(new SeiMessage(123, data2)); + assertThat(emittedMessages) + .containsExactly(new SeiMessage(/* presentationTimeUs= */ 123, data2)); } /** @@ -126,13 +130,13 @@ public final class ReorderingSeiMessageQueueTest { ParsableByteArray scratchData = new ParsableByteArray(); byte[] data1 = TestUtil.buildTestData(5); scratchData.reset(data1); - reorderingQueue.add(345, scratchData); + reorderingQueue.add(/* presentationTimeUs= */ 345, scratchData); verifyNoInteractions(mockSeiConsumer); byte[] data2 = TestUtil.buildTestData(10); scratchData.reset(data2); - reorderingQueue.add(123, scratchData); + reorderingQueue.add(/* presentationTimeUs= */ 123, scratchData); verify(mockSeiConsumer).consume(eq(123L), same(scratchData)); }