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