Skip to content

Commit 7233b3c

Browse files
committed
xds: fix checkstyle violations in ExternalProcessorClientInterceptorTest
TAG=agy CONV=86c74ebe-9fd7-4876-b27c-a4c1b230d346
1 parent 6d2335d commit 7233b3c

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

‎xds/src/test/java/io/grpc/xds/ExternalProcessorClientInterceptorTest.java‎

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5210,7 +5210,8 @@ public void halfClose() {
52105210
proxyCall.halfClose();
52115211

52125212
assertThat(dataPlaneHalfClosedLatch.await(5, java.util.concurrent.TimeUnit.SECONDS)).isTrue();
5213-
// Since both end_of_stream and end_of_stream_without_message are true, the body must NOT be delivered
5213+
// Since both end_of_stream and end_of_stream_without_message are true,
5214+
// the body must NOT be delivered
52145215
assertThat(serverReceivedMessages).isEmpty();
52155216

52165217
proxyCall.cancel("Cleanup", null);
@@ -5377,7 +5378,7 @@ public void halfClose() {
53775378
@Test
53785379
@SuppressWarnings("unchecked")
53795380
public void
5380-
clientInterceptor_extProcResponse_eosTrue_eosWithoutMessageFalse_emptyBody_deliveredAsMessage()
5381+
clientInterceptor_extProcResponse_eosTrue_eosWithoutMsgFalse_emptyBody_deliveredAsMessage()
53815382
throws Exception {
53825383
String uniqueExtProcServerName = InProcessServerBuilder.generateName();
53835384
String uniqueDataPlaneServerName = InProcessServerBuilder.generateName();
@@ -5512,7 +5513,8 @@ public void halfClose() {
55125513
proxyCall.halfClose();
55135514

55145515
assertThat(dataPlaneHalfClosedLatch.await(5, java.util.concurrent.TimeUnit.SECONDS)).isTrue();
5515-
// Since end_of_stream is true and end_of_stream_without_message is false, empty body is delivered as a message
5516+
// Since end_of_stream is true and end_of_stream_without_message is false,
5517+
// empty body is delivered as a message
55165518
assertThat(serverReceivedMessages).containsExactly("");
55175519

55185520
proxyCall.cancel("Cleanup", null);
@@ -5658,7 +5660,8 @@ public void halfClose() {
56585660

56595661
assertThat(messageDeliveredLatch.await(5, java.util.concurrent.TimeUnit.SECONDS)).isTrue();
56605662
assertThat(serverReceivedMessages).containsExactly("normal-body");
5661-
// Since end_of_stream was false, end_of_stream_without_message is ignored, so data plane must NOT be half-closed
5663+
// Since end_of_stream was false, end_of_stream_without_message is ignored,
5664+
// so data plane must NOT be half-closed
56625665
assertThat(dataPlaneHalfClosedLatch.getCount()).isEqualTo(1);
56635666

56645667
proxyCall.cancel("Cleanup", null);

0 commit comments

Comments
 (0)