cleanup

Author: felangel

pkgs/test_core/lib/src/runner/vm/platform.dart

@@ -458,23 +458,19 @@ Future<Set<String>> _filterCoveragePackages(
   List<RegExp>? coveragePackages,
   String? coverageLcov,
 ) async {
-  if (coveragePackages == null || coveragePackages.isEmpty) {
-    if (coverageLcov == null) {
-      // If no filters were provided and we using json workflow, report coverage
-      // for all packages. This is required to maintain backward compatibility
-      // particularly in cases where coverage is required for files outside of
-      // the lib directory. See https://github.com/dart-lang/test/issues/2581
-      return {};
-    }
-    return workspacePackageNames(await currentPackage);
+  if (coverageLcov == null && coveragePackages == null) {
+    // If no filters were provided and we using json workflow, report coverage
+    // for all packages. This is required to maintain backward compatibility
+    // particularly in cases where coverage is required for files outside of
+    // the lib directory. See https://github.com/dart-lang/test/issues/2581
+    return {};
   }
-
-  if (coveragePackages.isEmpty) {
+  if (coveragePackages == null || coveragePackages.isEmpty) {
     return workspacePackageNames(await currentPackage);
+  } else {
+    return (await currentPackageConfig).packages
+        .map((package) => package.name)
+        .where((name) => coveragePackages.any((re) => re.hasMatch(name)))
+        .toSet();
   }
-
-  return (await currentPackageConfig).packages
-      .map((package) => package.name)
-      .where((name) => coveragePackages.any((re) => re.hasMatch(name)))
-      .toSet();
 }