globalreachtech / tinyradius-netty / 19740227474 / 1

86%

master: 86%

push

github

feature-adding-instrumentation-hooks to RadiusClient (#410)

* feature-adding-instrumentation-hooks to RadiusClient

* close timer. Group hooks in an interface

* fixed javadoc

* Applying initial suggestions by Horaceli

* use external TimeoutHandler instead of maxAttempts and timeoutMs in 'communicate'

* make sure timeout hook is only invoked on legitimate timeouts

* - generify hook interface
- move timeout event hook to timeoutHandler to avoid special case of final attempt
- add defaults to maintain previous interface
- renamed TimeoutHandler.onTimeout() to scheduleTimeout()

* fix bad annotation

---------

Co-authored-by: frodriguezg <frodriguezg@indra.es>
Co-authored-by: Horace Li <horaceli@gmail.com>
Co-authored-by: Horace Li <horace.l@goodnotes.com>

432 of 563 branches covered (76.73%)

Branch coverage included in aggregate %.

1357 of 1509 relevant lines covered (89.93%)

4.26 hits per line