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

Unified Diff: tests/language/full_stacktrace2_test.dart

Issue 14009004: Remove methods on StackTrace class. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Address comments. Created 7 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
Index: tests/language/full_stacktrace2_test.dart
diff --git a/tests/language/full_stacktrace2_test.dart b/tests/language/full_stacktrace2_test.dart
index 7f45706adf45308d45de99752b69692634baf3af..840e9b7660e5a30069797af0f566e7dad7c69baa 100644
--- a/tests/language/full_stacktrace2_test.dart
+++ b/tests/language/full_stacktrace2_test.dart
@@ -4,8 +4,11 @@
import "package:expect/expect.dart";
+// This test assumes that all called functions occur in the stack trace.
+// This might not be the case if inlining methods.
Ivan Posva 2013/04/12 03:13:10 This is a current limitation of dart2js and not so
Lasse Reichstein Nielsen 2013/04/15 11:37:22 I'll remove this for now, and keep it in the statu
+
void func1() {
- throw new Exception("Test peanut gallery request for Full stacktrace");
+ throw new Exception("Test full stacktrace");
}
void func2() {
func1();
@@ -14,36 +17,15 @@ void func3() {
try {
func2();
} on Object catch(e, s) {
- print(e);
-
- var full_trace = s.fullStackTrace;
- Expect.isTrue(full_trace.contains("func1"));
- Expect.isTrue(full_trace.contains("func2"));
- Expect.isTrue(full_trace.contains("func3"));
- Expect.isTrue(full_trace.contains("func4"));
- Expect.isTrue(full_trace.contains("func5"));
- Expect.isTrue(full_trace.contains("func6"));
- Expect.isTrue(full_trace.contains("func7"));
- Expect.isTrue(full_trace.contains("main"));
-
- var trace = s.stackTrace;
- Expect.isTrue(trace.contains("func1"));
- Expect.isTrue(trace.contains("func2"));
- Expect.isTrue(trace.contains("func3"));
-
- Expect.isFalse(trace.contains("func4"));
- Expect.isFalse(trace.contains("func5"));
- Expect.isFalse(trace.contains("func6"));
- Expect.isFalse(trace.contains("func7"));
- Expect.isFalse(trace.contains("main"));
-
- print(s);
-
- print("Full stack trace");
- print(full_trace);
-
- print("Stack trace");
- print(trace);
+ var fullTrace = s.toString();
+ Expect.isTrue(fullTrace.contains("func1"));
+ Expect.isTrue(fullTrace.contains("func2"));
+ Expect.isTrue(fullTrace.contains("func3"));
+ Expect.isTrue(fullTrace.contains("func4"));
+ Expect.isTrue(fullTrace.contains("func5"));
+ Expect.isTrue(fullTrace.contains("func6"));
+ Expect.isTrue(fullTrace.contains("func7"));
+ Expect.isTrue(fullTrace.contains("main"));
throw; // This is a rethrow.
}
@@ -56,36 +38,15 @@ int func5() {
try {
func4();
} on Object catch(e, s) {
-
- var full_trace = s.fullStackTrace;
- print(full_trace);
- Expect.isTrue(full_trace.contains("func1"));
- Expect.isTrue(full_trace.contains("func2"));
- Expect.isTrue(full_trace.contains("func3"));
- Expect.isTrue(full_trace.contains("func4"));
- Expect.isTrue(full_trace.contains("func5"));
- Expect.isTrue(full_trace.contains("func6"));
- Expect.isTrue(full_trace.contains("func7"));
- Expect.isTrue(full_trace.contains("main"));
-
- var trace = s.stackTrace;
- Expect.isTrue(trace.contains("func1"));
- Expect.isTrue(trace.contains("func2"));
- Expect.isTrue(trace.contains("func3"));
- Expect.isTrue(trace.contains("func4"));
- Expect.isTrue(trace.contains("func5"));
-
- Expect.isFalse(trace.contains("func6"));
- Expect.isFalse(trace.contains("func7"));
- Expect.isFalse(trace.contains("main"));
-
- print(s);
-
- print("Full stack trace");
- print(full_trace);
-
- print("Stack trace");
- print(trace);
+ var fullTrace = s.toString();
+ Expect.isTrue(fullTrace.contains("func1"));
+ Expect.isTrue(fullTrace.contains("func2"));
+ Expect.isTrue(fullTrace.contains("func3"));
+ Expect.isTrue(fullTrace.contains("func4"));
+ Expect.isTrue(fullTrace.contains("func5"));
+ Expect.isTrue(fullTrace.contains("func6"));
+ Expect.isTrue(fullTrace.contains("func7"));
+ Expect.isTrue(fullTrace.contains("main"));
}
return 1;
}

Powered by Google App Engine
This is Rietveld 408576698