refactor: pass produces values directly to header via varargs

Signed-off-by: Segergren <olle.segergren@gmail.com>

Author: Segergren

spring-cloud-openfeign-core/src/main/java/org/springframework/cloud/openfeign/support/SpringMvcContract.java

@@ -30,7 +30,6 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
-import java.util.stream.Collectors;
 
 import feign.Contract;
 import feign.Feign;
@@ -419,11 +418,11 @@ private boolean queryMapParamPresent(MethodMetadata data) {
 	}
 
 	private void parseProduces(MethodMetadata md, RequestMapping annotation) {
-		String clientAccepts = Arrays.stream(annotation.produces())
+		String[] clientAccepts = Arrays.stream(annotation.produces())
 			.map(s -> emptyToNull(s))
 			.filter(Objects::nonNull)
-			.collect(Collectors.joining(", "));
-		if (StringUtils.hasText(clientAccepts)) {
+			.toArray(String[]::new);
+		if (clientAccepts.length > 0) {
 			md.template().header(ACCEPT, clientAccepts);
 		}
 	}

spring-cloud-openfeign-core/src/test/java/org/springframework/cloud/openfeign/support/SpringMvcContractTests.java

@@ -769,7 +769,7 @@ void testMultipleProducesValues() throws NoSuchMethodException {
 		Method method = TestTemplate_MultipleProduces.class.getDeclaredMethod("multipleProduces");
 		MethodMetadata data = contract.parseAndValidateMetadata(method.getDeclaringClass(), method);
 
-		assertThat(data.template().headers().get("Accept")).containsExactly("application/jose, application/json");
+		assertThat(data.template().headers().get("Accept")).containsExactly("application/jose", "application/json");
 	}
 
 	@Test
@@ -793,7 +793,7 @@ void testProducesWithBlankEntryIgnored() throws NoSuchMethodException {
 		Method method = TestTemplate_MultipleProduces.class.getDeclaredMethod("producesWithBlankEntry");
 		MethodMetadata data = contract.parseAndValidateMetadata(method.getDeclaringClass(), method);
 
-		assertThat(data.template().headers().get("Accept")).containsExactly("application/jose, application/json");
+		assertThat(data.template().headers().get("Accept")).containsExactly("application/jose", "application/json");
 	}
 
 	@Test