From 2aa0472971b2f0382412d5b1c7f87d65e9f754d5 Mon Sep 17 00:00:00 2001
From: tison <wander4096@gmail.com>
Date: Wed, 11 Jan 2023 10:07:12 +0800
Subject: [PATCH] chore: improve debug logging

Signed-off-by: tison <wander4096@gmail.com>
---
 .../io/korandoru/hawkeye/core/Selection.java  | 20 ++++++++++++++++---
 1 file changed, 17 insertions(+), 3 deletions(-)

diff --git a/hawkeye-core/src/main/java/io/korandoru/hawkeye/core/Selection.java b/hawkeye-core/src/main/java/io/korandoru/hawkeye/core/Selection.java
index 7fbbe424..2bc095b2 100644
--- a/hawkeye-core/src/main/java/io/korandoru/hawkeye/core/Selection.java
+++ b/hawkeye-core/src/main/java/io/korandoru/hawkeye/core/Selection.java
@@ -28,6 +28,7 @@
 import java.nio.file.PathMatcher;
 import java.nio.file.attribute.BasicFileAttributes;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.EnumSet;
 import java.util.List;
 import java.util.Set;
@@ -36,8 +37,10 @@
 import java.util.stream.Stream;
 import lombok.Getter;
 import lombok.SneakyThrows;
+import lombok.extern.slf4j.Slf4j;
 
 @Getter
+@Slf4j
 public final class Selection {
 
     private final File basedir;
@@ -59,9 +62,17 @@ public Selection(File basedir, String[] included, String[] excluded, boolean use
     @SneakyThrows
     public String[] getSelectedFiles() {
         if (selectedFiles.isDone()) {
-            return selectedFiles.get(0, TimeUnit.SECONDS);
+            final String[] files = selectedFiles.get(0, TimeUnit.SECONDS);
+            log.debug("Got previous selected files: {} (count: {})", Arrays.toString(files), files.length);
+            return files;
         }
 
+        log.debug(
+                "Selecting files with baseDir: {}, included: {}, excluded: {}",
+                basedir,
+                Arrays.toString(included),
+                Arrays.toString(excluded));
+
         final Path basePath = basedir.toPath();
 
         final List<String> excludesList = new ArrayList<>();
@@ -86,7 +97,7 @@ public String[] getSelectedFiles() {
         final Set<FileVisitOption> followLinksOption = EnumSet.of(FileVisitOption.FOLLOW_LINKS);
         Files.walkFileTree(basePath, followLinksOption, Integer.MAX_VALUE, new FileVisitor<>() {
             @Override
-            public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs)  {
+            public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs) {
                 final Path path = basePath.relativize(dir);
                 final boolean isExcluded = folderExcludes.stream().anyMatch(m -> m.matches(path));
                 final boolean isInvertExcluded = folderInvertExcludes.stream().anyMatch(m -> m.matches(path));
@@ -117,7 +128,10 @@ public FileVisitResult postVisitDirectory(Path dir, IOException exc) {
         });
 
         this.selectedFiles.complete(results.toArray(String[]::new));
-        return selectedFiles.get(0, TimeUnit.SECONDS);
+        final String[] files = selectedFiles.get(0, TimeUnit.SECONDS);
+        log.debug("Selected files: {} (count: {})", Arrays.toString(files), files.length);
+
+        return files;
     }
 
     private List<PathMatcher> buildFolderPathMaters(String[] patterns) {