Polish

reactor · Dec 12, 2024 · d473186 · d473186
1 parent 8d22dec
commit d473186
Show file tree

Hide file tree

Showing 6 changed files with 31 additions and 31 deletions.
diff --git a/reactor-netty-http/src/test/java/reactor/netty/http/HttpCompressionClientServerTests.java b/reactor-netty-http/src/test/java/reactor/netty/http/HttpCompressionClientServerTests.java
@@ -69,7 +69,7 @@ class HttpCompressionClientServerTests extends BaseHttpTest {
 
 	@Retention(RetentionPolicy.RUNTIME)
 	@Target(ElementType.METHOD)
-	@ParameterizedTest(name = "{displayName}({0}, {1})")
+	@ParameterizedTest
 	@MethodSource("data")
 	@interface ParameterizedCompressionTest {
 	}

diff --git a/reactor-netty-http/src/test/java/reactor/netty/http/HttpProtocolsTests.java b/reactor-netty-http/src/test/java/reactor/netty/http/HttpProtocolsTests.java
@@ -122,28 +122,28 @@ static void disposePool() {
 
 	@Retention(RetentionPolicy.RUNTIME)
 	@Target(ElementType.METHOD)
-	@ParameterizedTest(name = "{displayName}({0}, {1})")
+	@ParameterizedTest
 	@MethodSource("dataAllCombinations")
 	@interface ParameterizedAllCombinationsTest {
 	}
 
 	@Retention(RetentionPolicy.RUNTIME)
 	@Target(ElementType.METHOD)
-	@ParameterizedTest(name = "{displayName}({0}, {1})")
+	@ParameterizedTest
 	@MethodSource("dataCompatibleCombinations")
 	@interface ParameterizedCompatibleCombinationsTest {
 	}
 
 	@Retention(RetentionPolicy.RUNTIME)
 	@Target(ElementType.METHOD)
-	@ParameterizedTest(name = "{displayName}({0}, {1})")
+	@ParameterizedTest
 	@MethodSource("dataCompatibleCombinations_NoPool")
 	@interface ParameterizedCompatibleCombinationsNoPoolTest {
 	}
 
 	@Retention(RetentionPolicy.RUNTIME)
 	@Target(ElementType.METHOD)
-	@ParameterizedTest(name = "{displayName}({0}, {1})")
+	@ParameterizedTest
 	@MethodSource("dataCompatibleCombinations_CustomPool")
 	@interface ParameterizedCompatibleCombinationsCustomPoolTest {
 	}

diff --git a/reactor-netty-http/src/test/java/reactor/netty/http/server/ConnectionInfoTests.java b/reactor-netty-http/src/test/java/reactor/netty/http/server/ConnectionInfoTests.java
@@ -194,7 +194,7 @@ void forwardedHostIpV6() {
 				});
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(strings = {"http", "https", "wss"})
 	void forwardedProtoOnly(String protocol) {
 		testClientRequest(
@@ -210,7 +210,7 @@ void forwardedProtoOnly(String protocol) {
 				});
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedFor(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -224,7 +224,7 @@ void xForwardedFor(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedHost(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -241,7 +241,7 @@ void xForwardedHost(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedHostEmptyHostHeader(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -258,7 +258,7 @@ void xForwardedHostEmptyHostHeader(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedHostPortIncluded(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -274,7 +274,7 @@ void xForwardedHostPortIncluded(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedHostAndPort(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -292,7 +292,7 @@ void xForwardedHostAndPort(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedHostPortIncludedAndXForwardedPort(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -310,7 +310,7 @@ void xForwardedHostPortIncludedAndXForwardedPort(boolean useCustomForwardedHandl
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedPrefix(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -357,7 +357,7 @@ void xForwardedPrefixDelimited(String input, String output) {
 				false);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedPrefixEmpty(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -370,7 +370,7 @@ void xForwardedPrefixEmpty(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedPrefixMissing(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -381,7 +381,7 @@ void xForwardedPrefixMissing(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedMultipleHeaders(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -405,7 +405,7 @@ void xForwardedMultipleHeaders(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedHostAndEmptyPort(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -427,7 +427,7 @@ void xForwardedHostAndEmptyPort(boolean useCustomForwardedHandler) {
 				false);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedHostAndNonNumericPort(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -449,7 +449,7 @@ void xForwardedHostAndNonNumericPort(boolean useCustomForwardedHandler) {
 				true);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedForHostAndPort(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -469,7 +469,7 @@ void xForwardedForHostAndPort(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedForHostAndPortAndProto(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -491,7 +491,7 @@ void xForwardedForHostAndPortAndProto(boolean useCustomForwardedHandler) {
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedForMultipleHostAndPortAndProto(boolean useCustomForwardedHandler) {
 		testClientRequest(
@@ -513,7 +513,7 @@ void xForwardedForMultipleHostAndPortAndProto(boolean useCustomForwardedHandler)
 				useCustomForwardedHandler);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(booleans = {true, false})
 	void xForwardedForAndPortOnly(boolean useCustomForwardedHandler) throws SSLException {
 		SslContext clientSslContext = SslContextBuilder.forClient()
@@ -542,7 +542,7 @@ void xForwardedForAndPortOnly(boolean useCustomForwardedHandler) throws SSLExcep
 				true);
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@MethodSource("forwardedProtoOnlyParams")
 	void xForwardedProtoOnly(String protocol, boolean useCustomForwardedHandler) {
 		testClientRequest(

diff --git a/reactor-netty-http/src/test/java/reactor/netty/http/server/HttpServerPostFormTests.java b/reactor-netty-http/src/test/java/reactor/netty/http/server/HttpServerPostFormTests.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2021-2023 VMware, Inc. or its affiliates, All Rights Reserved.
+ * Copyright (c) 2021-2024 VMware, Inc. or its affiliates, All Rights Reserved.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -61,7 +61,7 @@ class HttpServerPostFormTests extends BaseHttpTest {
 
 	@Retention(RetentionPolicy.RUNTIME)
 	@Target(ElementType.METHOD)
-	@ParameterizedTest(name = "{displayName}({0}, {1})")
+	@ParameterizedTest
 	@MethodSource("data")
 	@interface ParameterizedPostFormTest {
 	}

diff --git a/reactor-netty-http/src/test/java/reactor/netty/http/server/HttpServerTests.java b/reactor-netty-http/src/test/java/reactor/netty/http/server/HttpServerTests.java
@@ -3126,7 +3126,7 @@ void testRemoveRoutes() {
 		            .verify(Duration.ofSeconds(5));
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(ints = {-1, 1, 2})
 	void testMaxKeepAliveRequests(int maxKeepAliveRequests) {
 		HttpServer server = createServer().handle((req, res) -> res.sendString(Mono.just("testMaxKeepAliveRequests")));
@@ -3176,7 +3176,7 @@ else if (maxKeepAliveRequests == 2) {
 		provider.disposeLater().block(Duration.ofSeconds(5));
 	}
 
-	@ParameterizedTest(name = "{displayName}({arguments})")
+	@ParameterizedTest
 	@ValueSource(ints = {-2, 0})
 	void testMaxKeepAliveRequestsBadValues(int maxKeepAliveRequests) {
 		assertThatExceptionOfType(IllegalArgumentException.class)

diff --git a/reactor-netty-http/src/test/java/reactor/netty/http/server/TrailerHeadersTests.java b/reactor-netty-http/src/test/java/reactor/netty/http/server/TrailerHeadersTests.java
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2021-2023 VMware, Inc. or its affiliates, All Rights Reserved.
+ * Copyright (c) 2021-2024 VMware, Inc. or its affiliates, All Rights Reserved.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -42,15 +42,15 @@ class TrailerHeadersTests {
 	static final String HEADER_VALUE = "test";
 	static final String SPACE = " ";
 
-	@ParameterizedTest(name = "{displayName}({0})")
+	@ParameterizedTest
 	@MethodSource("disallowedTrailerHeaderNames")
 	void testDisallowedTrailerHeaderNames(String declaredHeaderName) {
 		assertThatExceptionOfType(IllegalArgumentException.class)
 				.isThrownBy(() -> new HttpServerOperations.TrailerHeaders(declaredHeaderName).add(declaredHeaderName, HEADER_VALUE))
 				.withMessage(String.format(ERROR_MESSAGE, declaredHeaderName));
 	}
 
-	@ParameterizedTest(name = "{displayName}({index})")
+	@ParameterizedTest
 	@ValueSource(strings = {
 			HEADER_NAME_1,
 			COMMA + HEADER_NAME_1,
@@ -86,7 +86,7 @@ void testNameNotIncludedInTrailerHeader() {
 				.withMessage(String.format(ERROR_MESSAGE, HEADER_NAME_2));
 	}
 
-	@ParameterizedTest(name = "{displayName}({index})")
+	@ParameterizedTest
 	@ValueSource(strings = {COMMA, EMPTY, SPACE})
 	void testNothingIsIncludedInTrailerHeader(String declaredHeaderNames) {
 		assertThatExceptionOfType(IllegalArgumentException.class)