From 80e3152e8c8d77271def2b3996312ea2b3cb8bc3 Mon Sep 17 00:00:00 2001 From: olly Date: Fri, 13 Dec 2019 16:51:06 +0000 Subject: [PATCH] Make it easier to update extractor test dump files PiperOrigin-RevId: 285407744 --- .../exoplayer2/testutil/FakeExtractorOutput.java | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeExtractorOutput.java b/testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeExtractorOutput.java index 0502707682..74a554df76 100644 --- a/testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeExtractorOutput.java +++ b/testutils/src/main/java/com/google/android/exoplayer2/testutil/FakeExtractorOutput.java @@ -33,9 +33,11 @@ import java.io.PrintWriter; public final class FakeExtractorOutput implements ExtractorOutput, Dumper.Dumpable { /** - * If true, makes {@link #assertOutput(Context, String)} method write dump result to {@code - * /sdcard/Android/data/apk_package/ + dumpfile} file instead of comparing it with an existing - * file. + * If true, makes {@link #assertOutput(Context, String)} method write the output to the dump file, + * rather than validating that the output matches what the dump file already contains. + * + *

Enabling this option works when tests are run in Android Studio. It may not work when the + * tests are run in another environment. */ private static final boolean WRITE_DUMP = false; @@ -121,8 +123,8 @@ public final class FakeExtractorOutput implements ExtractorOutput, Dumper.Dumpab String actual = new Dumper().add(this).toString(); if (WRITE_DUMP) { - File directory = context.getExternalFilesDir(null); - File file = new File(directory, dumpFile); + File file = new File(System.getProperty("user.dir"), "src/test/assets"); + file = new File(file, dumpFile); file.getParentFile().mkdirs(); PrintWriter out = new PrintWriter(file); out.print(actual);