From 52d3caaddbbad2f5305203b1ef9a0059a5813186 Mon Sep 17 00:00:00 2001 From: Elvys Soares Date: Wed, 29 Jul 2020 09:37:55 -0300 Subject: [PATCH 1/2] Removed assertion roulette (test smell) --- .../test/java/cn/hutool/extra/template/BeetlUtilTest.java | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/hutool-extra/src/test/java/cn/hutool/extra/template/BeetlUtilTest.java b/hutool-extra/src/test/java/cn/hutool/extra/template/BeetlUtilTest.java index c7d27f621..d2e2d029d 100644 --- a/hutool-extra/src/test/java/cn/hutool/extra/template/BeetlUtilTest.java +++ b/hutool-extra/src/test/java/cn/hutool/extra/template/BeetlUtilTest.java @@ -21,15 +21,16 @@ import java.io.IOException; public class BeetlUtilTest { @Test - public void renderStrTest() throws IOException { + public void renderTest() throws IOException { GroupTemplate groupTemplate = BeetlUtil.createGroupTemplate(new StringTemplateResourceLoader(), Configuration.defaultConfiguration()); Template template = BeetlUtil.getTemplate(groupTemplate, "hello,${name}"); String result = BeetlUtil.render(template, Dict.create().set("name", "hutool")); - Assert.assertEquals("hello,hutool", result); + } + @Test + public void renderFromStrTest() { String renderFromStr = BeetlUtil.renderFromStr("hello,${name}", Dict.create().set("name", "hutool")); Assert.assertEquals("hello,hutool", renderFromStr); - } } From c6cf5f825b96d1c862db6b65f341910edfb8ec31 Mon Sep 17 00:00:00 2001 From: Golden Looly Date: Thu, 30 Jul 2020 08:09:50 +0800 Subject: [PATCH 2/2] Revert "Test improvement: removed assertion roulette (test smell)" --- .../java/cn/hutool/extra/template/BeetlUtilTest.java | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/hutool-extra/src/test/java/cn/hutool/extra/template/BeetlUtilTest.java b/hutool-extra/src/test/java/cn/hutool/extra/template/BeetlUtilTest.java index d2e2d029d..c7d27f621 100644 --- a/hutool-extra/src/test/java/cn/hutool/extra/template/BeetlUtilTest.java +++ b/hutool-extra/src/test/java/cn/hutool/extra/template/BeetlUtilTest.java @@ -21,16 +21,15 @@ import java.io.IOException; public class BeetlUtilTest { @Test - public void renderTest() throws IOException { + public void renderStrTest() throws IOException { GroupTemplate groupTemplate = BeetlUtil.createGroupTemplate(new StringTemplateResourceLoader(), Configuration.defaultConfiguration()); Template template = BeetlUtil.getTemplate(groupTemplate, "hello,${name}"); String result = BeetlUtil.render(template, Dict.create().set("name", "hutool")); - Assert.assertEquals("hello,hutool", result); - } - @Test - public void renderFromStrTest() { + Assert.assertEquals("hello,hutool", result); + String renderFromStr = BeetlUtil.renderFromStr("hello,${name}", Dict.create().set("name", "hutool")); Assert.assertEquals("hello,hutool", renderFromStr); + } }