Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add enableLoggingFail option for failed introspect to FixtureMonkeyOptions #1069

Merged
merged 3 commits into from
Nov 2, 2024
Merged

Add enableLoggingFail option for failed introspect to FixtureMonkeyOptions #1069

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
35637b7
Add enableLoggingFail option for failed introspect to FixtureMonkeyOp…
HanJaehee Oct 24, 2024
f8992b1
Move the enableLoggingFail option from ArbitraryGeneratorContext to A…
HanJaehee Oct 29, 2024
6f2ed2e
Add license and API annotation to ArbitraryGeneratorLoggingContext
Nov 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion ...pi/src/main/java/com/navercorp/fixturemonkey/api/generator/ArbitraryGeneratorContext.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,7 @@ public final class ArbitraryGeneratorContext implements Traceable {
private final int generateUniqueMaxTries;
private final AtomicReference<CombinableArbitrary<?>> generated =
new AtomicReference<>(CombinableArbitrary.NOT_GENERATED);
private final ArbitraryGeneratorLoggingContext loggingContext;

public ArbitraryGeneratorContext(
Property resolvedProperty,
Expand All @@ -69,7 +70,8 @@ public ArbitraryGeneratorContext(
BiFunction<ArbitraryGeneratorContext, ArbitraryProperty, CombinableArbitrary<?>> resolveArbitrary,
LazyArbitrary<PropertyPath> lazyPropertyPath,
MonkeyGeneratorContext monkeyGeneratorContext,
int generateUniqueMaxTries
int generateUniqueMaxTries,
ArbitraryGeneratorLoggingContext loggingContext
) {
this.resolvedProperty = resolvedProperty;
this.property = property;
Expand All @@ -79,6 +81,7 @@ public ArbitraryGeneratorContext(
this.lazyPropertyPath = lazyPropertyPath;
this.monkeyGeneratorContext = monkeyGeneratorContext;
this.generateUniqueMaxTries = generateUniqueMaxTries;
this.loggingContext = loggingContext;
}

public ArbitraryProperty getArbitraryProperty() {
Expand Down Expand Up @@ -149,6 +152,10 @@ public int getGenerateUniqueMaxTries() {
return generateUniqueMaxTries;
}

public ArbitraryGeneratorLoggingContext getLoggingContext() {
return loggingContext;
}

public CombinableArbitrary<?> getGenerated() {
return generated.get();
}
Expand Down
14 changes: 14 additions & 0 deletions ...main/java/com/navercorp/fixturemonkey/api/generator/ArbitraryGeneratorLoggingContext.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
package com.navercorp.fixturemonkey.api.generator;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A license is needed.

/*
 * Fixture Monkey
 *
 * Copyright (c) 2021-present NAVER Corp.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */



public final class ArbitraryGeneratorLoggingContext {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the annotation.
@API(since = "1.0.28", status = Status.EXPERIMENTAL)

private final boolean enableLoggingFail;

public ArbitraryGeneratorLoggingContext(boolean enableLoggingFail) {
this.enableLoggingFail = enableLoggingFail;
}

public boolean isEnableLoggingFail() {
return enableLoggingFail;
}
}
6 changes: 5 additions & 1 deletion ...src/main/java/com/navercorp/fixturemonkey/api/introspector/BeanArbitraryIntrospector.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@
import com.navercorp.fixturemonkey.api.arbitrary.CombinableArbitrary;
import com.navercorp.fixturemonkey.api.arbitrary.CombinableArbitraryDelegator;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorLoggingContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryProperty;
import com.navercorp.fixturemonkey.api.property.Property;
import com.navercorp.fixturemonkey.api.property.PropertyGenerator;
Expand Down Expand Up @@ -66,7 +67,10 @@ public ArbitraryIntrospectorResult introspect(ArbitraryGeneratorContext context)
try {
checkPrerequisite(type);
} catch (Exception ex) {
LOGGER.warn("Given type {} is failed to generate due to the exception. It may be null.", type, ex);
ArbitraryGeneratorLoggingContext loggingContext = context.getLoggingContext();
if (loggingContext.isEnableLoggingFail()) {
LOGGER.warn("Given type {} is failed to generate due to the exception. It may be null.", type, ex);
}
return ArbitraryIntrospectorResult.NOT_INTROSPECTED;
}
generated = CombinableArbitrary.from(() -> Reflections.newInstance(type));
Expand Down
6 changes: 5 additions & 1 deletion .../main/java/com/navercorp/fixturemonkey/api/introspector/BuilderArbitraryIntrospector.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@

import com.navercorp.fixturemonkey.api.arbitrary.CombinableArbitrary;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorLoggingContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryProperty;
import com.navercorp.fixturemonkey.api.lazy.LazyArbitrary;
import com.navercorp.fixturemonkey.api.property.CompositeProperty;
Expand Down Expand Up @@ -90,7 +91,10 @@ public ArbitraryIntrospectorResult introspect(ArbitraryGeneratorContext context)
}
);
} catch (Exception ex) {
LOGGER.warn("Given type {} is failed to generate due to the exception. It may be null.", type, ex);
ArbitraryGeneratorLoggingContext loggingContext = context.getLoggingContext();
if (loggingContext.isEnableLoggingFail()) {
LOGGER.warn("Given type {} is failed to generate due to the exception. It may be null.", type, ex);
}
return ArbitraryIntrospectorResult.NOT_INTROSPECTED;
}
Method builderMethod = BUILDER_CACHE.get(type);
Expand Down
14 changes: 9 additions & 5 deletions .../navercorp/fixturemonkey/api/introspector/ConstructorPropertiesArbitraryIntrospector.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@

import com.navercorp.fixturemonkey.api.arbitrary.CombinableArbitrary;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorLoggingContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryProperty;
import com.navercorp.fixturemonkey.api.property.ConstructorParameterPropertyGenerator;
import com.navercorp.fixturemonkey.api.property.Property;
Expand Down Expand Up @@ -68,11 +69,14 @@ public ArbitraryIntrospectorResult introspect(ArbitraryGeneratorContext context)

Entry<Constructor<?>, String[]> parameterNamesByConstructor = TypeCache.getParameterNamesByConstructor(type);
if (parameterNamesByConstructor == null) {
LOGGER.warn(
"Given type {} is failed to generate due to the exception. It may be null.",
type,
new IllegalArgumentException("Primary Constructor does not exist. type " + type.getSimpleName())
);
ArbitraryGeneratorLoggingContext loggingContext = context.getLoggingContext();
if (loggingContext.isEnableLoggingFail()) {
LOGGER.warn(
"Given type {} is failed to generate due to the exception. It may be null.",
type,
new IllegalArgumentException("Primary Constructor does not exist. type " + type.getSimpleName())
);
}
return ArbitraryIntrospectorResult.NOT_INTROSPECTED;
}

Expand Down
10 changes: 7 additions & 3 deletions ...-api/src/main/java/com/navercorp/fixturemonkey/api/introspector/FailoverIntrospector.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@

import com.navercorp.fixturemonkey.api.arbitrary.CombinableArbitrary;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorLoggingContext;

@API(since = "0.6.0", status = Status.MAINTAINED)
public final class FailoverIntrospector implements ArbitraryIntrospector {
Expand Down Expand Up @@ -57,7 +58,8 @@ public ArbitraryIntrospectorResult introspect(ArbitraryGeneratorContext context)
results.add(new FailoverIntrospectorResult(introspector, result));
}
} catch (Exception ex) {
if (enableLoggingFail) {
ArbitraryGeneratorLoggingContext loggingContext = context.getLoggingContext();
if (loggingContext.isEnableLoggingFail() || enableLoggingFail) {
LOGGER.warn(
String.format(
"\"%s\" is failed to introspect \"%s\" type.",
Expand Down Expand Up @@ -85,7 +87,8 @@ public Object combined() {
result = iterator.next();
return result.getResult().getValue().combined();
} catch (Exception ex) {
if (enableLoggingFail) {
ArbitraryGeneratorLoggingContext loggingContext = context.getLoggingContext();
if (loggingContext.isEnableLoggingFail() || enableLoggingFail) {
LOGGER.warn(
String.format(
"\"%s\" is failed to introspect \"%s\" type.",
Expand Down Expand Up @@ -115,7 +118,8 @@ public Object rawValue() {
result = iterator.next();
return result.getResult().getValue().rawValue();
} catch (Exception ex) {
if (enableLoggingFail) {
ArbitraryGeneratorLoggingContext loggingContext = context.getLoggingContext();
if (loggingContext.isEnableLoggingFail() || enableLoggingFail) {
LOGGER.warn(
String.format(
"\"%s\" is failed to introspect type \"%s\"",
Expand Down
6 changes: 5 additions & 1 deletion ...va/com/navercorp/fixturemonkey/api/introspector/FieldReflectionArbitraryIntrospector.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@
import com.navercorp.fixturemonkey.api.arbitrary.CombinableArbitrary;
import com.navercorp.fixturemonkey.api.arbitrary.CombinableArbitraryDelegator;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorLoggingContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryProperty;
import com.navercorp.fixturemonkey.api.property.Property;
import com.navercorp.fixturemonkey.api.property.PropertyGenerator;
Expand Down Expand Up @@ -62,7 +63,10 @@ public ArbitraryIntrospectorResult introspect(ArbitraryGeneratorContext context)
try {
checkPrerequisite(type);
} catch (Exception ex) {
LOGGER.warn("Given type {} is failed to generate due to the exception. It may be null.", type, ex);
ArbitraryGeneratorLoggingContext loggingContext = context.getLoggingContext();
if (loggingContext.isEnableLoggingFail()) {
LOGGER.warn("Given type {} is failed to generate due to the exception. It may be null.", type, ex);
}
return ArbitraryIntrospectorResult.NOT_INTROSPECTED;
}
generated = CombinableArbitrary.from(() -> Reflections.newInstance(type));
Expand Down
9 changes: 8 additions & 1 deletion ...monkey-api/src/main/java/com/navercorp/fixturemonkey/api/option/FixtureMonkeyOptions.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -169,6 +169,7 @@ public final class FixtureMonkeyOptions {
private final JavaConstraintGenerator javaConstraintGenerator;
private final InstantiatorProcessor instantiatorProcessor;
private final List<MatcherOperator<CandidateConcretePropertyResolver>> candidateConcretePropertyResolvers;
private final boolean enableLoggingFail;

public FixtureMonkeyOptions(
List<MatcherOperator<PropertyGenerator>> propertyGenerators,
Expand All @@ -189,7 +190,8 @@ public FixtureMonkeyOptions(
int generateUniqueMaxTries,
JavaConstraintGenerator javaConstraintGenerator,
InstantiatorProcessor instantiatorProcessor,
List<MatcherOperator<CandidateConcretePropertyResolver>> candidateConcretePropertyResolvers
List<MatcherOperator<CandidateConcretePropertyResolver>> candidateConcretePropertyResolvers,
boolean enableLoggingFail
) {
this.propertyGenerators = propertyGenerators;
this.defaultPropertyGenerator = defaultPropertyGenerator;
Expand All @@ -210,6 +212,7 @@ public FixtureMonkeyOptions(
this.javaConstraintGenerator = javaConstraintGenerator;
this.instantiatorProcessor = instantiatorProcessor;
this.candidateConcretePropertyResolvers = candidateConcretePropertyResolvers;
this.enableLoggingFail = enableLoggingFail;
}

public static FixtureMonkeyOptionsBuilder builder() {
Expand Down Expand Up @@ -351,6 +354,10 @@ public InstantiatorProcessor getInstantiatorProcessor() {
return instantiatorProcessor;
}

public boolean isEnableLoggingFail() {
return enableLoggingFail;
}

@Nullable
public CandidateConcretePropertyResolver getCandidateConcretePropertyResolver(Property property) {
List<CandidateConcretePropertyResolver> candidateConcretePropertyResolverList =
Expand Down
9 changes: 8 additions & 1 deletion ...api/src/main/java/com/navercorp/fixturemonkey/api/option/FixtureMonkeyOptionsBuilder.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -108,6 +108,7 @@ public final class FixtureMonkeyOptionsBuilder {
private boolean defaultNotNull = false;
private boolean nullableContainer = false;
private boolean nullableElement = false;
private boolean enableLoggingFail = true;
private UnaryOperator<NullInjectGenerator> defaultNullInjectGeneratorOperator = it -> it;
private ArbitraryValidator defaultArbitraryValidator = (obj) -> {
};
Expand Down Expand Up @@ -453,6 +454,11 @@ public FixtureMonkeyOptionsBuilder nullableElement(boolean nullableElement) {
return this;
}

public FixtureMonkeyOptionsBuilder enableLoggingFail(boolean enableLoggingFail) {
this.enableLoggingFail = enableLoggingFail;
return this;
}

public FixtureMonkeyOptionsBuilder defaultArbitraryValidator(ArbitraryValidator arbitraryValidator) {
this.defaultArbitraryValidator = arbitraryValidator;
return this;
Expand Down Expand Up @@ -664,7 +670,8 @@ public FixtureMonkeyOptions build() {
this.generateUniqueMaxTries,
resolvedJavaConstraintGenerator,
this.instantiatorProcessor,
this.candidateConcretePropertyResolvers
this.candidateConcretePropertyResolvers,
this.enableLoggingFail
);
}

Expand Down
5 changes: 5 additions & 0 deletions fixture-monkey/src/main/java/com/navercorp/fixturemonkey/FixtureMonkeyBuilder.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -450,6 +450,11 @@ public FixtureMonkeyBuilder nullableElement(boolean nullableElement) {
return this;
}

public FixtureMonkeyBuilder enableLoggingFail(boolean enableLoggingFail) {
this.fixtureMonkeyOptionsBuilder.enableLoggingFail(enableLoggingFail);
return this;
}

/**
* It is deprecated. Please use {@link InterfacePlugin#interfaceImplements(Matcher, List)} instead.
*/
Expand Down
7 changes: 6 additions & 1 deletion fixture-monkey/src/main/java/com/navercorp/fixturemonkey/tree/ObjectTree.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@
import com.navercorp.fixturemonkey.api.context.MonkeyGeneratorContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGenerator;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryGeneratorLoggingContext;
import com.navercorp.fixturemonkey.api.generator.ArbitraryProperty;
import com.navercorp.fixturemonkey.api.generator.CompositeArbitraryGenerator;
import com.navercorp.fixturemonkey.api.generator.IntrospectedArbitraryGenerator;
Expand Down Expand Up @@ -112,6 +113,9 @@ private ArbitraryGeneratorContext generateContext(
}

MonkeyGeneratorContext monkeyGeneratorContext = monkeyContext.retrieveGeneratorContext(rootProperty);
ArbitraryGeneratorLoggingContext loggingContext = new ArbitraryGeneratorLoggingContext(
fixtureMonkeyOptions.isEnableLoggingFail());

return new ArbitraryGeneratorContext(
resolvedParentProperty,
arbitraryProperty,
Expand All @@ -127,7 +131,8 @@ private ArbitraryGeneratorContext generateContext(
},
objectNode.getLazyPropertyPath(),
monkeyGeneratorContext,
fixtureMonkeyOptions.getGenerateUniqueMaxTries()
fixtureMonkeyOptions.getGenerateUniqueMaxTries(),
loggingContext
);
}

Expand Down
Loading