Skip to content

Commit 29ce5fc

Browse files
committed
Fix tests
1 parent 02b97ed commit 29ce5fc

File tree

1 file changed

+28
-5
lines changed

1 file changed

+28
-5
lines changed

client/src/test/java/org/asynchttpclient/AsyncStreamHandlerTest.java

Lines changed: 28 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,8 @@
2525
import org.testng.annotations.Test;
2626

2727
import java.util.Arrays;
28+
import java.util.HashMap;
29+
import java.util.Map;
2830
import java.util.concurrent.CountDownLatch;
2931
import java.util.concurrent.Future;
3032
import java.util.concurrent.TimeUnit;
@@ -40,7 +42,7 @@
4042

4143
public class AsyncStreamHandlerTest extends HttpTest {
4244

43-
private static final String RESPONSE = "param_1_";
45+
private static final String RESPONSE = "param_1=value_1";
4446

4547
private static HttpServer server;
4648

@@ -93,18 +95,25 @@ public void asyncStreamPOSTTest() throws Throwable {
9395
@Override
9496
public State onHeadersReceived(HttpHeaders headers) {
9597
assertContentTypesEquals(headers.get(CONTENT_TYPE), TEXT_HTML_CONTENT_TYPE_WITH_UTF_8_CHARSET);
98+
for (Map.Entry<String, String> header : headers) {
99+
if (header.getKey().startsWith("X-param")) {
100+
builder.append(header.getKey().substring(2)).append("=").append(header.getValue()).append("&");
101+
}
102+
}
96103
return State.CONTINUE;
97104
}
98105

99106
@Override
100107
public State onBodyPartReceived(HttpResponseBodyPart content) {
101-
builder.append(new String(content.getBodyPartBytes(), US_ASCII));
102108
return State.CONTINUE;
103109
}
104110

105111
@Override
106112
public String onCompleted() {
107-
return builder.toString().trim();
113+
if (builder.length() > 0) {
114+
builder.setLength(builder.length() - 1);
115+
}
116+
return builder.toString();
108117
}
109118
}).get(10, TimeUnit.SECONDS);
110119

@@ -174,17 +183,24 @@ public void asyncStreamFutureTest() throws Throwable {
174183
public State onHeadersReceived(HttpHeaders headers) {
175184
assertContentTypesEquals(headers.get(CONTENT_TYPE), TEXT_HTML_CONTENT_TYPE_WITH_UTF_8_CHARSET);
176185
onHeadersReceived.set(true);
186+
for (Map.Entry<String, String> header : headers) {
187+
if (header.getKey().startsWith("X-param")) {
188+
builder.append(header.getKey().substring(2)).append("=").append(header.getValue()).append("&");
189+
}
190+
}
177191
return State.CONTINUE;
178192
}
179193

180194
@Override
181195
public State onBodyPartReceived(HttpResponseBodyPart content) {
182-
builder.append(new String(content.getBodyPartBytes()));
183196
return State.CONTINUE;
184197
}
185198

186199
@Override
187200
public String onCompleted() {
201+
if (builder.length() > 0) {
202+
builder.setLength(builder.length() - 1);
203+
}
188204
return builder.toString().trim();
189205
}
190206

@@ -254,17 +270,24 @@ public void asyncStreamReusePOSTTest() throws Throwable {
254270
@Override
255271
public State onHeadersReceived(HttpHeaders headers) {
256272
responseHeaders.set(headers);
273+
for (Map.Entry<String, String> header : headers) {
274+
if (header.getKey().startsWith("X-param")) {
275+
builder.append(header.getKey().substring(2)).append("=").append(header.getValue()).append("&");
276+
}
277+
}
257278
return State.CONTINUE;
258279
}
259280

260281
@Override
261282
public State onBodyPartReceived(HttpResponseBodyPart content) {
262-
builder.append(new String(content.getBodyPartBytes()));
263283
return State.CONTINUE;
264284
}
265285

266286
@Override
267287
public String onCompleted() {
288+
if (builder.length() > 0) {
289+
builder.setLength(builder.length() - 1);
290+
}
268291
return builder.toString();
269292
}
270293
});

0 commit comments

Comments
 (0)