Review fixes for jdbc:library (#18062)

Co-authored-by: otelbot <197425009+otelbot@users.noreply.github.com>

Author: otelbot[bot]

instrumentation/jdbc/library/src/test/java/io/opentelemetry/instrumentation/jdbc/OpenTelemetryDriverTest.java

@@ -268,8 +268,8 @@ private static boolean hasTestDriver() {
   @Test
   void verifyGetPropertyInfoWithNullUrl() {
     assertThatThrownBy(() -> OpenTelemetryDriver.INSTANCE.getPropertyInfo(null, null))
-        .describedAs("url is required")
-        .isInstanceOf(IllegalArgumentException.class);
+        .isInstanceOf(IllegalArgumentException.class)
+        .hasMessage("url is required");
   }
 
   @DisplayName("verify get property info with empty url")

instrumentation/jdbc/library/src/test/java/io/opentelemetry/instrumentation/jdbc/internal/OpenTelemetryConnectionTest.java

@@ -17,6 +17,7 @@
 import io.opentelemetry.instrumentation.jdbc.internal.dbinfo.DbInfo;
 import io.opentelemetry.instrumentation.testing.junit.InstrumentationExtension;
 import io.opentelemetry.instrumentation.testing.junit.LibraryInstrumentationExtension;
+import java.sql.SQLException;
 import java.sql.Statement;
 import java.util.ArrayList;
 import java.util.List;
@@ -38,7 +39,7 @@ void resetTest() {
 
   @SuppressWarnings("unchecked")
   @Test
-  void testVerifyCreateStatementReturnsOtelWrapper() throws Exception {
+  void testVerifyCreateStatementReturnsOtelWrapper() throws SQLException {
     OpenTelemetry openTelemetry = OpenTelemetry.propagating(ContextPropagators.noop());
     OpenTelemetryConnection connection = getConnection(openTelemetry);
 
@@ -53,7 +54,7 @@ void testVerifyCreateStatementReturnsOtelWrapper() throws Exception {
 
   @SuppressWarnings("unchecked")
   @Test
-  void testVerifyPrepareStatementReturnsOtelWrapper() throws Exception {
+  void testVerifyPrepareStatementReturnsOtelWrapper() throws SQLException {
     OpenTelemetry openTelemetry = OpenTelemetry.propagating(ContextPropagators.noop());
     OpenTelemetryConnection connection = getConnection(openTelemetry);
 
@@ -80,7 +81,7 @@ void testVerifyPrepareStatementReturnsOtelWrapper() throws Exception {
 
   @SuppressWarnings("unchecked")
   @Test
-  void testVerifyPrepareCallReturnsOtelWrapper() throws Exception {
+  void testVerifyPrepareCallReturnsOtelWrapper() throws SQLException {
     OpenTelemetry openTelemetry = OpenTelemetry.propagating(ContextPropagators.noop());
     OpenTelemetryConnection connection = getConnection(openTelemetry);
 

instrumentation/jdbc/library/src/test/java/io/opentelemetry/instrumentation/jdbc/internal/WrapperTest.java

@@ -23,7 +23,7 @@ class WrapperTest {
   // we don't implement methods introduced in jdbc 4.5 (added in java 26) yet
   @EnabledForJreRange(max = JRE.JAVA_25)
   @Test
-  void wrapperImplementsAllMethods() throws Exception {
+  void wrapperImplementsAllMethods() throws NoSuchMethodException {
     validate(Statement.class, OpenTelemetryStatement.class);
     validate(PreparedStatement.class, OpenTelemetryPreparedStatement.class);
     validate(CallableStatement.class, OpenTelemetryCallableStatement.class);
@@ -35,7 +35,7 @@ void wrapperImplementsAllMethods() throws Exception {
     validate(ResultSet.class, OpenTelemetryResultSet.class);
   }
 
-  void validate(Class<?> jdbcClass, Class<?> wrapperClass) throws Exception {
+  void validate(Class<?> jdbcClass, Class<?> wrapperClass) throws NoSuchMethodException {
     for (Method method : jdbcClass.getMethods()) {
       Method result = wrapperClass.getMethod(method.getName(), method.getParameterTypes());
       if (!result.getDeclaringClass().getName().startsWith("io.opentelemetry")) {