You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2004/11/01 21:55:36 UTC

svn commit: rev 56280 - in geronimo/trunk/modules: assembly/src/plan connector/src/java/org/apache/geronimo/connector/deployment connector/src/schema connector/src/test-data/connector_1_0 connector/src/test-data/connector_1_5 connector/src/test-data/data connector/src/test/org/apache/geronimo/connector/deployment

Author: djencks
Date: Mon Nov  1 12:55:35 2004
New Revision: 56280

Modified:
   geronimo/trunk/modules/assembly/src/plan/j2ee-deployer-plan.xml
   geronimo/trunk/modules/assembly/src/plan/j2ee-server-plan.xml
   geronimo/trunk/modules/connector/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java
   geronimo/trunk/modules/connector/src/schema/geronimo-connector_1_5.xsd
   geronimo/trunk/modules/connector/src/test-data/connector_1_0/ra.xml
   geronimo/trunk/modules/connector/src/test-data/connector_1_5/ra.xml
   geronimo/trunk/modules/connector/src/test-data/data/external-application-plan.xml
   geronimo/trunk/modules/connector/src/test/org/apache/geronimo/connector/deployment/RAR_1_0ConfigBuilderTest.java
   geronimo/trunk/modules/connector/src/test/org/apache/geronimo/connector/deployment/RAR_1_5ConfigBuilderTest.java
Log:
GERONIMO-413. Transaction support is now defaulted from ra.xml if not in plan.  XA default config set in ConnectorModuleBuilder gbean attributes

Modified: geronimo/trunk/modules/assembly/src/plan/j2ee-deployer-plan.xml
==============================================================================
--- geronimo/trunk/modules/assembly/src/plan/j2ee-deployer-plan.xml	(original)
+++ geronimo/trunk/modules/assembly/src/plan/j2ee-deployer-plan.xml	Mon Nov  1 12:55:35 2004
@@ -114,6 +114,8 @@
         <attribute name="defaultMinSize" type="int">0</attribute>
         <attribute name="defaultBlockingTimeoutMilliseconds" type="int">5000</attribute>
         <attribute name="defaultIdleTimeoutMinutes" type="int">15</attribute>
+        <attribute name="defaultXATransactionCaching" type="boolean">true</attribute>
+        <attribute name="defaultXAThreadCaching" type="boolean">false</attribute>
     </gbean>
 
     <gbean name="geronimo.deployer:role=ModuleBuilder,type=AppClient,config=org/apache/geronimo/J2EEDeployer" class="org.apache.geronimo.client.builder.AppClientModuleBuilder">

Modified: geronimo/trunk/modules/assembly/src/plan/j2ee-server-plan.xml
==============================================================================
--- geronimo/trunk/modules/assembly/src/plan/j2ee-server-plan.xml	(original)
+++ geronimo/trunk/modules/assembly/src/plan/j2ee-server-plan.xml	Mon Nov  1 12:55:35 2004
@@ -273,6 +273,8 @@
         <attribute name="defaultMinSize" type="int">0</attribute>
         <attribute name="defaultBlockingTimeoutMilliseconds" type="int">5000</attribute>
         <attribute name="defaultIdleTimeoutMinutes" type="int">15</attribute>
+        <attribute name="defaultXATransactionCaching" type="boolean">true</attribute>
+        <attribute name="defaultXAThreadCaching" type="boolean">false</attribute>
     </gbean>
 
     <gbean name="geronimo.deployer:role=ModuleBuilder,type=AppClient,config=org/apache/geronimo/Server" class="org.apache.geronimo.client.builder.AppClientModuleBuilder">

Modified: geronimo/trunk/modules/connector/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java
==============================================================================
--- geronimo/trunk/modules/connector/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java	(original)
+++ geronimo/trunk/modules/connector/src/java/org/apache/geronimo/connector/deployment/ConnectorModuleBuilder.java	Mon Nov  1 12:55:35 2004
@@ -112,13 +112,17 @@
     private final int defaultMinSize;
     private final int defaultBlockingTimeoutMilliseconds;
     private final int defaultIdleTimeoutMinutes;
