languages/dart-sdk
languages/dart-sdk
1
0
Fork 0
mirror of https://github.com/dart-lang/sdk synced 2024-09-05 08:24:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

[infra] Handle test steps with no tested configuration

Browse source 
Change-Id: I95543f88a64884bf2334165baa986d381bbf7872
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/141241
Commit-Queue: Karl Klose <karlklose@google.com>
Reviewed-by: Karl Klose <karlklose@google.com>
This commit is contained in:
Karl Klose 2020-03-27 07:09:37 +00:00
parent 9b5aac9495
commit 7387682bec
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
tools/test.dart
View file

@ -112,7 +112,13 @@ Map<String, Set<Builder>> resolveNamedConfigurations(
}
foundBuilder = true;
for (final step in builder.steps.where((step) => step.isTestStep)) {
final namedConfiguration = step.testedConfiguration.name;
final testedConfiguration = step.testedConfiguration;
if (testedConfiguration == null) {
// This test step does not use a configuration; for example,
// because it is a simple script that does not produce results.
continue;
}
final namedConfiguration = testedConfiguration.name;
if (requestedNamedConfigurations.isEmpty ||
requestedNamedConfigurations.contains(namedConfiguration)) {
testedConfigurations