Cedric/Julien,
Do you think we should change AnnotationTransformer such that its applied after a method selector is done with the filtering logic of pruning the *in-eligible* methods? Please advise.
Thomas,
I would be curious to understand why you feel that its problematic for your AnnotationTransformer implementation to get access to all methods (even the ones which will be filtered out and not executed)?
I am not sure if it’s the intended behavior or not. Perhaps Cedric/Julien can add more context around it.
I currently don’t see any negative impact because of this behavior (Except for an additional execution time, because TestNG has to spend time on applying the transformer on all methods). Maybe you could help elaborate your specific issue on why this is problematic for you.
Thanks & Regards
Krishnan Mahadevan
"All the desirable things in life are either illegal, expensive, fattening or in love with someone else!"
My Scribblings @ http://wakened-cognition.blogspot.com/
My Technical Scribbings @ http://rationaleemotions.wordpress.com/
--
You received this message because you are subscribed to the Google Groups "testng-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to testng-users...@googlegroups.com.
To post to this group, send email to testng...@googlegroups.com.
Visit this group at https://groups.google.com/group/testng-users.
For more options, visit https://groups.google.com/d/optout.
Thomas,
I think you might want to consider using IMethodInterceptor (as you also pointed out) instead of the IAnnotationTransformer.
Thanks & Regards
Krishnan Mahadevan
"All the desirable things in life are either illegal, expensive, fattening or in love with someone else!"
My Scribblings @ http://wakened-cognition.blogspot.com/
My Technical Scribbings @ http://rationaleemotions.wordpress.com/
From: <testng...@googlegroups.com> on behalf of Thomas H <nose...@gmail.com>
Reply-To: <testng...@googlegroups.com>
Date: Wednesday, June 14, 2017 at 8:28 PM
To: testng-users <testng...@googlegroups.com>
--