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

Heinrich Bohne commented on NUMBERS-130:
----------------------------------------

Yes, it does not really matter; to be honest, I just wanted to make the IDE 
warnings go away (of course, this does not mean that I would not find a 
package-private access preferable over a {{public}} access without the IDE 
warnings, since I think it's generally a good idea to reduce scope to the 
smallest extent necessary, so it's not like I intended to sacrifice code design 
just to shut the IDE up, but I probably wouldn't have cared enough to do 
anything about it). But it seems that, in the cases you described, the 
practical benefits of leaving the test methods and classes {{public}} outweigh 
whatever would be gained from making them package-private (which would be 
probably more of a cosmetic nature than anything of real practical use).

> Test classes and methods can be package-private
> -----------------------------------------------
>
>                 Key: NUMBERS-130
>                 URL: https://issues.apache.org/jira/browse/NUMBERS-130
>             Project: Commons Numbers
>          Issue Type: Improvement
>    Affects Versions: 1.0
>            Reporter: Heinrich Bohne
>            Priority: Minor
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> With JUnit 5, test classes and methods don't need to be {{public}}, unlike 
> with JUnit 4. It suffices if they are package-private.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to