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

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

Issue 2626653002: Remove computeLibrarySourceErrors(). (Closed)
Patch Set: 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/non_hint_code_test.dart
diff --git a/pkg/analyzer/test/generated/non_hint_code_test.dart b/pkg/analyzer/test/generated/non_hint_code_test.dart
index 2ef678bcc17238a68a6dfd18430b13dc5641caaa..aa4526a4cd9612e88e91261a31a72ce1654a9156 100644
--- a/pkg/analyzer/test/generated/non_hint_code_test.dart
+++ b/pkg/analyzer/test/generated/non_hint_code_test.dart
@@ -32,7 +32,6 @@ class B extends A {
}
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -51,7 +50,6 @@ f() {
throw 'foo';
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -62,7 +60,6 @@ const bool DEBUG = true;
f() {
DEBUG ? 1 : 2;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -73,7 +70,6 @@ const bool DEBUG = false;
f() {
DEBUG ? 1 : 2;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -84,7 +80,6 @@ const bool DEBUG = true;
f() {
if(DEBUG) {} else {}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -97,7 +92,6 @@ class A {
f() {
if(A.DEBUG) {}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -116,7 +110,6 @@ library lib2;
class A {
static const bool DEBUG = false;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -135,7 +128,6 @@ library lib2;
class A {
static const bool DEBUG = false;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -146,7 +138,6 @@ const bool DEBUG = false;
f() {
if(DEBUG) {}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -157,7 +148,6 @@ const bool DEBUG = false;
f() {
while(DEBUG) {}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -169,7 +159,6 @@ class B extends A {}
f() {
try {} on B catch (e) {} on A catch (e) {} catch (e) {}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -189,7 +178,6 @@ f() {
break;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -200,7 +188,6 @@ const bool DEBUG = false;
f() {
bool b = DEBUG && false;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -211,7 +198,6 @@ const bool DEBUG = true;
f() {
bool b = DEBUG || true;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -224,7 +210,6 @@ f() {
}
int a = 1;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -241,7 +226,6 @@ class C {
}
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -256,7 +240,6 @@ g() {
f();
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -275,7 +258,6 @@ class C {
}
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -292,7 +274,6 @@ class C {
}
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -310,7 +291,6 @@ class C {
}
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -320,7 +300,6 @@ class C {
f(int x, int y) {
var v = x / y.toInt();
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -330,7 +309,6 @@ f(int x, int y) {
f(x, y) {
var v = (x / y).toInt();
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -343,7 +321,6 @@ class A {
f(A x, A y) {
var v = (x / y).toInt();
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -360,7 +337,6 @@ one.A a2;''');
r'''
library lib1;
class A {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -378,7 +354,6 @@ B b;''');
library lib1;
class A {}
class B {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -396,7 +371,6 @@ B b;''');
library lib1;
class A {}
class B {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -422,7 +396,6 @@ f(var message, var dynamic_) {
}
int s = message;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -439,7 +412,6 @@ f(var message) {
}
int s = message;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -455,7 +427,6 @@ f(var message) {
}
String s = message;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -465,28 +436,24 @@ f(var message) {
abstract class A {
int m();
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
void test_missingReturn_expressionFunctionBody() {
Source source = addSource("int f() => 0;");
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
void test_missingReturn_noReturnType() {
Source source = addSource("f() {}");
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
void test_missingReturn_voidReturnType() {
Source source = addSource("void f() {}");
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -497,7 +464,6 @@ m(x) {
for (var v = x; ; v++) {}
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -508,7 +474,6 @@ m(x) {
if (x?.y == null) {}
}
''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -519,7 +484,6 @@ class A {
bool operator ==(x) { return x; }
get hashCode => 0;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -539,7 +503,6 @@ class B implements A {
@override
int c;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -559,7 +522,6 @@ class B extends A {
@override
int c;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -573,7 +535,6 @@ class B implements A {
@override
int get m => 1;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -587,7 +548,6 @@ class B extends A {
@override
int get m => 1;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -601,7 +561,6 @@ class B implements A {
@override
int m() => 1;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -615,7 +574,6 @@ class B extends A {
@override
int m() => 1;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -629,7 +587,6 @@ class B extends A {
@override
int m() => 1;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -643,7 +600,6 @@ class B implements A {
@override
set m(int x) {}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -657,7 +613,6 @@ class B extends A {
@override
set m(int x) {}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -674,7 +629,6 @@ class Z {
y.x.add(new A());
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -693,7 +647,6 @@ f(var a) {
a++;
++a;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -713,7 +666,6 @@ class B {
++a;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -733,7 +685,6 @@ class B {
}
@proxy
class A {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -748,7 +699,6 @@ f(var a) {
return a.b;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -763,7 +713,6 @@ f(var a, var a2) {
a2 = new A();
a += a2;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -773,7 +722,6 @@ class D<T extends dynamic> {
fieldAccess(T t) => t.abc;
methodAccess(T t) => t.xyz(1, 2, 'three');
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -787,7 +735,6 @@ f() {
var a = new A();
a.b();
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -808,7 +755,6 @@ f(A a, B b) {
}
ab.m();
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -827,7 +773,6 @@ f(A a, B b) {
}
ab.m(0);
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -842,7 +787,6 @@ f(var a) {
a + 1;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -857,7 +801,6 @@ f(var a) {
a[0]++;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -872,7 +815,6 @@ f(var a) {
a[0];
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -887,7 +829,6 @@ f(var a) {
a[0] = 1;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -902,7 +843,6 @@ f(var a) {
a++;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -917,7 +857,6 @@ f(var a) {
++a;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -932,7 +871,6 @@ f(var a) {
a.b = 0;
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -948,7 +886,6 @@ class B<E> {
(e as A).a();
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -961,7 +898,6 @@ class B implements I {}
I m(A a, B b) {
return a == null ? b as I : a as I;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -971,7 +907,6 @@ I m(A a, B b) {
m(v) {
var b = v as Object;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -984,7 +919,6 @@ Future<int> f() => new Future.value(0);
void g(bool c) {
(c ? f(): new Future.value(0) as Future<int>).then((int value) {});
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -994,7 +928,6 @@ void g(bool c) {
m(v) {
var b = Object as dynamic;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1010,7 +943,6 @@ class B extends A {
print(y);
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1027,7 +959,6 @@ class B extends A {
return super.noSuchMethod(y);
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1041,7 +972,6 @@ class B extends A {
mmm();
noSuchMethod(y) => super.hashCode;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1055,7 +985,6 @@ class B extends A {
mmm();
noSuchMethod(y) => 42;
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1072,7 +1001,6 @@ library lib1;
class A {
const A() {}
}''');
- computeLibrarySourceErrors(source);
assertErrors(source);
verify([source, source2]);
}
@@ -1095,7 +1023,6 @@ class A {}''');
r'''
library lib2;
class B {}''');
- computeLibrarySourceErrors(source);
assertErrors(source);
assertNoErrors(source2);
assertNoErrors(source3);
@@ -1106,7 +1033,6 @@ class B {}''');
Source source = addSource(r'''
library L;
import 'dart:core';''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1127,7 +1053,6 @@ class One {}''');
r'''
library lib2;
class Two {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1154,7 +1079,6 @@ class Two {}''');
r'''
library lib3;
class Three {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1182,7 +1106,6 @@ class Two {}''');
library lib3;
export 'lib2.dart';
class Three {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1201,7 +1124,6 @@ class A {
r'''
library lib1;
const x = 0;''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1223,7 +1145,6 @@ class A {
library lib1;
class One {}
topLevelFunction() {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1247,7 +1168,6 @@ class A {
library lib1;
class One {}
topLevelFunction() {}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1260,7 +1180,6 @@ class A {
var a = f();
}
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1271,7 +1190,6 @@ int f() => 1;
g() {
var a = f();
}''');
- computeLibrarySourceErrors(source);
assertNoErrors(source);
verify([source]);
}
@@ -1280,13 +1198,11 @@ g() {
class PubSuggestionCodeTest extends ResolverTestCase {
void test_import_package() {
Source source = addSource("import 'package:somepackage/other.dart';");
- computeLibrarySourceErrors(source);
assertErrors(source, [CompileTimeErrorCode.URI_DOES_NOT_EXIST]);
}
void test_import_packageWithDotDot() {
Source source = addSource("import 'package:somepackage/../other.dart';");
- computeLibrarySourceErrors(source);
assertErrors(source, [
CompileTimeErrorCode.URI_DOES_NOT_EXIST,
HintCode.PACKAGE_IMPORT_CONTAINS_DOT_DOT
@@ -1295,7 +1211,6 @@ class PubSuggestionCodeTest extends ResolverTestCase {
void test_import_packageWithLeadingDotDot() {
Source source = addSource("import 'package:../other.dart';");
- computeLibrarySourceErrors(source);
assertErrors(source, [
CompileTimeErrorCode.URI_DOES_NOT_EXIST,
HintCode.PACKAGE_IMPORT_CONTAINS_DOT_DOT
@@ -1307,7 +1222,6 @@ class PubSuggestionCodeTest extends ResolverTestCase {
cacheSource("/myproj/lib/other.dart", "");
Source source =
addNamedSource("/myproj/web/test.dart", "import '../lib/other.dart';");
- computeLibrarySourceErrors(source);
assertErrors(
source, [HintCode.FILE_IMPORT_OUTSIDE_LIB_REFERENCES_FILE_INSIDE]);
}
@@ -1316,7 +1230,6 @@ class PubSuggestionCodeTest extends ResolverTestCase {
cacheSource("/myproj/lib/other.dart", "");
Source source =
addNamedSource("/myproj/web/test.dart", "import '../lib/other.dart';");
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -1325,7 +1238,6 @@ class PubSuggestionCodeTest extends ResolverTestCase {
cacheSource("/myproj/web/other.dart", "");
Source source =
addNamedSource("/myproj/lib/test.dart", "import '../web/other.dart';");
- computeLibrarySourceErrors(source);
assertErrors(
source, [HintCode.FILE_IMPORT_INSIDE_LIB_REFERENCES_FILE_OUTSIDE]);
}
@@ -1334,7 +1246,6 @@ class PubSuggestionCodeTest extends ResolverTestCase {
cacheSource("/myproj/web/other.dart", "");
Source source =
addNamedSource("/myproj/lib/test.dart", "import '../web/other.dart';");
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -1343,7 +1254,6 @@ class PubSuggestionCodeTest extends ResolverTestCase {
cacheSource("/myproj/lib/other.dart", "");
Source source =
addNamedSource("/myproj/lib/test.dart", "import 'other.dart';");
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -1352,7 +1262,6 @@ class PubSuggestionCodeTest extends ResolverTestCase {
cacheSource("/myproj/lib/bar/other.dart", "");
Source source = addNamedSource(
"/myproj/lib/foo/test.dart", "import '../bar/other.dart';");
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
@@ -1361,7 +1270,6 @@ class PubSuggestionCodeTest extends ResolverTestCase {
cacheSource("/myproj/web/other.dart", "");
Source source =
addNamedSource("/myproj/lib2/test.dart", "import '../web/other.dart';");
- computeLibrarySourceErrors(source);
assertNoErrors(source);
}
}
« no previous file with comments | « pkg/analyzer/test/generated/non_error_resolver_test.dart ('k') | pkg/analyzer/test/generated/resolver_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698