Added uncovered files detection issue #529

Author: Victowolf

pkgs/coverage/analysis_options.yaml

@@ -1,6 +1,8 @@
 include: package:dart_flutter_team_lints/analysis_options.yaml
 
 analyzer:
+  errors:
+    unnecessary_this: ignore
   language:
     strict-casts: true
 

pkgs/coverage/lib/src/formatter.dart

@@ -2,6 +2,8 @@
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
+// ignore_for_file: unnecessary_this
+
 import 'package:glob/glob.dart';
 import 'package:path/path.dart' as p;
 
@@ -78,26 +80,27 @@ extension FileHitMapsFormatter on Map<String, HitMap> {
     String? basePath,
     List<String>? reportOn,
     Set<Glob>? ignoreGlobs,
+    bool Function(String path)? includeUncovered,
   }) {
     final pathFilter = _getPathFilter(
       reportOn: reportOn,
       ignoreGlobs: ignoreGlobs,
     );
     final buf = StringBuffer();
+
+    // Get all Dart files in the project
+    final allDartFiles = resolver.listAllDartFiles().toSet();
+    final coveredFiles = this.keys.toSet();
+    final uncoveredFiles = allDartFiles.difference(coveredFiles);
+
     for (final entry in entries) {
       final v = entry.value;
       final lineHits = v.lineHits;
       final funcHits = v.funcHits;
       final funcNames = v.funcNames;
       final branchHits = v.branchHits;
       var source = resolver.resolve(entry.key);
-      if (source == null) {
-        continue;
-      }
-
-      if (!pathFilter(source)) {
-        continue;
-      }
+      if (source == null || !pathFilter(source)) continue;
 
       if (basePath != null) {
         source = p.relative(source, from: basePath);
@@ -129,6 +132,21 @@ extension FileHitMapsFormatter on Map<String, HitMap> {
       buf.write('end_of_record\n');
     }
 
+    // Add uncovered files if allowed
+    for (final file in uncoveredFiles) {
+      if (includeUncovered != null && !includeUncovered(file)) continue;
+      var source = resolver.resolve(file);
+      if (source == null || !pathFilter(source)) continue;
+      if (basePath != null) {
+        source = p.relative(source, from: basePath);
+      }
+
+      buf.write('SF:$source\n');
+      buf.write('LF:0\n');
+      buf.write('LH:0\n');
+      buf.write('end_of_record\n');
+    }
+
     return buf.toString();
   }
 
@@ -144,12 +162,19 @@ extension FileHitMapsFormatter on Map<String, HitMap> {
     Set<Glob>? ignoreGlobs,
     bool reportFuncs = false,
     bool reportBranches = false,
+    bool Function(String path)? includeUncovered,
   }) async {
     final pathFilter = _getPathFilter(
       reportOn: reportOn,
       ignoreGlobs: ignoreGlobs,
     );
     final buf = StringBuffer();
+
+    // Get all Dart files in the project
+    final allDartFiles = resolver.listAllDartFiles().toSet();
+    final coveredFiles = this.keys.toSet();
+    final uncoveredFiles = allDartFiles.difference(coveredFiles);
+
     for (final entry in entries) {
       final v = entry.value;
       if (reportFuncs && v.funcHits == null) {
@@ -171,18 +196,10 @@ extension FileHitMapsFormatter on Map<String, HitMap> {
               ? v.branchHits!
               : v.lineHits;
       final source = resolver.resolve(entry.key);
-      if (source == null) {
-        continue;
-      }
-
-      if (!pathFilter(source)) {
-        continue;
-      }
+      if (source == null || !pathFilter(source)) continue;
 
       final lines = await loader.load(source);
-      if (lines == null) {
-        continue;
-      }
+      if (lines == null) continue;
       buf.writeln(source);
       for (var line = 1; line <= lines.length; line++) {
         var prefix = _prefix;
@@ -193,6 +210,20 @@ extension FileHitMapsFormatter on Map<String, HitMap> {
       }
     }
 
+    // Add uncovered files if allowed
+    for (final file in uncoveredFiles) {
+      if (includeUncovered != null && !includeUncovered(file)) continue;
+      final source = resolver.resolve(file);
+      if (source == null || !pathFilter(source)) continue;
+
+      final lines = await loader.load(source);
+      if (lines == null) continue;
+      buf.writeln(source);
+      for (final line in lines) {
+        buf.writeln('       |$line');
+      }
+    }
+
     return buf.toString();
   }
 }
@@ -221,3 +252,4 @@ _PathFilter _getPathFilter({List<String>? reportOn, Set<Glob>? ignoreGlobs}) {
     return true;
   };
 }
+

pkgs/coverage/lib/src/resolver.dart

@@ -9,6 +9,19 @@ import 'package:path/path.dart' as p;
 
 /// [Resolver] resolves imports with respect to a given environment.
 class Resolver {
+  /// Returns a list of all Dart files in the project.
+  List<String> listAllDartFiles({String directoryPath = '.'}) {
+    final dir = Directory(directoryPath);
+    if (!dir.existsSync()) return [];
+
+    return dir
+        .listSync(recursive: true)
+        .whereType<File>()
+        .where((file) => file.path.endsWith('.dart'))
+        .map((file) => file.path)
+        .toList();
+  }
+
   @Deprecated('Use Resolver.create')
   Resolver({this.packagesPath, this.sdkRoot})
       : _packages = packagesPath != null ? _parsePackages(packagesPath) : null,

pkgs/coverage/test/run_and_collect_test.dart

@@ -69,6 +69,10 @@ class ThrowingResolver implements Resolver {
 
   @override
   String? get sdkRoot => throw UnimplementedError();
+
+  @override
+  List<String> listAllDartFiles({String directoryPath = '.'}) =>
+      throw UnimplementedError();
 }
 
 void checkIgnoredLinesInFilesCache(