pkg/stack_trace: fix vm_test

test/vm_test.dart

Index: test/vm_test.dart
diff --git a/test/vm_test.dart b/test/vm_test.dart
index f0967b51024138a3fafc9ffb698b837fbb38e431..ad6c1dd148c4be137cce8824d40c0d0e45b114e5 100644
--- a/test/vm_test.dart
+++ b/test/vm_test.dart
@@ -12,6 +12,10 @@ import 'package:path/path.dart' as path;
 import 'package:stack_trace/stack_trace.dart';
 import 'package:test/test.dart';
 
+String getStackTraceString() => StackTrace.current.toString();

[nweiz, 2016/02/01 22:46:46]

Add comments explaining why these have to be wrapped in functions.

[kevmoo, 2016/02/01 22:48:55]

Done.

+
+StackTrace getStackTraceObject() => StackTrace.current;
+
 Frame getCaller([int level]) {
   if (level == null) return new Frame.caller();
   return new Frame.caller(level);
@@ -26,7 +30,7 @@ Trace nestedGetCurrentTrace(int level) => getCurrentTrace(level);
 void main() {
   group('Trace', () {
     test('.parse parses a real stack trace correctly', () {
-      var string = StackTrace.current.toString();
+      var string = getStackTraceString();
       var trace = new Trace.parse(string);
       expect(path.url.basename(trace.frames.first.uri.path),
           equals('vm_test.dart'));
@@ -34,7 +38,7 @@ void main() {
     });
 
     test('converts from a native stack trace correctly', () {
-      var trace = new Trace.from(StackTrace.current);
+      var trace = new Trace.from(getStackTraceObject());
       expect(path.url.basename(trace.frames.first.uri.path),
           equals('vm_test.dart'));
       expect(trace.frames.first.member, equals('getStackTraceObject'));