+    private final boolean defaultXATransactionCaching;
+    private final boolean defaultXAThreadCaching;
     private final Kernel kernel;
 
-    public ConnectorModuleBuilder(int defaultMaxSize, int defaultMinSize, int defaultBlockingTimeoutMilliseconds, int defaultIdleTimeoutMinutes, Kernel kernel) {
+    public ConnectorModuleBuilder(int defaultMaxSize, int defaultMinSize, int defaultBlockingTimeoutMilliseconds, int defaultIdleTimeoutMinutes, boolean defaultXATransactionCaching, boolean defaultXAThreadCaching, Kernel kernel) {
         this.defaultMaxSize = defaultMaxSize;
         this.defaultMinSize = defaultMinSize;
         this.defaultBlockingTimeoutMilliseconds = defaultBlockingTimeoutMilliseconds;
         this.defaultIdleTimeoutMinutes = defaultIdleTimeoutMinutes;
+        this.defaultXATransactionCaching = defaultXATransactionCaching;
+        this.defaultXAThreadCaching = defaultXAThreadCaching;
         this.kernel = kernel;
     }
 
@@ -347,6 +351,12 @@
 
     private void addConnectorGBeans(EARContext earContext, J2eeContext j2eeContext, ConnectorType10 connector, GerConnectorType geronimoConnector, ClassLoader cl) throws DeploymentException {
         ResourceadapterType10 resourceAdapter = connector.getResourceadapter();
+        String managedConnectionFactoryClass = resourceAdapter.getManagedconnectionfactoryClass().getStringValue().trim();
+        String connectionFactoryInterface = resourceAdapter.getConnectionfactoryInterface().getStringValue().trim();
+        String connectionFactoryImplClass = resourceAdapter.getConnectionfactoryImplClass().getStringValue().trim();
+        String connectionInterface = resourceAdapter.getConnectionInterface().getStringValue().trim();
+        String connectionImplClass = resourceAdapter.getConnectionImplClass().getStringValue().trim();
+        String transactionSupport = resourceAdapter.getTransactionSupport().getStringValue().trim();
         GerResourceadapterType[] geronimoResourceAdapters = geronimoConnector.getResourceadapterArray();
         for (int k = 0; k < geronimoResourceAdapters.length; k++) {
             GerResourceadapterType geronimoResourceAdapter = geronimoResourceAdapters[k];
@@ -357,14 +367,9 @@
 
                 for (int j = 0; j < geronimoConnectionDefinition.getConnectiondefinitionInstanceArray().length; j++) {
                     GerConnectiondefinitionInstanceType connectionfactoryInstance = geronimoConnectionDefinition.getConnectiondefinitionInstanceArray()[j];
-                    String managedConnectionFactoryClass = resourceAdapter.getManagedconnectionfactoryClass().getStringValue();
-                    String connectionFactoryInterface = resourceAdapter.getConnectionfactoryInterface().getStringValue();
-                    String connectionFactoryImplClass = resourceAdapter.getConnectionfactoryImplClass().getStringValue();
-                    String connectionInterface = resourceAdapter.getConnectionInterface().getStringValue();
-                    String connectionImplClass = resourceAdapter.getConnectionImplClass().getStringValue();
                     ConfigProperty[] configProperties = getConfigProperties(resourceAdapter.getConfigPropertyArray(), connectionfactoryInstance.getConfigPropertySettingArray());
 
-                    addOutboundGBeans(earContext, j2eeContext, null, connectionfactoryInstance, configProperties, managedConnectionFactoryClass, connectionFactoryInterface, connectionFactoryImplClass, connectionInterface, connectionImplClass, cl);
+                    addOutboundGBeans(earContext, j2eeContext, null, connectionfactoryInstance, configProperties, managedConnectionFactoryClass, connectionFactoryInterface, connectionFactoryImplClass, connectionInterface, connectionImplClass, transactionSupport, cl);
                 }
             }
         }
@@ -372,6 +377,7 @@
 
     private void addConnectorGBeans(EARContext earContext, J2eeContext moduleJ2eeContext, ConnectorType connector, GerConnectorType geronimoConnector, ClassLoader cl) throws DeploymentException {
         ResourceadapterType resourceadapter = connector.getResourceadapter();
+        String transactionSupport = resourceadapter.getOutboundResourceadapter().getTransactionSupport().getStringValue().trim();
         GerResourceadapterType[] geronimoResourceAdapters = geronimoConnector.getResourceadapterArray();
         for (int k = 0; k < geronimoResourceAdapters.length; k++) {
             GerResourceadapterType geronimoResourceAdapter = geronimoResourceAdapters[k];
@@ -433,17 +439,17 @@
                         if (connectionDefinition == null) {
                             throw new DeploymentException("No connection definition for ConnectionFactory class: " + connectionFactoryInterfaceName);
                         }
+                        String managedConnectionFactoryClass = connectionDefinition.getManagedconnectionfactoryClass().getStringValue().trim();
+                        String connectionFactoryInterface = connectionDefinition.getConnectionfactoryInterface().getStringValue().trim();
+                        String connectionFactoryImplClass = connectionDefinition.getConnectionfactoryImplClass().getStringValue().trim();
+                        String connectionInterface = connectionDefinition.getConnectionInterface().getStringValue().trim();
+                        String connectionImplClass = connectionDefinition.getConnectionImplClass().getStringValue().trim();
 
                         for (int j = 0; j < geronimoConnectionDefinition.getConnectiondefinitionInstanceArray().length; j++) {
                             GerConnectiondefinitionInstanceType connectionfactoryInstance = geronimoConnectionDefinition.getConnectiondefinitionInstanceArray()[j];
-                            String managedConnectionFactoryClass = connectionDefinition.getManagedconnectionfactoryClass().getStringValue();
-                            String connectionFactoryInterface = connectionDefinition.getConnectionfactoryInterface().getStringValue();
-                            String connectionFactoryImplClass = connectionDefinition.getConnectionfactoryImplClass().getStringValue();
-                            String connectionInterface = connectionDefinition.getConnectionInterface().getStringValue();
-                            String connectionImplClass = connectionDefinition.getConnectionImplClass().getStringValue();
                             ConfigProperty[] configProperties = getConfigProperties(connectionDefinition.getConfigPropertyArray(), connectionfactoryInstance.getConfigPropertySettingArray());
 
-                            addOutboundGBeans(earContext, moduleJ2eeContext, resourceAdapterObjectName, connectionfactoryInstance, configProperties, managedConnectionFactoryClass, connectionFactoryInterface, connectionFactoryImplClass, connectionInterface, connectionImplClass, cl);
+                            addOutboundGBeans(earContext, moduleJ2eeContext, resourceAdapterObjectName, connectionfactoryInstance, configProperties, managedConnectionFactoryClass, connectionFactoryInterface, connectionFactoryImplClass, connectionInterface, connectionImplClass, transactionSupport, cl);
                         }
                     }
                 }
@@ -639,7 +645,7 @@
         gbean.setAttribute(name, value);
     }
 
