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

elharo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-wagon.git


The following commit(s) were added to refs/heads/master by this push:
     new 666f55e5 Remove unused code (#177)
666f55e5 is described below

commit 666f55e55ba0f7660e73fc73e8006617ac392db5
Author: Elliotte Rusty Harold <[email protected]>
AuthorDate: Sat Jun 21 20:05:14 2025 +0000

    Remove unused code (#177)
---
 .../main/java/org/apache/maven/wagon/StreamingWagonTestCase.java    | 2 --
 .../java/org/apache/maven/wagon/providers/file/FileWagonTest.java   | 6 ++----
 .../org/apache/maven/wagon/providers/webdav/WebDavWagonTest.java    | 2 +-
 3 files changed, 3 insertions(+), 7 deletions(-)

diff --git 
a/wagon-provider-test/src/main/java/org/apache/maven/wagon/StreamingWagonTestCase.java
 
b/wagon-provider-test/src/main/java/org/apache/maven/wagon/StreamingWagonTestCase.java
index 30246302..8a405b4e 100644
--- 
a/wagon-provider-test/src/main/java/org/apache/maven/wagon/StreamingWagonTestCase.java
+++ 
b/wagon-provider-test/src/main/java/org/apache/maven/wagon/StreamingWagonTestCase.java
@@ -69,8 +69,6 @@ public void testFailedGetToStream() throws Exception {
             stream = new FileOutputStream(destFile);
             wagon.getToStream("fubar.txt", stream);
             fail("File was found when it shouldn't have been");
-            stream.close();
-            stream = null;
         } catch (ResourceDoesNotExistException e) {
             // expected
             assertTrue(true);
diff --git 
a/wagon-providers/wagon-file/src/test/java/org/apache/maven/wagon/providers/file/FileWagonTest.java
 
b/wagon-providers/wagon-file/src/test/java/org/apache/maven/wagon/providers/file/FileWagonTest.java
index e1b551f4..e976404e 100644
--- 
a/wagon-providers/wagon-file/src/test/java/org/apache/maven/wagon/providers/file/FileWagonTest.java
+++ 
b/wagon-providers/wagon-file/src/test/java/org/apache/maven/wagon/providers/file/FileWagonTest.java
@@ -45,16 +45,14 @@ protected String getTestRepositoryUrl() throws IOException {
     }
 
     /**
-     * This test is introduced to allow for null file wagons.
-     * Which is used heavily in the maven component ITs.
+     * This test is introduced to allow for null file wagons
+     * which are used heavily in the maven component ITs.
      *
      * @throws ConnectionException
      * @throws AuthenticationException
      */
     public void testNullFileWagon() throws ConnectionException, 
AuthenticationException {
         Wagon wagon = new FileWagon();
-        Resource resource = new Resource();
-        resource.setContentLength(100000);
         Repository repository = new Repository();
         wagon.connect(repository);
         wagon.disconnect();
diff --git 
a/wagon-providers/wagon-webdav-jackrabbit/src/test/java/org/apache/maven/wagon/providers/webdav/WebDavWagonTest.java
 
b/wagon-providers/wagon-webdav-jackrabbit/src/test/java/org/apache/maven/wagon/providers/webdav/WebDavWagonTest.java
index 223e2888..49b08764 100644
--- 
a/wagon-providers/wagon-webdav-jackrabbit/src/test/java/org/apache/maven/wagon/providers/webdav/WebDavWagonTest.java
+++ 
b/wagon-providers/wagon-webdav-jackrabbit/src/test/java/org/apache/maven/wagon/providers/webdav/WebDavWagonTest.java
@@ -267,7 +267,7 @@ public void testWagonWebDavGetFileList() throws Exception {
 
         
/////////////////////////////////////////////////////////////////////////////
         try {
-            list = wagon.getFileList(dirName + "/test-dir-bogus");
+            wagon.getFileList(dirName + "/test-dir-bogus");
             fail("Exception expected");
         } catch (ResourceDoesNotExistException e) {
 

Reply via email to