cstamas commented on code in PR #1755:
URL: https://github.com/apache/maven-resolver/pull/1755#discussion_r2699420773


##########
maven-resolver-test-http/src/main/java/org/eclipse/aether/internal/test/util/http/HttpTransporterTest.java:
##########
@@ -991,7 +992,7 @@ protected void testPut_ProxyAuthenticated() throws 
Exception {
         transporter.put(task);
         assertEquals(0L, listener.getDataOffset());
         assertEquals(6L, listener.getDataLength());
-        assertEquals(1, listener.getStartedCount());
+        assertTrue(listener.getStartedCount() > 0 && 
listener.getStartedCount() <= 2, "Started count: " + 
listener.getStartedCount());

Review Comment:
   That was the exact reason why it used tmp file, I wanted higher level APIs 
(ie listeners most importantly) protect from low level stuff, and make them 
behave _in same way_ (re emitted events).



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to