| Index: tools/push-to-trunk/test_scripts.py
|
| diff --git a/tools/push-to-trunk/test_scripts.py b/tools/push-to-trunk/test_scripts.py
|
| index bce0dc42cafa1d906860c92514b258778c3003d8..28d67b8ddea9fd6365c349957289772fc8812e80 100644
|
| --- a/tools/push-to-trunk/test_scripts.py
|
| +++ b/tools/push-to-trunk/test_scripts.py
|
| @@ -1187,7 +1187,8 @@ LOG=N
|
| Git("diff --name-only hash2 hash2^", TEST_CONFIG[VERSION_FILE]),
|
| Git("checkout -f hash2 -- %s" % TEST_CONFIG[VERSION_FILE], "",
|
| cb=ResetVersion(3, 1, 1)),
|
| - Git("log -1 --format=%B hash2", "Version 3.3.1.1 (merged 12)"),
|
| + Git("log -1 --format=%B hash2",
|
| + "Version 3.3.1.1 (merged 12)\n\nReview URL: fake.com\n"),
|
| Git("log -1 --format=%s hash2", ""),
|
| Git("svn find-rev hash2", "234"),
|
| Git("checkout -f HEAD -- %s" % TEST_CONFIG[VERSION_FILE], "",
|
| @@ -1197,6 +1198,7 @@ LOG=N
|
| Git("diff --name-only hash3 hash3^", TEST_CONFIG[VERSION_FILE]),
|
| Git("checkout -f hash3 -- %s" % TEST_CONFIG[VERSION_FILE], "",
|
| cb=ResetVersion(21, 2)),
|
| + Git("log -1 --format=%B hash3", ""),
|
| Git("log -1 --format=%s hash3", ""),
|
| Git("svn find-rev hash3", "123"),
|
| Git("checkout -f HEAD -- %s" % TEST_CONFIG[VERSION_FILE], "",
|
| @@ -1206,6 +1208,7 @@ LOG=N
|
| Git("diff --name-only hash6 hash6^", TEST_CONFIG[VERSION_FILE]),
|
| Git("checkout -f hash6 -- %s" % TEST_CONFIG[VERSION_FILE], "",
|
| cb=ResetVersion(22, 3)),
|
| + Git("log -1 --format=%B hash6", ""),
|
| Git("log -1 --format=%s hash6", ""),
|
| Git("svn find-rev hash6", "345"),
|
| Git("checkout -f HEAD -- %s" % TEST_CONFIG[VERSION_FILE], "",
|
| @@ -1243,11 +1246,17 @@ LOG=N
|
|
|
| expected_json = [
|
| {"bleeding_edge": "", "patches_merged": "", "version": "3.22.3",
|
| - "chromium_revision": "4567", "branch": "trunk", "revision": "345"},
|
| + "chromium_revision": "4567", "branch": "trunk", "revision": "345",
|
| + "review_link": "",
|
| + "revision_link": "https://code.google.com/p/v8/source/detail?r=345"},
|
| {"patches_merged": "", "bleeding_edge": "", "version": "3.21.2",
|
| - "chromium_revision": "", "branch": "3.21", "revision": "123"},
|
| + "chromium_revision": "", "branch": "3.21", "revision": "123",
|
| + "review_link": "",
|
| + "revision_link": "https://code.google.com/p/v8/source/detail?r=123"},
|
| {"patches_merged": "12", "bleeding_edge": "", "version": "3.3.1.1",
|
| - "chromium_revision": "", "branch": "3.3", "revision": "234"}
|
| + "chromium_revision": "", "branch": "3.3", "revision": "234",
|
| + "review_link": "fake.com",
|
| + "revision_link": "https://code.google.com/p/v8/source/detail?r=234"},
|
| ]
|
| self.assertEquals(expected_json, json.loads(FileToText(json_output)))
|
|
|
|
|