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

Mike Foody commented on CSV-173:
--------------------------------

Adding a record interface would also allow Mockito and other mocking tools to 
easily mock records. Given the choice though I'd go all the way and create the 
factory as well. I'd rather not mock a simple class like CSVRecord if I can 
avoid it.

> Update CSVRecord Constructor
> ----------------------------
>
>                 Key: CSV-173
>                 URL: https://issues.apache.org/jira/browse/CSV-173
>             Project: Commons CSV
>          Issue Type: Improvement
>          Components: Parser
>    Affects Versions: 1.2
>            Reporter: Mike Foody
>            Priority: Minor
>
> Thanks for the solid library. This is a simple change but the fact that 
> CSVRecord is final and doesn't have a public constructor means you end up 
> having to write more code than seems necessary just to instantiate instances 
> for test cases. If it was made non final then most mocking frameworks would 
> happily mock it or if it had a public constructor it could just be 
> instantiated. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to