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

Behrang Saeedzadeh commented on IO-513:
---------------------------------------

Hi Gary,

I added unit tests for NPE.

Also what pattern do you suggest for unit test method names?

Looks like some test methods are {{camelCase}} (e.g. 
{{testAsBufferedOutputStreamWithBufferSize}}) and some are 
{{camelCase_MixedWithUnderscore}} (e.g. 
{{testToByteArray_InputStream_SizeIllegal}}).

Or are you suggesting to rename, for example, 
{{testResourceToString_ExistingResourceAtRootPackage}} to 
{{testResourceToString_ShouldReadExistingResourceAtRootPackage}}?

Cheers,
Behrang

> Add convenience methods for reading class path resources
> --------------------------------------------------------
>
>                 Key: IO-513
>                 URL: https://issues.apache.org/jira/browse/IO-513
>             Project: Commons IO
>          Issue Type: New Feature
>          Components: Utilities
>            Reporter: Behrang Saeedzadeh
>            Priority: Minor
>              Labels: beginner, features, github-import, newbie
>
> Add convenience methods to {{IOUtils}} for reading class path resources and 
> returning them as {{String}}, {{byte[]}}, and {{URL}} respectively.
> Github PR: https://github.com/apache/commons-io/pull/17



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

Reply via email to