add test client error

Author: shalk

testing-common/src/main/java/io/opentelemetry/instrumentation/testing/junit/http/AbstractHttpClientTest.java

@@ -1029,6 +1029,26 @@ void spanEndsAfterHeadersReceived() throws Exception {
         });
   }
 
+  @Test
+  void requestClientError() throws Exception {
+    assumeTrue(options.getTestClientError());
+
+    URI uri = resolveAddress("/client-error");
+    String method = "GET";
+    int responseCode = doRequest(method, uri);
+
+    assertThat(responseCode).isEqualTo(400);
+
+    testing.waitAndAssertTraces(
+        trace ->
+            trace.hasSpansSatisfyingExactly(
+                span ->
+                    assertClientSpan(span, uri, method, 400, null)
+                        .hasNoParent()
+                        .hasStatus(StatusData.error()),
+                span -> assertServerSpan(span).hasParent(trace.getSpan(0))));
+  }
+
   // Visible for spock bridge.
   SpanDataAssert assertClientSpan(
       SpanDataAssert span,

testing-common/src/main/java/io/opentelemetry/instrumentation/testing/junit/http/HttpClientTestOptions.java

@@ -89,6 +89,8 @@ public boolean isLowLevelInstrumentation() {
 
   public abstract boolean getTestNonStandardHttpMethod();
 
+  public abstract boolean getTestClientError();
+
   public abstract Function<URI, String> getHttpProtocolVersion();
 
   @Nullable
@@ -131,6 +133,7 @@ default Builder withDefaults() {
           .setTestCallbackWithParent(true)
           .setTestErrorWithCallback(true)
           .setTestNonStandardHttpMethod(true)
+          .setTestClientError(false)
           .setHttpProtocolVersion(uri -> "1.1");
     }
 
@@ -174,6 +177,8 @@ default Builder withDefaults() {
 
     Builder setTestNonStandardHttpMethod(boolean value);
 
+    Builder setTestClientError(boolean value);
+
     Builder setHttpProtocolVersion(Function<URI, String> value);
 
     @CanIgnoreReturnValue