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

Unified Diff: pkg/analyzer/test/generated/strong_mode_test.dart

Issue 2625783002: Explicitly compute analysis results for sources to check errors. (Closed)
Patch Set: Rollback await(ing) of assertErrors(). Format. Created 3 years, 11 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
Index: pkg/analyzer/test/generated/strong_mode_test.dart
diff --git a/pkg/analyzer/test/generated/strong_mode_test.dart b/pkg/analyzer/test/generated/strong_mode_test.dart
index cf4cd1d93a8926f5b748578de9dbd6ec60b0aee4..73561292fb44a59a8034b19f13b18e3329304b0e 100644
--- a/pkg/analyzer/test/generated/strong_mode_test.dart
+++ b/pkg/analyzer/test/generated/strong_mode_test.dart
@@ -5,6 +5,7 @@
library analyzer.test.generated.strong_mode_test;
import 'dart:async';
+
import 'package:analyzer/dart/ast/ast.dart';
import 'package:analyzer/dart/ast/standard_resolution_map.dart';
import 'package:analyzer/dart/element/element.dart';
@@ -685,7 +686,8 @@ class StrongModeDownwardsInferenceTest extends ResolverTestCase {
}
''');
resolve2(source);
- await assertNoErrors(source);
+ await computeAnalysisResult(source);
+ assertNoErrors(source);
verify([source]);
}
@@ -1328,16 +1330,6 @@ void test/*<S>*/(/*=T*/ pf/*<T>*/(/*=T*/ e)) {
expectIdentifierType('paramTearOffInst', "(int) → int");
}
- Future<Null> _objectMethodOnFunctions_helper2(String code) async {
- await resolveTestUnit(code);
- expectIdentifierType('t0', "String");
- expectIdentifierType('t1', "() → String");
- expectIdentifierType('t2', "int");
- expectIdentifierType('t3', "String");
- expectIdentifierType('t4', "() → String");
- expectIdentifierType('t5', "int");
- }
-
void setUp() {
super.setUp();
AnalysisOptionsImpl options = new AnalysisOptionsImpl();
@@ -1814,7 +1806,8 @@ class C {
class D extends C {
String f/*<S>*/(/*=S*/ x) => null;
}''');
- await assertErrors(source, [StrongModeCode.INVALID_METHOD_OVERRIDE]);
+ await computeAnalysisResult(source);
+ assertErrors(source, [StrongModeCode.INVALID_METHOD_OVERRIDE]);
verify([source]);
}
@@ -1828,7 +1821,8 @@ class C {
class D extends C {
/*=T*/ f/*<T extends B>*/(/*=T*/ x) => null;
}''');
- await assertErrors(source, [StrongModeCode.INVALID_METHOD_OVERRIDE]);
+ await computeAnalysisResult(source);
+ assertErrors(source, [StrongModeCode.INVALID_METHOD_OVERRIDE]);
verify([source]);
}
@@ -1840,7 +1834,8 @@ class C {
class D extends C {
/*=S*/ f/*<T, S>*/(/*=T*/ x) => null;
}''');
- await assertErrors(source, [StrongModeCode.INVALID_METHOD_OVERRIDE]);
+ await computeAnalysisResult(source);
+ assertErrors(source, [StrongModeCode.INVALID_METHOD_OVERRIDE]);
verify([source]);
}
@@ -2081,7 +2076,8 @@ class A {
}
dynamic set g(int x) => null;
''');
- await assertErrors(source, [
+ await computeAnalysisResult(source);
+ assertErrors(source, [
StaticWarningCode.NON_VOID_RETURN_FOR_SETTER,
StaticWarningCode.NON_VOID_RETURN_FOR_SETTER
]);
@@ -2096,7 +2092,8 @@ class A {
}
void set g(int x) => returnsVoid();
''');
- await assertNoErrors(source);
+ await computeAnalysisResult(source);
+ assertNoErrors(source);
verify([source]);
}
@@ -2109,7 +2106,8 @@ class A {
}
set g(int x) => 42;
''');
- await assertErrors(source, [
+ await computeAnalysisResult(source);
+ assertErrors(source, [
StaticTypeWarningCode.RETURN_OF_INVALID_TYPE,
]);
verify([source]);
@@ -2123,7 +2121,8 @@ class A {
}
set g(int x) => returnsVoid();
''');
- await assertNoErrors(source);
+ await computeAnalysisResult(source);
+ assertNoErrors(source);
verify([source]);
}
@@ -2134,7 +2133,8 @@ class A {
}
Object set g(x) => null;
''');
- await assertErrors(source, [
+ await computeAnalysisResult(source);
+ assertErrors(source, [
StaticWarningCode.NON_VOID_RETURN_FOR_SETTER,
StaticWarningCode.NON_VOID_RETURN_FOR_SETTER
]);
@@ -2160,6 +2160,16 @@ main() {
await resolveTestUnit(code);
expectInitializerType('foo', 'int', isNull);
}
+
+ Future<Null> _objectMethodOnFunctions_helper2(String code) async {
+ await resolveTestUnit(code);
+ expectIdentifierType('t0', "String");
+ expectIdentifierType('t1', "() → String");
+ expectIdentifierType('t2', "int");
+ expectIdentifierType('t3', "String");
+ expectIdentifierType('t4', "() → String");
+ expectIdentifierType('t5', "int");
+ }
}
@reflectiveTest
« no previous file with comments | « pkg/analyzer/test/generated/static_warning_code_test.dart ('k') | pkg/analyzer/test/src/dart/constant/evaluation_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698