That's a reasonable idea.
Russell Butek
[EMAIL PROTECTED]
Glen Daniels <[EMAIL PROTECTED]> on 04/18/2002 09:02:05 AM
Please respond to [EMAIL PROTECTED]
To: "'[EMAIL PROTECTED]'" <[EMAIL PROTECTED]>
cc:
Subject: RE: cvs commit: xml-axis/java/src/org/apache/axis/description
Ser viceDesc.java
I think I'd rather see the tests have a doSkeletons property set to "yes"
or "no", which gets used on all the tests, then allow the user to set that
in their build.properties or on the command line. Then we can just set the
gump build to run the functional-tests twice, once with "yes" and once with
"no", in fact we could even make a "full-functional-tests" target which
does that. What do you think?
--Glen
> -----Original Message-----
> From: Russell Butek [mailto:[EMAIL PROTECTED]]
> Sent: Thursday, April 18, 2002 9:46 AM
> To: [EMAIL PROTECTED]
> Subject: RE: cvs commit: xml-axis/java/src/org/apache/axis/description
> Ser viceDesc.java
>
>
> Thanks, Glen, that did it! Now I can set half the tests to
> skeletonDeploy
> as we discussed. I'll just randomly pick the first half (+
> the types test
> that caused us so much problems). This isn't the best
> solution, not even a
> very good one, but it'll have to do until we can spend the
> time to revamp
> it.
>
> Russell Butek
> [EMAIL PROTECTED]
>
>
> Glen Daniels <[EMAIL PROTECTED]> on 04/17/2002 11:21:31 PM
>
> Please respond to [EMAIL PROTECTED]
>
> To: "'[EMAIL PROTECTED]'" <[EMAIL PROTECTED]>
> cc:
> Subject: RE: cvs commit:
> xml-axis/java/src/org/apache/axis/description
> Ser viceDesc.java
>
>
>
>
> Russell, this should fix the problem with skeletons + test.wsdl.types.
>
> --Glen
>
> > -----Original Message-----
> > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]]
> > Sent: Thursday, April 18, 2002 12:19 AM
> > To: [EMAIL PROTECTED]
> > Subject: cvs commit: xml-axis/java/src/org/apache/axis/description
> > ServiceDesc.java
> >
> >
> > gdaniels 02/04/17 21:19:14
> >
> > Modified: java/src/org/apache/axis/description ServiceDesc.java
> > Log:
> > If the introspected type is a Holder, use the held class
> > when determining
> > the type QName.
> >
> > Revision Changes Path
> > 1.18 +9 -1
> > xml-axis/java/src/org/apache/axis/description/ServiceDesc.java
> >
> > Index: ServiceDesc.java
> >
> ===================================================================
> > RCS file:
> > /home/cvs/xml-axis/java/src/org/apache/axis/description/Servic
> > eDesc.java,v
> > retrieving revision 1.17
> > retrieving revision 1.18
> > diff -u -r1.17 -r1.18
> > --- ServiceDesc.java 15 Apr 2002 02:35:57 -0000 1.17
> > +++ ServiceDesc.java 18 Apr 2002 04:19:14 -0000 1.18
> > @@ -60,6 +60,7 @@
> > import org.apache.axis.wsdl.Skeleton;
> >
> > import javax.xml.rpc.namespace.QName;
> > +import javax.xml.rpc.holders.Holder;
> > import java.util.*;
> > import java.lang.reflect.Method;
> > import java.lang.reflect.InvocationTargetException;
> > @@ -425,8 +426,15 @@
> > // If no type is specified, just use
> > the Java type
> > QName typeQName = param.getTypeQName();
> > if (typeQName == null) {
> > + if
> (Holder.class.isAssignableFrom(type)) {
> > + typeQName = tm.getTypeQName(
> > +
> > JavaUtils.getHolderValueType(type));
> > + } else {
> > + typeQName = tm.getTypeQName(type);
> > + }
> > +
> > param.setJavaType(type);
> > -
> param.setTypeQName(tm.getTypeQName(type));
> > + param.setTypeQName(typeQName);
> > } else {
> > // A type was specified - see if
> they match
> > Class paramClass = tm.getClassForQName(
> >
> >
> >
> >
>
>
