TinyRadius icon indicating copy to clipboard operation
TinyRadius copied to clipboard

Java 8/Junit 5 Migration

Open gribnut opened this issue 3 years ago • 2 comments

Was having a fair number of warnings/issues compiling for Java 8/11 so thought I'd update some of the code for use with current supported JVMs. Mainly updates to migrate toward Java 8/11 code style.

  • Set type and checks for raw maps, lists, sets, classes, etc.
  • Migrate to Junit 5
  • Some additional unit tests for RadiusAttribute subclasses (additional unit tests still needed)
  • Minor code changes/cleanup for Java 8+

I only use library with RADIUS client so server code not tested. Feel free to merge changes or toss as see fit. Happy to make additional changes if needed.

gribnut avatar Dec 27 '21 23:12 gribnut

Appreciate the effort. I'll need some time to review all the changes.

ctran avatar Jan 04 '22 03:01 ctran

Merged recent CI changes and fixed sonatype lift warnings (the ones marked new anyway).

gribnut avatar Jan 16 '22 18:01 gribnut