use stack_trace

Author: Renan Araujo

lib/src/cli/cli.dart

@@ -5,6 +5,7 @@ import 'package:lcov_parser/lcov_parser.dart';
 import 'package:mason/mason.dart';
 import 'package:path/path.dart' as p;
 import 'package:pubspec_parse/pubspec_parse.dart';
+import 'package:stack_trace/stack_trace.dart';
 import 'package:universal_io/io.dart';
 import 'package:very_good_cli/src/commands/test/templates/test_runner_bundle.dart';
 import 'package:very_good_test_runner/very_good_test_runner.dart';

lib/src/cli/flutter_cli.dart

@@ -332,10 +332,13 @@ Future<int> _flutterTest({
           stderr('$clearLine${event.stackTrace}');
         }
 
-        final pathFromStackTrace = getPathFromStackTrace(event.stackTrace);
+        final traceLocation = _getTraceLocation(
+          cwd: cwd,
+          stackTrace: event.stackTrace,
+        );
 
         final testErrorDescription =
-            pathFromStackTrace ?? event.error.replaceAll('\n', ' ');
+            traceLocation ?? event.error.replaceAll('\n', ' ');
 
         failedTestErrorMessages[event.testID] = testErrorDescription;
       }
@@ -388,7 +391,6 @@ Future<int> _flutterTest({
               '$clearLine - $errorMessage',
             );
           }
-
           stderr(lines.toString());
         }
       }
@@ -462,39 +464,17 @@ extension on String {
   }
 }
 
-String? getPathFromStackTrace(String stackTrace) {
-  final trimmedStackTrace = stackTrace.trim();
-
-  if (trimmedStackTrace.isEmpty) {
-    return null;
-  }
-
-  final splittedStackTrace =
-      trimmedStackTrace.split('\n').where((element) => element.isNotEmpty);
-
-  if (splittedStackTrace.isEmpty) {
-    return null;
-  }
-
-  final lastLine = splittedStackTrace.last.trim();
-
-  if (lastLine.isEmpty) {
-    return null;
-  }
-
-  final lastLineIterator = lastLine.split(' ').iterator;
-
-  if (!lastLineIterator.moveNext()) {
-    return null;
-  }
-
-  final path = p.normalize(lastLineIterator.current);
-  if (!File(path).existsSync()) {
+String? _getTraceLocation({
+  required String cwd,
+  required String stackTrace,
+}) {
+  try {
+    final trace = Trace.parse(stackTrace);
+    if (trace.frames.isEmpty) {
+      return null;
+    }
+    return trace.frames.last.location;
+  } on FormatException {
     return null;
   }
-  if (!lastLineIterator.moveNext()) {
-    return path;
-  }
-
-  return '$path:${lastLineIterator.current}';
 }

pubspec.yaml

@@ -16,6 +16,7 @@ dependencies:
   path: ^1.8.0
   pub_updater: ^0.2.1
   pubspec_parse: ^1.2.0
+  stack_trace: 1.10.0
   universal_io: ^2.0.4
   usage: ^4.0.2
   very_good_analysis: ^2.4.0

test/src/cli/flutter_cli_test.dart

@@ -381,6 +381,9 @@ void main() {
         verify(
           () => logger.write(any(that: contains('Some tests failed.'))),
         ).called(1);
+        verify(
+          () => logger.err(any(that: contains('- test/example_test.dart 4:5'))),
+        ).called(1);
       });
 
       test('completes when there is a test directory (skipping)', () async {
@@ -534,6 +537,9 @@ void main() {
         verify(
           () => logger.write(any(that: contains('-1: Some tests failed.'))),
         ).called(1);
+        verify(
+              () => logger.err(any(that: contains('- test/example_test.dart 5:5'))),
+        ).called(1);
       });
 
       test('completes and truncates really long test name', () async {