From 0154cf09bbe655e4511a12b67c2870d945d478ca Mon Sep 17 00:00:00 2001 From: mcilwain Date: Fri, 23 Mar 2018 12:52:33 -0700 Subject: [PATCH] Change dash to underscore in param name for consistency ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=190261145 --- java/google/registry/tools/VerifyOteCommand.java | 4 ++-- javatests/google/registry/tools/VerifyOteCommandTest.java | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/java/google/registry/tools/VerifyOteCommand.java b/java/google/registry/tools/VerifyOteCommand.java index 2799ce91f..bf629c142 100644 --- a/java/google/registry/tools/VerifyOteCommand.java +++ b/java/google/registry/tools/VerifyOteCommand.java @@ -48,7 +48,7 @@ final class VerifyOteCommand implements ServerSideCommand { private List mainParameters = new ArrayList<>(); @Parameter( - names = "--check-all", + names = "--check_all", description = "Check the OT&E pass status of all active registrars") private boolean checkAll; @@ -74,7 +74,7 @@ final class VerifyOteCommand implements ServerSideCommand { } checkArgument( 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) { // 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 diff --git a/javatests/google/registry/tools/VerifyOteCommandTest.java b/javatests/google/registry/tools/VerifyOteCommandTest.java index 2d16b3468..62040d6c0 100644 --- a/javatests/google/registry/tools/VerifyOteCommandTest.java +++ b/javatests/google/registry/tools/VerifyOteCommandTest.java @@ -78,6 +78,6 @@ public class VerifyOteCommandTest extends CommandTestCase { assertThrows(IllegalArgumentException.class, () -> runCommand("")); assertThat(thrown) .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."); } }