commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Simone Tripodi (Commented) (JIRA)" <>
Subject [jira] [Commented] (SANDBOX-379) [BeanUtils2] Implement describe() on DefaultBeanAccessor
Date Fri, 03 Feb 2012 10:05:53 GMT


Simone Tripodi commented on SANDBOX-379:

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.

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

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

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

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).

At least we have an agreement, looking forward your patch!
> [BeanUtils2] Implement describe() on DefaultBeanAccessor 
> ---------------------------------------------------------
>                 Key: SANDBOX-379
>                 URL:
>             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:!default.jspa
For more information on JIRA, see:


View raw message