Add the ability to ignore lines from coverage depending on the comment

[pasindud]

Use // coverage:ignore-line to ignore one line.
Use // coverage:ignore-start and // coverage:ignore-end to ignore range of lines inclusive.
Use // coverage:ignore-file to ignore the whole file.

[grouma]

We should have a CHANGELOG entry and rev the package version.

[pasindud]

Should we add a entry to README also about this ?

[grouma]

Should we add a entry to README also about this ?

Probably a good idea. Otherwise LGTM.

[elias8]

Is this available on the current stable which is 1.22.5?

[grouma]

This should be available in stable as it depends on version 14.1: https://github.com/flutter/flutter/blob/78910062997c3a836feee883712c241a5fd22983/packages/flutter_tools/pubspec.yaml#L17

[lukepighetti]

I'm on Flutter Channel dev, 1.26.0-8.0.pre and have // coverage:ignore-file at the top of a file and it's still showing up in code coverage reports. I'm generating coverage with flutter test --coverage

Is this known to work? Am I missing a configuration step?

Entire file

// coverage:ignore-file
import 'package:permission_handler/permission_handler.dart';
import 'package:permission_handler/permission_handler.dart' as permissionHandler
    show openAppSettings;

/// A mockable interface for `permission_handler` package.
class PermissionsService {
  /// Check the status of a specific [Permission]
  Future<PermissionStatus> status(Permission permission) {
    return permission.status;
  }

  /// Open the app settings.
  Future<bool> openAppSettings() {
    return permissionHandler.openAppSettings();
  }

  /// Request permissions for a single permission.
  Future<PermissionStatus> request(Permission permission) {
    return permission.request();
  }
}

[grouma]

@lukepighetti I tried to reproduce your issue and I'm unable to do so. I simply ran flutter test --coverage on the hello_world example within the Flutter repo. I can add and ignore files as expected.