-    private ObjectName configureConnectionManager(EARContext earContext, J2eeContext j2eeContext, GerConnectiondefinitionInstanceType connectionfactoryInstance, ClassLoader cl) throws DeploymentException {
+    private ObjectName configureConnectionManager(EARContext earContext, J2eeContext j2eeContext, String ddTransactionSupport, GerConnectiondefinitionInstanceType connectionfactoryInstance, ClassLoader cl) throws DeploymentException {
         if (connectionfactoryInstance.getConnectionmanagerRef() != null) {
             //we don't configure anything, just use the supplied gbean
             try {
@@ -667,15 +673,21 @@
         } else if (connectionManager.isSetXaTransaction()) {
             transactionSupport = new XATransactions(connectionManager.getXaTransaction().isSetTransactionCaching(),
                     connectionManager.getXaTransaction().isSetThreadCaching());
+        } else if ("NoTransaction".equals(ddTransactionSupport)) {
+            transactionSupport = NoTransactions.INSTANCE;
+        } else if ("LocalTransaction".equals(ddTransactionSupport)) {
+            transactionSupport = LocalTransactions.INSTANCE;
+        } else if ("XATransaction".equals(ddTransactionSupport)) {
+            transactionSupport = new XATransactions(defaultXATransactionCaching, defaultXAThreadCaching);
         } else {
+            //this should not happen
             throw new DeploymentException("Unexpected transaction support element");
         }
         PoolingSupport pooling = null;
         if (connectionManager.getSinglePool() != null) {
             GerSinglepoolType pool = connectionManager.getSinglePool();
 
-            pooling = new SinglePool(
-                    pool.isSetMaxSize() ? pool.getMaxSize() : defaultMaxSize,
+            pooling = new SinglePool(pool.isSetMaxSize() ? pool.getMaxSize() : defaultMaxSize,
                     pool.isSetMinSize() ? pool.getMinSize() : defaultMinSize,
                     pool.isSetBlockingTimeoutMilliseconds() ? pool.getBlockingTimeoutMilliseconds() : defaultBlockingTimeoutMilliseconds,
                     pool.isSetIdleTimeoutMinutes() ? pool.getIdleTimeoutMinutes() : defaultIdleTimeoutMinutes,
@@ -684,8 +696,7 @@
                     pool.getSelectOneAssumeMatch() != null);
         } else if (connectionManager.getPartitionedPool() != null) {
             GerPartitionedpoolType pool = connectionManager.getPartitionedPool();
-            pooling = new PartitionedPool(
-                    pool.isSetMaxSize() ? pool.getMaxSize() : defaultMaxSize,
+            pooling = new PartitionedPool(pool.isSetMaxSize() ? pool.getMaxSize() : defaultMaxSize,
                     pool.isSetMinSize() ? pool.getMinSize() : defaultMinSize,
                     pool.isSetBlockingTimeoutMilliseconds() ? pool.getBlockingTimeoutMilliseconds() : defaultBlockingTimeoutMilliseconds,
                     pool.isSetIdleTimeoutMinutes() ? pool.getIdleTimeoutMinutes() : defaultIdleTimeoutMinutes,
@@ -718,9 +729,9 @@
         return connectionManagerObjectName;
     }
 
-    private void addOutboundGBeans(EARContext earContext, J2eeContext j2eeContext, ObjectName resourceAdapterObjectName, GerConnectiondefinitionInstanceType connectiondefinitionInstance, ConfigProperty[] configProperties, String managedConnectionFactoryClass, String connectionFactoryInterface, String connectionFactoryImplClass, String connectionInterface, String connectionImplClass, ClassLoader cl) throws DeploymentException {
+    private void addOutboundGBeans(EARContext earContext, J2eeContext j2eeContext, ObjectName resourceAdapterObjectName, GerConnectiondefinitionInstanceType connectiondefinitionInstance, ConfigProperty[] configProperties, String managedConnectionFactoryClass, String connectionFactoryInterface, String connectionFactoryImplClass, String connectionInterface, String connectionImplClass, String transactionSupport, ClassLoader cl) throws DeploymentException {
         // ConnectionManager
-        ObjectName connectionManagerObjectName = configureConnectionManager(earContext, j2eeContext, connectiondefinitionInstance, cl);
+        ObjectName connectionManagerObjectName = configureConnectionManager(earContext, j2eeContext, transactionSupport, connectiondefinitionInstance, cl);
 
         // ManagedConnectionFactory
         GBeanInfoBuilder managedConnectionFactoryInfoFactory = new GBeanInfoBuilder("org.apache.geronimo.connector.outbound.ManagedConnectionFactoryWrapper", cl);
@@ -858,13 +869,15 @@
         infoBuilder.addAttribute("defaultMinSize", int.class, true);
         infoBuilder.addAttribute("defaultBlockingTimeoutMilliseconds", int.class, true);
         infoBuilder.addAttribute("defaultIdleTimeoutMinutes", int.class, true);
+        infoBuilder.addAttribute("defaultXATransactionCaching", boolean.class, true);
+        infoBuilder.addAttribute("defaultXAThreadCaching", boolean.class, true);
 
         infoBuilder.addAttribute("kernel", Kernel.class, false);
 
         infoBuilder.addInterface(ModuleBuilder.class);
         infoBuilder.addInterface(ResourceReferenceBuilder.class);
 
-        infoBuilder.setConstructor(new String[]{"defaultMaxSize", "defaultMinSize", "defaultBlockingTimeoutMilliseconds", "defaultIdleTimeoutMinutes", "kernel"});
+        infoBuilder.setConstructor(new String[]{"defaultMaxSize", "defaultMinSize", "defaultBlockingTimeoutMilliseconds", "defaultIdleTimeoutMinutes", "defaultXATransactionCaching", "defaultXAThreadCaching", "kernel"});
         GBEAN_INFO = infoBuilder.getBeanInfo();
     }
 

Modified: geronimo/trunk/modules/connector/src/schema/geronimo-connector_1_5.xsd
==============================================================================
--- geronimo/trunk/modules/connector/src/schema/geronimo-connector_1_5.xsd	(original)
+++ geronimo/trunk/modules/connector/src/schema/geronimo-connector_1_5.xsd	Mon Nov  1 12:55:35 2004
@@ -264,7 +264,7 @@
             <xsd:element name="realm-bridge"
                 type="xsd:string"
                 minOccurs="0"/>
-            <xsd:choice>
+            <xsd:choice minOccurs="0">
                 <xsd:element name="no-transaction"/>
                 <xsd:element name="local-transaction"/>
                 <xsd:element name="xa-transaction" type="ger:xatransaction-Type"/>

Modified: geronimo/trunk/modules/connector/src/test-data/connector_1_0/ra.xml
==============================================================================
--- geronimo/trunk/modules/connector/src/test-data/connector_1_0/ra.xml	(original)
+++ geronimo/trunk/modules/connector/src/test-data/connector_1_0/ra.xml	Mon Nov  1 12:55:35 2004
@@ -33,7 +33,7 @@
         <connectionfactory-impl-class>org.apache.geronimo.connector.mock.MockConnectionFactory</connectionfactory-impl-class>
         <connection-interface>javax.resource.cci.Connection</connection-interface>
         <connection-impl-class>org.apache.geronimo.connector.mock.MockConnection</connection-impl-class>
-        <transaction-support></transaction-support>
+        <transaction-support>XATransaction</transaction-support>
         <config-property>
             <config-property-name>OutboundStringProperty1</config-property-name>
             <config-property-type>java.lang.String</config-property-type>

Modified: geronimo/trunk/modules/connector/src/test-data/connector_1_5/ra.xml
==============================================================================
--- geronimo/trunk/modules/connector/src/test-data/connector_1_5/ra.xml	(original)
+++ geronimo/trunk/modules/connector/src/test-data/connector_1_5/ra.xml	Mon Nov  1 12:55:35 2004
@@ -75,7 +75,7 @@
                 <connection-impl-class>org.apache.geronimo.connector.mock.MockConnection</connection-impl-class>
 
             </connection-definition>
-            <transaction-support>LocalTransaction</transaction-support>
+            <transaction-support>XATransaction</transaction-support>
             <authentication-mechanism>
                 <description>description</description>
                 <authentication-mechanism-type>BasicPassword</authentication-mechanism-type>

Modified: geronimo/trunk/modules/connector/src/test-data/data/external-application-plan.xml
==============================================================================
--- geronimo/trunk/modules/connector/src/test-data/data/external-application-plan.xml	(original)
+++ geronimo/trunk/modules/connector/src/test-data/data/external-application-plan.xml	Mon Nov  1 12:55:35 2004
@@ -65,17 +65,13 @@
                                 </single-pool>
                             </connectionmanager>
                         </connectiondefinition-instance>
+                        <!--test defaults for pool and tx-->
                         <connectiondefinition-instance>
                             <name>ThirdTestOutboundConnectionFactory</name>
                             <config-property-setting name="OutboundStringProperty">StringValue3</config-property-setting>
                             <connectionmanager>
                                 <realm-bridge>TargetRealm</realm-bridge>
-                                <xa-transaction>
-                                    <transaction-caching/>
-                                </xa-transaction>
                                 <partitioned-pool>
-                                    <max-size>10</max-size>
-                                    <blocking-timeout-milliseconds>5000</blocking-timeout-milliseconds>
                                     <match-one/>
                                     <partition-by-subject/>
                                 </partitioned-pool>
@@ -139,6 +135,7 @@
                             </connectionmanager>
                         </connectiondefinition-instance>
                     </connection-definition>
+                    <!--test defaults for pool and tx-->
                     <connection-definition>
                         <connectionfactory-interface>org.apache.geronimo.connector.mock.ConnectionFactoryExtension</connectionfactory-interface>
                         <connectiondefinition-instance>
@@ -146,12 +143,7 @@
                             <config-property-setting name="OutboundStringProperty1">StringValue3</config-property-setting>
                             <connectionmanager>
                                 <realm-bridge>TargetRealm</realm-bridge>
-                                <xa-transaction>
-                                    <transaction-caching/>
-                                </xa-transaction>
                                 <partitioned-pool>
-                                    <max-size>10</max-size>
-                                    <blocking-timeout-milliseconds>5000</blocking-timeout-milliseconds>
                                     <match-one/>
                                     <partition-by-subject/>
                                 </partitioned-pool>
@@ -162,18 +154,18 @@
             </resourceadapter>
 
             <adminobject>
-                 <adminobject-interface>org.apache.geronimo.connector.mock.MockAdminObject</adminobject-interface>
-                 <adminobject-class>org.apache.geronimo.connector.mock.MockAdminObjectImpl</adminobject-class>
-                 <adminobject-instance>
-                     <message-destination-name>tweedledee</message-destination-name>
-                     <config-property-setting name="Tweedle">Dee-value</config-property-setting>
-                 </adminobject-instance>
-                 <adminobject-instance>
-                     <message-destination-name>tweedledum</message-destination-name>
-                     <config-property-setting name="Tweedle">Dum-value</config-property-setting>
-                 </adminobject-instance>
-             </adminobject>
-             
+                <adminobject-interface>org.apache.geronimo.connector.mock.MockAdminObject</adminobject-interface>
+                <adminobject-class>org.apache.geronimo.connector.mock.MockAdminObjectImpl</adminobject-class>
+                <adminobject-instance>
+                    <message-destination-name>tweedledee</message-destination-name>
+                    <config-property-setting name="Tweedle">Dee-value</config-property-setting>
+                </adminobject-instance>
+                <adminobject-instance>
+                    <message-destination-name>tweedledum</message-destination-name>
+                    <config-property-setting name="Tweedle">Dum-value</config-property-setting>
+                </adminobject-instance>
+            </adminobject>
+
         </connector>
 
     </module>

Modified: geronimo/trunk/modules/connector/src/test/org/apache/geronimo/connector/deployment/RAR_1_0ConfigBuilderTest.java
==============================================================================
--- geronimo/trunk/modules/connector/src/test/org/apache/geronimo/connector/deployment/RAR_1_0ConfigBuilderTest.java	(original)
+++ geronimo/trunk/modules/connector/src/test/org/apache/geronimo/connector/deployment/RAR_1_0ConfigBuilderTest.java	Mon Nov  1 12:55:35 2004
@@ -67,6 +67,12 @@
     private URL geronimoDD;
     XmlOptions xmlOptions;
     private List errors;
+    private boolean defaultXATransactionCaching = true;
+    private boolean defaultXAThreadCaching = false;
+    private int defaultMaxSize = 10;
+    private int defaultMinSize = 0;
+    private int defaultBlockingTimeoutMilliseconds = 5000;
+    private int defaultidleTimeoutMinutes = 15;
 
 
     public void testLoadJ2eeDeploymentDescriptor() throws Exception {
@@ -176,7 +182,7 @@
         ObjectName connectionTrackerName = new ObjectName("geronimo.connector:service=ConnectionTracker");
 
         Kernel kernel = new Kernel("testServer");
-        ConnectorModuleBuilder moduleBuilder = new ConnectorModuleBuilder(10, 0, 5000, 15, kernel);
+        ConnectorModuleBuilder moduleBuilder = new ConnectorModuleBuilder(defaultMaxSize, defaultMinSize, defaultBlockingTimeoutMilliseconds, defaultidleTimeoutMinutes, defaultXATransactionCaching, defaultXAThreadCaching, kernel);
         File rarFile = action.getRARFile();
 
         ClassLoader oldCl = Thread.currentThread().getContextClassLoader();
@@ -230,7 +236,7 @@
         JarFile rarFile = null;
         try {
             rarFile = DeploymentUtil.createJarFile(new File(basedir, "target/test-ear-noger.ear"));
-            EARConfigBuilder configBuilder = new EARConfigBuilder(j2eeServer, null, connectionTrackerName, null, null, null, null, null, null, new ConnectorModuleBuilder(10, 0, 5000, 15, kernel), null, null, kernel);
+            EARConfigBuilder configBuilder = new EARConfigBuilder(j2eeServer, null, connectionTrackerName, null, null, null, null, null, null, new ConnectorModuleBuilder(defaultMaxSize, defaultMinSize, defaultBlockingTimeoutMilliseconds, defaultidleTimeoutMinutes, defaultXATransactionCaching, defaultXAThreadCaching, kernel), null, null, kernel);
             File tempDir = null;
             try {
                 tempDir = DeploymentUtil.createTempDir();

Modified: geronimo/trunk/modules/connector/src/test/org/apache/geronimo/connector/deployment/RAR_1_5ConfigBuilderTest.java
==============================================================================
--- geronimo/trunk/modules/connector/src/test/org/apache/geronimo/connector/deployment/RAR_1_5ConfigBuilderTest.java	(original)
+++ geronimo/trunk/modules/connector/src/test/org/apache/geronimo/connector/deployment/RAR_1_5ConfigBuilderTest.java	Mon Nov  1 12:55:35 2004
@@ -65,6 +65,12 @@
     private URL geronimoDD;
     private XmlOptions xmlOptions;
     private List errors;
+    private boolean defaultXATransactionCaching = true;
+    private boolean defaultXAThreadCaching = false;
+    private int defaultMaxSize = 10;
+    private int defaultMinSize = 0;
+    private int defaultBlockingTimeoutMilliseconds = 5000;
+    private int defaultidleTimeoutMinutes = 15;
 
     public void testLoadJ2eeDeploymentDescriptor() throws Exception {
         InputStream j2eeInputStream = j2eeDD.openStream();
@@ -121,7 +127,7 @@
         ObjectName connectionTrackerName = new ObjectName("geronimo.connector:service=ConnectionTracker");
 
         Kernel kernel = new Kernel("testServer");
-        ConnectorModuleBuilder moduleBuilder = new ConnectorModuleBuilder(10, 0, 5000, 15, kernel);
+        ConnectorModuleBuilder moduleBuilder = new ConnectorModuleBuilder(defaultMaxSize, defaultMinSize, defaultBlockingTimeoutMilliseconds, defaultidleTimeoutMinutes, defaultXATransactionCaching, defaultXAThreadCaching, kernel);
         File rarFile = action.getRARFile();
 
         ClassLoader oldCl = Thread.currentThread().getContextClassLoader();