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

Side by Side Diff: tools/release/test_scripts.py

Issue 1195693003: Revert of [release-tools] Fix possible race condition in retrieval of latest rolled version (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 6 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 unified diff | Download patch
« no previous file with comments | « tools/release/auto_roll.py ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2013 the V8 project authors. All rights reserved. 2 # Copyright 2013 the V8 project authors. All rights reserved.
3 # Redistribution and use in source and binary forms, with or without 3 # Redistribution and use in source and binary forms, with or without
4 # modification, are permitted provided that the following conditions are 4 # modification, are permitted provided that the following conditions are
5 # met: 5 # met:
6 # 6 #
7 # * Redistributions of source code must retain the above copyright 7 # * Redistributions of source code must retain the above copyright
8 # notice, this list of conditions and the following disclaimer. 8 # notice, this list of conditions and the following disclaimer.
9 # * Redistributions in binary form must reproduce the above 9 # * Redistributions in binary form must reproduce the above
10 # copyright notice, this list of conditions and the following 10 # copyright notice, this list of conditions and the following
(...skipping 1094 matching lines...) Expand 10 before | Expand all | Expand 10 after
1105 self.Expect([ 1105 self.Expect([
1106 URL("https://codereview.chromium.org/search", 1106 URL("https://codereview.chromium.org/search",
1107 "owner=author%40chromium.org&limit=30&closed=3&format=json", 1107 "owner=author%40chromium.org&limit=30&closed=3&format=json",
1108 ("{\"results\": [{\"subject\": \"different\"}]}")), 1108 ("{\"results\": [{\"subject\": \"different\"}]}")),
1109 Cmd("git fetch origin +refs/tags/*:refs/tags/*", ""), 1109 Cmd("git fetch origin +refs/tags/*:refs/tags/*", ""),
1110 Cmd("git rev-list --max-age=740800 --tags", 1110 Cmd("git rev-list --max-age=740800 --tags",
1111 "bad_tag\nhash_234\nhash_123"), 1111 "bad_tag\nhash_234\nhash_123"),
1112 Cmd("git describe --tags bad_tag", ""), 1112 Cmd("git describe --tags bad_tag", ""),
1113 Cmd("git describe --tags hash_234", "3.22.4"), 1113 Cmd("git describe --tags hash_234", "3.22.4"),
1114 Cmd("git describe --tags hash_123", "3.22.3"), 1114 Cmd("git describe --tags hash_123", "3.22.3"),
1115 Cmd("git pull", "", cwd=TEST_CONFIG["CHROMIUM"]),
1116 Cmd("git describe --tags abcd123455", "3.22.4"), 1115 Cmd("git describe --tags abcd123455", "3.22.4"),
1117 Cmd("git describe --tags hash_234", "3.22.4"), 1116 Cmd("git describe --tags hash_234", "3.22.4"),
1118 Cmd("git describe --tags hash_123", "3.22.3"), 1117 Cmd("git describe --tags hash_123", "3.22.3"),
1119 ]) 1118 ])
1120 1119
1121 result = auto_roll.AutoRoll(TEST_CONFIG, self).Run( 1120 result = auto_roll.AutoRoll(TEST_CONFIG, self).Run(
1122 AUTO_PUSH_ARGS + ["-c", TEST_CONFIG["CHROMIUM"]]) 1121 AUTO_PUSH_ARGS + ["-c", TEST_CONFIG["CHROMIUM"]])
1123 self.assertEquals(0, result) 1122 self.assertEquals(0, result)
1124 1123
1125 def testAutoRoll(self): 1124 def testAutoRoll(self):
1126 TEST_CONFIG["CHROMIUM"] = self.MakeEmptyTempDirectory() 1125 TEST_CONFIG["CHROMIUM"] = self.MakeEmptyTempDirectory()
1127 TextToFile(self.FAKE_DEPS, os.path.join(TEST_CONFIG["CHROMIUM"], "DEPS")) 1126 TextToFile(self.FAKE_DEPS, os.path.join(TEST_CONFIG["CHROMIUM"], "DEPS"))
1128 1127
1129 self.Expect([ 1128 self.Expect([
1130 URL("https://codereview.chromium.org/search", 1129 URL("https://codereview.chromium.org/search",
1131 "owner=author%40chromium.org&limit=30&closed=3&format=json", 1130 "owner=author%40chromium.org&limit=30&closed=3&format=json",
1132 ("{\"results\": [{\"subject\": \"different\"}]}")), 1131 ("{\"results\": [{\"subject\": \"different\"}]}")),
1133 Cmd("git fetch origin +refs/tags/*:refs/tags/*", ""), 1132 Cmd("git fetch origin +refs/tags/*:refs/tags/*", ""),
1134 Cmd("git rev-list --max-age=740800 --tags", 1133 Cmd("git rev-list --max-age=740800 --tags",
1135 "bad_tag\nhash_234\nhash_123"), 1134 "bad_tag\nhash_234\nhash_123"),
1136 Cmd("git describe --tags bad_tag", ""), 1135 Cmd("git describe --tags bad_tag", ""),
1137 Cmd("git describe --tags hash_234", "3.22.4"), 1136 Cmd("git describe --tags hash_234", "3.22.4"),
1138 Cmd("git describe --tags hash_123", "3.22.3"), 1137 Cmd("git describe --tags hash_123", "3.22.3"),
1139 Cmd("git pull", "", cwd=TEST_CONFIG["CHROMIUM"]),
1140 Cmd("git describe --tags abcd123455", "3.22.3.1"), 1138 Cmd("git describe --tags abcd123455", "3.22.3.1"),
1141 Cmd("git describe --tags hash_234", "3.22.4"), 1139 Cmd("git describe --tags hash_234", "3.22.4"),
1142 ]) 1140 ])
1143 1141
1144 result = auto_roll.AutoRoll(TEST_CONFIG, self).Run( 1142 result = auto_roll.AutoRoll(TEST_CONFIG, self).Run(
1145 AUTO_PUSH_ARGS + ["-c", TEST_CONFIG["CHROMIUM"], "--roll"]) 1143 AUTO_PUSH_ARGS + ["-c", TEST_CONFIG["CHROMIUM"], "--roll"])
1146 self.assertEquals(0, result) 1144 self.assertEquals(0, result)
1147 1145
1148 def testMergeToBranch(self): 1146 def testMergeToBranch(self):
1149 TEST_CONFIG["ALREADY_MERGING_SENTINEL_FILE"] = self.MakeEmptyTempFile() 1147 TEST_CONFIG["ALREADY_MERGING_SENTINEL_FILE"] = self.MakeEmptyTempFile()
(...skipping 395 matching lines...) Expand 10 before | Expand all | Expand 10 after
1545 1543
1546 Review URL: https://codereview.chromium.org/83173002 1544 Review URL: https://codereview.chromium.org/83173002
1547 1545
1548 ------------------------------------------------------------------------""") 1546 ------------------------------------------------------------------------""")
1549 self.assertEquals( 1547 self.assertEquals(
1550 """Prepare push to trunk. Now working on version 3.23.11. 1548 """Prepare push to trunk. Now working on version 3.23.11.
1551 1549
1552 R=danno@chromium.org 1550 R=danno@chromium.org
1553 1551
1554 Committed: https://code.google.com/p/v8/source/detail?r=17997""", body) 1552 Committed: https://code.google.com/p/v8/source/detail?r=17997""", body)
OLDNEW
« no previous file with comments | « tools/release/auto_roll.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698