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) {