You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@avalon.apache.org by bl...@apache.org on 2003/03/28 20:17:32 UTC
cvs commit: avalon/src/test/org/apache/avalon/framework/configuration/test DefaultConfigurationBuilderTestCase.java
bloritsch 2003/03/28 11:17:32
Modified: src/java/org/apache/avalon/framework/configuration
ConfigurationUtil.java
src/test/org/apache/avalon/framework/configuration/test
DefaultConfigurationBuilderTestCase.java
Log:
remove assignments to dummy variables and unneeded imports
Revision Changes Path
1.13 +1 -2 avalon/src/java/org/apache/avalon/framework/configuration/ConfigurationUtil.java
Index: ConfigurationUtil.java
===================================================================
RCS file: /home/cvs/avalon/src/java/org/apache/avalon/framework/configuration/ConfigurationUtil.java,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -r1.12 -r1.13
--- ConfigurationUtil.java 12 Mar 2003 12:06:54 -0000 1.12
+++ ConfigurationUtil.java 28 Mar 2003 19:17:32 -0000 1.13
@@ -64,7 +64,6 @@
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Iterator;
-import org.xml.sax.SAXException;
/**
* This class has a bunch of utility methods to work
1.8 +6 -6 avalon/src/test/org/apache/avalon/framework/configuration/test/DefaultConfigurationBuilderTestCase.java
Index: DefaultConfigurationBuilderTestCase.java
===================================================================
RCS file: /home/cvs/avalon/src/test/org/apache/avalon/framework/configuration/test/DefaultConfigurationBuilderTestCase.java,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -r1.7 -r1.8
--- DefaultConfigurationBuilderTestCase.java 22 Mar 2003 10:59:16 -0000 1.7
+++ DefaultConfigurationBuilderTestCase.java 28 Mar 2003 19:17:32 -0000 1.8
@@ -107,7 +107,7 @@
assertEquals( "config", conf.getName() );
assertEquals( "getNamespace() should default to \"\"", "", conf.getNamespace() );
try {
- String value = conf.getValue();
+ conf.getValue();
fail( "Should throw a ConfigurationException, as this element"+
"contains child elements, not a value" );
} catch ( ConfigurationException e )
@@ -140,7 +140,7 @@
conf.getChild( "foo" ).getChild("bar").getChild("baz").getName()
);
try {
- String value = conf.getChild("nonexistent").getValue();
+ conf.getChild("nonexistent").getValue();
fail( "Auto-created child nodes should not have a value" );
} catch ( ConfigurationException e )
{}
@@ -188,7 +188,7 @@
assertEquals( "conf:config", conf.getName() );
assertEquals( "getNamespace() should default to \"\"", "", conf.getNamespace() );
try {
- String value = conf.getValue();
+ conf.getValue();
fail( "Should throw a ConfigurationException, as this element"+
"contains child elements, not a value" );
} catch ( ConfigurationException e )
@@ -225,7 +225,7 @@
conf.getChild( "foo" ).getChild("bar").getChild("baz").getName()
);
try {
- String value = conf.getChild("nonexistent").getValue();
+ conf.getChild("nonexistent").getValue();
fail( "Auto-created child nodes should not have a value" );
} catch ( ConfigurationException e )
{}
@@ -251,7 +251,7 @@
assertEquals( "config", conf.getName() );
assertEquals( "Namespace not set correctly", "http://conf.com", conf.getNamespace() );
try {
- String value = conf.getValue();
+ conf.getValue();
fail( "Should throw a ConfigurationException, as this element"+
"contains child elements, not a value" );
} catch ( ConfigurationException e )
@@ -288,7 +288,7 @@
conf.getChild( "foo" ).getChild("bar").getChild("baz").getName()
);
try {
- String value = conf.getChild("nonexistent").getValue();
+ conf.getChild("nonexistent").getValue();
fail( "Auto-created child nodes should not have a value" );
} catch ( ConfigurationException e )
{}
---------------------------------------------------------------------
To unsubscribe, e-mail: cvs-unsubscribe@avalon.apache.org
For additional commands, e-mail: cvs-help@avalon.apache.org