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

Unified Diff: test/browser/language_tests.js

Issue 1521283003: Run browser tests on Atom/Electron (Closed) Base URL: https://github.com/dart-lang/dev_compiler.git@master
Patch Set: Created 5 years 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
Index: test/browser/language_tests.js
diff --git a/test/browser/language_tests.js b/test/browser/language_tests.js
index 02dc2104c30ed15e67b6039a87f90c83616e3266..2d05cfed8149ff293918b1f9783367f8e9a62fce 100644
--- a/test/browser/language_tests.js
+++ b/test/browser/language_tests.js
@@ -317,14 +317,24 @@
'no_main_test_01_multi',
]);
+ let helpers = new Set([
+ 'library_prefixes_test1',
+ 'library_prefixes_test2',
+ 'top_level_prefixed_library_test',
+ ]);
suite('language', () => {
- let languageTestPattern = new RegExp('language/(.*_test.*)\\.js');
- for (let testFile of Object.keys(window.__karma__.files)) {
+ let languageTestPattern = new RegExp('language/(.*_test.*)');
+ for (let testFile of dart_library.libraries()) {
let match = languageTestPattern.exec(testFile);
if (match != null) {
let name = match[1];
+ if (helpers.has(name)) {
+ // These are not top-level tests. They are used by other tests.
+ continue;
+ }
+
// These two tests are special because they use package:unittest.
// We run them below.
if (name == 'async_await_test' || name.startsWith('async_star_test')) {
« lib/runtime/dart_library.js ('K') | « package.json ('k') | tool/browser_test.sh » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698