Remove redundant assertThat(x).isNotNull() before a dereference

trask · trask · commit b522da4cc914 · 2026-04-22T08:09:46.000-07:00
When the next line dereferences x (e.g. x.foo()), an NPE produces an
equivalent failure, so the preceding isNotNull() assertion adds no
diagnostic value.
diff --git a/instrumentation-docs/src/test/java/io/opentelemetry/instrumentation/docs/InstrumentationAnalyzerTest.java b/instrumentation-docs/src/test/java/io/opentelemetry/instrumentation/docs/InstrumentationAnalyzerTest.java
@@ -51,7 +51,6 @@ void testConvertToInstrumentationModule() {
             .findFirst()
             .orElse(null);
 
-    assertThat(log4jModule).isNotNull();
     assertThat(log4jModule.getNamespace()).isEqualTo("log4j");
     assertThat(log4jModule.getGroup()).isEqualTo("log4j");
     assertThat(log4jModule.getSrcPath()).isEqualTo("instrumentation/log4j/log4j-appender-2.17");
@@ -64,7 +63,6 @@ void testConvertToInstrumentationModule() {
             .findFirst()
             .orElse(null);
 
-    assertThat(springModule).isNotNull();
     assertThat(springModule.getNamespace()).isEqualTo("spring");
     assertThat(springModule.getGroup()).isEqualTo("spring");
     assertThat(springModule.getSrcPath()).isEqualTo("instrumentation/spring/spring-web");
diff --git a/instrumentation-docs/src/test/java/io/opentelemetry/instrumentation/docs/parsers/EmittedScopeParserTest.java b/instrumentation-docs/src/test/java/io/opentelemetry/instrumentation/docs/parsers/EmittedScopeParserTest.java
@@ -160,7 +160,6 @@ void testGetScopeWithMatchingName(@TempDir Path tempDir) throws IOException {
 
     InstrumentationScopeInfo scopeInfo = EmittedScopeParser.getScope(fileManager, module);
 
-    assertThat(scopeInfo).isNotNull();
     assertThat(scopeInfo.getName()).isEqualTo("io.opentelemetry.test-lib-1.0");
     assertThat(scopeInfo.getSchemaUrl()).isNull();
   }
@@ -187,7 +186,6 @@ void testGetScopeWithSchemaUrl(@TempDir Path tempDir) throws IOException {
 
     InstrumentationScopeInfo scopeInfo = EmittedScopeParser.getScope(fileManager, module);
 
-    assertThat(scopeInfo).isNotNull();
     assertThat(scopeInfo.getName()).isEqualTo("io.opentelemetry.spring-web-6.0");
     assertThat(scopeInfo.getSchemaUrl()).isEqualTo("https://opentelemetry.io/schemas/1.21.0");
   }
@@ -259,7 +257,6 @@ void testGetScopeMultipleScopesOneMatches(@TempDir Path tempDir) throws IOExcept
 
     InstrumentationScopeInfo scopeInfo = EmittedScopeParser.getScope(fileManager, module);
 
-    assertThat(scopeInfo).isNotNull();
     assertThat(scopeInfo.getName()).isEqualTo("io.opentelemetry.hibernate-6.0");
     assertThat(scopeInfo.getSchemaUrl()).isEqualTo("https://opentelemetry.io/schemas/1.21.0");
   }
@@ -289,7 +286,6 @@ void testGetScopeWithStringAttributes(@TempDir Path tempDir) throws IOException
 
     InstrumentationScopeInfo scopeInfo = EmittedScopeParser.getScope(fileManager, module);
 
-    assertThat(scopeInfo).isNotNull();
     assertThat(scopeInfo.getName()).isEqualTo("io.opentelemetry.jdbc");
     assertThat(scopeInfo.getAttributes()).isNotNull();
     assertThat(scopeInfo.getAttributes().get(stringKey("test.key"))).isEqualTo("test-value");
@@ -324,7 +320,6 @@ void testGetScopeWithMixedTypeAttributes(@TempDir Path tempDir) throws IOExcepti
 
     InstrumentationScopeInfo scopeInfo = EmittedScopeParser.getScope(fileManager, module);
 
-    assertThat(scopeInfo).isNotNull();
     assertThat(scopeInfo.getAttributes()).isNotNull();
     assertThat(scopeInfo.getAttributes().get(stringKey("string.key"))).isEqualTo("string-value");
     assertThat(scopeInfo.getAttributes().get(longKey("int.key"))).isEqualTo(123L);
diff --git a/instrumentation/aws-lambda/aws-lambda-events-common-2.2/library/src/test/java/io/opentelemetry/instrumentation/awslambdaevents/common/v2_2/internal/SerializationUtilTest.java b/instrumentation/aws-lambda/aws-lambda-events-common-2.2/library/src/test/java/io/opentelemetry/instrumentation/awslambdaevents/common/v2_2/internal/SerializationUtilTest.java
@@ -194,7 +194,6 @@ private static Map<Class<?>, String> buildEventExamples() {
   }
 
   private static void assertScheduledEvent(ScheduledEvent event) {
-    assertThat(event).isNotNull();
     assertThat(event.getSource()).isEqualTo("aws.events");
     assertThat(event.getDetailType()).isEqualTo("Scheduled Event");
     assertThat(event.getAccount()).isEqualTo("123456789012");
@@ -205,7 +204,6 @@ private static void assertScheduledEvent(ScheduledEvent event) {
   }
 
   private static void assertKinesisEvent(KinesisEvent event) {
-    assertThat(event).isNotNull();
     assertThat(event.getRecords()).isNotNull();
     assertThat(event.getRecords()).hasSize(2);
     KinesisEvent.KinesisEventRecord record = event.getRecords().get(0);
@@ -223,7 +221,6 @@ private static void assertKinesisEvent(KinesisEvent event) {
   }
 
   private static void assertSqsEvent(SQSEvent event) {
-    assertThat(event).isNotNull();
     assertThat(event.getRecords()).isNotNull();
     assertThat(event.getRecords()).hasSize(2);
     SQSEvent.SQSMessage record = event.getRecords().get(0);
@@ -236,7 +233,6 @@ private static void assertSqsEvent(SQSEvent event) {
   }
 
   private static void assertS3Event(S3Event event) {
-    assertThat(event).isNotNull();
     assertThat(event.getRecords()).isNotNull();
     assertThat(event.getRecords()).hasSize(1);
     S3EventNotification.S3EventNotificationRecord record = event.getRecords().get(0);
@@ -249,7 +245,6 @@ private static void assertS3Event(S3Event event) {
   }
 
   private static void assertSnsEvent(SNSEvent event) {
-    assertThat(event).isNotNull();
     assertThat(event.getRecords()).isNotNull();
     assertThat(event.getRecords()).hasSize(1);
     SNSEvent.SNSRecord record = event.getRecords().get(0);
diff --git a/instrumentation/aws-sdk/aws-sdk-2.2/testing/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/AbstractAws2ClientCoreTest.java b/instrumentation/aws-sdk/aws-sdk-2.2/testing/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/AbstractAws2ClientCoreTest.java
@@ -125,11 +125,9 @@ void prepTest() {
   }
 
   private void validateOperationResponse(String operation, Object response) {
-    assertThat(response).isNotNull();
     assertThat(response.getClass().getSimpleName()).startsWith(operation);
 
     RecordedRequest request = server.takeRequest();
-    assertThat(request).isNotNull();
     assertThat(request.request().headers().get("X-Amzn-Trace-Id")).isNotNull();
     assertThat(request.request().headers().get("traceparent")).isNull();
 
diff --git a/instrumentation/aws-sdk/aws-sdk-2.2/testing/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/AbstractAws2ClientTest.java b/instrumentation/aws-sdk/aws-sdk-2.2/testing/src/main/java/io/opentelemetry/instrumentation/awssdk/v2_2/AbstractAws2ClientTest.java
@@ -291,7 +291,6 @@ private void clientAssertions(
     assertThat(response).isNotNull();
 
     RecordedRequest request = server.takeRequest();
-    assertThat(request).isNotNull();
     assertThat(request.request().headers().get("X-Amzn-Trace-Id")).isNotNull();
     assertThat(request.request().headers().get("traceparent")).isNull();
 
diff --git a/instrumentation/clickhouse/clickhouse-client-v1-0.5/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/clickhouse/clientv1/v0_5/ClickHouseClientV1Test.java b/instrumentation/clickhouse/clickhouse-client-v1-0.5/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/clickhouse/clientv1/v0_5/ClickHouseClientV1Test.java
@@ -299,7 +299,6 @@ void testAsyncExecuteQuery() throws Exception {
             .execute();
 
     ClickHouseResponse result = response.get();
-    assertThat(result).isNotNull();
     result.close();
 
     testing.waitAndAssertTraces(
diff --git a/instrumentation/couchbase/couchbase-common-2.0/javaagent-unit-tests/src/test/java/io/opentelemetry/javaagent/instrumentation/couchbase/common/v2_0/CouchbaseQuerySanitizerTest.java b/instrumentation/couchbase/couchbase-common-2.0/javaagent-unit-tests/src/test/java/io/opentelemetry/javaagent/instrumentation/couchbase/common/v2_0/CouchbaseQuerySanitizerTest.java
@@ -25,7 +25,6 @@ class CouchbaseQuerySanitizerTest {
   @MethodSource("providesArguments")
   void testShouldNormalizeStringQuery(Parameter parameter) {
     String normalized = CouchbaseQuerySanitizer.analyze(parameter.query).getQueryText();
-    assertThat(normalized).isNotNull();
     // the analytics query ends up with trailing ';' in earlier couchbase version, but no trailing
     // ';' in later couchbase version
     assertThat(normalized.replaceFirst(";$", "")).isEqualTo(parameter.expected);
diff --git a/instrumentation/iceberg-1.8/testing/src/main/java/io/opentelemetry/instrumentation/iceberg/v1_8/AbstractIcebergTest.java b/instrumentation/iceberg-1.8/testing/src/main/java/io/opentelemetry/instrumentation/iceberg/v1_8/AbstractIcebergTest.java
@@ -84,7 +84,6 @@ void testCreateTelemetry() throws IOException {
     scan = configure(scan);
 
     try (CloseableIterable<FileScanTask> tasks = scan.planFiles()) {
-      assertThat(tasks).isNotNull();
       assertThat(tasks.iterator()).isNotNull();
     }
 
diff --git a/instrumentation/internal/internal-class-loader/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/classloader/ClassLoadingTest.java b/instrumentation/internal/internal-class-loader/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/classloader/ClassLoadingTest.java
@@ -20,7 +20,6 @@ void testDelegatesToBootstrapClassLoaderForAgentClasses() throws ClassNotFoundEx
         Class.forName(
             "io.opentelemetry.javaagent.bootstrap.Java8BytecodeBridge", false, classLoader);
 
-    assertThat(clazz).isNotNull();
     assertThat(clazz.getClassLoader()).isNull();
   }
 
@@ -30,7 +29,6 @@ void testDelegatesToBootstrapClassLoaderForAgentClassesTwoArguments()
     NonDelegatingUrlClassLoader classLoader = new NonDelegatingUrlClassLoader();
     Class<?> clazz =
         classLoader.loadClass("io.opentelemetry.javaagent.bootstrap.Java8BytecodeBridge", false);
-    assertThat(clazz).isNotNull();
     assertThat(clazz.getClassLoader()).isNull();
   }
 
diff --git a/instrumentation/internal/internal-class-loader/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/classloader/JbossClassloadingTest.java b/instrumentation/internal/internal-class-loader/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/classloader/JbossClassloadingTest.java
@@ -39,7 +39,6 @@ void testDelegatesToBootstrapClassLoaderForAgentClasses()
         Class.forName(
             "io.opentelemetry.javaagent.bootstrap.Java8BytecodeBridge", false, classLoader);
 
-    assertThat(clazz).isNotNull();
     assertThat(clazz.getClassLoader()).isNull();
   }
 }
diff --git a/instrumentation/internal/internal-class-loader/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/classloader/OsgiClassloadingTest.java b/instrumentation/internal/internal-class-loader/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/classloader/OsgiClassloadingTest.java
@@ -32,7 +32,6 @@ class OsgiClassloadingTest {
   void testOsgiDelegatesToBootstrapClassloaderForAgentClasses(ClassLoader loader)
       throws ClassNotFoundException {
     Class<?> clazz = loader.loadClass("io.opentelemetry.javaagent.bootstrap.Java8BytecodeBridge");
-    assertThat(clazz).isNotNull();
     assertThat(clazz.getClassLoader()).isNull();
   }
 
diff --git a/instrumentation/internal/internal-reflection/javaagent-integration-tests/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/reflection/ReflectionTest.java b/instrumentation/internal/internal-reflection/javaagent-integration-tests/src/test/java/io/opentelemetry/javaagent/instrumentation/internal/reflection/ReflectionTest.java
@@ -54,7 +54,6 @@ void testGeneratedSerialVersionUid() {
   void testInjectedClassProxyUnwrap() throws Exception {
     TestClass testClass = new TestClass();
     Class<?> helperType = testClass.testHelperClass();
-    assertThat(helperType).isNotNull();
 
     Object instance = helperType.getConstructor().newInstance();
     if (InstrumentationProxy.class.isAssignableFrom(helperType)) {
diff --git a/instrumentation/jmx-metrics/library/src/test/java/io/opentelemetry/instrumentation/jmx/internal/engine/AttributeExtractorTest.java b/instrumentation/jmx-metrics/library/src/test/java/io/opentelemetry/instrumentation/jmx/internal/engine/AttributeExtractorTest.java
@@ -131,95 +131,83 @@ void testSetup() {
   void testByteAttribute() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("ByteAttribute");
     AttributeInfo info = extractor.getAttributeInfo(theServer, objectName);
-    assertThat(info).isNotNull();
     assertThat(info.usesDoubleValues()).isFalse();
   }
 
   @Test
   void testByteAttributeValue() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("ByteAttribute");
     Number number = extractor.extractNumericalAttribute(theServer, objectName);
-    assertThat(number).isNotNull();
     assertThat(number.longValue()).isEqualTo(10);
   }
 
   @Test
   void testShortAttribute() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("ShortAttribute");
     AttributeInfo info = extractor.getAttributeInfo(theServer, objectName);
-    assertThat(info).isNotNull();
     assertThat(info.usesDoubleValues()).isFalse();
   }
 
   @Test
   void testShortAttributeValue() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("ShortAttribute");
     Number number = extractor.extractNumericalAttribute(theServer, objectName);
-    assertThat(number).isNotNull();
     assertThat(number.longValue()).isEqualTo(11);
   }
 
   @Test
   void testIntAttribute() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("IntAttribute");
     AttributeInfo info = extractor.getAttributeInfo(theServer, objectName);
-    assertThat(info).isNotNull();
     assertThat(info.usesDoubleValues()).isFalse();
   }
 
   @Test
   void testIntAttributeValue() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("IntAttribute");
     Number number = extractor.extractNumericalAttribute(theServer, objectName);
-    assertThat(number).isNotNull();
     assertThat(number.longValue()).isEqualTo(12);
   }
 
   @Test
   void testLongAttribute() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("LongAttribute");
     AttributeInfo info = extractor.getAttributeInfo(theServer, objectName);
-    assertThat(info).isNotNull();
     assertThat(info.usesDoubleValues()).isFalse();
   }
 
   @Test
   void testLongAttributeValue() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("LongAttribute");
     Number number = extractor.extractNumericalAttribute(theServer, objectName);
-    assertThat(number).isNotNull();
     assertThat(number.longValue()).isEqualTo(13);
   }
 
   @Test
   void testFloatAttribute() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("FloatAttribute");
     AttributeInfo info = extractor.getAttributeInfo(theServer, objectName);
-    assertThat(info).isNotNull();
     assertThat(info.usesDoubleValues()).isTrue();
   }
 
   @Test
   void testFloatAttributeValue() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("FloatAttribute");
     Number number = extractor.extractNumericalAttribute(theServer, objectName);
-    assertThat(number).isNotNull();
     assertThat(number.doubleValue()).isEqualTo(14.0); // accurate representation
   }
 
   @Test
   void testDoubleAttribute() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("DoubleAttribute");
     AttributeInfo info = extractor.getAttributeInfo(theServer, objectName);
-    assertThat(info).isNotNull();
     assertThat(info.usesDoubleValues()).isTrue();
   }
 
   @Test
   void testDoubleAttributeValue() {
     BeanAttributeExtractor extractor = BeanAttributeExtractor.fromName("DoubleAttribute");
     Number number = extractor.extractNumericalAttribute(theServer, objectName);
-    assertThat(number).isNotNull();
     assertThat(number.doubleValue()).isEqualTo(15.0); // accurate representation
   }
 
@@ -260,7 +248,6 @@ void testNegativeFilter(String attributeName) {
 
     test1.negativeValues = true;
     Number rawValue = rawExtractor.extractNumericalAttribute(theServer, objectName);
-    assertThat(rawValue).isNotNull();
     assertThat(rawValue.doubleValue()).isNegative();
     assertThat(filteringExtractor.extractNumericalAttribute(theServer, objectName))
         .describedAs("negative value should be filtered")
diff --git a/instrumentation/jmx-metrics/library/src/test/java/io/opentelemetry/instrumentation/jmx/internal/engine/RuleParserTest.java b/instrumentation/jmx-metrics/library/src/test/java/io/opentelemetry/instrumentation/jmx/internal/engine/RuleParserTest.java
@@ -100,7 +100,6 @@ void testConf2() {
         .containsKeys("ATTRIBUTE1", "ATTRIBUTE2", "ATTRIBUTE3", "ATTRIBUTE4");
 
     Metric m1 = attr.get("ATTRIBUTE1");
-    assertThat(m1).isNotNull();
     assertThat(m1.getMetric()).isEqualTo("METRIC_NAME1");
     assertThat(m1.getMetricType()).isEqualTo(MetricInfo.Type.GAUGE);
     assertThat(m1.getSourceUnit()).isNull();
@@ -111,7 +110,6 @@ void testConf2() {
         .isNull();
 
     Metric m2 = attr.get("ATTRIBUTE2");
-    assertThat(m2).isNotNull();
     assertThat(m2.getMetric()).isEqualTo("METRIC_NAME2");
     assertThat(m2.getDesc()).isEqualTo("DESCRIPTION2");
     assertThat(m2.getSourceUnit()).isEqualTo("SOURCE_UNIT2");
@@ -157,7 +155,6 @@ void testConf3() {
     assertThat(attr.get("ATTRIBUTE32")).isNull();
     assertThat(attr.get("ATTRIBUTE33")).isNull();
     Metric attribute34 = attr.get("ATTRIBUTE34");
-    assertThat(attribute34).isNotNull();
     assertThat(attribute34.getMetric()).isEqualTo("METRIC_NAME34");
     assertThat(attr.get("ATTRIBUTE35")).isNull();
   }
@@ -204,7 +201,6 @@ void testConf4() throws Exception {
     assertThat(jmxDef.getMetricType()).isEqualTo(MetricInfo.Type.UPDOWNCOUNTER);
 
     MetricDef metricDef = jmxDef.buildMetricDef();
-    assertThat(metricDef).isNotNull();
 
     assertThat(metricDef.getMetricExtractors())
         .hasSize(3)
@@ -274,7 +270,6 @@ void testConf5() throws Exception {
     assertThat(defs).hasSize(1);
 
     MetricDef metricDef = defs.get(0).buildMetricDef();
-    assertThat(metricDef).isNotNull();
     assertThat(metricDef.getMetricExtractors()).hasSize(1);
 
     MetricExtractor m1 = metricDef.getMetricExtractors().get(0);
@@ -309,7 +304,6 @@ void testConf6() throws Exception {
     assertThat(rules).hasSize(1);
 
     MetricDef metricDef = rules.get(0).buildMetricDef();
-    assertThat(metricDef).isNotNull();
     assertThat(metricDef.getMetricExtractors()).hasSize(1);
 
     MetricExtractor m1 = metricDef.getMetricExtractors().get(0);
@@ -344,7 +338,6 @@ void testConf7() throws Exception {
     assertThat(defs).hasSize(1);
 
     MetricDef metricDef = defs.get(0).buildMetricDef();
-    assertThat(metricDef).isNotNull();
     assertThat(metricDef.getMetricExtractors()).hasSize(1);
 
     // Test that the MBean attribute is correctly parsed
@@ -375,7 +368,6 @@ void testConf8() throws Exception {
     assertThat(defs).hasSize(1);
 
     MetricDef metricDef = defs.get(0).buildMetricDef();
-    assertThat(metricDef).isNotNull();
     assertThat(metricDef.getMetricExtractors()).hasSize(1);
 
     MetricExtractor m1 = metricDef.getMetricExtractors().get(0);
@@ -463,7 +455,6 @@ void testStateMetricConf() throws Exception {
                   stateAttribute.acquireAttributeValue(mockConnection, objectName);
 
               BeanAttributeExtractor attributeExtractor = me.getMetricValueExtractor();
-              assertThat(attributeExtractor).isNotNull();
               assertThat(attributeExtractor.getSampleValue(null, null))
                   .describedAs("sampled value must be an integer")
                   .isInstanceOf(Integer.class);
@@ -603,7 +594,6 @@ private static void checkConstantMetricAttribute(
   private static JmxConfig parseConf(String s) {
     InputStream is = new ByteArrayInputStream(s.getBytes(UTF_8));
     JmxConfig jmxConfig = parser.loadConfig(is);
-    assertThat(jmxConfig).isNotNull();
 
     // building metric definitions allows triggering validation rules
     for (JmxRule rule : jmxConfig.getRules()) {
diff --git a/instrumentation/jmx-metrics/library/src/test/java/io/opentelemetry/instrumentation/jmx/internal/engine/UnitConverterTest.java b/instrumentation/jmx-metrics/library/src/test/java/io/opentelemetry/instrumentation/jmx/internal/engine/UnitConverterTest.java
@@ -47,7 +47,6 @@ private static void testConversion(
       Long originalValue, String originalUnit, Double expectedConvertedValue, String targetUnit) {
     UnitConverter converter = UnitConverter.getInstance(originalUnit, targetUnit);
 
-    assertThat(converter).isNotNull();
     Number actualValue = converter.convert(originalValue);
 
     assertThat(expectedConvertedValue).isEqualTo(actualValue);
diff --git a/instrumentation/nats/nats-2.17/testing/src/main/java/io/opentelemetry/instrumentation/nats/v2_17/NatsTestHelper.java b/instrumentation/nats/nats-2.17/testing/src/main/java/io/opentelemetry/instrumentation/nats/v2_17/NatsTestHelper.java
@@ -51,7 +51,6 @@ static AttributeAssertion[] messagingAttributes(String operation, String subject
 
   static void assertTraceparentHeader(Subscription subscription) throws InterruptedException {
     Message published = subscription.nextMessage(Duration.ofSeconds(10));
-    assertThat(published).isNotNull();
     assertThat(published.getHeaders().get("traceparent")).isNotEmpty();
   }
 
diff --git a/instrumentation/netty/netty-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/netty/v4_0/ChannelPipelineTest.java b/instrumentation/netty/netty-4.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/netty/v4_0/ChannelPipelineTest.java
@@ -41,7 +41,6 @@ private static Class<?> getDefaultChannelPipelineClass() {
 
   @NotNull
   private static Constructor<?> getConstructor() throws NoSuchMethodException {
-    assertThat(defaultChannelPipelineClass).isNotNull();
     Constructor<?> constructor = defaultChannelPipelineClass.getDeclaredConstructor(Channel.class);
     constructor.setAccessible(true);
     return constructor;
diff --git a/instrumentation/opentelemetry-api/opentelemetry-api-1.56/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_56/incubator/ExtendedOpenTelemetryTest.java b/instrumentation/opentelemetry-api/opentelemetry-api-1.56/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_56/incubator/ExtendedOpenTelemetryTest.java
@@ -32,7 +32,6 @@ void getConfig() {
     assertThat(openTelemetry).isInstanceOf(ExtendedOpenTelemetry.class);
     DeclarativeConfigProperties instrumentationConfig =
         ((ExtendedOpenTelemetry) openTelemetry).getConfigProvider().getInstrumentationConfig();
-    assertThat(instrumentationConfig).isNotNull();
 
     assertThat(instrumentationConfig.getStructured("java").getStructured("foo").getString("bar"))
         .isEqualTo("baz");
diff --git a/instrumentation/opentelemetry-api/opentelemetry-api-1.59/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_59/ValueAttributeTest.java b/instrumentation/opentelemetry-api/opentelemetry-api-1.59/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/opentelemetryapi/v1_59/ValueAttributeTest.java
diff --git a/instrumentation/redisson/redisson-common/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/redisson/AbstractRedissonClientTest.java b/instrumentation/redisson/redisson-common/testing/src/main/java/io/opentelemetry/javaagent/instrumentation/redisson/AbstractRedissonClientTest.java
diff --git a/instrumentation/runtime-telemetry/library/src/testJava17/java/io/opentelemetry/instrumentation/runtimetelemetry/RuntimeTelemetryBuilderTest.java b/instrumentation/runtime-telemetry/library/src/testJava17/java/io/opentelemetry/instrumentation/runtimetelemetry/RuntimeTelemetryBuilderTest.java
diff --git a/instrumentation/runtime-telemetry/library/src/testJava17/java/io/opentelemetry/instrumentation/runtimetelemetry/RuntimeTelemetryTest.java b/instrumentation/runtime-telemetry/library/src/testJava17/java/io/opentelemetry/instrumentation/runtimetelemetry/RuntimeTelemetryTest.java
diff --git a/instrumentation/spring/spring-jms/spring-jms-2.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/spring/jms/v2_0/SpringTemplateTest.java b/instrumentation/spring/spring-jms/spring-jms-2.0/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/spring/jms/v2_0/SpringTemplateTest.java
diff --git a/instrumentation/twilio-6.6/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/twilio/TwilioClientTest.java b/instrumentation/twilio-6.6/javaagent/src/test/java/io/opentelemetry/javaagent/instrumentation/twilio/TwilioClientTest.java
diff --git a/javaagent-bootstrap/src/test/java/io/opentelemetry/javaagent/bootstrap/AgentClassLoaderTest.java b/javaagent-bootstrap/src/test/java/io/opentelemetry/javaagent/bootstrap/AgentClassLoaderTest.java
diff --git a/muzzle/src/test/java/io/opentelemetry/javaagent/tooling/muzzle/ReferenceCollectorTest.java b/muzzle/src/test/java/io/opentelemetry/javaagent/tooling/muzzle/ReferenceCollectorTest.java

Original file line number	Diff line number	Diff line change
`@@ -125,11 +125,9 @@ void prepTest() {`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`private void validateOperationResponse(String operation, Object response) {`
`128`		`- assertThat(response).isNotNull();`
`129`	`128`	`assertThat(response.getClass().getSimpleName()).startsWith(operation);`
`130`	`129`
`131`	`130`	`RecordedRequest request = server.takeRequest();`
`132`		`- assertThat(request).isNotNull();`
`133`	`131`	`assertThat(request.request().headers().get("X-Amzn-Trace-Id")).isNotNull();`
`134`	`132`	`assertThat(request.request().headers().get("traceparent")).isNull();`
`135`	`133`
Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,6 @@ void testCreateTelemetry() throws IOException {`
`84`	`84`	`scan = configure(scan);`
`85`	`85`
`86`	`86`	`try (CloseableIterable<FileScanTask> tasks = scan.planFiles()) {`
`87`		`- assertThat(tasks).isNotNull();`
`88`	`87`	`assertThat(tasks.iterator()).isNotNull();`
`89`	`88`	`}`
`90`	`89`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,6 @@ void testDelegatesToBootstrapClassLoaderForAgentClasses() throws ClassNotFoundEx`
`20`	`20`	`Class.forName(`
`21`	`21`	`"io.opentelemetry.javaagent.bootstrap.Java8BytecodeBridge", false, classLoader);`
`22`	`22`
`23`		`- assertThat(clazz).isNotNull();`
`24`	`23`	`assertThat(clazz.getClassLoader()).isNull();`
`25`	`24`	`}`
`26`	`25`
`@@ -30,7 +29,6 @@ void testDelegatesToBootstrapClassLoaderForAgentClassesTwoArguments()`
`30`	`29`	`NonDelegatingUrlClassLoader classLoader = new NonDelegatingUrlClassLoader();`
`31`	`30`	`Class<?> clazz =`
`32`	`31`	`classLoader.loadClass("io.opentelemetry.javaagent.bootstrap.Java8BytecodeBridge", false);`
`33`		`- assertThat(clazz).isNotNull();`
`34`	`32`	`assertThat(clazz.getClassLoader()).isNull();`
`35`	`33`	`}`
`36`	`34`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,6 @@ void testDelegatesToBootstrapClassLoaderForAgentClasses()`
`39`	`39`	`Class.forName(`
`40`	`40`	`"io.opentelemetry.javaagent.bootstrap.Java8BytecodeBridge", false, classLoader);`
`41`	`41`
`42`		`- assertThat(clazz).isNotNull();`
`43`	`42`	`assertThat(clazz.getClassLoader()).isNull();`
`44`	`43`	`}`
`45`	`44`	`}`