antelder 2002/06/25 09:59:42 Modified: java/src/org/apache/wsif/base WSIFDefaultPort.java Added: java/test/util PortFinalizeTest.java Log: Fix defect 128114 - WSIFDefaultPort finalize method should call close on the port Revision Changes Path 1.2 +12 -0 xml-axis-wsif/java/src/org/apache/wsif/base/WSIFDefaultPort.java Index: WSIFDefaultPort.java =================================================================== RCS file: /home/cvs/xml-axis-wsif/java/src/org/apache/wsif/base/WSIFDefaultPort.java,v retrieving revision 1.1 retrieving revision 1.2 diff -u -r1.1 -r1.2 --- WSIFDefaultPort.java 12 Jun 2002 10:27:23 -0000 1.1 +++ WSIFDefaultPort.java 25 Jun 2002 16:59:42 -0000 1.2 @@ -62,6 +62,7 @@ import java.util.List; import org.apache.wsif.WSIFException; +import org.apache.wsif.WSIFConstants; import org.apache.wsif.WSIFMessage; import org.apache.wsif.WSIFPort; import org.apache.wsif.logging.Tr; @@ -83,6 +84,17 @@ Tr.exit(); } + public void finalize() throws Throwable { + Tr.entry(this); + try { + close(); + } catch (WSIFException ex) { + Tr.exception( WSIFConstants.TYPE_EVENT, ex ); + } + super.finalize(); + Tr.exit(); + } + /** * Utility method to return key suitable for hash table. */ 1.1 xml-axis-wsif/java/test/util/PortFinalizeTest.java Index: PortFinalizeTest.java =================================================================== /* * The Apache Software License, Version 1.1 * * * Copyright (c) 2002 The Apache Software Foundation. All rights * reserved. * * Redistribution and use in source and binary forms, with or without * modification, are permitted provided that the following conditions * are met: * * 1. Redistributions of source code must retain the above copyright * notice, this list of conditions and the following disclaimer. * * 2. Redistributions in binary form must reproduce the above copyright * notice, this list of conditions and the following disclaimer in * the documentation and/or other materials provided with the * distribution. * * 3. The end-user documentation included with the redistribution, * if any, must include the following acknowledgment: * "This product includes software developed by the * Apache Software Foundation (http://www.apache.org/)." * Alternately, this acknowledgment may appear in the software itself, * if and wherever such third-party acknowledgments normally appear. * * 4. The names "WSIF" and "Apache Software Foundation" must * not be used to endorse or promote products derived from this * software without prior written permission. For written * permission, please contact [EMAIL PROTECTED] * * 5. Products derived from this software may not be called "Apache", * nor may "Apache" appear in their name, without prior written * permission of the Apache Software Foundation. * * THIS SOFTWARE IS PROVIDED ``AS IS'' AND ANY EXPRESSED OR IMPLIED * WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE * DISCLAIMED. IN NO EVENT SHALL THE APACHE SOFTWARE FOUNDATION OR * ITS CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF * USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT * OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF * SUCH DAMAGE. * ==================================================================== * * This software consists of voluntary contributions made by many * individuals on behalf of the Apache Software Foundation and was * originally based on software copyright (c) 2001, 2002, International * Business Machines, Inc., http://www.apache.org. For more * information on the Apache Software Foundation, please see * <http://www.apache.org/>. */ package util; import junit.framework.Test; import junit.framework.TestCase; import junit.framework.TestSuite; import org.apache.wsif.WSIFOperation; import org.apache.wsif.base.WSIFDefaultPort; /** * Junit test for the finalize method on the WSIFDefaultPort. * * @author Ant Elder <[EMAIL PROTECTED]> */ public class PortFinalizeTest extends TestCase { public PortFinalizeTest(String name) { super(name); } public static void main(String[] args) { junit.textui.TestRunner.run(suite()); } public static Test suite() { return new TestSuite(PortFinalizeTest.class); } public void setUp() { TestUtilities.setUpExtensionsAndProviders(); } public void testFinalize() throws Throwable { DummyPort p = new DummyPort(); assertTrue( "freshly created port should not be closed!", !DummyPort.isClosed() ); //p = null; //System.gc(); //System.runFinalization(); // unfortunately the above code doesn't seem to work in WSAD5 // which makes this test not so useful. TODO - How can I fix that? p.finalize(); assertTrue( "port should now be closed!", DummyPort.isClosed() ); } } class DummyPort extends WSIFDefaultPort { static private boolean closed; public DummyPort() { closed = false; } public WSIFOperation createOperation(String s1) { return null; } public WSIFOperation createOperation(String s1, String s2, String s3) { return null; } public void close() { closed = true; } static public boolean isClosed() { return closed; } }