Repository: wicket Updated Branches: refs/heads/wicket-7.x b2252c329 -> 200c7f34f
WICKET-6243 better testing session locale changes in auto link rendering Project: http://git-wip-us.apache.org/repos/asf/wicket/repo Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/200c7f34 Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/200c7f34 Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/200c7f34 Branch: refs/heads/wicket-7.x Commit: 200c7f34f10e8bb99ef10c8b8365a84211fe8f1a Parents: b2252c3 Author: Pedro Henrique Oliveira dos Santos <[email protected]> Authored: Wed Sep 14 00:15:00 2016 -0300 Committer: Pedro Henrique Oliveira dos Santos <[email protected]> Committed: Wed Sep 14 00:15:00 2016 -0300 ---------------------------------------------------------------------- .../markup/resolver/AutoLinkResolverTest.java | 42 +++++++++++++++----- .../apache/wicket/markup/resolver/resource.ext | 0 2 files changed, 33 insertions(+), 9 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/wicket/blob/200c7f34/wicket-core/src/test/java/org/apache/wicket/markup/resolver/AutoLinkResolverTest.java ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/markup/resolver/AutoLinkResolverTest.java b/wicket-core/src/test/java/org/apache/wicket/markup/resolver/AutoLinkResolverTest.java index 32ed85c..d517fcc 100644 --- a/wicket-core/src/test/java/org/apache/wicket/markup/resolver/AutoLinkResolverTest.java +++ b/wicket-core/src/test/java/org/apache/wicket/markup/resolver/AutoLinkResolverTest.java @@ -16,29 +16,53 @@ */ package org.apache.wicket.markup.resolver; -import static org.junit.Assert.assertThat; +import static org.hamcrest.CoreMatchers.containsString; import java.util.Locale; -import org.apache.wicket.util.tester.WicketTester; -import org.hamcrest.CoreMatchers; +import org.apache.wicket.util.tester.WicketTestCase; import org.junit.Test; -public class AutoLinkResolverTest +/** + * @author Pedro Santos + */ +public class AutoLinkResolverTest extends WicketTestCase { + private static final Locale DEFAULT_LOCALE = Locale.US; + private static final Locale EXISTENT_RESOURCE_LOCALE = Locale.CANADA; + private static final Locale NON_EXISTENT_RESOURCE_LOCALE = Locale.FRANCE; @Test public void shouldAutoLinkLocalizedResources() { - Locale.setDefault(Locale.US); + PageWithAutoLinkedLocalResource instance = new PageWithAutoLinkedLocalResource(); - WicketTester tester = new WicketTester(); + tester.getSession().setLocale(DEFAULT_LOCALE); - tester.getSession().setLocale(Locale.CANADA); + tester.startPage(instance); - tester.startPage(PageWithAutoLinkedLocalResource.class); + tester.getSession().setLocale(EXISTENT_RESOURCE_LOCALE); + + tester.startPage(instance); - assertThat(tester.getLastResponseAsString(), CoreMatchers.containsString("en_CA")); + assertThat(tester.getLastResponseAsString(), + containsString(EXISTENT_RESOURCE_LOCALE.getCountry())); } + @Test + public void shouldAutoLinkExistentLocalizedResources() + { + tester.getSession().setLocale(NON_EXISTENT_RESOURCE_LOCALE); + + tester.startPage(PageWithAutoLinkedLocalResource.class); + + tester.getSession().setLocale(EXISTENT_RESOURCE_LOCALE); + + // works if the page is recreated only + // TODO: render existent resource's URL if previously not shown by this page instance? + tester.startPage(PageWithAutoLinkedLocalResource.class); + + assertThat(tester.getLastResponseAsString(), + containsString(EXISTENT_RESOURCE_LOCALE.getCountry())); + } } http://git-wip-us.apache.org/repos/asf/wicket/blob/200c7f34/wicket-core/src/test/java/org/apache/wicket/markup/resolver/resource.ext ---------------------------------------------------------------------- diff --git a/wicket-core/src/test/java/org/apache/wicket/markup/resolver/resource.ext b/wicket-core/src/test/java/org/apache/wicket/markup/resolver/resource.ext deleted file mode 100644 index e69de29..0000000
