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

Unified Diff: lib/src/runner/loader.dart

Issue 1668573003: Add configuration support for choosing test paths. (Closed) Base URL: git@github.com:dart-lang/test@master
Patch Set: cr 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 | « lib/src/runner/configuration/values.dart ('k') | test/runner/configuration/top_level_error_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: lib/src/runner/loader.dart
diff --git a/lib/src/runner/loader.dart b/lib/src/runner/loader.dart
index 33092e1925f5bfc78cb9f54c38ad0e52cf55fdc9..b7594db40536b4bc904c9b47078422a91d145e44 100644
--- a/lib/src/runner/loader.dart
+++ b/lib/src/runner/loader.dart
@@ -62,8 +62,8 @@ class Loader {
/// Loads all test suites in [dir].
///
- /// This will load tests from files that end in "_test.dart". Any tests that
- /// fail to load will be emitted as [LoadException]s.
+ /// This will load tests from files that match the configuration's filename
+ /// glob. Any tests that fail to load will be emitted as [LoadException]s.
///
/// This emits [LoadSuite]s that must then be run to emit the actual
/// [RunnerSuite]s defined in the file.
@@ -72,7 +72,7 @@ class Loader {
.map((entry) {
if (entry is! File) return new Stream.fromIterable([]);
- if (!entry.path.endsWith("_test.dart")) {
+ if (!_config.filename.matches(p.basename(entry.path))) {
return new Stream.fromIterable([]);
}
« no previous file with comments | « lib/src/runner/configuration/values.dart ('k') | test/runner/configuration/top_level_error_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698