Merge pull request #260 from dart-lang/handle-null

Handle null coverage range
dart-lang · Jul 18, 2019 · ee986a9 · ee986a9
2 parents 93b7d82 + a847925
commit ee986a9
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 0.13.1 - 2019-07-18
+
+ * Handle scenario where the VM returns empty coverage information for a range.
+
 ## 0.13.0 - 2019-07-10
 
  * BREAKING CHANGE: Skips collecting coverage for `dart:` libaries by default,

diff --git a/lib/src/collect.dart b/lib/src/collect.dart
@@ -164,6 +164,9 @@ Future<List<Map<String, dynamic>>> _getCoverageJson(VmService service,
 
     // Collect hits and misses.
     final coverage = range.coverage;
+
+    if (coverage == null) continue;
+
     for (final tokenPos in coverage.hits) {
       final line = _getLineFromTokenPos(script, tokenPos);
       if (line == null) {

diff --git a/pubspec.yaml b/pubspec.yaml
@@ -1,5 +1,5 @@
 name: coverage
-version: 0.13.0
+version: 0.13.1
 author: Dart Team <misc@dartlang.org>
 description: Coverage data manipulation and formatting
 homepage: https://github.com/dart-lang/coverage