Return-Path: Delivered-To: apmail-xml-axis-dev-archive@xml.apache.org Received: (qmail 53045 invoked by uid 500); 4 Dec 2002 22:52:33 -0000 Mailing-List: contact axis-dev-help@xml.apache.org; run by ezmlm Precedence: bulk Reply-To: axis-dev@xml.apache.org list-help: list-unsubscribe: list-post: Delivered-To: mailing list axis-dev@xml.apache.org Received: (qmail 53035 invoked from network); 4 Dec 2002 22:52:33 -0000 Message-ID: <01a101c29be7$d4b27110$a535ee20@55344H6> From: "Jeremy Hughes" To: References: <20021204201358.57492.qmail@icarus.apache.org> Subject: Re: cvs commit: xml-axis-wsif/java/test/util WSIFTestRunner.java Date: Wed, 4 Dec 2002 22:52:30 -0000 X-Priority: 3 X-MSMail-Priority: Normal X-Mailer: Microsoft Outlook Express 5.50.4807.1700 X-MIMEOLE: Produced By Microsoft MimeOLE V5.50.4807.1700 X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N Alek, This test was written initially to ensure none of our text files we check in had rogue ^M characters. Agreed it's probably dangerous because it has a side effect. Some of us use the Eclipse dev environment ... the Windows version which has a tendancy to add ^M eg when creating a new file. This test was just to keep us honest as ^M can annoy non-windows developers. Jeremy ----- Original Message ----- From: To: Sent: Wednesday, December 04, 2002 8:13 PM Subject: cvs commit: xml-axis-wsif/java/test/util WSIFTestRunner.java aslom 2002/12/04 12:13:58 Modified: java/test/util WSIFTestRunner.java Log: commented out FileCharacterTest as it can modify source files replacing ^M and this is not what expected of unit tests ... Revision Changes Path 1.24 +2 -2 xml-axis-wsif/java/test/util/WSIFTestRunner.java Index: WSIFTestRunner.java =================================================================== RCS file: /home/cvs/xml-axis-wsif/java/test/util/WSIFTestRunner.java,v retrieving revision 1.23 retrieving revision 1.24 diff -u -r1.23 -r1.24 --- WSIFTestRunner.java 4 Dec 2002 20:11:47 -0000 1.23 +++ WSIFTestRunner.java 4 Dec 2002 20:13:57 -0000 1.24 @@ -88,7 +88,7 @@ * Run JUnit tests on WSIF code. * Add new tests to the suite() method. * Comment out lines for tests not required. - * + * * @author Owen Burroughs */ @@ -106,7 +106,7 @@ // Check all files for the dreaded ^M character! If you set // fix=on in the wsif.test.properties file, this test will // not only check the files but also fix them. - suite.addTest(new TestSuite(FileCharacterTest.class)); + //suite.addTest(new TestSuite(FileCharacterTest.class)); // Test the initialisation of WSIF providers. This test needs to be // the FIRST of the functional tests as after this, WSIF will already