Reorder the assetions in tests
So it's more clear what's actually broke in the error log. PiperOrigin-RevId: 667964963
This commit is contained in:
parent
a4d0735d4c
commit
36d61000fd
@ -367,20 +367,20 @@ public class VideoTimestampConsistencyTest {
|
|||||||
private void compareTimestamps(
|
private void compareTimestamps(
|
||||||
List<EditedMediaItem> mediaItems, List<Long> expectedTimestamps, boolean containsImage)
|
List<EditedMediaItem> mediaItems, List<Long> expectedTimestamps, boolean containsImage)
|
||||||
throws Exception {
|
throws Exception {
|
||||||
|
ImmutableList<Long> timestampsFromTransformer = getTimestampsFromTransformer(mediaItems);
|
||||||
|
assertThat(timestampsFromTransformer).isEqualTo(expectedTimestamps);
|
||||||
|
|
||||||
ImmutableList<Long> timestampsFromCompositionPlayer =
|
ImmutableList<Long> timestampsFromCompositionPlayer =
|
||||||
getTimestampsFromCompositionPlayer(mediaItems);
|
getTimestampsFromCompositionPlayer(mediaItems);
|
||||||
ImmutableList<Long> timestampsFromTransformer = getTimestampsFromTransformer(mediaItems);
|
assertThat(timestampsFromCompositionPlayer).isEqualTo(expectedTimestamps);
|
||||||
|
|
||||||
assertThat(timestampsFromCompositionPlayer).isEqualTo(timestampsFromTransformer);
|
|
||||||
|
|
||||||
if (!containsImage) {
|
if (!containsImage) {
|
||||||
// ExoPlayer doesn't support image playback with effects.
|
// ExoPlayer doesn't support image playback with effects.
|
||||||
ImmutableList<Long> timestampsFromExoPlayer =
|
ImmutableList<Long> timestampsFromExoPlayer =
|
||||||
getTimestampsFromExoPlayer(
|
getTimestampsFromExoPlayer(
|
||||||
Lists.transform(mediaItems, editedMediaItem -> editedMediaItem.mediaItem));
|
Lists.transform(mediaItems, editedMediaItem -> editedMediaItem.mediaItem));
|
||||||
assertThat(timestampsFromCompositionPlayer).isEqualTo(timestampsFromExoPlayer);
|
assertThat(timestampsFromExoPlayer).isEqualTo(expectedTimestamps);
|
||||||
}
|
}
|
||||||
assertThat(timestampsFromTransformer).isEqualTo(expectedTimestamps);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private ImmutableList<Long> getTimestampsFromTransformer(List<EditedMediaItem> editedMediaItems)
|
private ImmutableList<Long> getTimestampsFromTransformer(List<EditedMediaItem> editedMediaItems)
|
||||||
|
Loading…
x
Reference in New Issue
Block a user