Return-Path: Delivered-To: apmail-incubator-cxf-dev-archive@locus.apache.org Received: (qmail 98111 invoked from network); 12 Jun 2007 01:58:48 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 12 Jun 2007 01:58:48 -0000 Received: (qmail 62586 invoked by uid 500); 12 Jun 2007 01:58:51 -0000 Delivered-To: apmail-incubator-cxf-dev-archive@incubator.apache.org Received: (qmail 62549 invoked by uid 500); 12 Jun 2007 01:58:51 -0000 Mailing-List: contact cxf-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: cxf-dev@incubator.apache.org Delivered-To: mailing list cxf-dev@incubator.apache.org Received: (qmail 62540 invoked by uid 99); 12 Jun 2007 01:58:51 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 11 Jun 2007 18:58:50 -0700 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_HELO_PASS,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (herse.apache.org: domain of james.mao@iona.com designates 12.170.54.180 as permitted sender) Received: from [12.170.54.180] (HELO amer-mx1.iona.com) (12.170.54.180) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 11 Jun 2007 18:58:44 -0700 X-IronPort-AV: E=Sophos;i="4.16,409,1175486400"; d="scan'208";a="582089" Received: from amer-ems1.ionaglobal.com ([10.65.6.25]) by amer-mx1.iona.com with ESMTP; 11 Jun 2007 21:58:22 -0400 Received: from [127.0.0.1] ([10.129.9.178]) by amer-ems1.IONAGLOBAL.COM with Microsoft SMTPSVC(6.0.3790.1830); Mon, 11 Jun 2007 21:58:16 -0400 Message-ID: <466DFDBC.4090309@iona.com> Date: Tue, 12 Jun 2007 09:58:20 +0800 From: James Mao User-Agent: Thunderbird 1.5.0.12 (Windows/20070509) MIME-Version: 1.0 To: cxf-dev@incubator.apache.org Subject: Re: Rename AbstractEndpointFactory?? - [was Re: svn commit: r545465 ...] References: <7b774c950706110947jad3f9ddw2a1e6ca67ea977ab@mail.gmail.com> In-Reply-To: <7b774c950706110947jad3f9ddw2a1e6ca67ea977ab@mail.gmail.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 12 Jun 2007 01:58:17.0723 (UTC) FILETIME=[252E14B0:01C7AC95] X-Virus-Checked: Checked by ClamAV on apache.org You mean java2wsdl use the EndpointFactoryBean to create the endpoint, right? I'm OK with changing the name btw. But this class will be in the simple frontend? if so, java2wsdl still can not use it directly unless it's in rt/core or api Java2wsdl would like not to depend on specific frontend, otherwise it will have cycle dependencies. (codegen -> java2wsdl -> jaxws -> testutils -> codegen) same as (codegen -> java2wsdl -> simple -> testutils -> codegen) Actually I was trying to modify the AbstractEndpointFactory directlry and finally i realized that what i really want is making the java2wsdl not depend on the simple frontend (jaxws frontend as well) And to introduce some simple class to extend what we already have always better than modify directly, right? I still need the JaxwsServiceBuilder and SimpleServiceBuilder to build the specific service model org.apache.cxf.simple.SimpleServiceBuilder use ReflectionServiceFactoryBean as the service factory, org.apache.cxf.jaxws.JaxwsServiceBuilder use JaxWsServiceFactoryBean as the service factory, And those builders should be in their own frontend. You also noticed that the package name and the class name of the frontend ServiceBuilder, it's quite easy for java2wsdl to load them without depending on the specific frontend at compile time just like a frontend plugin to build the service model from the service class. James > OK, I see you just want to create an Endpoint... > > Would anyone object if I renamed AbstractEndpointFactory to > EndpointFactoryBean and made it non abstract then? This would allow > you to > reuse the Endpoint creation code, but without the server/client creation > logic easily without the introduction of several new classes. > > - Dan > > On 6/10/07, James Mao wrote: >> >> Dan, >> >> Previous, java2wsdl depends on rt/frontend/jaxws and rt/frontend/simple, >> after this commit, we don't need to do that anymore, >> The frontends just like the plugins to java2wsdl, >> >> In Java2wsdl we have a mechanism to auto discover the specific frontend, >> and then use the reflection to load the service builder to build the >> service model from the service class >> >> The AbstractEndpointFactory has a createEndpoint, we use the method to >> create the endpoint and get the service model, >> We extend the factory so that we can pass in the argument for the tools, >> e.g serviceFactory.setPopulateFromClass(true); in >> JaxwsServiceBuilder >> , which may not be used in the rt >> >> And most important, i think this commit is for preparing moving the >> wsdlgen maven plugin into the codegen plugin, before the commit, i can >> not resolve the cycle dependencies. >> >> >> James >> >> > Can we please stick SerivceBuilder and AbstractServiceFactory in a >> tool >> > specific package? Right now it looks quite confusing with both >> > AbstractServiceFactory and *ServiceFactoryBeans when the two things >> are >> > quite unrelated... >> > >> > Also the hierarchy of extending AbstractEndpointFactory seems odd to >> > me. Are >> > you just trying to create a ServiceInfo for the tools? Why introduce >> > SimpleServiceBuilder and JaxwsServiceBuilder when you already have >> > ReflectionServiceFactory and JaxWsServiceFactory? >> > >> > Thanks, >> > - Dan >> > >> > On 6/8/07, mmao@apache.org wrote: >> >> >> >> Author: mmao >> >> Date: Fri Jun 8 02:47:24 2007 >> >> New Revision: 545465 >> >> >> >> URL: http://svn.apache.org/viewvc?view=rev&rev=545465 >> >> Log: >> >> * Java2Wsdl should not depends on specific frontend, >> >> it should treat frontend like a plugin, >> >> however the unit tests still depend on them. >> >> * Move the frontend specific service builder to their own frontend >> >> module >> >> >> >> >> >> Added: >> >> >> >> >> >> >> incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/ServiceBuilder.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JaxwsServiceBuilder.java >> >> >> >> >> - copied, changed from r545422, >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilder.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/simple/ >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/simple/SimpleServiceBuilder.java >> >> >> >> >> - copied, changed from r545422, >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/SimpleServiceBuilder.java >> >> >> >> >> Removed: >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilder.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilder.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/SimpleServiceBuilder.java >> >> >> >> >> Modified: >> >> incubator/cxf/trunk/tools/javato/pom.xml >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/JavaToProcessor.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactory.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/FaultBeanGeneratorTest.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/WrapperBeanGeneratorTest.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactoryTest.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderNoAnnoTest.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderRPCTest.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderTest.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/RequestWrapperTest.java >> >> >> >> >> >> >> >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/ResponseWrapperTest.java >> >> >> >> >> >> >> Added: >> >> >> incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/ServiceBuilder.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/ServiceBuilder.java?view=auto&rev=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/ServiceBuilder.java >> >> >> >> >> (added) >> >> +++ >> >> >> incubator/cxf/trunk/api/src/main/java/org/apache/cxf/service/ServiceBuilder.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -0,0 +1,37 @@ >> >> +/** >> >> + * Licensed to the Apache Software Foundation (ASF) under one >> >> + * or more contributor license agreements. See the NOTICE file >> >> + * distributed with this work for additional information >> >> + * regarding copyright ownership. The ASF licenses this file >> >> + * to you under the Apache License, Version 2.0 (the >> >> + * "License"); you may not use this file except in compliance >> >> + * with the License. You may obtain a copy of the License at >> >> + * >> >> + * http://www.apache.org/licenses/LICENSE-2.0 >> >> + * >> >> + * Unless required by applicable law or agreed to in writing, >> >> + * software distributed under the License is distributed on an >> >> + * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY >> >> + * KIND, either express or implied. See the License for the >> >> + * specific language governing permissions and limitations >> >> + * under the License. >> >> + */ >> >> + >> >> +package org.apache.cxf.service; >> >> + >> >> +import java.io.File; >> >> + >> >> +import org.apache.cxf.Bus; >> >> +import org.apache.cxf.service.model.ServiceInfo; >> >> + >> >> +public interface ServiceBuilder { >> >> + ServiceInfo createService(); >> >> + >> >> + void setBindingId(String id); >> >> + void setTransportId(String id); >> >> + void setBus(Bus bus); >> >> + void setAddress(String addr); >> >> + void setServiceClass(Class clz); >> >> + >> >> + File getOutputFile(); >> >> +} >> >> >> >> Copied: >> >> >> incubator/cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JaxwsServiceBuilder.java >> >> >> >> >> (from r545422, >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilder.java) >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JaxwsServiceBuilder.java?view=diff&rev=545465&p1=incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilder.java&r1=545422&p2=incubator/cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JaxwsServiceBuilder.java&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilder.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/rt/frontend/jaxws/src/main/java/org/apache/cxf/jaxws/JaxwsServiceBuilder.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -17,21 +17,22 @@ >> >> * under the License. >> >> */ >> >> >> >> -package org.apache.cxf.tools.java2wsdl.processor.internal.jaxws; >> >> +package org.apache.cxf.jaxws; >> >> >> >> import java.io.File; >> >> -import javax.jws.WebService; >> >> >> >> import org.apache.cxf.common.util.StringUtils; >> >> +import org.apache.cxf.frontend.AbstractServiceFactory; >> >> +import org.apache.cxf.jaxws.support.JaxWsImplementorInfo; >> >> import org.apache.cxf.jaxws.support.JaxWsServiceFactoryBean; >> >> -import >> >> org.apache.cxf.tools.java2wsdl.processor.internal.ServiceBuilder; >> >> -import org.apache.cxf.tools.util.AnnotationUtil; >> >> >> >> -public class JaxwsServiceBuilder extends ServiceBuilder { >> >> +public class JaxwsServiceBuilder extends AbstractServiceFactory { >> >> + >> >> + final JaxWsServiceFactoryBean serviceFactory; >> >> >> >> public JaxwsServiceBuilder() { >> >> super(); >> >> - JaxWsServiceFactoryBean serviceFactory = new >> >> JaxWsServiceFactoryBean(); >> >> + serviceFactory = new JaxWsServiceFactoryBean(); >> >> //As this is a javatowsdl tool, explictly populate service >> >> model >> >> from class >> >> serviceFactory.setPopulateFromClass(true); >> >> >> >> @@ -39,11 +40,8 @@ >> >> } >> >> >> >> public File getOutputFile() { >> >> - WebService webService = >> >> AnnotationUtil.getPrivClassAnnotation(getServiceClass(), >> >> WebService.class); >> >> - if (webService == null) { >> >> - return null; >> >> - } >> >> - String wsdlLocation = webService.wsdlLocation(); >> >> + JaxWsImplementorInfo jaxwsImpl = >> >> serviceFactory.getJaxWsImplementorInfo(); >> >> + String wsdlLocation = jaxwsImpl.getWsdlLocation(); >> >> if (!StringUtils.isEmpty(wsdlLocation)) { >> >> return new File(wsdlLocation); >> >> } >> >> >> >> Copied: >> >> >> incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/simple/SimpleServiceBuilder.java >> >> >> >> >> (from r545422, >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/SimpleServiceBuilder.java) >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/simple/SimpleServiceBuilder.java?view=diff&rev=545465&p1=incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/SimpleServiceBuilder.java&r1=545422&p2=incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/simple/SimpleServiceBuilder.java&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/simple/SimpleServiceBuilder.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/rt/frontend/simple/src/main/java/org/apache/cxf/simple/SimpleServiceBuilder.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -17,12 +17,12 @@ >> >> * under the License. >> >> */ >> >> >> >> -package org.apache.cxf.tools.java2wsdl.processor.internal.simple; >> >> +package org.apache.cxf.simple; >> >> >> >> +import org.apache.cxf.frontend.AbstractServiceFactory; >> >> import org.apache.cxf.service.factory.ReflectionServiceFactoryBean; >> >> -import >> >> org.apache.cxf.tools.java2wsdl.processor.internal.ServiceBuilder; >> >> >> >> -public class SimpleServiceBuilder extends ServiceBuilder { >> >> +public class SimpleServiceBuilder extends AbstractServiceFactory { >> >> >> >> public SimpleServiceBuilder() { >> >> super(); >> >> >> >> Modified: incubator/cxf/trunk/tools/javato/pom.xml >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/pom.xml?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- incubator/cxf/trunk/tools/javato/pom.xml (original) >> >> +++ incubator/cxf/trunk/tools/javato/pom.xml Fri Jun 8 02:47:24 2007 >> >> @@ -80,14 +80,15 @@ >> >> >> >> >> >> org.apache.cxf >> >> + cxf-rt-core >> >> + ${project.version} >> >> + >> >> + >> >> + >> >> + org.apache.cxf >> >> cxf-rt-frontend-jaxws >> >> ${project.version} >> >> - >> >> - >> >> - org.apache.cxf >> >> - cxf-tools-common >> >> - >> >> - >> >> + test >> >> >> >> >> >> >> >> @@ -113,7 +114,10 @@ >> >> jaxb-impl >> >> >> >> >> >> - >> >> + >> >> + javax.xml.ws >> >> + jaxws-api >> >> + >> >> >> >> >> >> >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/JavaToProcessor.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/JavaToProcessor.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/JavaToProcessor.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/JavaToProcessor.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -32,6 +32,7 @@ >> >> import org.apache.cxf.common.i18n.Message; >> >> import org.apache.cxf.common.logging.LogUtils; >> >> import org.apache.cxf.common.util.StringUtils; >> >> +import org.apache.cxf.service.ServiceBuilder; >> >> import org.apache.cxf.service.model.EndpointInfo; >> >> import org.apache.cxf.service.model.ServiceInfo; >> >> import org.apache.cxf.tools.common.Processor; >> >> @@ -42,7 +43,6 @@ >> >> import org.apache.cxf.tools.java2wsdl.generator.WSDLGeneratorFactory; >> >> import >> >> org.apache.cxf.tools.java2wsdl.generator.wsdl11.FaultBeanGenerator; >> >> import >> >> org.apache.cxf.tools.java2wsdl.generator.wsdl11.WrapperBeanGenerator; >> >> -import >> >> org.apache.cxf.tools.java2wsdl.processor.internal.ServiceBuilder; >> >> import >> >> >> org.apache.cxf.tools.java2wsdl.processor.internal.ServiceBuilderFactory >> ; >> >> import org.apache.cxf.tools.util.AnnotationUtil; >> >> import org.apache.cxf.wsdl.WSDLConstants; >> >> @@ -79,7 +79,7 @@ >> >> System.setProperty("java.class.path", newCp + >> >> File.pathSeparator + oldClassPath); >> >> } >> >> ServiceBuilder builder = getServiceBuilder(); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> customize(service); >> >> >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactory.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactory.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactory.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/main/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactory.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -19,6 +19,7 @@ >> >> >> >> package org.apache.cxf.tools.java2wsdl.processor.internal; >> >> >> >> +import org.apache.cxf.service.ServiceBuilder; >> >> import org.apache.cxf.tools.common.ToolException; >> >> import org.apache.cxf.tools.java2wsdl.processor.FrontendFactory; >> >> >> >> @@ -51,12 +52,12 @@ >> >> throw new ToolException("Can not find the ServiceBulider >> >> for >> >> style: " + s, e); >> >> } >> >> builder.setServiceClass(serviceClass); >> >> - builder.getServiceFactory().setServiceClass(serviceClass); >> >> + //builder.getServiceFactory().setServiceClass(serviceClass); >> >> return builder; >> >> } >> >> >> >> protected String getBuilderClassName(FrontendFactory.Style s) { >> >> - String pkgName = getClass().getPackage().getName(); >> >> + String pkgName = "org.apache.cxf"; >> >> return pkgName + "." + s.toString().toLowerCase() + "." + >> s + >> >> "ServiceBuilder"; >> >> } >> >> >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/FaultBeanGeneratorTest.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/FaultBeanGeneratorTest.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/FaultBeanGeneratorTest.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/FaultBeanGeneratorTest.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -50,7 +50,7 @@ >> >> } >> >> >> >> private ServiceInfo getServiceInfo() { >> >> - return processor.getServiceBuilder().build(); >> >> + return processor.getServiceBuilder().createService(); >> >> } >> >> >> >> @Test >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/WrapperBeanGeneratorTest.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/WrapperBeanGeneratorTest.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/WrapperBeanGeneratorTest.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/generator/wsdl11/WrapperBeanGeneratorTest.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -50,7 +50,7 @@ >> >> } >> >> >> >> private ServiceInfo getServiceInfo() { >> >> - return processor.getServiceBuilder().build(); >> >> + return processor.getServiceBuilder().createService(); >> >> } >> >> >> >> @Test >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactoryTest.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactoryTest.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactoryTest.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/ServiceBuilderFactoryTest.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -19,17 +19,19 @@ >> >> >> >> package org.apache.cxf.tools.java2wsdl.processor.internal; >> >> >> >> -import junit.framework.TestCase; >> >> - >> >> +import org.apache.cxf.jaxws.JaxwsServiceBuilder; >> >> +import org.apache.cxf.service.ServiceBuilder; >> >> +import org.apache.cxf.simple.SimpleServiceBuilder; >> >> import org.apache.cxf.tools.fortest.classnoanno.docbare.Stock; >> >> import org.apache.cxf.tools.fortest.simple.Hello; >> >> import org.apache.cxf.tools.java2wsdl.processor.FrontendFactory; >> >> -import >> >> >> org.apache.cxf.tools.java2wsdl.processor.internal.jaxws.JaxwsServiceBuilder >> >> >> >> >> ; >> >> -import >> >> >> org.apache.cxf.tools.java2wsdl.processor.internal.simple.SimpleServiceBuilder >> >> >> >> >> ; >> >> +import org.junit.Assert; >> >> +import org.junit.Test; >> >> >> >> -public class ServiceBuilderFactoryTest extends TestCase { >> >> +public class ServiceBuilderFactoryTest extends Assert { >> >> ServiceBuilderFactory factory = >> >> ServiceBuilderFactory.getInstance(); >> >> - >> >> + >> >> + @Test >> >> public void testGetBuilderClassName() { >> >> assertNotNull(factory); >> >> assertEquals(JaxwsServiceBuilder.class.getName(), >> >> @@ -39,6 +41,7 @@ >> >> factory.getBuilderClassName( >> >> FrontendFactory.Style.Simple)); >> >> } >> >> >> >> + @Test >> >> public void testGetJaxwsBuilder() { >> >> factory.setServiceClass(Stock.class); >> >> ServiceBuilder builder = factory.newBuilder(); >> >> @@ -46,6 +49,7 @@ >> >> assertTrue(builder instanceof JaxwsServiceBuilder); >> >> } >> >> >> >> + @Test >> >> public void testGetSimpleBuilder() { >> >> factory.setServiceClass(Hello.class); >> >> ServiceBuilder builder = factory.newBuilder(); >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderNoAnnoTest.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderNoAnnoTest.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderNoAnnoTest.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderNoAnnoTest.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -22,6 +22,7 @@ >> >> import java.io.File; >> >> >> >> import org.apache.cxf.BusFactory; >> >> +import org.apache.cxf.jaxws.JaxwsServiceBuilder; >> >> import org.apache.cxf.service.model.ServiceInfo; >> >> import org.apache.cxf.tools.common.ProcessorTestBase; >> >> import >> org.apache.cxf.tools.java2wsdl.generator.wsdl11.WSDL11Generator; >> >> @@ -48,7 +49,7 @@ >> >> public void testGeneratedWithElementryClass() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.classnoanno.docbare.Stock.class); >> >> builder.setAddress("http://localhost"); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> generator.setServiceModel(service); >> >> File output = getOutputFile("stock_noanno_bare.wsdl"); >> >> generator.generate(output); >> >> @@ -63,7 +64,7 @@ >> >> public void testGeneratedWithDocWrappedClass() throws >> Exception { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.classnoanno.docwrapped.Stock.class); >> >> builder.setAddress("http://localhost"); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> generator.setServiceModel(service); >> >> File output = getOutputFile("stock_noanno_wrapped.wsdl"); >> >> generator.generate(output); >> >> @@ -83,7 +84,7 @@ >> >> public void testGeneratedWithRPCClass() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.classnoanno.rpc.Stock.class); >> >> builder.setAddress("http://localhost"); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> generator.setServiceModel(service); >> >> File output = getOutputFile("stock_noanno_rpc.wsdl"); >> >> generator.generate(output); >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderRPCTest.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderRPCTest.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderRPCTest.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderRPCTest.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -22,6 +22,7 @@ >> >> import java.io.File; >> >> >> >> import org.apache.cxf.BusFactory; >> >> +import org.apache.cxf.jaxws.JaxwsServiceBuilder; >> >> import org.apache.cxf.service.model.ServiceInfo; >> >> import org.apache.cxf.tools.common.ProcessorTestBase; >> >> import >> org.apache.cxf.tools.java2wsdl.generator.wsdl11.WSDL11Generator; >> >> @@ -49,7 +50,7 @@ >> >> @Test >> >> public void testGreeter() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.jaxws.rpc.Greeter.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> generator.setServiceModel(service); >> >> File output = getOutputFile("rpc_greeter.wsdl"); >> >> assertNotNull(output); >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderTest.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderTest.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderTest.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/JaxwsServiceBuilderTest.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -26,6 +26,7 @@ >> >> >> >> import org.apache.cxf.BusFactory; >> >> import org.apache.cxf.helpers.IOUtils; >> >> +import org.apache.cxf.jaxws.JaxwsServiceBuilder; >> >> import org.apache.cxf.service.model.BindingInfo; >> >> import org.apache.cxf.service.model.ServiceInfo; >> >> import org.apache.cxf.tools.common.ProcessorTestBase; >> >> @@ -70,7 +71,7 @@ >> >> @Test >> >> public void testDocLitWrappedWithWrapperClass() { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.withannotation.doc.StockWrapped.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> generator.setServiceModel(service); >> >> File output = >> >> getOutputFile("doc_lit_wrapped_with_wrapperclass.wsdl"); >> >> assertNotNull(output); >> >> @@ -85,7 +86,7 @@ >> >> @Test >> >> public void testDocLitWrappedWithoutWrapperClass() throws >> >> Exception { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.withannotation.doc.HelloWrapped.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File output = >> >> getOutputFile("doc_lit_wrapped_no_wrapperclass.wsdl"); >> >> @@ -102,7 +103,7 @@ >> >> @Test >> >> public void testDocLitWrapped() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.hello_world_doc_lit.Greeter.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> generator.setServiceModel(service); >> >> File output = getOutputFile("hello_doc_lit.wsdl"); >> >> assertNotNull(output); >> >> @@ -119,7 +120,7 @@ >> >> //contain namespace >> >> public void testDocWrappedWithLocalName() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.withannotation.doc.Stock.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File output = >> getOutputFile("doc_lit_wrapped_localName.wsdl"); >> >> @@ -137,7 +138,7 @@ >> >> //contain namespace >> >> public void testDocWrappedNoWebParam() throws Exception { >> >> builder.setServiceClass( >> >> >> org.apache.cxf.tools.fortest.withannotation.doc.HelloWithNoWebParam.class >> >> >> >> >> ); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File output = >> >> getOutputFile("doc_lit_wrapped_no_webparam.wsdl"); >> >> @@ -155,7 +156,7 @@ >> >> //"invalid XML schema: "header" must refer to an existing >> element" >> >> public void testHolder() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.holder.HolderService.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File output = getOutputFile("holder.wsdl"); >> >> @@ -171,7 +172,7 @@ >> >> @Test >> >> public void testAsync() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.hello_world_async_soap_http.GreeterAsync.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> generator.setServiceModel(service); >> >> File output = getOutputFile("hello_async.wsdl"); >> >> assertNotNull(output); >> >> @@ -187,7 +188,7 @@ >> >> public void testRPCLit() throws Exception { >> >> builder.setServiceClass(GreeterRPCLit.class); >> >> builder.setAddress("http://localhost"); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File file = getOutputFile("rpc_lit.wsdl"); >> >> @@ -205,7 +206,7 @@ >> >> public void testDocWrapparBare() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.hello_world_doc_wrapped_bare.Greeter.class); >> >> builder.setAddress("http://localhost"); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File file = getOutputFile("doc_wrapped_bare.wsdl"); >> >> @@ -219,7 +220,7 @@ >> >> @Ignore >> >> public void testRPCWithoutParentBindingAnnotation() throws >> >> Exception >> >> { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.withannotation.rpc.Hello.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File file = getOutputFile("rpc_lit_service_no_anno.wsdl"); >> >> @@ -234,7 +235,7 @@ >> >> @Ignore >> >> public void testSOAPBindingRPCOnMethod() throws Exception { >> >> builder.setServiceClass( >> >> >> org.apache.cxf.tools.fortest.withannotation.rpc.HelloWrongAnnotation.class >> >> >> >> >> ); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File file = getOutputFile("rpc_on_method.wsdl"); >> >> @@ -248,7 +249,7 @@ >> >> public void testSoapHeader() throws Exception { >> >> >> >> builder.setServiceClass( >> >> org.apache.samples.headers.HeaderTester.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File file = getOutputFile("soap_header.wsdl"); >> >> @@ -267,7 +268,7 @@ >> >> Class clz = AnnotationUtil.loadClass(" >> >> org.apache.cxf.tools.fortest.cxf188.Demo", getClass() >> >> .getClassLoader()); >> >> builder.setServiceClass(clz); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> >> >> generator.setServiceModel(service); >> >> File file = getOutputFile("cxf188.wsdl"); >> >> @@ -279,7 +280,7 @@ >> >> @Test >> >> public void testRpcLitNoSEI() throws Exception { >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.withannotation.rpc.EchoImpl.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> assertNotNull(service); >> >> >> >> assertEquals(new QName("http://cxf.apache.org/echotest", >> >> "EchoService"), >> >> @@ -309,7 +310,7 @@ >> >> generator.setAllowImports(true); >> >> >> >> builder.setServiceClass( >> >> org.apache.cxf.tools.fortest.cxf669.HelloImpl.class); >> >> - ServiceInfo service = builder.build(); >> >> + ServiceInfo service = builder.createService(); >> >> assertNotNull(service); >> >> assertEquals(new QName("http://foo.com/HelloWorldService", >> >> "HelloService"), service.getName()); >> >> assertEquals(new QName("http://foo.com/HelloWorld", >> >> "HelloWorld"), service.getInterface().getName()); >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/RequestWrapperTest.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/RequestWrapperTest.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/RequestWrapperTest.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/RequestWrapperTest.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -23,6 +23,7 @@ >> >> import java.util.List; >> >> >> >> import junit.framework.TestCase; >> >> +import org.apache.cxf.jaxws.JaxwsServiceBuilder; >> >> import org.apache.cxf.service.model.MessageInfo; >> >> import org.apache.cxf.service.model.OperationInfo; >> >> import org.apache.cxf.service.model.ServiceInfo; >> >> @@ -36,7 +37,7 @@ >> >> >> >> private OperationInfo getOperation(Class clz, String opName) { >> >> builder.setServiceClass(clz); >> >> - ServiceInfo serviceInfo = builder.build(); >> >> + ServiceInfo serviceInfo = builder.createService(); >> >> >> >> for (OperationInfo op : >> >> serviceInfo.getInterface().getOperations()) >> >> { >> >> if (op.getUnwrappedOperation() != null >> >> >> >> Modified: >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/ResponseWrapperTest.java >> >> >> >> >> URL: >> >> >> http://svn.apache.org/viewvc/incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/ResponseWrapperTest.java?view=diff&rev=545465&r1=545464&r2=545465 >> >> >> >> >> >> >> >> ============================================================================== >> >> >> >> >> --- >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/ResponseWrapperTest.java >> >> >> >> >> (original) >> >> +++ >> >> >> incubator/cxf/trunk/tools/javato/src/test/java/org/apache/cxf/tools/java2wsdl/processor/internal/jaxws/ResponseWrapperTest.java >> >> >> >> >> Fri Jun 8 02:47:24 2007 >> >> @@ -22,6 +22,7 @@ >> >> import java.lang.reflect.Method; >> >> >> >> import junit.framework.TestCase; >> >> +import org.apache.cxf.jaxws.JaxwsServiceBuilder; >> >> import org.apache.cxf.service.model.MessageInfo; >> >> import org.apache.cxf.service.model.OperationInfo; >> >> import org.apache.cxf.service.model.ServiceInfo; >> >> @@ -33,7 +34,7 @@ >> >> >> >> private OperationInfo getOperation(Class clz, String opName) { >> >> builder.setServiceClass(clz); >> >> - ServiceInfo serviceInfo = builder.build(); >> >> + ServiceInfo serviceInfo = builder.createService(); >> >> >> >> for (OperationInfo op : >> >> serviceInfo.getInterface().getOperations()) >> >> { >> >> if (op.getUnwrappedOperation() != null >> >> >> >> >> >> >> > >> > >> > > >