Skip to content
  • Peter Vigier's avatar
    Feature/fix sonar warnings (#1226) · 44a85f20
    Peter Vigier authored
    * Fixed sonar warnings
    
    - "Cognitive Complexity"
    - "Do not use replaceAll when not using a regex"
    - java:S5869 - Character classes in regular expressions should not contain the same character twice
    - Improved bad name
    - Typos
    - reduced code duplications
    - Replaced hand-made wait-utility with Awaitility
    - Log messages
    - Duplicate code
    - Typos
    - Removed Thread.sleep, instead relaxed check condition
    - Removed use of deprecated API
    - Removed use of deprecated API
    - Added supress-warnings as I do not see a better way to write the tests
    - Removed Thread.sleep / redundant functionality to Awaitility
    - Fixed other warnings (use isZero, isEmpty, hasToString)
    - Removed/Reduced duplicate code
    - Added generics
    - Fixed asserts
    - removed: field.setAccessible(true) actually should not be needed for public static fields!
    - Too long constructor passes arguments in wrong order - how surprisingly...
    - Clean-up use of varargs arguments
    - Fixed regex
    - Fixed typos an...
    44a85f20
After you've reviewed these contribution guidelines, you'll be all set to contribute to this project.