From df02518dc98bd5109dd4a3473884fbce0fc8b837 Mon Sep 17 00:00:00 2001 From: Pasindu De Silva Date: Fri, 19 Jun 2020 01:34:42 +0800 Subject: [PATCH] Add the ability to ignore lines depending on comments (#58656) https://github.com/dart-lang/coverage - 0.14.0 added the ability to be able to ignore coverage depending on comments. // coverage:ignore-line to ignore one line. // coverage:ignore-start and // coverage:ignore-end to ignore range of lines inclusive. // coverage:ignore-file to ignore the whole file. --- .../lib/src/test/coverage_collector.dart | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/packages/flutter_tools/lib/src/test/coverage_collector.dart b/packages/flutter_tools/lib/src/test/coverage_collector.dart index da9960ab46..5976482f79 100644 --- a/packages/flutter_tools/lib/src/test/coverage_collector.dart +++ b/packages/flutter_tools/lib/src/test/coverage_collector.dart @@ -66,7 +66,11 @@ class CoverageCollector extends TestWatcher { assert(data != null); _logMessage('($observatoryUri): collected coverage data; merging...'); - _addHitmap(await coverage.createHitmap(data['coverage'] as List>)); + _addHitmap(await coverage.createHitmap( + data['coverage'] as List>, + packagesPath: globalPackagesPath, + checkIgnoredLines: true, + )); _logMessage('($observatoryUri): done merging coverage data into global coverage map.'); } @@ -98,7 +102,11 @@ class CoverageCollector extends TestWatcher { assert(data != null); _logMessage('pid $pid ($observatoryUri): collected coverage data; merging...'); - _addHitmap(await coverage.createHitmap(data['coverage'] as List>)); + _addHitmap(await coverage.createHitmap( + data['coverage'] as List>, + packagesPath: globalPackagesPath, + checkIgnoredLines: true, + )); _logMessage('pid $pid ($observatoryUri): done merging coverage data into global coverage map.'); }