Skip to content

Commit 00d2e91

Browse files
gmessnergmessner
committed
Replaced all instances of str.length() == 0 with str.isEmpty() per lamdaV suggestion.
1 parent 2267922 commit 00d2e91

18 files changed

+59
-59
lines changed

‎src/test/java/org/gitlab4j/api/TestAccessToken.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,15 +42,15 @@ public TestAccessToken() {
4242
public static void setup() {
4343

4444
String problems = "";
45-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
45+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
4646
problems += "TEST_HOST_URL cannot be empty\n";
4747
}
4848

49-
if (TEST_ACCESS_TOKEN == null || TEST_ACCESS_TOKEN.trim().length() == 0) {
49+
if (TEST_ACCESS_TOKEN == null || TEST_ACCESS_TOKEN.trim().isEmpty()) {
5050
problems += "TEST_ACCESS_TOKEN cannot be empty\n";
5151
}
5252

53-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
53+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
5454
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
5555
}
5656

‎src/test/java/org/gitlab4j/api/TestCommitsApi.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -58,19 +58,19 @@ public TestCommitsApi() {
5858
public static void setup() {
5959

6060
String problems = "";
61-
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().length() == 0) {
61+
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().isEmpty()) {
6262
problems += "TEST_NAMESPACE cannot be empty\n";
6363
}
6464

65-
if (TEST_PROJECT_NAME == null || TEST_PROJECT_NAME.trim().length() == 0) {
65+
if (TEST_PROJECT_NAME == null || TEST_PROJECT_NAME.trim().isEmpty()) {
6666
problems += "TEST_PROJECT_NAME cannot be empty\n";
6767
}
6868

69-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
69+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
7070
problems += "TEST_HOST_URL cannot be empty\n";
7171
}
7272

73-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
73+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
7474
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
7575
}
7676

‎src/test/java/org/gitlab4j/api/TestDeployKeysApi.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,15 @@ public TestDeployKeysApi() {
3838
public static void setup() {
3939

4040
String problems = "";
41-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
41+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
4242
problems += "TEST_HOST_URL cannot be empty\n";
4343
}
4444

45-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
45+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
4646
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
4747
}
4848

49-
if (TEST_USERNAME == null || TEST_USERNAME.trim().length() == 0) {
49+
if (TEST_USERNAME == null || TEST_USERNAME.trim().isEmpty()) {
5050
problems += "TEST_USER_NAME cannot be empty\n";
5151
}
5252

‎src/test/java/org/gitlab4j/api/TestEventsApi.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,19 +51,19 @@ public TestEventsApi() {
5151
public static void setup() {
5252

5353
String problems = "";
54-
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().length() == 0) {
54+
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().isEmpty()) {
5555
problems += "TEST_NAMESPACE cannot be empty\n";
5656
}
5757

58-
if (TEST_PROJECT_NAME == null || TEST_PROJECT_NAME.trim().length() == 0) {
58+
if (TEST_PROJECT_NAME == null || TEST_PROJECT_NAME.trim().isEmpty()) {
5959
problems += "TEST_PROJECT_NAME cannot be empty\n";
6060
}
6161

62-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
62+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
6363
problems += "TEST_HOST_URL cannot be empty\n";
6464
}
6565

66-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
66+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
6767
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
6868
}
6969

‎src/test/java/org/gitlab4j/api/TestGitLabApi.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,11 +45,11 @@ public TestGitLabApi() {
4545
public static void setup() {
4646

4747
String problems = "";
48-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
48+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
4949
problems += "TEST_HOST_URL cannot be empty\n";
5050
}
5151

52-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
52+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
5353
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
5454
}
5555

‎src/test/java/org/gitlab4j/api/TestGroupApi.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,15 +61,15 @@ public TestGroupApi() {
6161
public static void setup() {
6262

6363
String problems = "";
64-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
64+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
6565
problems += "TEST_HOST_URL cannot be empty\n";
6666
}
6767

68-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
68+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
6969
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
7070
}
7171

72-
if (TEST_USERNAME == null || TEST_USERNAME.trim().length() == 0) {
72+
if (TEST_USERNAME == null || TEST_USERNAME.trim().isEmpty()) {
7373
problems += "TEST_USER_NAME cannot be empty\n";
7474
}
7575

‎src/test/java/org/gitlab4j/api/TestIgnoreCertificateErrors.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,11 @@ public TestIgnoreCertificateErrors() {
4040
public static void setup() {
4141

4242
String problems = "";
43-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
43+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
4444
problems += "TEST_HOST_URL cannot be empty\n";
4545
}
4646

47-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
47+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
4848
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
4949
}
5050

‎src/test/java/org/gitlab4j/api/TestIssuesApi.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -85,15 +85,15 @@ public TestIssuesApi() {
8585
public static void setup() {
8686

8787
String problems = "";
88-
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().length() == 0) {
88+
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().isEmpty()) {
8989
problems += "TEST_NAMESPACE cannot be empty\n";
9090
}
9191

92-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
92+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
9393
problems += "TEST_HOST_URL cannot be empty\n";
9494
}
9595

96-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
96+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
9797
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
9898
}
9999

‎src/test/java/org/gitlab4j/api/TestNamespaceApi.java‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,15 +46,15 @@ public TestNamespaceApi() {
4646
public static void setup() {
4747

4848
String problems = "";
49-
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().length() == 0) {
49+
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().isEmpty()) {
5050
problems += "TEST_NAMESPACE cannot be empty\n";
5151
}
5252

53-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
53+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
5454
problems += "TEST_HOST_URL cannot be empty\n";
5555
}
5656

57-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
57+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
5858
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
5959
}
6060

‎src/test/java/org/gitlab4j/api/TestNotesApi.java‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,19 +52,19 @@ public TestNotesApi() {
5252
public static void setup() {
5353

5454
String problems = "";
55-
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().length() == 0) {
55+
if (TEST_NAMESPACE == null || TEST_NAMESPACE.trim().isEmpty()) {
5656
problems += "TEST_NAMESPACE cannot be empty\n";
5757
}
5858

59-
if (TEST_PROJECT_NAME == null || TEST_PROJECT_NAME.trim().length() == 0) {
59+
if (TEST_PROJECT_NAME == null || TEST_PROJECT_NAME.trim().isEmpty()) {
6060
problems += "TEST_PROJECT_NAME cannot be empty\n";
6161
}
6262

63-
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().length() == 0) {
63+
if (TEST_HOST_URL == null || TEST_HOST_URL.trim().isEmpty()) {
6464
problems += "TEST_HOST_URL cannot be empty\n";
6565
}
6666

67-
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().length() == 0) {
67+
if (TEST_PRIVATE_TOKEN == null || TEST_PRIVATE_TOKEN.trim().isEmpty()) {
6868
problems += "TEST_PRIVATE_TOKEN cannot be empty\n";
6969
}
7070

0 commit comments

Comments
 (0)