add db client metrics for JDBC, vertx, r2dbc, cassandra, add tests

Author: zeitlinger

instrumentation/jdbc/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/jdbc/test/JdbcInstrumentationTest.java

@@ -385,7 +385,12 @@ public void testBasicStatement(
                             equalTo(maybeStable(DB_SQL_TABLE), table))));
 
     assertDurationMetric(
-        testing, "io.opentelemetry.jdbc", DB_SYSTEM, DB_COLLECTION_NAME, DB_NAMESPACE, DB_OPERATION_NAME);
+        testing,
+        "io.opentelemetry.jdbc",
+        DB_SYSTEM,
+        DB_COLLECTION_NAME,
+        DB_NAMESPACE,
+        DB_OPERATION_NAME);
   }
 
   static Stream<Arguments> preparedStatementStream() throws SQLException {

instrumentation/jdbc/library/src/test/java/io/opentelemetry/instrumentation/jdbc/datasource/JdbcTelemetryTest.java

@@ -52,7 +52,13 @@ void buildWithDefaults() throws SQLException {
                         .hasAttribute(equalTo(maybeStable(DB_STATEMENT), "SELECT ?;"))));
 
     assertDurationMetric(
-        testing, "io.opentelemetry.jdbc", DB_NAMESPACE, DB_OPERATION_NAME, DB_SYSTEM, SERVER_ADDRESS, SERVER_PORT);
+        testing,
+        "io.opentelemetry.jdbc",
+        DB_NAMESPACE,
+        DB_OPERATION_NAME,
+        DB_SYSTEM,
+        SERVER_ADDRESS,
+        SERVER_PORT);
   }
 
   @Test

instrumentation/jdbc/testing/src/main/java/io/opentelemetry/instrumentation/jdbc/DbClientMetricsTestUtil.java

@@ -16,23 +16,26 @@ public class DbClientMetricsTestUtil {
   private DbClientMetricsTestUtil() {}
 
   public static void assertDurationMetric(
-      InstrumentationExtension testing, String instrumentationName,  AttributeKey<?>... expectedKeys) {
+      InstrumentationExtension testing,
+      String instrumentationName,
+      AttributeKey<?>... expectedKeys) {
     if (!emitStableDatabaseSemconv()) {
       return;
     }
     testing.waitAndAssertMetrics(
         instrumentationName,
         metrics ->
-            metrics.hasName("db.client.operation.duration")
-                        .hasUnit("s")
-                        .hasDescription("Duration of database client operations.")
-                        .hasHistogramSatisfying(
-                            histogram ->
-                                histogram.hasPointsSatisfying(
-                                    point ->
-                                        point.hasAttributesSatisfying(
-                                            attributes ->
-                                                assertThat(attributes.asMap())
-                                                    .containsOnlyKeys(expectedKeys)))));
+            metrics
+                .hasName("db.client.operation.duration")
+                .hasUnit("s")
+                .hasDescription("Duration of database client operations.")
+                .hasHistogramSatisfying(
+                    histogram ->
+                        histogram.hasPointsSatisfying(
+                            point ->
+                                point.hasAttributesSatisfying(
+                                    attributes ->
+                                        assertThat(attributes.asMap())
+                                            .containsOnlyKeys(expectedKeys)))));
   }
 }