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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py

Issue 2188733003: Implement functionality to cc relevant owners. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: corrections Created 4 years, 5 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 | « third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
index 149541fb4d9f655121b9ccdcd3c494f6fa908949..3a36d50fdb9712c627a2fea3725a646ed97d817c 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater_unittest.py
@@ -68,3 +68,19 @@ Total: 8 tryjobs
})
self.assertTrue(updater.is_manual_test(fs, imported_dir + 'wpt/a', 'x-manual.html'))
self.assertFalse(updater.is_manual_test(fs, imported_dir + 'wpt/a', 'y-manual.html'))
+
+ def test_generate_email_list(self):
+ updater = DepsUpdater(MockHost())
+ owners = {'foo/bar': 'me@gmail.com', 'foo/baz': 'you@gmail.com', 'foo/bat': 'noone@gmail.com'}
+ results = """third_party/WebKit/LayoutTests/foo/bar/file.html
+third_party/WebKit/LayoutTests/foo/bar/otherfile.html
+third_party/WebKit/LayoutTests/foo/baz/files.html"""
+ self.assertEqual(updater.generate_email_list(results, owners), ['me@gmail.com', 'you@gmail.com'])
+
+ def test_parse_directory_owners(self):
+ updater = DepsUpdater(MockHost())
+ data_file = [{'notification-email': 'charizard@gmail.com', 'directory': 'foo/bar'},
+ {'notification-email': 'blastoise@gmail.com', 'directory': 'foo/baz'},
+ {'notification-email': '', 'directory': 'gol/bat'}]
+ self.assertEqual(updater.parse_directory_owners(data_file),
+ {'foo/bar': 'charizard@gmail.com', 'foo/baz': 'blastoise@gmail.com'})
« no previous file with comments | « third_party/WebKit/Tools/Scripts/webkitpy/w3c/deps_updater.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698