Commit 3ab9aa85 authored by romax's avatar romax Committed by Commit bot

[Offline Pages] Fix test harness failure related with batch size.

Fixed the test failure due to not setting default batch size in script.

BUG=673519

Review-Url: https://codereview.chromium.org/2567983005
Cr-Commit-Position: refs/heads/master@{#438032}
parent 4fe55265
...@@ -119,6 +119,7 @@ public class OfflinePageSavePageLaterEvaluationTest ...@@ -119,6 +119,7 @@ public class OfflinePageSavePageLaterEvaluationTest
ThreadUtils.runOnUiThread(new Runnable() { ThreadUtils.runOnUiThread(new Runnable() {
@Override @Override
public void run() { public void run() {
assert mBridge != null;
mBridge.getRequestsInQueue(new Callback<SavePageRequest[]>() { mBridge.getRequestsInQueue(new Callback<SavePageRequest[]>() {
@Override @Override
public void onResult(SavePageRequest[] results) { public void onResult(SavePageRequest[] results) {
...@@ -481,6 +482,9 @@ public class OfflinePageSavePageLaterEvaluationTest ...@@ -481,6 +482,9 @@ public class OfflinePageSavePageLaterEvaluationTest
Log.e(TAG, e.getMessage(), e); Log.e(TAG, e.getMessage(), e);
fail(String.format( fail(String.format(
"Config file %s is not found, aborting the test.", CONFIG_FILE_PATH)); "Config file %s is not found, aborting the test.", CONFIG_FILE_PATH));
} catch (NumberFormatException e) {
Log.e(TAG, e.getMessage(), e);
fail("Error parsing config file, aborting test.");
} finally { } finally {
if (inputStream != null) { if (inputStream != null) {
inputStream.close(); inputStream.close();
......
...@@ -89,6 +89,7 @@ def main(args): ...@@ -89,6 +89,7 @@ def main(args):
output_dir=os.path.expanduser('~/offline_eval_output'), output_dir=os.path.expanduser('~/offline_eval_output'),
user_request=DEFAULT_USER_REQUEST, user_request=DEFAULT_USER_REQUEST,
user_test_scheduler=DEFAULT_USE_TEST_SCHEDULER, user_test_scheduler=DEFAULT_USE_TEST_SCHEDULER,
schedule_batch_size=DEFAULT_BATCH_SIZE,
verbose=DEFAULT_VERBOSE) verbose=DEFAULT_VERBOSE)
def get_adb_command(args): def get_adb_command(args):
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment