diff options
author | 2017-05-29 11:00:40 +0200 | |
---|---|---|
committer | 2017-05-29 14:08:41 +0200 | |
commit | bf6ef0f6051e7c8d9e87c42dd55b8a3a36044a78 (patch) | |
tree | 185f4340f22c2cdfa20dd71e4f7f2e10b0fde3c6 /src/test/java/com/google/devtools/common/options/OptionsTest.java | |
parent | 506fb1a07fb8fac54ef649b050156e13f6098f46 (diff) |
Migrate tests to Truth.
RELNOTES: None.
PiperOrigin-RevId: 157378037
Diffstat (limited to 'src/test/java/com/google/devtools/common/options/OptionsTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/common/options/OptionsTest.java | 46 |
1 files changed, 26 insertions, 20 deletions
diff --git a/src/test/java/com/google/devtools/common/options/OptionsTest.java b/src/test/java/com/google/devtools/common/options/OptionsTest.java index f6c75d5086..24b435f614 100644 --- a/src/test/java/com/google/devtools/common/options/OptionsTest.java +++ b/src/test/java/com/google/devtools/common/options/OptionsTest.java @@ -115,9 +115,9 @@ public class OptionsTest { assertThat(webFlags.host).isEqualTo("www.google.com"); assertThat(webFlags.port).isEqualTo(80); - assertThat(webFlags.isDebugging).isEqualTo(false); + assertThat(webFlags.isDebugging).isFalse(); assertThat(webFlags.triState).isEqualTo(TriState.AUTO); - assertThat(remainingArgs.length).isEqualTo(0); + assertThat(remainingArgs).hasLength(0); } @Test @@ -145,8 +145,8 @@ public class OptionsTest { assertThat(likeLeft).isEqualTo(left); assertThat(left).isNotEqualTo(right); assertThat(right).isNotEqualTo(left); - assertThat(left).isNotEqualTo(null); - assertThat(likeLeft).isNotEqualTo(null); + assertThat(left).isNotNull(); + assertThat(likeLeft).isNotNull(); assertThat(likeLeft.hashCode()).isEqualTo(likeLeft.hashCode()); assertThat(likeLeft.hashCode()).isEqualTo(left.hashCode()); // Strictly speaking this is not required for hashCode to be correct, @@ -186,8 +186,8 @@ public class OptionsTest { assertThat(webFlags.host).isEqualTo("google.com"); assertThat(webFlags.port).isEqualTo(8080); - assertThat(webFlags.isDebugging).isEqualTo(true); - assertThat(remainingArgs.length).isEqualTo(0); + assertThat(webFlags.isDebugging).isTrue(); + assertThat(remainingArgs).hasLength(0); } @Test @@ -201,8 +201,8 @@ public class OptionsTest { assertThat(webFlags.host).isEqualTo("google.com"); assertThat(webFlags.port).isEqualTo(8080); - assertThat(webFlags.isDebugging).isEqualTo(true); - assertThat(remainingArgs.length).isEqualTo(0); + assertThat(webFlags.isDebugging).isTrue(); + assertThat(remainingArgs).hasLength(0); } @Test @@ -210,7 +210,7 @@ public class OptionsTest { Options<HttpOptions> options = Options.parse(HttpOptions.class, new String[]{"--nodebug", "--notristate"}); HttpOptions webFlags = options.getOptions(); - assertThat(webFlags.isDebugging).isEqualTo(false); + assertThat(webFlags.isDebugging).isFalse(); assertThat(webFlags.triState).isEqualTo(TriState.NO); } @@ -219,7 +219,7 @@ public class OptionsTest { Options<HttpOptions> options = Options.parse(HttpOptions.class, new String[]{"-d-", "-t-"}); HttpOptions webFlags = options.getOptions(); - assertThat(webFlags.isDebugging).isEqualTo(false); + assertThat(webFlags.isDebugging).isFalse(); assertThat(webFlags.triState).isEqualTo(TriState.NO); } @@ -228,7 +228,7 @@ public class OptionsTest { Options<HttpOptions> options = Options.parse(HttpOptions.class, new String[]{"--debug=0", "--tristate=0"}); HttpOptions webFlags = options.getOptions(); - assertThat(webFlags.isDebugging).isEqualTo(false); + assertThat(webFlags.isDebugging).isFalse(); assertThat(webFlags.triState).isEqualTo(TriState.NO); } @@ -237,7 +237,7 @@ public class OptionsTest { Options<HttpOptions> options = Options.parse(HttpOptions.class, new String[]{"--debug=1", "--tristate=1"}); HttpOptions webFlags = options.getOptions(); - assertThat(webFlags.isDebugging).isEqualTo(true); + assertThat(webFlags.isDebugging).isTrue(); assertThat(webFlags.triState).isEqualTo(TriState.YES); } @@ -270,7 +270,7 @@ public class OptionsTest { Options.parse(HttpOptions.class, args); fail(); } catch (OptionsParsingException e) { - assertThat(e.getMessage()).isEqualTo("Unrecognized option: --unknown"); + assertThat(e).hasMessageThat().isEqualTo("Unrecognized option: --unknown"); } } @@ -281,7 +281,7 @@ public class OptionsTest { Options.parse(HttpOptions.class, args); fail(); } catch (OptionsParsingException e) { - assertThat(e.getMessage()).isEqualTo("Expected value after --port"); + assertThat(e).hasMessageThat().isEqualTo("Expected value after --port"); } } @@ -314,7 +314,8 @@ public class OptionsTest { Options.parse(HttpOptions.class, new String[]{"--debug=not_a_boolean"}); fail(); } catch (OptionsParsingException e) { - assertThat(e.getMessage()) + assertThat(e) + .hasMessageThat() .isEqualTo( "While parsing option --debug=not_a_boolean: " + "\'not_a_boolean\' is not a boolean"); @@ -327,7 +328,9 @@ public class OptionsTest { Options.parse(HttpOptions.class, new String[]{"--nodebug=1"}); fail(); } catch (OptionsParsingException e) { - assertThat(e.getMessage()).isEqualTo("Unexpected value after boolean option: --nodebug=1"); + assertThat(e) + .hasMessageThat() + .isEqualTo("Unexpected value after boolean option: --nodebug=1"); } } @@ -429,7 +432,8 @@ public class OptionsTest { Options.parse(UsesCustomConverter.class, args); fail(); } catch (OptionsParsingException e) { - assertThat(e.getMessage()) + assertThat(e) + .hasMessageThat() .isEqualTo( "While parsing option --url=a_malformed:url: " + "Could not convert 'a_malformed:url': " @@ -449,7 +453,7 @@ public class OptionsTest { Options.parse(HttpOptions.class, new String[]{"--no-debug"}); fail(); } catch (OptionsParsingException e) { - assertThat(e.getMessage()).isEqualTo("Unrecognized option: --no-debug"); + assertThat(e).hasMessageThat().isEqualTo("Unrecognized option: --no-debug"); } } @@ -477,8 +481,10 @@ public class OptionsTest { Options.parse(K.class, NO_ARGS).getOptions(); fail(); } catch (OptionsParser.ConstructionException e) { - assertThat(e.getCause()).isInstanceOf(IllegalStateException.class); - assertThat(e.getCause().getMessage()) + assertThat(e).hasCauseThat().isInstanceOf(IllegalStateException.class); + assertThat(e) + .hasCauseThat() + .hasMessageThat() .isEqualTo( "OptionsParsingException while retrieving default for " + "int1: 'null' is not an int"); |