ant-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Antoine Levy-Lambert" <anto...@gmx.de>
Subject Re: Location in non-Task tasks
Date Sun, 10 Sep 2006 19:26:49 GMT
Bonjour Stefan,

-------- Original-Nachricht --------
Datum: Sat, 09 Sep 2006 22:03:16 +0200
Von: Stefan Bodewig <bodewig@apache.org>
An: dev@ant.apache.org
Betreff: Location in non-Task tasks

> Hi,
> 
> I just added the feature to print the line number an error occured on
> in AntUnit's plainlistener.
> 
> I must admit that my error messages are sometimes not chosen well
> enough so having the line number to know which assertion in a longer
> testcase failed helps me a lot in J/NUnit.  I wanted to provide the
> same functionality to AntUnit.
> 
> Unfortunately AssertTask extends ConditionBase and thus gets wrapped
> in a TaskAdapter.  This means it doesn't know where it is (inside of
> the build file) and cannnot add the information to the exception it
> throws.
> 
> I'd like to do the following:
> 
> (1) push the Location handling from Task down to ProjectComponent
> 
> (2) Modify TaskAdapter to reflect Location into the wrapped task much
> like it reflects the Project instance into it right now.
> 
> Does anybody object here?  An alternative to (1) would be to copy the
> logic from Task to ConditionBase.
> 
> (3) I'd like to take advantage of these changes in AssertTask.

+1 for all this


> 
> The later would either mean 
> 
> * AntUnit 1.0beta1 had to wait until after Ant 1.7.0beta2 (or RC1 or
>   whatever) was released
> 
> or
> 
> * we'd release AntUnit 1.0beta1 without my change (3) and modify
>   AssertTask once a matching Ant release was out.
> 
up to you. I might start a vote thread for Ant 1.7.0beta2.

Antoine

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


Mime
View raw message