Pankraz76
diff --git a/‎src/main/java/com/puppycrawl/tools/checkstyle/Checker.java
Lines changed: 41 additions & 14 deletions b/‎src/main/java/com/puppycrawl/tools/checkstyle/Checker.java
Lines changed: 41 additions & 14 deletions
diff --git a/‎src/test/java/com/puppycrawl/tools/checkstyle/CheckerTest.java
Lines changed: 0 additions & 37 deletions b/‎src/test/java/com/puppycrawl/tools/checkstyle/CheckerTest.java
Lines changed: 0 additions & 37 deletions
@@ -212,6 +212,44 @@ public void setBasedir(String basedir) {
 
     @Override
     public int process(List<File> files) throws CheckstyleException {
+        return processFilesAudition(files);
+    }
+
+    @Override
+    public int process(Collection<Path> paths) throws CheckstyleException {
+        return processFilesAudition(paths.stream()
+                .map(Path::toFile)
+                .collect(Collectors.toUnmodifiableList()));
+    }
+
+    /**
+     * Core method that processes files through all configured checks.
+     *
+     * <p>This method:
+     * <ol>
+     *   <li>Updates cache with external resources if caching is enabled</li>
+     *   <li>Notifies listeners that audit is starting</li>
+     *   <li>Initializes all file set checks</li>
+     *   <li>Processes only files matching configured extensions</li>
+     *   <li>Performs cleanup of file set checks</li>
+     *   <li>Notifies listeners that audit is complete</li>
+     * </ol>
+     *
+     * <p>The actual file processing is handled by {@link #processFiles(List)} which:
+     * <ul>
+     *   <li>Applies before-execution file filters</li>
+     *   <li>Handles caching of file timestamps</li>
+     *   <li>Processes each file through all checks</li>
+     *   <li>Manages error reporting</li>
+     * </ul>
+     *
+     * @param files List of files to process (non-null, may be empty)
+     * @return Total count of errors found
+     * @throws CheckstyleException if an error occurs during processing
+     * @see #process(List)
+     * @see #process(Collection)
+     */
+    private int processFilesAudition(List<File> files) throws CheckstyleException {
         if (cacheFile != null) {
             cacheFile.putExternalResources(getExternalResourceLocations());
         }
@@ -222,29 +260,18 @@ public int process(List<File> files) throws CheckstyleException {
             fsc.beginProcessing(charset);
         }
 
-        final List<File> targetFiles = files.stream()
+        processFiles(files.stream()
                 .filter(file -> CommonUtil.matchesFileExtension(file, fileExtensions))
-                .collect(Collectors.toUnmodifiableList());
-        processFiles(targetFiles);
+                .collect(Collectors.toUnmodifiableList()));
 
         // Finish up
         // It may also log!!!
         fileSetChecks.forEach(FileSetCheck::finishProcessing);
-
-        // It may also log!!!
         fileSetChecks.forEach(FileSetCheck::destroy);
-
-        final int errorCount = counter.getCount();
         fireAuditFinished();
-        return errorCount;
+        return counter.getCount();
     }
 
-    @Override
-    public int process(Collection<Path> paths) throws CheckstyleException {
-        return process(paths.stream()
-                .map(Path::toFile)
-                .collect(Collectors.toUnmodifiableList()));
-    }
 
     /**
      * Returns a set of external configuration resource locations which are used by all file set
 
@@ -1779,43 +1779,6 @@ public void testFileExtensionsPath() throws Exception {
                     .isNull();
         }
 
-        @Test
-        public void testProcessPathCollection() throws Exception {
-            final Checker checker = new Checker();
-            final DebugAuditAdapter auditAdapter = new DebugAuditAdapter();
-            checker.addListener(auditAdapter);
-
-            final File tempFile1 = createTempFile("test1", ".java");
-            final File tempFile2 = createTempFile("test2", ".java");
-
-            final Checker spyChecker = Mockito.spy(checker);
-            doReturn(0).when(spyChecker).process(Collections.<File>emptyList());
-
-            spyChecker.process(Arrays.asList(
-                    tempFile1.toPath(),
-                    tempFile2.toPath()
-            ));
-
-            final ArgumentCaptor<List<File>> filesCaptor = ArgumentCaptor.forClass(List.class);
-            Mockito.verify(spyChecker).process(filesCaptor.capture());
-            final List<File> processedFiles = filesCaptor.getValue();
-            assertWithMessage("Processed files count mismatch")
-                    .that(processedFiles)
-                    .hasSize(2);
-            assertWithMessage("First file mismatch")
-                    .that(processedFiles.get(0))
-                    .isEqualTo(tempFile1);
-            assertWithMessage("Second file mismatch")
-                    .that(processedFiles.get(1))
-                    .isEqualTo(tempFile2);
-            assertWithMessage("Audit started event not fired")
-                    .that(auditAdapter.wasCalled())
-                    .isTrue();
-            assertWithMessage("Incorrect number of files processed")
-                    .that(auditAdapter.getNumFilesStarted())
-                    .isEqualTo(2);
-        }
-
     }
 
     public static class DefaultLoggerWithCounter extends DefaultLogger {