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

Unified Diff: dart_test.yaml

Issue 1707173002: Add browser tags to all of our tests. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: Created 4 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 | pubspec.yaml » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: dart_test.yaml
diff --git a/dart_test.yaml b/dart_test.yaml
index 1df839806bed8d8211e3691e1a3b0b2c798a70cc..cafb0cf859daf135f21368ec1fc38342dcb38dac 100644
--- a/dart_test.yaml
+++ b/dart_test.yaml
@@ -1,2 +1,16 @@
# Terse traces hide frames from test itself, which is bad.
verbose_trace: true
+
+tags:
+ browser: {timeout: 2x}
+ dart2js:
+ add_tags: [browser]
+ timeout: 2x
kevmoo 2016/02/17 23:47:30 2x here is a bit redundant, no?
nweiz 2016/02/18 00:02:58 No, it's cumulative with the browser timeout. So d
+
+ firefox: {add_tags: [dart2js]}
+ chrome: {add_tags: [dart2js]}
+ safari: {add_tags: [dart2js]}
+ ie: {add_tags: [dart2js]}
+ phantomjs: {add_tags: [dart2js]}
+ content-shell: {add_tags: [browser]}
+ dartium: {add_tags: [browser]}
« no previous file with comments | « no previous file | pubspec.yaml » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698