diff options
author | Jim Hahn <jrh3@att.com> | 2019-06-13 12:34:17 -0400 |
---|---|---|
committer | Jim Hahn <jrh3@att.com> | 2019-06-13 16:10:38 -0400 |
commit | fe5d78724f723a451ddc0d7cc41d6fc60092b314 (patch) | |
tree | 9913cc94014b3e99774e9ec5e39a7211046a765a /utils/src/test | |
parent | ea262e6da52fd4da0733f02998f87aebaf502ddb (diff) |
More sonar fixes in policy/common
Note: this does not increase code coverage, but should fix other
code issues.
Resolved cyclomatic complexity issue in ParameterValidationResult.
Refactored duplicate code in GroupValidationResult.
Removed IOException from NetworkUtil "throws".
Replaced null/empty string tests with StringUtils.isBlank().
Added @FunctionalInterface where needed.
Replaced anonymous classes with lambda expressions.
Replaced duplicate strings with a constant.
Added private constructors for utility classes.
Removed sleep() from tests.
Removed unused parameter from method call.
Made some protected methods private.
Compute integrity monitor's state-transition table once.
Use for-loop instead of iterator.
Moved constructors.
Fixed some checkstyle issues (tabs => spaces, trailing spaces).
Change-Id: I9a962ca45c4ff3f212c6014da799d06f07b232ef
Issue-ID: POLICY-1791
Signed-off-by: Jim Hahn <jrh3@att.com>
Diffstat (limited to 'utils/src/test')
0 files changed, 0 insertions, 0 deletions