Apply suggestions from code review

Author: trask

instrumentation/jmx-metrics/library/src/test/java/io/opentelemetry/instrumentation/jmx/engine/UnitConverterTest.java

@@ -67,9 +67,9 @@ void shouldHandleUnsupportedConversion(String sourceUnit, String targetUnit) {
   private static Stream<Arguments> shouldSkipConversionWhenSourceUnitNotSpecifiedProvider() {
     return Stream.of(
         // sourceUnit, targetUnit
-        Arguments.of(null, "s"), // null -> "s"
-        Arguments.of("", "s"), // "" -> "s"
-        Arguments.of("1", "")); // empty target unit
+        Arguments.of(null, "s"),
+        Arguments.of("", "s"),
+        Arguments.of("1", ""));
   }
 
   @ParameterizedTest

javaagent-tooling/src/test/java/io/opentelemetry/javaagent/tooling/OpenTelemetryInstallerTest.java

@@ -49,7 +49,7 @@ private static Stream<Arguments> defaultEnabledInDeclarativeConfigPropertiesBrid
         // propertyValue, expected, fail
         Arguments.of("default", true, false),
         Arguments.of("none", false, false),
-        Arguments.of(null, true, false), // null value means property is not set
+        Arguments.of(null, true, false),
         Arguments.of("invalid", false, true));
   }
 

javaagent-tooling/src/test/java/io/opentelemetry/javaagent/tooling/SpanLoggingCustomizerProviderTest.java

@@ -27,7 +27,7 @@ private static Stream<Arguments> addSpanLoggingExporterProvider() {
         // propertyValue, alreadyAdded, expected
         Arguments.of("true", false, true),
         Arguments.of("false", false, false),
-        Arguments.of(null, false, false), // null value means property is not set
+        Arguments.of(null, false, false),
         Arguments.of("invalid", false, false),
         Arguments.of("true", true, true));
   }