Chromium Code Reviews| Index: build/android/pylib/junit/test_runner.py |
| diff --git a/build/android/pylib/junit/test_runner.py b/build/android/pylib/junit/test_runner.py |
| index a97841264dfaf13e314bdd553e5dda3226cc610b..54cf04a2f079810bd707f7ddb6b5a8083ba700b6 100644 |
| --- a/build/android/pylib/junit/test_runner.py |
| +++ b/build/android/pylib/junit/test_runner.py |
| @@ -47,7 +47,8 @@ class JavaTestRunner(object): |
| # Add JVM arguments. |
| jvm_args = [] |
| if self._coverage_dir: |
| - jvm_args.append('-Demma.coverage.out.file=%s' % self._coverage_dir) |
| + jvm_args.append('-Demma.coverage.out.file=%s' % os.path.join( |
|
jbudorick
2016/05/24 12:35:18
This should check that self._coverage_dir exists a
BigBossZhiling
2016/05/24 17:41:33
That's a good point, but I just tested on it and i
jbudorick
2016/05/24 19:15:09
I'd still prefer we create it here.
BigBossZhiling
2016/05/24 19:28:52
Done.
|
| + self._coverage_dir, '%s.ec' % self._test_suite)) |
| if jvm_args: |
| command.extend(['--jvm-args', '"%s"' % ' '.join(jvm_args)]) |