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

Unified Diff: build/find_isolated_tests.py

Issue 180153005: Add a script that can discover *.isolated files in a directory. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 10 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/find_isolated_tests.py
diff --git a/build/find_isolated_tests.py b/build/find_isolated_tests.py
new file mode 100755
index 0000000000000000000000000000000000000000..bab49f5f3ab56a09bb194251328d864f8f8c2d03
--- /dev/null
+++ b/build/find_isolated_tests.py
@@ -0,0 +1,76 @@
+#!/usr/bin/env python
+# Copyright 2014 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+"""
+Scans build output directory for .isolated files, calculates their SHA1
+hashes, stores final list in JSON document and then removes *.isolated files
+found (to ensure no stale *.isolated stay around on the next build).
+
+Used to figure out what tests were build in isolated mode to trigger these
+tests to run on swarming.
+
+For more info see:
+https://sites.google.com/a/chromium.org/dev/developers/testing/isolated-testing
+"""
+
+import glob
+import hashlib
+import json
+import optparse
+import os
+import re
+import sys
+
+
+def hash_file(filepath):
+ """Calculates the hash of a file without reading it all in memory at once."""
+ digest = hashlib.sha1()
+ with open(filepath, 'rb') as f:
+ while True:
+ chunk = f.read(1024*1024)
+ if not chunk:
+ break
+ digest.update(chunk)
+ return digest.hexdigest()
+
+
+def main():
+ parser = optparse.OptionParser(usage='%prog path [options]')
M-A Ruel 2014/02/28 18:56:04 I like description=sys.modules[__name__].__doc__,
Vadim Sh. 2014/02/28 19:12:18 Done.
+ parser.add_option('--output-json', help='File to dump JSON results into.')
+ options, args = parser.parse_args()
+
+ if len(args) != 1:
+ parser.error('Missing path.')
M-A Ruel 2014/02/28 18:56:04 s/path/build output directory/
Vadim Sh. 2014/02/28 19:12:18 Done.
+ outdir = args[0]
+
+ result = {}
+
+ # Get the file hash values and output the pair.
+ for filepath in sorted(glob.glob(os.path.join(outdir, '*.isolated'))):
+ test_name = os.path.splitext(os.path.basename(filepath))[0]
+ if re.match(r'^.+?\.\d$', test_name):
+ # It's a split .isolated file, e.g. foo.0.isolated. Ignore these.
+ continue
+
+ sha1_hash = hash_file(filepath)
+ os.remove(filepath)
+ if not options.output_json:
M-A Ruel 2014/02/28 18:56:04 If you don't plan to use the --output_json flag, d
Vadim Sh. 2014/02/28 19:12:18 That is actually exactly what I'm going to use. Ma
M-A Ruel 2014/02/28 19:14:00 Ok. Note that I do like flags in general, so you c
Vadim Sh. 2014/02/28 19:23:20 Good idea. Done.
+ print('%s %s' % (test_name, sha1_hash))
+ # TODO(csharp): Remove both the message and the deletion once the isolate
+ # tracked dependencies are inputs for the isolated files.
+ print('Deleted %s to ensure it is regenerated by the next build.' %
+ filepath)
+ else:
+ result[test_name] = sha1_hash
+
+ if options.output_json:
+ with open(options.output_json, 'wb') as f:
+ json.dump(result, f)
+
+ return 0
+
+
+if __name__ == '__main__':
+ sys.exit(main())
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698