Skip to content

Commit 402dcaf

Browse files
committed
Rename validate method
1 parent 1c0f550 commit 402dcaf

File tree

2 files changed

+7
-7
lines changed

2 files changed

+7
-7
lines changed

src/main/java/de/donnerbart/split/TestSplitMain.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ public static void main(final @Nullable String @NotNull [] args) throws Exceptio
3131
Objects.requireNonNullElse(arguments.workingDirectory, Paths.get(System.getProperty("user.dir")))
3232
.toAbsolutePath()
3333
.normalize();
34-
if (!validate(arguments, workingDirectory)) {
34+
if (!validateArguments(arguments, workingDirectory)) {
3535
System.exit(1);
3636
}
3737
final var testSplit = new TestSplit(arguments.splitIndex,
@@ -46,7 +46,7 @@ public static void main(final @Nullable String @NotNull [] args) throws Exceptio
4646
}
4747

4848
@VisibleForTesting
49-
static boolean validate(final @NotNull Arguments arguments, final @NotNull Path workingDirectory) {
49+
static boolean validateArguments(final @NotNull Arguments arguments, final @NotNull Path workingDirectory) {
5050
if (arguments.splitTotal < 1) {
5151
System.out.println("--split-total must be greater than 0");
5252
return false;

src/test/java/de/donnerbart/split/TestSplitMainTest.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,30 +20,30 @@ class TestSplitMainTest {
2020
@Test
2121
void validateArguments() {
2222
jCommander.parse("-g", "**/*Test.java", "-t", "1", "-i", "0");
23-
assertThat(TestSplitMain.validate(arguments, tmp)).isTrue();
23+
assertThat(TestSplitMain.validateArguments(arguments, tmp)).isTrue();
2424
}
2525

2626
@Test
2727
void validateArguments_withZeroSplitTotal() {
2828
jCommander.parse("-g", "**/*Test.java", "-t", "0", "-i", "0");
29-
assertThat(TestSplitMain.validate(arguments, tmp)).isFalse();
29+
assertThat(TestSplitMain.validateArguments(arguments, tmp)).isFalse();
3030
}
3131

3232
@Test
3333
void validateArguments_withNegativeSplitTotal() {
3434
jCommander.parse("-g", "**/*Test.java", "-t", "-1", "-i", "0");
35-
assertThat(TestSplitMain.validate(arguments, tmp)).isFalse();
35+
assertThat(TestSplitMain.validateArguments(arguments, tmp)).isFalse();
3636
}
3737

3838
@Test
3939
void validateArguments_withTooSmallSplitIndex() {
4040
jCommander.parse("-g", "**/*Test.java", "-t", "1", "-i", "1");
41-
assertThat(TestSplitMain.validate(arguments, tmp)).isFalse();
41+
assertThat(TestSplitMain.validateArguments(arguments, tmp)).isFalse();
4242
}
4343

4444
@Test
4545
void validateArguments_withInvalidWorkingDirectory() {
4646
jCommander.parse("-g", "**/*Test.java", "-t", "1", "-i", "0");
47-
assertThat(TestSplitMain.validate(arguments, tmp.resolve("does-not-exist"))).isFalse();
47+
assertThat(TestSplitMain.validateArguments(arguments, tmp.resolve("does-not-exist"))).isFalse();
4848
}
4949
}

0 commit comments

Comments
 (0)