add test

Author: jfbreault

instrumentation/spring/spring-boot-autoconfigure/src/main/java/io/opentelemetry/instrumentation/spring/autoconfigure/instrumentation/logging/LogbackAppenderInstaller.java

@@ -37,7 +37,7 @@ private static boolean isLogbackAppenderAddable(
             applicationEnvironmentPreparedEvent,
             "otel.instrumentation.logback-appender.enabled",
             true);
-    return !otelSdkDisableProperty || logbackInstrumentationEnabledProperty;
+    return !otelSdkDisableProperty && logbackInstrumentationEnabledProperty;
   }
 
   private static void reInitializeOpenTelemetryAppender(

instrumentation/spring/spring-boot-autoconfigure/src/testLogbackAppender/java/io/opentelemetry/instrumentation/spring/autoconfigure/instrumentation/logging/LogbackAppenderTest.java

@@ -110,4 +110,40 @@ void shouldNotInitializeAppenderWhenDisabled() {
 
     assertThat(testing.logRecords()).isEmpty();
   }
+
+  @Test
+  void shouldNotInitializeAppenderWhenSdkDisabled() {
+    Map<String, Object> properties = new HashMap<>();
+    properties.put("logging.config", "classpath:logback-test.xml");
+    properties.put("otel.sdk.disabled", "true");
+
+    SpringApplication app =
+        new SpringApplication(
+            TestingOpenTelemetryConfiguration.class, OpenTelemetryAppenderAutoConfiguration.class);
+    app.setDefaultProperties(properties);
+    ConfigurableApplicationContext context = app.run();
+    cleanup.deferCleanup(context);
+
+    LoggerFactory.getLogger("test").info("test log message");
+
+    assertThat(testing.logRecords()).isEmpty();
+  }
+
+  @Test
+  void shouldNotInitializeAppenderWhenInstrumentationDisabled() {
+    Map<String, Object> properties = new HashMap<>();
+    properties.put("logging.config", "classpath:logback-test.xml");
+    properties.put("otel.instrumentation.logback-appender.enabled", "false");
+
+    SpringApplication app =
+        new SpringApplication(
+            TestingOpenTelemetryConfiguration.class, OpenTelemetryAppenderAutoConfiguration.class);
+    app.setDefaultProperties(properties);
+    ConfigurableApplicationContext context = app.run();
+    cleanup.deferCleanup(context);
+
+    LoggerFactory.getLogger("test").info("test log message");
+
+    assertThat(testing.logRecords()).isEmpty();
+  }
 }