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

Unified Diff: frog/tests/leg/src/CompilerTest.dart

Issue 10170025: Use the correct current element when computing the type of function elements. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Add a test. Created 8 years, 8 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 | lib/compiler/implementation/elements/elements.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: frog/tests/leg/src/CompilerTest.dart
diff --git a/frog/tests/leg/src/CompilerTest.dart b/frog/tests/leg/src/CompilerTest.dart
new file mode 100644
index 0000000000000000000000000000000000000000..f4e02e2ed713eb4ac32022469a56be2a3beab7bc
--- /dev/null
+++ b/frog/tests/leg/src/CompilerTest.dart
@@ -0,0 +1,46 @@
+// Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+#import("../../../../lib/compiler/implementation/leg.dart");
+#import("../../../../lib/compiler/implementation/elements/elements.dart");
+#import("../../../../lib/compiler/implementation/tree/tree.dart");
+#import("../../../../lib/compiler/implementation/util/util.dart");
+#import("mock_compiler.dart");
+#import("parser_helper.dart");
+
+class CallbackMockCompiler extends MockCompiler {
+ CallbackMockCompiler();
+
+ var onError;
+ var onWarning;
+
+ setOnError(var f) => onError = f;
+ setOnWarning(var f) => onWarning = f;
+
+ void reportWarning(Node node, var message) {
+ if (onWarning !== null) onWarning(this, node, message);
+ super.reportWarning(node, message);
+ }
+
+ void reportError(Node node, var message) {
+ if (onError !== null) onError(this, node, message);
+ super.reportError(node, message);
+ }
+}
+
+testErrorHandling() {
+ // Test that compiler.currentElement is set correctly when
+ // reporting errors/warnings.
+ CallbackMockCompiler compiler = new CallbackMockCompiler();
+ ResolverVisitor visitor = compiler.resolverVisitor();
+ compiler.parseScript('NoSuchPrefix.NoSuchType foo() {}');
+ FunctionElement foo = compiler.mainApp.find(buildSourceString('foo'));
+ compiler.setOnWarning((c, n, m) => Expect.equals(foo, compiler.currentElement));
+ foo.computeType(compiler);
+ Expect.equals(1, compiler.warnings.length);
+}
+
+main() {
+ testErrorHandling();
+}
« no previous file with comments | « no previous file | lib/compiler/implementation/elements/elements.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698