Forgot to add a type token.

Author: dualspiral

src/main/java/uk/co/drnaylor/quickstart/ModuleContainer.java

@@ -7,13 +7,11 @@
 import com.google.common.base.Preconditions;
 import com.google.common.collect.Maps;
 import com.google.common.reflect.ClassPath;
-import com.google.common.reflect.TypeToken;
 import ninja.leaping.configurate.ConfigurationNode;
 import ninja.leaping.configurate.loader.ConfigurationLoader;
 import ninja.leaping.configurate.objectmapping.ObjectMappingException;
 import uk.co.drnaylor.quickstart.annotations.ModuleData;
 import uk.co.drnaylor.quickstart.config.AbstractConfigAdapter;
-import uk.co.drnaylor.quickstart.config.LoadingStatusTypeSerializer;
 import uk.co.drnaylor.quickstart.constructors.ModuleConstructor;
 import uk.co.drnaylor.quickstart.constructors.SimpleModuleConstructor;
 import uk.co.drnaylor.quickstart.enums.ConstructionPhase;
@@ -399,7 +397,6 @@ public ModuleContainer build() throws QuickStartModuleDiscoveryException {
                 classLoader = getClass().getClassLoader();
             }
 
-            configurationLoader.getDefaultOptions().getSerializers().registerType(TypeToken.of(LoadingStatus.class), new LoadingStatusTypeSerializer());
             Metadata.getStartupMessage().ifPresent(x -> Logger.getLogger("QuickStart").info(x));
             return new ModuleContainer(configurationLoader, classLoader, packageToScan, constructor);
         }

src/main/java/uk/co/drnaylor/quickstart/SystemConfig.java

@@ -13,6 +13,7 @@
 import uk.co.drnaylor.quickstart.enums.LoadingStatus;
 
 import java.io.IOException;
+import java.util.HashMap;
 import java.util.Map;
 
 /**
@@ -31,7 +32,7 @@ void attachModulesConfig(Map<String, LoadingStatus> defaults) throws IOException
         Preconditions.checkNotNull(defaults);
         Preconditions.checkState(configAdapter == null);
 
-        configAdapter = new ModulesConfigAdapter(defaults);
+        configAdapter = new ModulesConfigAdapter(new HashMap<>(defaults));
         this.attachConfigAdapter(modulesNode, configAdapter);
     }
 

src/main/java/uk/co/drnaylor/quickstart/config/LoadingStatusTypeSerializer.java

@@ -4,25 +4,25 @@
  */
 package uk.co.drnaylor.quickstart.config;
 
-import com.google.common.collect.ImmutableMap;
+import com.google.common.collect.Maps;
 import com.google.common.reflect.TypeToken;
 import ninja.leaping.configurate.ConfigurationNode;
 import ninja.leaping.configurate.objectmapping.ObjectMappingException;
 import uk.co.drnaylor.quickstart.enums.LoadingStatus;
 
 import java.util.HashMap;
-import java.util.Map;
 
 /**
  * Configuration adapter that handles the module statuses.
  */
-public final class ModulesConfigAdapter extends AbstractConfigAdapter<Map<String, LoadingStatus>> {
+public final class ModulesConfigAdapter extends AbstractConfigAdapter<HashMap<String, LoadingStatus>> {
 
     public static final String modulesKey = "modules";
 
-    private final Map<String, LoadingStatus> defaults;
+    private final TypeToken<HashMap<String, LoadingStatus>> tt = new TypeToken<HashMap<String, LoadingStatus>>() {};
+    private final HashMap<String, LoadingStatus> defaults;
 
-    public ModulesConfigAdapter(Map<String, LoadingStatus> defaults) {
+    public ModulesConfigAdapter(HashMap<String, LoadingStatus> defaults) {
         this.defaults = defaults;
     }
 
@@ -33,7 +33,7 @@ protected ConfigurationNode generateDefaults(ConfigurationNode node) {
     }
 
     @Override
-    protected Map<String, LoadingStatus> convertFromConfigurateNode(ConfigurationNode node) {
+    protected HashMap<String, LoadingStatus> convertFromConfigurateNode(ConfigurationNode node) {
         HashMap<String, LoadingStatus> value = null;
         try {
             value = node.getValue(new TypeToken<HashMap<String, LoadingStatus>>() {});
@@ -42,15 +42,15 @@ protected Map<String, LoadingStatus> convertFromConfigurateNode(ConfigurationNod
         }
 
         if (value == null) {
-            return ImmutableMap.copyOf(defaults);
+            return Maps.newHashMap(defaults);
         }
 
         return value;
     }
 
     @Override
     @SuppressWarnings("unchecked")
-    protected ConfigurationNode insertIntoConfigurateNode(Map<String, LoadingStatus> data) {
-        return this.getNewNode().setValue(data);
+    protected ConfigurationNode insertIntoConfigurateNode(HashMap<String, LoadingStatus> data) throws ObjectMappingException {
+        return this.getNewNode().setValue(tt, data);
     }
 }

src/main/java/uk/co/drnaylor/quickstart/config/ModulesConfigAdapter.java

@@ -19,6 +19,7 @@
 import uk.co.drnaylor.quickstart.tests.scaffolding.FakeLoaderTests;
 
 import java.lang.reflect.Constructor;
+import java.util.HashMap;
 import java.util.Map;
 
 public class ModuleConfigurationAdapterTests extends FakeLoaderTests {
@@ -34,7 +35,7 @@ public void beforeTests() throws Exception {
         ctor.setAccessible(true);
         config = (SystemConfig<ConfigurationNode, ConfigurationLoader<ConfigurationNode>>) ctor.newInstance(loader);
 
-        Map<String, LoadingStatus> m = Maps.newHashMap();
+        HashMap<String, LoadingStatus> m = Maps.newHashMap();
         m.put("d", LoadingStatus.DISABLED);
         m.put("e", LoadingStatus.ENABLED);
         m.put("f", LoadingStatus.FORCELOAD);