ant-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Isaac Jurado <dipto...@gmail.com>
Subject Re: Creating task/macro wrappers
Date Mon, 25 Feb 2019 18:59:10 GMT
So, if I understood that right, the idea would be to create two
<presetdef>: one with "timeout" and another one without?

If that is the case, as the number of "omittable" attributes grows,
the amount of <presetdef> combinations increases exponentially.

I may be pretty much stretching Ant beyond its limits (in terms of XML
syntax).  And the behavior of the <testng> task implemented in Java
doesn't help much either, not interpreting empty strings as null
values.

But thanks anyway.

On Mon, Feb 25, 2019 at 7:53 PM Matt Benson <gudnabrsam@gmail.com> wrote:
>
> You could conditionally <presetdef> with or without the desired attributes,
> depending on the structure of your build. :|
>
> Matt
>
> On Mon, Feb 25, 2019 at 12:39 PM Isaac Jurado <diptongo@gmail.com> wrote:
>
> > Thanks for the info, I didn't know about <presetdef> either.
> >
> > By the looks of it, seems that <presetdef> is the opposite of what I'm
> > trying to achieve.
> >
> > I'll try to explain myself again with a very narrowed and concrete
> > example.  As I mentioned, I'm trying to wrap <testng> because I want
> > to teak some of its attributes, while leaving others "untouched" but
> > still exposed through my wrapper macro.
> >
> > One of these "untouched" attributes would be "timeout", defined here:
> >
> >
> > https://github.com/cbeust/testng/blob/master/src/main/java/org/testng/TestNGAntTask.java#L285-L287
> >
> > So, without a wrapper, using <testng timeout="4"> is different from
> > just <testng>.  In this case, the default value of timeout is null,
> > which cannot be represented in a build.xml file. Using the following:
> >
> > <macrodef name="mymacro">
> >   <attribute name="timeout" default=""/>
> >   <testng timeout="@{timeout}">...</testng>
> > </macrodef>
> >
> > Doesn't work with the default value because TestNGAntTask.setTimeout()
> > accepts an Integer which Ant cannot parse.
> >
> > Then, using <augment> seems like the way to go.  However, <testng> is
> > a task which executes immediately.  So none of the following options
> > work:
> >
> > <macrodef name="mymacro">
> >   <attribute name="timeout" default=""/>
> >   <sequential>
> >     <augment unless:blank="timeout" id="invocation" timeout="@{timeout}"/>
> >     <testng ref="invocation">...</testng>
> >   </sequential>
> > </macrodef>
> >
> > <macrodef name="mymacro">
> >   <attribute name="timeout" default=""/>
> >   <sequential>
> >     <testng ref="invocation">...</testng>
> >     <augment unless:blank="timeout" id="invocation" timeout="@{timeout}"/>
> >  </sequential>
> > </macrodef>
> >
> > The first one fails because the reference is defined after <agument>,
> > and the second one doesn't achieve the desired result because <testng>
> > is executed before being augmented.
> >
> > I guess the only two solutions left are, as Al Le suggested, use
> > Javascript or Java directly.  Which is a pity because reproducing the
> > functionality of <element optional="true" implicit="true"/> with
> > Javascript (e.g. via <scriptdef>) seems very tedious.
> >
> > Fortunately, none of the problematic TestNG's attributes are essential
> > right now, so I may end up implementing the wrapper in Java.
> >
> > Thank you all for your suggestions.
> >
> > On Mon, Feb 25, 2019 at 3:20 PM Jan Matèrne (jhm) <apache@materne.de>
> > wrote:
> > >
> > > Not sure why you want to wrap an existing Ant task ...
> > > If you want to provide default settings, you could use <presetdef>
> > >
> > > Jan
> > >
> > > > -----Urspr√ľngliche Nachricht-----
> > > > Von: Isaac Jurado [mailto:diptongo@gmail.com]
> > > > Gesendet: Sonntag, 24. Februar 2019 18:27
> > > > An: Ant Users List; gudnabrsam@gmail.com
> > > > Betreff: Re: Creating task/macro wrappers
> > > >
> > > > Thanks!  I didn't notice the <augment> task existed.
> > > >
> > > > However, I don't seem to wrap my head around it.  In particular, it
> > > > doesn't seem to support forward references.  This is what I tried:
> > > >
> > > > <macrodef name="mywrapper">
> > > >   <attribute name="timeOut" default=""/>
> > > >   <sequential>
> > > >     <augment unless:blank="timeOut" id="my-testng-invocation"
> > > > timeOut="@{timeOut}"/>
> > > >     <testng id="my-testng-invocation">...</testng>
> > > >   </sequentail>
> > > > </macrodef>
> > > >
> > > > The error I get is:
> > > >
> > > > java.lang.IllegalStateException: Unknown reference "my-testng-
> > > > invocation"
> > > >
> > > > Best regards.
> > > >
> > > > On Sat, Feb 23, 2019 at 5:41 PM Matt Benson <gudnabrsam@gmail.com>
> > > > wrote:
> > > > >
> > > > > Look into the <augment> task with conditionally executed (using
> > > > > if/unless namespaces, antcontrib <if>, or similar) <sequential>
> > > > blocks.
> > > > >
> > > > > HTH,
> > > > > Matt
> > > > >
> > > > > On Sat, Feb 23, 2019, 3:37 AM Isaac Jurado <diptongo@gmail.com>
> > > > wrote:
> > > > >
> > > > > > Hello Ant users,
> > > > > >
> > > > > > I'm trying to create a <macrodef> wrapper over <testng>.
 The idea
