Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(157)

Unified Diff: tools/release/test_scripts.py

Issue 1095483002: Store hashes of current and previous shipped V8 version (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Removed more than 80 character line Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/release/releases.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/release/test_scripts.py
diff --git a/tools/release/test_scripts.py b/tools/release/test_scripts.py
index 95cdcc0d978686d64bd24eb5072f6b408de24293..dc6f097384bfb0cf3d3816f17b202a53b1db63ac 100644
--- a/tools/release/test_scripts.py
+++ b/tools/release/test_scripts.py
@@ -1422,13 +1422,20 @@ Cr-Commit-Position: refs/heads/4.2.71@{#1}
URL("http://omahaproxy.appspot.com/all.json", """[{
"os": "win",
"versions": [{
- "version": "1.1.1.1",
- "v8_version": "2.2.2.2",
+ "version": "2.2.2.2",
+ "v8_version": "22.2.2.2",
"current_reldate": "04/09/15",
"os": "win",
- "channel": "canary"
+ "channel": "canary",
+ "previous_version": "1.1.1.0"
}]
}]"""),
+ URL("http://omahaproxy.appspot.com/v8.json?version=1.1.1.0", """{
+ "chromium_version": "1.1.1.0",
+ "v8_version": "11.1.1.0"
+ }"""),
+ Cmd("git rev-list -1 11.1.1", "v8_previous_version_hash"),
+ Cmd("git rev-list -1 22.2.2.2", "v8_version_hash"),
Cmd("git checkout -f origin/master", ""),
Cmd("git branch -D %s" % TEST_CONFIG["BRANCHNAME"], "")
])
@@ -1449,10 +1456,13 @@ Cr-Commit-Position: refs/heads/4.2.71@{#1}
expected_json = {"chrome_releases":{
"canaries": [
{
- "chrome_version": "1.1.1.1",
+ "chrome_version": "2.2.2.2",
"os": "win",
"release_date": "04/09/15",
- "v8_version": "2.2.2.2",
+ "v8_version": "22.2.2.2",
+ "v8_version_hash": "v8_version_hash",
+ "v8_previous_version": "11.1.1.0",
+ "v8_previous_version_hash": "v8_previous_version_hash"
}]},
"releases":[
{
« no previous file with comments | « tools/release/releases.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698