Add enableLoggingFail option to the FailoverIntrospector

naver · Oct 4, 2024 · 6f748f0 · 6f748f0
1 parent 1309439
commit 6f748f0
Showing 1 changed file with 36 additions and 24 deletions.
diff --git a/...-api/src/main/java/com/navercorp/fixturemonkey/api/introspector/FailoverIntrospector.java b/...-api/src/main/java/com/navercorp/fixturemonkey/api/introspector/FailoverIntrospector.java
@@ -36,9 +36,15 @@ public final class FailoverIntrospector implements ArbitraryIntrospector {
 	private static final Logger LOGGER = LoggerFactory.getLogger(FailoverIntrospector.class);
 
 	private final List<ArbitraryIntrospector> introspectors;
+	private final boolean enableLoggingFail;
 
 	public FailoverIntrospector(List<ArbitraryIntrospector> introspectors) {
+		this(introspectors, true);
+	}
+
+	public FailoverIntrospector(List<ArbitraryIntrospector> introspectors, boolean enableLoggingFail) {
 		this.introspectors = introspectors;
+		this.enableLoggingFail = enableLoggingFail;
 	}
 
 	@Override
@@ -51,14 +57,16 @@ public ArbitraryIntrospectorResult introspect(ArbitraryGeneratorContext context)
 					results.add(new FailoverIntrospectorResult(introspector, result));
 				}
 			} catch (Exception ex) {
-				LOGGER.warn(
-					String.format(
-						"\"%s\" is failed to introspect \"%s\" type.",
-						introspector.getClass().getSimpleName(),
-						((Class<?>)context.getResolvedProperty().getType()).getName()
-					),
-					ex
-				);
+				if (enableLoggingFail) {
+					LOGGER.warn(
+						String.format(
+							"\"%s\" is failed to introspect \"%s\" type.",
+							introspector.getClass().getSimpleName(),
+							((Class<?>)context.getResolvedProperty().getType()).getName()
+						),
+						ex
+					);
+				}
 				// omitted
 			}
 		}
@@ -77,14 +85,16 @@ public Object combined() {
 							result = iterator.next();
 							return result.getResult().getValue().combined();
 						} catch (Exception ex) {
-							LOGGER.warn(
-								String.format(
-									"\"%s\" is failed to introspect \"%s\" type.",
-									Objects.requireNonNull(result).getIntrospector().getClass().getSimpleName(),
-									((Class<?>)context.getResolvedProperty().getType()).getName()
-								),
-								ex
-							);
+							if (enableLoggingFail) {
+								LOGGER.warn(
+									String.format(
+										"\"%s\" is failed to introspect \"%s\" type.",
+										Objects.requireNonNull(result).getIntrospector().getClass().getSimpleName(),
+										((Class<?>)context.getResolvedProperty().getType()).getName()
+									),
+									ex
+								);
+							}
 							// omitted
 						}
 					}
@@ -105,14 +115,16 @@ public Object rawValue() {
 							result = iterator.next();
 							return result.getResult().getValue().rawValue();
 						} catch (Exception ex) {
-							LOGGER.warn(
-								String.format(
-									"\"%s\" is failed to introspect type \"%s\"",
-									Objects.requireNonNull(result).getIntrospector().getClass().getSimpleName(),
-									((Class<?>)context.getResolvedProperty().getType()).getName()
-								),
-								ex
-							);
+							if (enableLoggingFail) {
+								LOGGER.warn(
+									String.format(
+										"\"%s\" is failed to introspect type \"%s\"",
+										Objects.requireNonNull(result).getIntrospector().getClass().getSimpleName(),
+										((Class<?>)context.getResolvedProperty().getType()).getName()
+									),
+									ex
+								);
+							}
 							// omitted
 						}
 					}