Skip to content

Commit 5c0d4a4

Browse files
committed
Fixed sonar issues
1 parent e936f18 commit 5c0d4a4

2 files changed

Lines changed: 11 additions & 8 deletions

File tree

bugdna-core/src/main/java/io/github/bugdna/BugDnaAssertions.java

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@
77
*/
88
public final class BugDnaAssertions {
99

10+
private static final String EXPECTED_MUST_NOT_BE_NULL = "expected must not be null";
11+
1012
private BugDnaAssertions() {
1113
}
1214

@@ -44,7 +46,7 @@ private FingerprintAssert(Fingerprint fingerprint) {
4446
public FingerprintAssert hasCategory(FailureCategory expected) {
4547
FailureCategory requiredExpected = Objects.requireNonNull(
4648
expected,
47-
"expected must not be null"
49+
EXPECTED_MUST_NOT_BE_NULL
4850
);
4951
if (fingerprint.getCategory() != requiredExpected) {
5052
fail("category", requiredExpected, fingerprint.getCategory());
@@ -61,7 +63,7 @@ public FingerprintAssert hasCategory(FailureCategory expected) {
6163
public FingerprintAssert hasFamily(FailureFamily expected) {
6264
FailureFamily requiredExpected = Objects.requireNonNull(
6365
expected,
64-
"expected must not be null"
66+
EXPECTED_MUST_NOT_BE_NULL
6567
);
6668
if (fingerprint.getFamily() != requiredExpected) {
6769
fail("family", requiredExpected, fingerprint.getFamily());
@@ -78,7 +80,7 @@ public FingerprintAssert hasFamily(FailureFamily expected) {
7880
public FingerprintAssert hasRootCause(Class<? extends Throwable> expected) {
7981
Class<? extends Throwable> requiredExpected = Objects.requireNonNull(
8082
expected,
81-
"expected must not be null"
83+
EXPECTED_MUST_NOT_BE_NULL
8284
);
8385
return hasRootCause(requiredExpected.getName());
8486
}
@@ -92,7 +94,7 @@ public FingerprintAssert hasRootCause(Class<? extends Throwable> expected) {
9294
public FingerprintAssert hasRootCause(String expected) {
9395
String requiredExpected = Objects.requireNonNull(
9496
expected,
95-
"expected must not be null"
97+
EXPECTED_MUST_NOT_BE_NULL
9698
);
9799
if (!fingerprint.getRootCause().equals(requiredExpected)) {
98100
fail("root cause", requiredExpected, fingerprint.getRootCause());
@@ -109,7 +111,7 @@ public FingerprintAssert hasRootCause(String expected) {
109111
public FingerprintAssert hasId(String expected) {
110112
String requiredExpected = Objects.requireNonNull(
111113
expected,
112-
"expected must not be null"
114+
EXPECTED_MUST_NOT_BE_NULL
113115
);
114116
if (!fingerprint.getId().equals(requiredExpected)) {
115117
fail("id", requiredExpected, fingerprint.getId());
@@ -126,7 +128,7 @@ public FingerprintAssert hasId(String expected) {
126128
public FingerprintAssert hasSignature(String expected) {
127129
String requiredExpected = Objects.requireNonNull(
128130
expected,
129-
"expected must not be null"
131+
EXPECTED_MUST_NOT_BE_NULL
130132
);
131133
if (!fingerprint.getSignature().equals(requiredExpected)) {
132134
fail("signature", requiredExpected, fingerprint.getSignature());
@@ -143,7 +145,7 @@ public FingerprintAssert hasSignature(String expected) {
143145
public FingerprintAssert hasQualifiedSignature(String expected) {
144146
String requiredExpected = Objects.requireNonNull(
145147
expected,
146-
"expected must not be null"
148+
EXPECTED_MUST_NOT_BE_NULL
147149
);
148150
if (!fingerprint.getQualifiedSignature().equals(requiredExpected)) {
149151
fail(

bugdna-core/src/test/java/io/github/bugdna/BugDnaAssertionsTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,10 +44,11 @@ void failsWithHelpfulMessageWhenValuesDiffer() {
4444
"find",
4545
10
4646
));
47+
BugDnaAssertions.FingerprintAssert assertion = assertThat(fingerprint);
4748

4849
AssertionError error = assertThrows(
4950
AssertionError.class,
50-
() -> assertThat(fingerprint).hasCategory(FailureCategory.NETWORK)
51+
() -> assertion.hasCategory(FailureCategory.NETWORK)
5152
);
5253

5354
assertEquals(

0 commit comments

Comments
 (0)