You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2023/06/11 14:24:03 UTC

[commons-fileupload] branch master updated: Set maven-checkstyle-plugin includeTestSourceDirectory to true

This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-fileupload.git


The following commit(s) were added to refs/heads/master by this push:
     new 4451a27  Set maven-checkstyle-plugin includeTestSourceDirectory to true
4451a27 is described below

commit 4451a271b1b8d2bd95491921a91d484e6e4ea2dc
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Jun 11 10:23:59 2023 -0400

    Set maven-checkstyle-plugin includeTestSourceDirectory to true
    
    Fix compile issue
---
 .../main/java/org/apache/commons/fileupload2/FileItemStreamImpl.java    | 2 +-
 .../src/main/java/org/apache/commons/fileupload2/MultipartStream.java   | 2 +-
 pom.xml                                                                 | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/commons-fileupload2-core/src/main/java/org/apache/commons/fileupload2/FileItemStreamImpl.java b/commons-fileupload2-core/src/main/java/org/apache/commons/fileupload2/FileItemStreamImpl.java
index 0357d77..634f35b 100644
--- a/commons-fileupload2-core/src/main/java/org/apache/commons/fileupload2/FileItemStreamImpl.java
+++ b/commons-fileupload2-core/src/main/java/org/apache/commons/fileupload2/FileItemStreamImpl.java
@@ -161,7 +161,7 @@ class FileItemStreamImpl implements FileItemStream {
     @Override
     public InputStream getInputStream() throws IOException {
         if (inputStreamClosed) {
-            throw new FileItemStream.ItemSkippedException();
+            throw new FileItemStream.ItemSkippedException("getInputStream()");
         }
         return inputStream;
     }
diff --git a/commons-fileupload2-core/src/main/java/org/apache/commons/fileupload2/MultipartStream.java b/commons-fileupload2-core/src/main/java/org/apache/commons/fileupload2/MultipartStream.java
index c25bea8..4dda7cb 100644
--- a/commons-fileupload2-core/src/main/java/org/apache/commons/fileupload2/MultipartStream.java
+++ b/commons-fileupload2-core/src/main/java/org/apache/commons/fileupload2/MultipartStream.java
@@ -231,7 +231,7 @@ public final class MultipartStream {
 
         private void checkOpen() throws ItemSkippedException {
             if (closed) {
-                throw new FileItemStream.ItemSkippedException("checkOpen");
+                throw new FileItemStream.ItemSkippedException("checkOpen()");
             }
         }
 
diff --git a/pom.xml b/pom.xml
index 3095a5f..5fc4696 100644
--- a/pom.xml
+++ b/pom.xml
@@ -402,7 +402,7 @@
           <configuration>
             <configLocation>${commons.parent.dir}/src/checkstyle/fileupload_checks.xml</configLocation>
             <suppressionsLocation>${commons.parent.dir}/src/checkstyle/checkstyle-suppressions.xml</suppressionsLocation>
-            <includeTestSourceDirectory>false</includeTestSourceDirectory>
+            <includeTestSourceDirectory>true</includeTestSourceDirectory>
             <enableRulesSummary>false</enableRulesSummary>
           </configuration>
         </plugin>