Author: norman
Date: Sun Oct 25 09:36:01 2009
New Revision: 829518
URL: http://svn.apache.org/viewvc?rev=829518&view=rev
Log:
Rename ConfigurationConverter to ConfigurationAdapter
Move some more Test to smtpserver-protocol-library
Added:
james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationAdapter.java
- copied, changed from r829396,
james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationConverter.java
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/SpamTrapHandlerTest.java
- copied, changed from r829399,
james/server/trunk/smtpserver-function/src/test/java/org/apache/james/smtpserver/SpamTrapHandlerTest.java
Removed:
james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationConverter.java
james/server/trunk/smtpserver-function/src/test/java/org/apache/james/smtpserver/SpamTrapHandlerTest.java
Modified:
james/server/trunk/pop3server-function/src/main/java/org/apache/james/pop3server/POP3Server.java
james/server/trunk/remotemanager-function/src/main/java/org/apache/james/remotemanager/RemoteManager.java
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/DNSRBLHandlerTest.java
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/ResolvableEhloHeloHandlerTest.java
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServer.java
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServerComposed.java
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/mina/AvalonAsyncSMTPServer.java
Copied:
james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationAdapter.java
(from r829396,
james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationConverter.java)
URL:
http://svn.apache.org/viewvc/james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationAdapter.java?p2=james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationAdapter.java&p1=james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationConverter.java&r1=829396&r2=829518&rev=829518&view=diff
==============================================================================
---
james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationConverter.java
(original)
+++
james/server/trunk/common-util/src/main/java/org/apache/james/util/ConfigurationAdapter.java
Sun Oct 25 09:36:01 2009
@@ -29,11 +29,11 @@
/**
* Bridges commons configuration to Avalon.
*/
-public class ConfigurationConverter extends XMLConfiguration {
+public class ConfigurationAdapter extends XMLConfiguration {
private static final long serialVersionUID = 6920719067623856243L;
- public
ConfigurationConverter(org.apache.avalon.framework.configuration.Configuration
avalonConfig) throws ConfigurationException {
+ public
ConfigurationAdapter(org.apache.avalon.framework.configuration.Configuration
avalonConfig) throws ConfigurationException {
String config = ConfigurationUtil.toString(avalonConfig);
load(new ByteArrayInputStream(config.getBytes()));
Modified:
james/server/trunk/pop3server-function/src/main/java/org/apache/james/pop3server/POP3Server.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/pop3server-function/src/main/java/org/apache/james/pop3server/POP3Server.java?rev=829518&r1=829517&r2=829518&view=diff
==============================================================================
---
james/server/trunk/pop3server-function/src/main/java/org/apache/james/pop3server/POP3Server.java
(original)
+++
james/server/trunk/pop3server-function/src/main/java/org/apache/james/pop3server/POP3Server.java
Sun Oct 25 09:36:01 2009
@@ -34,7 +34,7 @@
import org.apache.james.socket.AbstractProtocolServer;
import org.apache.james.socket.api.ProtocolHandler;
import org.apache.james.socket.shared.ProtocolHandlerChainImpl;
-import org.apache.james.util.ConfigurationConverter;
+import org.apache.james.util.ConfigurationAdapter;
/**
* <p>Accepts POP3 connections on a server socket and dispatches them to
POP3Handlers.</p>
@@ -131,7 +131,7 @@
handlerChain.setLog(new AvalonLogger(getLogger()));
//read from the XML configuration and create and configure each of the
handlers
- ConfigurationConverter jamesConfiguration = new
ConfigurationConverter(handlerConfiguration.getChild("handlerchain"));
+ ConfigurationAdapter jamesConfiguration = new
ConfigurationAdapter(handlerConfiguration.getChild("handlerchain"));
if (jamesConfiguration.getString("@coreHandlersPackage") == null)
jamesConfiguration.addProperty("/ @coreHandlersPackage",
CoreCmdHandlerLoader.class.getName());
handlerChain.configure(jamesConfiguration);
Modified:
james/server/trunk/remotemanager-function/src/main/java/org/apache/james/remotemanager/RemoteManager.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/remotemanager-function/src/main/java/org/apache/james/remotemanager/RemoteManager.java?rev=829518&r1=829517&r2=829518&view=diff
==============================================================================
---
james/server/trunk/remotemanager-function/src/main/java/org/apache/james/remotemanager/RemoteManager.java
(original)
+++
james/server/trunk/remotemanager-function/src/main/java/org/apache/james/remotemanager/RemoteManager.java
Sun Oct 25 09:36:01 2009
@@ -37,7 +37,7 @@
import org.apache.james.socket.AbstractProtocolServer;
import org.apache.james.socket.api.ProtocolHandler;
import org.apache.james.socket.shared.ProtocolHandlerChainImpl;
-import org.apache.james.util.ConfigurationConverter;
+import org.apache.james.util.ConfigurationAdapter;
/**
* Provides a really rude network interface to administer James.
@@ -154,7 +154,7 @@
handlerChain.setLog(new AvalonLogger(getLogger()));
//read from the XML configuration and create and configure each of the
handlers
- ConfigurationConverter jamesConfiguration = new
ConfigurationConverter(handlerConfiguration.getChild("handlerchain"));
+ ConfigurationAdapter jamesConfiguration = new
ConfigurationAdapter(handlerConfiguration.getChild("handlerchain"));
if (jamesConfiguration.getString("@coreHandlersPackage") == null)
jamesConfiguration.addProperty("/ @coreHandlersPackage",
CoreCmdHandlerLoader.class.getName());
handlerChain.configure(jamesConfiguration);
Modified:
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/DNSRBLHandlerTest.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/DNSRBLHandlerTest.java?rev=829518&r1=829517&r2=829518&view=diff
==============================================================================
---
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/DNSRBLHandlerTest.java
(original)
+++
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/DNSRBLHandlerTest.java
Sun Oct 25 09:36:01 2009
@@ -121,14 +121,14 @@
*/
private void setupMockedSMTPSession(final MailAddress rcpt) {
mockedSMTPSession = new BaseFakeSMTPSession() {
- HashMap state = new HashMap();
- HashMap connectionState = new HashMap();
+ HashMap<String,Object> state = new HashMap<String,Object>();
+ HashMap<String,Object> connectionState = new
HashMap<String,Object>();
public String getRemoteIPAddress() {
return remoteIp;
}
- public Map getState() {
+ public Map<String,Object> getState() {
return state;
}
@@ -144,7 +144,7 @@
return 0;
}
- public Map getConnectionState() {
+ public Map<String,Object> getConnectionState() {
return connectionState;
}
@@ -249,207 +249,170 @@
rbl.configure(new Configuration() {
public Configuration subset(String prefix) {
- // TODO Auto-generated method stub
return null;
}
public void setProperty(String key, Object
value) {
- // TODO Auto-generated method stub
}
public boolean isEmpty() {
- // TODO Auto-generated method stub
return true;
}
public String[] getStringArray(String key) {
- // TODO Auto-generated method stub
return null;
}
public String getString(String key, String
defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public String getString(String key) {
- // TODO Auto-generated method stub
return null;
}
public Short getShort(String key, Short
defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public short getShort(String key, short
defaultValue) {
- // TODO Auto-generated method stub
return 0;
}
public short getShort(String key) {
- // TODO Auto-generated method stub
return 0;
}
public Object getProperty(String key) {
- // TODO Auto-generated method stub
return null;
}
public Properties getProperties(String key) {
- // TODO Auto-generated method stub
return null;
}
public Long getLong(String key, Long
defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public long getLong(String key, long
defaultValue) {
- // TODO Auto-generated method stub
return 0;
}
public long getLong(String key) {
- // TODO Auto-generated method stub
return 0;
}
- public List getList(String key, List
defaultValue) {
- // TODO Auto-generated method stub
+ @SuppressWarnings("unchecked")
+ public List getList(String key, List defaultValue) {
return null;
}
- public List getList(String key) {
- // TODO Auto-generated method stub
+ @SuppressWarnings("unchecked")
+ public List getList(String key) {
return null;
}
- public Iterator getKeys(String prefix) {
- // TODO Auto-generated method stub
+ @SuppressWarnings("unchecked")
+ public Iterator getKeys(String prefix) {
return null;
}
- public Iterator getKeys() {
- // TODO Auto-generated method stub
+ @SuppressWarnings("unchecked")
+ public Iterator getKeys() {
return null;
}
public Integer getInteger(String key, Integer
defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public int getInt(String key, int defaultValue)
{
- // TODO Auto-generated method stub
return 0;
}
public int getInt(String key) {
- // TODO Auto-generated method stub
return 0;
}
public Float getFloat(String key, Float
defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public float getFloat(String key, float
defaultValue) {
- // TODO Auto-generated method stub
return 0;
}
public float getFloat(String key) {
- // TODO Auto-generated method stub
return 0;
}
public Double getDouble(String key, Double
defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public double getDouble(String key, double
defaultValue) {
- // TODO Auto-generated method stub
return 0;
}
public double getDouble(String key) {
- // TODO Auto-generated method stub
return 0;
}
public Byte getByte(String key, Byte
defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public byte getByte(String key, byte
defaultValue) {
- // TODO Auto-generated method stub
return 0;
}
public byte getByte(String key) {
- // TODO Auto-generated method stub
return 0;
}
public Boolean getBoolean(String key, Boolean
defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public boolean getBoolean(String key, boolean
defaultValue) {
- // TODO Auto-generated method stub
return false;
}
public boolean getBoolean(String key) {
- // TODO Auto-generated method stub
return false;
}
public BigInteger getBigInteger(String key,
BigInteger defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public BigInteger getBigInteger(String key) {
- // TODO Auto-generated method stub
return null;
}
public BigDecimal getBigDecimal(String key,
BigDecimal defaultValue) {
- // TODO Auto-generated method stub
return null;
}
public BigDecimal getBigDecimal(String key) {
- // TODO Auto-generated method stub
return null;
}
public boolean containsKey(String key) {
- // TODO Auto-generated method stub
return false;
}
public void clearProperty(String key) {
- // TODO Auto-generated method stub
}
public void clear() {
- // TODO Auto-generated method stub
}
public void addProperty(String key, Object
value) {
- // TODO Auto-generated method stub
}
});
Modified:
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/ResolvableEhloHeloHandlerTest.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/ResolvableEhloHeloHandlerTest.java?rev=829518&r1=829517&r2=829518&view=diff
==============================================================================
---
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/ResolvableEhloHeloHandlerTest.java
(original)
+++
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/ResolvableEhloHeloHandlerTest.java
Sun Oct 25 09:36:01 2009
@@ -50,8 +50,8 @@
SMTPSession session = new BaseFakeSMTPSession() {
- HashMap connectionMap = new HashMap();
- HashMap map = new HashMap();
+ HashMap<String,Object> connectionMap = new
HashMap<String,Object>();
+ HashMap<String,Object> map = new HashMap<String,Object>();
public boolean isAuthSupported() {
return authRequired;
@@ -61,7 +61,7 @@
return user;
}
- public Map getConnectionState() {
+ public Map<String,Object> getConnectionState() {
return connectionMap;
}
@@ -69,7 +69,7 @@
return relaying;
}
- public Map getState() {
+ public Map<String,Object> getState() {
return map;
}
Copied:
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/SpamTrapHandlerTest.java
(from r829399,
james/server/trunk/smtpserver-function/src/test/java/org/apache/james/smtpserver/SpamTrapHandlerTest.java)
URL:
http://svn.apache.org/viewvc/james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/SpamTrapHandlerTest.java?p2=james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/SpamTrapHandlerTest.java&p1=james/server/trunk/smtpserver-function/src/test/java/org/apache/james/smtpserver/SpamTrapHandlerTest.java&r1=829399&r2=829518&rev=829518&view=diff
==============================================================================
---
james/server/trunk/smtpserver-function/src/test/java/org/apache/james/smtpserver/SpamTrapHandlerTest.java
(original)
+++
james/server/trunk/smtp-protocol-library/src/test/java/org/apache/james/smtpserver/protocol/core/fastfail/SpamTrapHandlerTest.java
Sun Oct 25 09:36:01 2009
@@ -18,18 +18,16 @@
****************************************************************/
-package org.apache.james.smtpserver;
+package org.apache.james.smtpserver.protocol.core.fastfail;
import java.util.ArrayList;
import javax.mail.internet.ParseException;
-import org.apache.avalon.framework.container.ContainerUtil;
import org.apache.james.smtpserver.protocol.BaseFakeSMTPSession;
import org.apache.james.smtpserver.protocol.SMTPSession;
import org.apache.james.smtpserver.protocol.core.fastfail.SpamTrapHandler;
import org.apache.james.smtpserver.protocol.hook.HookReturnCode;
-import org.apache.james.test.mock.avalon.MockLogger;
import org.apache.mailet.MailAddress;
import junit.framework.TestCase;
@@ -52,11 +50,10 @@
String ip2 = "192.168.100.2";
long blockTime = 2000;
- ArrayList rcpts = new ArrayList();
+ ArrayList<String> rcpts = new ArrayList<String>();
rcpts.add(SPAM_TRAP_RECIP1);
SpamTrapHandler handler = new SpamTrapHandler();
- ContainerUtil.enableLogging(handler, new MockLogger());
handler.setBlockTime(blockTime);
handler.setSpamTrapRecipients(rcpts);
Modified:
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServer.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServer.java?rev=829518&r1=829517&r2=829518&view=diff
==============================================================================
---
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServer.java
(original)
+++
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServer.java
Sun Oct 25 09:36:01 2009
@@ -38,7 +38,7 @@
import org.apache.james.socket.AbstractProtocolServer;
import org.apache.james.socket.api.ProtocolHandler;
import org.apache.james.socket.shared.ProtocolHandlerChainImpl;
-import org.apache.james.util.ConfigurationConverter;
+import org.apache.james.util.ConfigurationAdapter;
import org.apache.mailet.MailetContext;
/**
@@ -243,7 +243,7 @@
handlerChain.setLog(new AvalonLogger(getLogger()));
//read from the XML configuration and create and configure each of the
handlers
- ConfigurationConverter jamesConfiguration = new
ConfigurationConverter(handlerConfiguration.getChild("handlerchain"));
+ ConfigurationAdapter jamesConfiguration = new
ConfigurationAdapter(handlerConfiguration.getChild("handlerchain"));
if (jamesConfiguration.getString("@coreHandlersPackage") == null)
jamesConfiguration.addProperty("/ @coreHandlersPackage",
CoreCmdHandlerLoader.class.getName());
handlerChain.configure(jamesConfiguration);
Modified:
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServerComposed.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServerComposed.java?rev=829518&r1=829517&r2=829518&view=diff
==============================================================================
---
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServerComposed.java
(original)
+++
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/SMTPServerComposed.java
Sun Oct 25 09:36:01 2009
@@ -45,7 +45,7 @@
import org.apache.james.socket.api.ProtocolHandlerFactory;
import org.apache.james.socket.api.ProtocolServer;
import org.apache.james.socket.shared.ProtocolHandlerChainImpl;
-import org.apache.james.util.ConfigurationConverter;
+import org.apache.james.util.ConfigurationAdapter;
import org.apache.mailet.MailetContext;
/**
@@ -269,7 +269,7 @@
handlerChain.setLog(new AvalonLogger(getLogger()));
//read from the XML configuration and create and configure each of the
handlers
- ConfigurationConverter jamesConfiguration = new
ConfigurationConverter(handlerConfiguration.getChild("handlerchain"));
+ ConfigurationAdapter jamesConfiguration = new
ConfigurationAdapter(handlerConfiguration.getChild("handlerchain"));
if (jamesConfiguration.getString("@coreHandlersPackage") == null)
jamesConfiguration.addProperty("/ @coreHandlersPackage",
CoreCmdHandlerLoader.class.getName());
handlerChain.configure(jamesConfiguration);
Modified:
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/mina/AvalonAsyncSMTPServer.java
URL:
http://svn.apache.org/viewvc/james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/mina/AvalonAsyncSMTPServer.java?rev=829518&r1=829517&r2=829518&view=diff
==============================================================================
---
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/mina/AvalonAsyncSMTPServer.java
(original)
+++
james/server/trunk/smtpserver-function/src/main/java/org/apache/james/smtpserver/mina/AvalonAsyncSMTPServer.java
Sun Oct 25 09:36:01 2009
@@ -37,7 +37,7 @@
import org.apache.james.services.FileSystem;
import org.apache.james.services.MailServer;
import org.apache.james.smtpserver.protocol.SMTPServerMBean;
-import org.apache.james.util.ConfigurationConverter;
+import org.apache.james.util.ConfigurationAdapter;
import org.apache.mailet.MailetContext;
import org.guiceyfruit.jsr250.Jsr250Module;
@@ -73,7 +73,7 @@
*/
public void configure(Configuration config) throws ConfigurationException {
try {
- this.config = new ConfigurationConverter(config);
+ this.config = new ConfigurationAdapter(config);
} catch (org.apache.commons.configuration.ConfigurationException e) {
throw new ConfigurationException("Unable to convert
configuration", e);
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]