Apply suggestions from code review

Co-authored-by: Jay DeLuca <[email protected]>

Author: laurit

instrumentation/jaxws/jaxws-2.0-common-testing/src/main/java/io/opentelemetry/javaagent/instrumentation/jaxws/v2_0/AbstractJaxWs2Test.java

@@ -110,18 +110,18 @@ void successfulRequest(TestMethod testMethod) {
     testing()
         .waitAndAssertTraces(
             trace -> {
-              List<Consumer<SpanDataAssert>> assertions = new ArrayList<>();
-              assertions.add(
-                  span ->
-                      span.hasName(serverSpanName(testMethod.methodName()))
-                          .hasNoParent()
-                          .hasKind(SpanKind.SERVER)
-                          .hasStatus(StatusData.unset()));
-              assertions.add(
-                  span ->
-                      span.hasName("HelloService/" + testMethod.methodName())
-                          .hasParent(trace.getSpan(0))
-                          .hasKind(SpanKind.INTERNAL));
+              List<Consumer<SpanDataAssert>> assertions =
+                  new ArrayList<>(
+                      Arrays.asList(
+                          span ->
+                              span.hasName(serverSpanName(testMethod.methodName()))
+                                  .hasNoParent()
+                                  .hasKind(SpanKind.SERVER)
+                                  .hasStatus(StatusData.unset()),
+                          span ->
+                              span.hasName("HelloService/" + testMethod.methodName())
+                                  .hasParent(trace.getSpan(0))
+                                  .hasKind(SpanKind.INTERNAL)));
               if (hasAnnotationHandlerSpan(testMethod)) {
                 assertions.add(
                     span ->
@@ -152,20 +152,20 @@ void failingRequest(TestMethod testMethod) {
     testing()
         .waitAndAssertTraces(
             trace -> {
-              List<Consumer<SpanDataAssert>> assertions = new ArrayList<>();
-              assertions.add(
-                  span ->
-                      span.hasName(serverSpanName(testMethod.methodName()))
-                          .hasNoParent()
-                          .hasKind(SpanKind.SERVER)
-                          .hasStatus(StatusData.error()));
-              assertions.add(
-                  span ->
-                      span.hasName("HelloService/" + testMethod.methodName())
-                          .hasParent(trace.getSpan(0))
-                          .hasKind(SpanKind.INTERNAL)
-                          .hasStatus(StatusData.error())
-                          .hasException(expectedException));
+              List<Consumer<SpanDataAssert>> assertions =
+                  new ArrayList<>(
+                      Arrays.asList(
+                          span ->
+                              span.hasName(serverSpanName(testMethod.methodName()))
+                                  .hasNoParent()
+                                  .hasKind(SpanKind.SERVER)
+                                  .hasStatus(StatusData.error()),
+                          span ->
+                              span.hasName("HelloService/" + testMethod.methodName())
+                                  .hasParent(trace.getSpan(0))
+                                  .hasKind(SpanKind.INTERNAL)
+                                  .hasStatus(StatusData.error())
+                                  .hasException(expectedException)));
               if (hasAnnotationHandlerSpan(testMethod)) {
                 assertions.add(
                     span ->