Author: simonetripodi
Date: Tue Feb 15 16:48:40 2011
New Revision: 1070960
URL: http://svn.apache.org/viewvc?rev=1070960&view=rev
Log:
first checkin of IncludeTestCase, an adapted version of the proper one on /trunk
old test continue to pass, it was needed adding the PrefixedRulesBinder to
maintain the pattern specified in the XML document
Added:
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
(with props)
commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/
commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTestCase.java
(with props)
Modified:
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/IncludeRule.java
Modified:
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/IncludeRule.java
URL:
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/IncludeRule.java?rev=1070960&r1=1070959&r2=1070960&view=diff
==============================================================================
---
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/IncludeRule.java
(original)
+++
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/IncludeRule.java
Tue Feb 15 16:48:40 2011
@@ -55,7 +55,8 @@ final class IncludeRule extends Rule {
if (!this.memoryRulesBinder.getIncludedFiles().add(fileName)) {
throw new DigesterLoadingException("Circular file inclusion
detected for file: " + fileName);
}
- this.memoryRulesBinder.install(new FromXmlRulesModule(fileName));
+ // this.memoryRulesBinder.install(new
FromXmlRulesModule(fileName));
+ this.install(new FromXmlRulesModule(fileName));
}
// The class attribute gives the name of a class that implements
@@ -73,7 +74,7 @@ final class IncludeRule extends Rule {
RulesModule rulesSource = (RulesModule) cls.newInstance();
- this.targetRulesBinder.install(rulesSource);
+ this.install(rulesSource);
} catch (Exception e) {
this.targetRulesBinder.addError("Impossible to include
programmatic rules from class '%s': %s",
className,
@@ -82,4 +83,10 @@ final class IncludeRule extends Rule {
}
}
+ private void install(RulesModule rulesModule) {
+ // that's an hack, shall not be taken in consideration!!! :)
+ rulesModule.configure(
+ new PrefixedRulesBinder(this.targetRulesBinder,
this.memoryRulesBinder.getPatternStack().toString()));
+ }
+
}
Added:
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
URL:
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java?rev=1070960&view=auto
==============================================================================
---
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
(added)
+++
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
Tue Feb 15 16:48:40 2011
@@ -0,0 +1,57 @@
+package org.apache.commons.digester3.xmlrules.metaparser;
+
+import org.apache.commons.digester3.RulesBinder;
+import org.apache.commons.digester3.RulesModule;
+import org.apache.commons.digester3.rulesbinder.ConverterBuilder;
+import org.apache.commons.digester3.rulesbinder.LinkedRuleBuilder;
+
+final class PrefixedRulesBinder implements RulesBinder {
+
+ private final RulesBinder wrappedRulesBinder;
+
+ private final String prefix;
+
+ public PrefixedRulesBinder(RulesBinder wrappedRulesBinder, String prefix) {
+ this.wrappedRulesBinder = wrappedRulesBinder;
+ this.prefix = prefix;
+ }
+
+ /**
+ * {@inheritDoc}
+ */
+ public void addError(String messagePattern, Object... arguments) {
+ this.wrappedRulesBinder.addError(messagePattern, arguments);
+ }
+
+ /**
+ * {@inheritDoc}
+ */
+ public void addError(Throwable t) {
+ this.wrappedRulesBinder.addError(t);
+ }
+
+ /**
+ * {@inheritDoc}
+ */
+ public void install(RulesModule rulesModule) {
+ this.wrappedRulesBinder.install(rulesModule);
+ }
+
+ /**
+ * {@inheritDoc}
+ */
+ public LinkedRuleBuilder forPattern(String pattern) {
+ if (this.prefix != null && this.prefix.length() > 0) {
+ pattern = this.prefix + '/' + pattern;
+ }
+ return this.wrappedRulesBinder.forPattern(pattern);
+ }
+
+ /**
+ * {@inheritDoc}
+ */
+ public <T> ConverterBuilder<T> convert(Class<T> type) {
+ return this.wrappedRulesBinder.convert(type);
+ }
+
+}
Propchange:
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
------------------------------------------------------------------------------
svn:eol-style = native
Propchange:
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
------------------------------------------------------------------------------
svn:keywords = Date Author Id Revision HeadURL
Propchange:
commons/sandbox/digester3/trunk/src/main/java/org/apache/commons/digester3/xmlrules/metaparser/PrefixedRulesBinder.java
------------------------------------------------------------------------------
svn:mime-type = text/plain
Added:
commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTestCase.java
URL:
http://svn.apache.org/viewvc/commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTestCase.java?rev=1070960&view=auto
==============================================================================
---
commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTestCase.java
(added)
+++
commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTestCase.java
Tue Feb 15 16:48:40 2011
@@ -0,0 +1,73 @@
+/* $Id$
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.commons.digester3.xmlrules;
+
+import static org.junit.Assert.*;
+
+import java.io.StringReader;
+import java.util.ArrayList;
+import java.util.List;
+
+import org.apache.commons.digester3.AbstractRulesModule;
+import org.apache.commons.digester3.AbstractTestCase;
+import org.apache.commons.digester3.Digester;
+import org.apache.commons.digester3.Rule;
+import org.junit.Test;
+
+/**
+ *
+ */
+public final class IncludeTestCase extends AbstractTestCase {
+
+ public static class TestDigesterRulesModule extends AbstractRulesModule {
+
+ @Override
+ protected void configure() {
+ forPattern("bar").addRule(new Rule() {
+
+ @Override
+ public void body(String namespace, String name, String text)
throws Exception {
+ ((ArrayList<String>) this.getDigester().peek()).add(text);
+ }
+
+ });
+ }
+
+ }
+
+ @Test
+ public void testBasicInclude() throws Exception {
+ String rulesXml = "<?xml version='1.0'?>"
+ + "<digester-rules>"
+ + " <pattern value='root/foo'>"
+ + " <include
class='org.apache.commons.digester3.xmlrules.IncludeTestCase$TestDigesterRulesModule'/>"
+ + " </pattern>"
+ + "</digester-rules>";
+
+ String xml = "<?xml version='1.0'
?><root><foo><bar>short</bar></foo></root>";
+
+ List<String> list = new ArrayList<String>();
+ Digester digester = newBasicDigester(new FromXmlRulesModule(new
StringReader(rulesXml)));
+ digester.push(list);
+ digester.parse(new StringReader(xml));
+
+ assertEquals("Number of entries", 1, list.size());
+ assertEquals("Entry value", "short", list.get(0));
+ }
+
+}
Propchange:
commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTestCase.java
------------------------------------------------------------------------------
svn:eol-style = native
Propchange:
commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTestCase.java
------------------------------------------------------------------------------
svn:keywords = Date Author Id Revision HeadURL
Propchange:
commons/sandbox/digester3/trunk/src/test/java/org/apache/commons/digester3/xmlrules/IncludeTestCase.java
------------------------------------------------------------------------------
svn:mime-type = text/plain