You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by ad...@apache.org on 2016/06/17 16:34:51 UTC

[16/21] wicket git commit: WICKET-6173 WICKET-6172 makes TagTester.createTagsByAttribute stop working

WICKET-6173 WICKET-6172 makes TagTester.createTagsByAttribute stop
working

Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/86009a44
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/86009a44
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/86009a44

Branch: refs/heads/WICKET-6183
Commit: 86009a4436cf76b76a12e3c7f4dcb886c2041cf0
Parents: 37906ba
Author: Andrea Del Bene <ad...@apache.org>
Authored: Tue May 31 22:36:59 2016 +0200
Committer: Andrea Del Bene <ad...@apache.org>
Committed: Thu Jun 9 11:17:06 2016 +0200

----------------------------------------------------------------------
 .../java/org/apache/wicket/util/tester/TagTester.java |  3 ++-
 .../org/apache/wicket/util/tester/TagTesterTest.java  | 14 ++++++++++++--
 2 files changed, 14 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/86009a44/wicket-core/src/main/java/org/apache/wicket/util/tester/TagTester.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/util/tester/TagTester.java b/wicket-core/src/main/java/org/apache/wicket/util/tester/TagTester.java
index e2cc83c..c6818f6 100644
--- a/wicket-core/src/main/java/org/apache/wicket/util/tester/TagTester.java
+++ b/wicket-core/src/main/java/org/apache/wicket/util/tester/TagTester.java
@@ -24,6 +24,7 @@ import java.util.regex.Pattern;
 import org.apache.wicket.WicketRuntimeException;
 import org.apache.wicket.markup.parser.XmlPullParser;
 import org.apache.wicket.markup.parser.XmlTag;
+import org.apache.wicket.markup.parser.filter.HtmlHandler;
 import org.apache.wicket.util.lang.Args;
 import org.apache.wicket.util.string.Strings;
 import org.apache.wicket.util.value.IValueMap;
@@ -671,7 +672,7 @@ public class TagTester
 						openTag = null;
 						closeTag = null;
 					}
-					else if (openTag != null)
+					else if (openTag != null && !HtmlHandler.requiresCloseTag(openTag.getName()))
 					{
 						TagTester tester = new TagTester(parser, openTag, openTag);
 						testers.add(tester);

http://git-wip-us.apache.org/repos/asf/wicket/blob/86009a44/wicket-core/src/test/java/org/apache/wicket/util/tester/TagTesterTest.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/test/java/org/apache/wicket/util/tester/TagTesterTest.java b/wicket-core/src/test/java/org/apache/wicket/util/tester/TagTesterTest.java
index 9a60a88..e7863a8 100644
--- a/wicket-core/src/test/java/org/apache/wicket/util/tester/TagTesterTest.java
+++ b/wicket-core/src/test/java/org/apache/wicket/util/tester/TagTesterTest.java
@@ -21,11 +21,11 @@ import static org.hamcrest.Matchers.equalTo;
 import static org.hamcrest.Matchers.is;
 import static org.hamcrest.Matchers.notNullValue;
 
+import java.util.List;
+
 import org.junit.Assert;
 import org.junit.Test;
 
-import java.util.List;
-
 /**
  * Test of TagTester
  */
@@ -330,4 +330,14 @@ public class TagTesterTest extends Assert
 		TagTester tagTester2 = TagTester.createTagByAttribute(MARKUP_1, "id", "test");
 		assertEquals(tagTester.getMarkup(), tagTester2.getValue());
 	}
+	
+	/**
+	 * https://issues.apache.org/jira/browse/WICKET-6173
+	 */
+	@Test
+	public void valueFromTagsByAttribute()
+	{
+		TagTester tagTester = TagTester.createTagsByAttribute(MARKUP_1, "id", "test2");
+		assertEquals("mock", tagTester.getValue());
+	}
 }