Mailing-List: contact dev-help@cxf.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cxf.apache.org
Received-SPF: pass (nike.apache.org: domain of willem.jiang@gmail.com
 designates 74.125.83.41 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=message-id:date:from:user-agent:mime-version:to:cc:subject
         :references:in-reply-to:content-type:content-transfer-encoding;
        b=JgXUybKmEt8BtPzHHlJhsAqduuSu+d+UGq1zSiqA+NiMMEQ3dT4vwVEUBHXSrU0/qJ
         2EvJTpTFzYrPQ+eBZ3T8Txt79CrX91XAWopO290rFjbV4+Wi9niPPk5LyXr59Gn9tQni
         r9wY6pvciF2agKuY1EZPB/aQ/7+83E+cXaJgQ=
Message-ID: <4D189F68.7090106@gmail.com>
Date: Mon, 27 Dec 2010 22:15:04 +0800
From: Willem Jiang <willem.jiang@gmail.com>
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US;
 rv:1.9.2.13) Gecko/20101207 Lightning/1.0b2 Thunderbird/3.1.7
MIME-Version: 1.0
To: dkulp@apache.org
CC: dev@cxf.apache.org
Subject: Re: svn commit: r1053030 -
 /cxf/branches/2.3.x-fixes/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/PolicyAnnotationTest.java
References: <EC7E5C55-0863-4F43-BD12-881EE3F6AA55@apache.org>
In-Reply-To: <EC7E5C55-0863-4F43-BD12-881EE3F6AA55@apache.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

Hi Dan,

I had to admit that the issue is introduced by the new added test 
PolicyEngineImplInitTest.
I think you already fix it just before I commit my fix :)

Willem

On 12/27/10 8:53 PM, dkulp@apache.org wrote:
>
>
> I'm -1 to this as it fixes the wrong thing. We need to find the test that is leaving the bus and make sure the cleanup is done.
>
> Daniel Kulp
> http://dankulp.com/blog
>
>
> On Dec 27, 2010, at 4:37 AM, ningjiang@apache.org wrote:
>
>> Author: ningjiang
>> Date: Mon Dec 27 09:37:49 2010
>> New Revision: 1053030
>>
>> URL: http://svn.apache.org/viewvc?rev53030&view=rev
>> Log:
>> Fix the test failed of PolicyAnnotationTest with JDK1.5
>>
>> Modified:
>>     cxf/branches/2.3.x-fixes/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/PolicyAnnotationTest.java
>>
>> Modified: cxf/branches/2.3.x-fixes/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/PolicyAnnotationTest.java
>> URL: http://svn.apache.org/viewvc/cxf/branches/2.3.x-fixes/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/PolicyAnnotationTest.java?rev53030&r153029&r253030&view=diff
>> ============================================================================
>> --- cxf/branches/2.3.x-fixes/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/PolicyAnnotationTest.java (original)
>> +++ cxf/branches/2.3.x-fixes/rt/ws/policy/src/test/java/org/apache/cxf/ws/policy/PolicyAnnotationTest.java Mon Dec 27 09:37:49 2010
>> @@ -53,6 +53,8 @@ public class PolicyAnnotationTest extend
>>
>>      @org.junit.Test
>>      public void testAnnotations() throws Exception {
>> +        // Need to clean up the Default Bus instance frist
>> +        BusFactory.setDefaultBus(null);
>>          Bus bus =usFactory.getDefaultBus();
>>          JaxWsServerFactoryBean factory =ew JaxWsServerFactoryBean();
>>          factory.setBus(bus);
>>
>>
>
>


-- 
Willem
----------------------------------
FuseSource
Web: http://www.fusesource.com
Blog:    http://willemjiang.blogspot.com (English)
          http://jnn.javaeye.com (Chinese)
Twitter: willemjiang