Polishing.

Replace if statement with ternary operator.

See #4994

Author: mp911de

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/observability/DefaultMongoHandlerObservationConvention.java

@@ -33,6 +33,7 @@
  *
  * @author Greg Turnquist
  * @author Mark Paluch
+ * @author Michal Domagala
  * @since 4.0
  */
 class DefaultMongoHandlerObservationConvention implements MongoHandlerObservationConvention {
@@ -54,21 +55,16 @@ public KeyValues getLowCardinalityKeyValues(MongoHandlerContext context) {
 			if (!ObjectUtils.isEmpty(user)) {
 				keyValues = keyValues.and(LowCardinalityCommandKeyNames.DB_USER.withValue(user));
 			}
-
 		}
 
 		if (!ObjectUtils.isEmpty(context.getDatabaseName())) {
 			keyValues = keyValues.and(LowCardinalityCommandKeyNames.DB_NAME.withValue(context.getDatabaseName()));
 		}
 
-		if (!ObjectUtils.isEmpty(context.getCollectionName())) {
-			keyValues = keyValues
-					.and(LowCardinalityCommandKeyNames.MONGODB_COLLECTION.withValue(context.getCollectionName()));
-		} else {
-			keyValues = keyValues.and(LowCardinalityCommandKeyNames.MONGODB_COLLECTION.withValue(KeyValue.NONE_VALUE));
-		}
+		keyValues = keyValues.and(LowCardinalityCommandKeyNames.MONGODB_COLLECTION.withValue(
+				ObjectUtils.isEmpty(context.getCollectionName()) ? KeyValue.NONE_VALUE : context.getCollectionName()));
 
-		if(context.getCommandStartedEvent() == null) {
+		if (context.getCommandStartedEvent() == null) {
 			throw new IllegalStateException("not command started event present");
 		}
 

spring-data-mongodb/src/test/java/org/springframework/data/mongodb/observability/MongoObservationCommandListenerTests.java

@@ -51,6 +51,7 @@
  * @author Greg Turnquist
  * @author Mark Paluch
  * @author François Kha
+ * @author Michal Domagala
  */
 class MongoObservationCommandListenerTests {
 
@@ -99,8 +100,8 @@ void commandStartedShouldNotInstrumentWhenNoParentSampleInRequestContext() {
 		assertThat(meterRegistry).hasMeterWithName("spring.data.mongodb.command.active");
 	}
 
-	@Test
-	void commandStartedShouldIncludeCollectionIfMissing() {
+	@Test // GH-4994
+	void commandStartedShouldAlwaysIncludeCollection() {
 
 		// when
 		listener.commandStarted(new CommandStartedEvent(new MapRequestContext(), 0, 0, null, "some name", "hello", null));
@@ -110,7 +111,6 @@ void commandStartedShouldIncludeCollectionIfMissing() {
 		assertThat(meterRegistry).hasMeterWithNameAndTags(
 				"spring.data.mongodb.command.active",
 				Tags.of("db.mongodb.collection", "none"));
-
 	}
 
 	@Test