Changing to default interface implementations for Theme

Additionally adding test that assert if one can replace a theme with
theirs.

Author: cardil

src/main/java/pl/wavesoftware/utils/stringify/impl/DefaultConfiguration.java

@@ -19,7 +19,6 @@
 import pl.wavesoftware.utils.stringify.api.Configuration;
 import pl.wavesoftware.utils.stringify.api.Mode;
 import pl.wavesoftware.utils.stringify.impl.beans.BeansModule;
-import pl.wavesoftware.utils.stringify.impl.theme.ThemeModule;
 import pl.wavesoftware.utils.stringify.spi.BeanFactory;
 import pl.wavesoftware.utils.stringify.spi.theme.Theme;
 
@@ -33,7 +32,7 @@ final class DefaultConfiguration implements Configuration {
 
   private Mode mode = Mode.DEFAULT_MODE;
   private BeanFactory beanFactory = DEFAULT_BEAN_FACTORY;
-  private Theme theme = ThemeModule.INSTANCE.defaultTheme();
+  private Theme theme = new Theme() {};
 
   @Override
   public Configuration mode(Mode mode) {
@@ -57,6 +56,7 @@ DefaultConfiguration dup() {
     DefaultConfiguration dup = new DefaultConfiguration();
     dup.beanFactory = beanFactory;
     dup.mode = mode;
+    dup.theme = theme;
     return dup;
   }
 

src/main/java/pl/wavesoftware/utils/stringify/impl/DefaultInspectionContext.java

@@ -22,6 +22,7 @@
 
 import java.util.IdentityHashMap;
 import java.util.Map;
+import java.util.function.Supplier;
 
 /**
  * @author <a href="mailto:krzysztof.suszynski@wavesoftware.pl">Krzysztof Suszynski</a>
@@ -31,11 +32,11 @@ final class DefaultInspectionContext implements InspectionContext {
   private static final Object CONTAIN = new Object();
 
   private final Map<Object, Object> resolved = new IdentityHashMap<>();
-  private final Theme theme;
+  private final Supplier<Theme> theme;
 
   private RootInpector rootInpector;
 
-  DefaultInspectionContext(Theme theme) {
+  DefaultInspectionContext(Supplier<Theme> theme) {
     this.theme = theme;
   }
 
@@ -56,7 +57,7 @@ public RootInpector rootInspector() {
 
   @Override
   public Theme theme() {
-    return theme;
+    return theme.get();
   }
 
   public void rootInpector(RootInpector rootInpector) {

src/main/java/pl/wavesoftware/utils/stringify/impl/DefaultStringify.java

@@ -63,7 +63,7 @@ public CharSequence resolve() {
     ComplexObjectStyle style = inspectionContext.theme().complexObject();
     StringBuilder sb = new StringBuilder();
     sb.append(style.begin());
-    sb.append(style.name(target::getClass, target::hashCode));
+    sb.append(style.name(target));
     CharSequence props = propertiesForToString();
     if (props.length() != 0) {
       sb.append(style.nameSeparator());
@@ -78,16 +78,17 @@ private CharSequence propertiesForToString() {
     props = inspectTargetAsClass(target.getClass());
     ComplexObjectStyle style = inspectionContext.theme().complexObject();
     StringBuilder sb = new StringBuilder();
+    CharSequence propertySeparator = style.propertySeparator();
     for (Map.Entry<String, CharSequence> entry : props.entrySet()) {
       String fieldName = entry.getKey();
       CharSequence fieldStringValue = entry.getValue();
       sb.append(fieldName);
       sb.append(style.propertyEquals());
       sb.append(fieldStringValue);
-      sb.append(style.propertySeparator());
+      sb.append(propertySeparator);
     }
     if (!props.isEmpty()) {
-      for (int i = 0; i < style.propertySeparator().length(); i++) {
+      for (int i = 0; i < propertySeparator.length(); i++) {
         sb.deleteCharAt(sb.length() - 1);
       }
     }

src/main/java/pl/wavesoftware/utils/stringify/impl/InspectorBasedToStringResolver.java

@@ -43,7 +43,7 @@ final class ToStringResolverImpl implements ToStringResolver, Configuration {
     this(
       target,
       configuration,
-      new DefaultInspectionContext(configuration.getTheme()),
+      new DefaultInspectionContext(configuration::getTheme),
       BeansModule.INSTANCE.cachedBeanFactory(
         configuration::getBeanFactory, target
       ),

src/main/java/pl/wavesoftware/utils/stringify/impl/ToStringResolverImpl.java

@@ -34,6 +34,6 @@ public boolean consentTo(Object candidate, InspectionContext context) {
   public CharSequence inspect(Object object, InspectionContext context) {
     return context.theme()
       .jpaLazy()
-      .representation(object::hashCode);
+      .representation(object);
   }
 }

src/main/java/pl/wavesoftware/utils/stringify/impl/inspector/InspectionContext.java

@@ -30,6 +30,6 @@ public boolean consentTo(Object candidate, InspectionContext context) {
   public CharSequence inspect(Object object, InspectionContext context) {
     return context.theme()
       .recursion()
-      .representation(object::getClass, object::hashCode);
+      .representation(object);
   }
 }