zydronium/google-nomulus
1
0
Fork 0
mirror of https://github.com/google/nomulus.git synced 2025-05-14 00:17:20 +02:00
Code Issues Projects Releases Packages Wiki Activity

Change dash to underscore in param name for consistency

Browse source 
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=190261145
This commit is contained in:
mcilwain 2018-03-23 12:52:33 -07:00 • committed by jianglai
parent 89d8ba93f2
commit 0154cf09bb
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
java/google/registry/tools/VerifyOteCommand.java
View file

@ -48,7 +48,7 @@ final class VerifyOteCommand implements ServerSideCommand {
private List<String> mainParameters = new ArrayList<>(); private List<String> mainParameters = new ArrayList<>();
@Parameter( @Parameter(
names = "--check-all", names = "--check_all",
description = "Check the OT&E pass status of all active registrars") description = "Check the OT&E pass status of all active registrars")
private boolean checkAll; private boolean checkAll;
@ -74,7 +74,7 @@ final class VerifyOteCommand implements ServerSideCommand {
} }
checkArgument( checkArgument(
mainParameters.isEmpty() == checkAll, mainParameters.isEmpty() == checkAll,
"Must provide at least one registrar name, or supply --check-all with no names."); "Must provide at least one registrar name, or supply --check_all with no names.");
for (String clientId : mainParameters) { for (String clientId : mainParameters) {
// OT&E registrars are created with clientIDs of registrarName-[1-4], but this command is // OT&E registrars are created with clientIDs of registrarName-[1-4], but this command is
// passed registrarName. So check the existence of the first persisted registrar to see if // passed registrarName. So check the existence of the first persisted registrar to see if

2
javatests/google/registry/tools/VerifyOteCommandTest.java
View file

@ -78,6 +78,6 @@ public class VerifyOteCommandTest extends CommandTestCase<VerifyOteCommand> {
assertThrows(IllegalArgumentException.class, () -> runCommand("")); assertThrows(IllegalArgumentException.class, () -> runCommand(""));
assertThat(thrown) assertThat(thrown)
.hasMessageThat() .hasMessageThat()
.contains("Must provide at least one registrar name, or supply --check-all with no names."); .contains("Must provide at least one registrar name, or supply --check_all with no names.");
} }
} }