feature/FREEMARKER-54: ConfigurationFactoryBean tests
Project: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/commit/2783387e Tree: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/tree/2783387e Diff: http://git-wip-us.apache.org/repos/asf/incubator-freemarker/diff/2783387e Branch: refs/heads/3 Commit: 2783387eb495331042e72449f02b2a18527a9e13 Parents: 57270ec Author: Woonsan Ko <woon...@apache.org> Authored: Thu Jun 15 00:37:24 2017 -0400 Committer: Woonsan Ko <woon...@apache.org> Committed: Thu Jun 15 00:37:24 2017 -0400 ---------------------------------------------------------------------- .../freemarker/spring/ConfigurationFactoryBean.java | 2 ++ .../freemarker/spring/ConfigurationFactoryBeanTest.java | 12 ++++++++++++ 2 files changed, 14 insertions(+) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/2783387e/freemarker-spring/src/main/java/org/apache/freemarker/spring/ConfigurationFactoryBean.java ---------------------------------------------------------------------- diff --git a/freemarker-spring/src/main/java/org/apache/freemarker/spring/ConfigurationFactoryBean.java b/freemarker-spring/src/main/java/org/apache/freemarker/spring/ConfigurationFactoryBean.java index ead823d..2761b9a 100644 --- a/freemarker-spring/src/main/java/org/apache/freemarker/spring/ConfigurationFactoryBean.java +++ b/freemarker-spring/src/main/java/org/apache/freemarker/spring/ConfigurationFactoryBean.java @@ -44,6 +44,8 @@ public class ConfigurationFactoryBean extends ExtendableBuilder<ConfigurationFac private Map<String, String> settings = new LinkedHashMap<>(); public ConfigurationFactoryBean() { + // By default, set the default version constant. + // #setIncompatibleImprovements(Version) can be used to change it. super(Configuration.DEFAULT_INCOMPATIBLE_IMPROVEMENTS); delegate = new AbstractFactoryBean<Configuration>() { http://git-wip-us.apache.org/repos/asf/incubator-freemarker/blob/2783387e/freemarker-spring/src/test/java/org/apache/freemarker/spring/ConfigurationFactoryBeanTest.java ---------------------------------------------------------------------- diff --git a/freemarker-spring/src/test/java/org/apache/freemarker/spring/ConfigurationFactoryBeanTest.java b/freemarker-spring/src/test/java/org/apache/freemarker/spring/ConfigurationFactoryBeanTest.java index 7dcbcc5..82a5e74 100644 --- a/freemarker-spring/src/test/java/org/apache/freemarker/spring/ConfigurationFactoryBeanTest.java +++ b/freemarker-spring/src/test/java/org/apache/freemarker/spring/ConfigurationFactoryBeanTest.java @@ -18,13 +18,16 @@ */ package org.apache.freemarker.spring; +import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertTrue; +import java.util.HashMap; import java.util.LinkedHashMap; import java.util.Map; import org.apache.freemarker.core.Configuration; import org.apache.freemarker.core.Configuration.ExtendableBuilder; +import org.apache.freemarker.core.Version; import org.junit.After; import org.junit.Before; import org.junit.Test; @@ -55,9 +58,15 @@ public class ConfigurationFactoryBeanTest { final Map<String, String> settings = new LinkedHashMap<>(); settings.put(ExtendableBuilder.LOCALIZED_TEMPLATE_LOOKUP_KEY_CAMEL_CASE, "true"); + final Map<String, Object> sharedVars = new HashMap<>(); + sharedVars.put("sharedVar1", "sharedVal1"); + sharedVars.put("sharedVar2", "sharedVal2"); + BeanDefinition beanDef = BeanDefinitionBuilder.genericBeanDefinition(ConfigurationFactoryBean.class.getName()) + .addPropertyValue("incompatibleImprovements", new Version(3, 0, 0)) .addPropertyValue("settings", settings) + .addPropertyValue("sharedVariables", sharedVars) .getBeanDefinition(); appContext.registerBeanDefinition("freemarkerConfig", beanDef); @@ -68,7 +77,10 @@ public class ConfigurationFactoryBeanTest { assertTrue("Not a Configuration object: " + bean, bean instanceof Configuration); Configuration config = (Configuration) bean; + assertEquals(new Version(3, 0, 0), config.getIncompatibleImprovements()); assertTrue(config.getLocalizedTemplateLookup()); + assertEquals("sharedVal1", config.getSharedVariables().get("sharedVar1")); + assertEquals("sharedVal2", config.getSharedVariables().get("sharedVar2")); } }