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

Unified Diff: pkg/dev_compiler/test/codegen_test.dart

Issue 2990213002: fix DDC buildbot so it is green (Closed)
Patch Set: Created 3 years, 4 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 | tests/language_2/language_2.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: pkg/dev_compiler/test/codegen_test.dart
diff --git a/pkg/dev_compiler/test/codegen_test.dart b/pkg/dev_compiler/test/codegen_test.dart
index 75c674bb09feb8b97592c4cb03ab7bf9c5148217..99c9f08b737e70dcfd0f4ed3b5f6a9ace0b4f579 100644
--- a/pkg/dev_compiler/test/codegen_test.dart
+++ b/pkg/dev_compiler/test/codegen_test.dart
@@ -168,8 +168,10 @@ main(List<String> arguments) {
// This covers tests where the intent of the test is to validate that
// some static error is produced.
- var intentionalCompileError = contents.contains(': compile-time error') ||
- contents.contains('/*@compile-error=');
+ var intentionalCompileError =
+ (contents.contains(': compile-time error') ||
+ contents.contains('/*@compile-error=')) &&
+ !status.contains(Expectation.missingCompileTimeError);
var crashing = status.contains(Expectation.crash);
if (module == null) {
@@ -188,7 +190,11 @@ main(List<String> arguments) {
expect(crashing, isFalse, reason: "test $name no longer crashes.");
- var knownCompileError = status.contains(Expectation.compileTimeError);
+ var knownCompileError = status.contains(Expectation.compileTimeError) ||
+ status.contains(Expectation.fail);
+ // TODO(jmesserly): we could also invert negative_test, however analyzer
+ // in test.dart does not do this.
+ // name.endsWith('negative_test') && !status.contains(Expectation.fail)
if (module.isValid) {
expect(knownCompileError, isFalse,
reason: "test $name expected static errors, but compiled.");
@@ -275,6 +281,7 @@ void _writeRuntimeStatus(Map<String, Set<Expectation>> testFiles) {
if (status.contains(Expectation.compileTimeError) ||
status.contains(Expectation.crash) ||
status.contains(Expectation.skip) ||
+ status.contains(Expectation.fail) ||
status.contains(Expectation.skipByDesign)) {
return;
}
« no previous file with comments | « no previous file | tests/language_2/language_2.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698