Use JUnit @Nested classes for tests instead of comment sections

[bvkatwijk]

Fix #2951

• Update Generator.sc so that generated tests use @Nested classes
• Use @Nested classes in all standalone tests
• Use @Nested classes in inherited tests
  • Not sure if this is possible at the moment - Inheritance and nested classes don't mix well, especially in java 8 which is the current language level
• Decide on style for @Nested classes
  • Class name sometimes clashes with other class names, so I appended Test and used @DisplayName

Notes:

• Used a script to do some bulk refactoring (sed may behave differently across OSes):

find ./src -name "*Test.java" -type f -exec sed -E -i '' \
    's|// -- ([a-zA-Z0-9]+)$|}\n\n    @Nested\n    @DisplayName("\1")\n    class \CAPS\1Test {|' {} +
find ./src -type f -name '*Test.java' -exec perl -i -pe 's/\bCAPS([a-zA-Z])([a-zA-Z]*)/\u\1\2/g' {} +

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.74%. Comparing base (e43e44b) to head (1feab74).
Report is 85 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #2960   +/-   ##
=========================================
  Coverage     92.74%   92.74%           
- Complexity     5247     5262   +15     
=========================================
  Files            89       89           
  Lines         12538    12552   +14     
  Branches       1604     1598    -6     
=========================================
+ Hits          11628    11641   +13     
- Misses          721      723    +2     
+ Partials        189      188    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bvkatwijk]

I changed these to non-static since static inner classes are not supported in JDK 8

[pivovarit]

note: there's now main branch, which is ex-version/0.x - for now, there's no reason to target version/1.x