Merge pull request #47072 from jonatan-ivanov

* pr/47072:
  Use isNoop() instead of Observation.NOOP

Closes gh-47072

Author: snicoll

module/spring-boot-micrometer-observation/src/test/java/org/springframework/boot/micrometer/observation/autoconfigure/ObservationAutoConfigurationTests.java

@@ -99,11 +99,11 @@ void autoConfiguresObservationPredicates() {
 			ObservationRegistry observationRegistry = context.getBean(ObservationRegistry.class);
 			// This is allowed by ObservationPredicates.customPredicate
 			Observation observation = Observation.start("observation1", observationRegistry);
-			assertThat(observation).isNotEqualTo(Observation.NOOP);
+			assertThat(observation.isNoop()).isFalse();
 			observation.stop();
 			// This isn't allowed by ObservationPredicates.customPredicate
 			observation = Observation.start("observation2", observationRegistry);
-			assertThat(observation).isEqualTo(Observation.NOOP);
+			assertThat(observation.isNoop()).isTrue();
 			observation.stop();
 		});
 	}
@@ -150,7 +150,7 @@ void shouldNotDisableSpringSecurityObservationsByDefault() {
 		this.contextRunner.run((context) -> {
 			ObservationRegistry observationRegistry = context.getBean(ObservationRegistry.class);
 			Observation observation = Observation.start("spring.security.filterchains", observationRegistry);
-			assertThat(observation).isNotEqualTo(Observation.NOOP);
+			assertThat(observation.isNoop()).isFalse();
 			observation.stop();
 		});
 	}
@@ -160,7 +160,7 @@ void shouldDisableSpringSecurityObservationsIfPropertyIsSet() {
 		this.contextRunner.withPropertyValues("management.observations.enable.spring.security=false").run((context) -> {
 			ObservationRegistry observationRegistry = context.getBean(ObservationRegistry.class);
 			Observation observation = Observation.start("spring.security.filterchains", observationRegistry);
-			assertThat(observation).isEqualTo(Observation.NOOP);
+			assertThat(observation.isNoop()).isTrue();
 			observation.stop();
 		});
 	}