> > > > > > is to have my macro expose the same attributes that <testng>
> > > > > > supports, with as close semantics as possible.
> > > > > >
> > > > > > Unfortunately, I have no idea how to reproduce the presence/absence
> > > > > > behavior.  For instance, <testng> accepts a "timeout"
integer
> > > > > > attribute.  So in my macro I would do:
> > > > > >
> > > > > > <attribute name="timeout" default=""/>
> > > > > >
> > > > > > And then calling TestNG would be:
> > > > > >
> > > > > > <testng timeout="@{timeout}">...</testng>
> > > > > >
> > > > > > But this turns out not to be the same behavior as NOT specifying
> > > > the
> > > > > > timeout attribute.  In this case, Ant fails because empty string
> > > > > > does not parse to an integer.
> > > > > >
> > > > > > Is there a way to achieve my goal?  I would like to avoid writing
> > > > > > Java code.  But I don't mind resorting to Javascript in the
build
> > > > > > file (<script> or <scriptdef>).
> > > > > >
> > > > > > Thanks in advance.
> > > > > >
> > > > > > --
> > > > > > Isaac Jurado
> > > > > >
> > > > > > "The noblest pleasure is the joy of understanding"
> > > > > > Leonardo da Vinci
> > > > > >
> > > > > > -------------------------------------------------------------------
> > > > -
> > > > > > - To unsubscribe, e-mail: user-unsubscribe@ant.apache.org For
> > > > > > additional commands, e-mail: user-help@ant.apache.org
> > > > > >
> > > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Isaac Jurado
> > > >
> > > > "The noblest pleasure is the joy of understanding"
> > > > Leonardo da Vinci
> > > >
> > > > ---------------------------------------------------------------------
> > > > To unsubscribe, e-mail: user-unsubscribe@ant.apache.org For additional
> > > > commands, e-mail: user-help@ant.apache.org
> > >
> > >
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: user-unsubscribe@ant.apache.org
> > > For additional commands, e-mail: user-help@ant.apache.org
> > >
> >
> >
> > --
> > Isaac Jurado
> >
> > "The noblest pleasure is the joy of understanding"
> > Leonardo da Vinci
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: user-unsubscribe@ant.apache.org
> > For additional commands, e-mail: user-help@ant.apache.org
> >
> >



-- 
Isaac Jurado

"The noblest pleasure is the joy of understanding"
Leonardo da Vinci

---------------------------------------------------------------------
To unsubscribe, e-mail: user-unsubscribe@ant.apache.org
For additional commands, e-mail: user-help@ant.apache.org


Mime
View raw message