[ 
https://issues.apache.org/jira/browse/SANDBOX-379?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13199626#comment-13199626
 ] 

Simone Tripodi commented on SANDBOX-379:
----------------------------------------

{quote}
As I said above, {{describe()}}, {{populate()}}, {{clone()}}, etc are the 
endpoints of a fluent call. So I thought putting them together in one test is a 
good idea.
{quote}

I don't, each method could involve a good number of test. See your contribution 
only for {{describe()}}.

{quote}
Okay, so do we create a test case for every method on {{BeanAccessor}}?
{quote}

Isn't what we are already doing? You already provided tests according to this...

{quote}
I think that would be a good idea. I don't like huge test classes. And a test 
class for all methods on {{BeanAccessor}} would be gigantic (as you mentioned).
{quote}

At least we have an agreement, looking forward your patch!
                
> [BeanUtils2] Implement describe() on DefaultBeanAccessor 
> ---------------------------------------------------------
>
>                 Key: SANDBOX-379
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-379
>             Project: Commons Sandbox
>          Issue Type: Improvement
>          Components: BeanUtils2
>    Affects Versions: Nightly Builds
>            Reporter: Benedikt Ritter
>         Attachments: SANDBOX-379.txt
>
>
> Implement the above mentioned method an corresponding unit tests

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to