From 6be9df9bec4bfa043f60b5bd0df4154f22eda70c Mon Sep 17 00:00:00 2001 From: Jean-Christophe GAY Date: Sat, 7 Feb 2015 15:08:07 +0100 Subject: [PATCH] Use short description by default Printing all modules status can be difficult to read for projects will lots of modules. --- .../java/fr/jcgay/maven/notifier/ConfigurationParser.java | 2 +- .../fr/jcgay/maven/notifier/ConfigurationParserTest.groovy | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/java/fr/jcgay/maven/notifier/ConfigurationParser.java b/src/main/java/fr/jcgay/maven/notifier/ConfigurationParser.java index c62922a..564981b 100644 --- a/src/main/java/fr/jcgay/maven/notifier/ConfigurationParser.java +++ b/src/main/java/fr/jcgay/maven/notifier/ConfigurationParser.java @@ -120,7 +120,7 @@ private boolean isWindows(String os) { public static enum Property { IMPLEMENTATION("notifier.implementation"), - SHORT_DESCRIPTION("notifier.message.short", "false"), + SHORT_DESCRIPTION("notifier.message.short", "true"), NOTIFY_WITH("notifyWith"); private final String key; diff --git a/src/test/groovy/fr/jcgay/maven/notifier/ConfigurationParserTest.groovy b/src/test/groovy/fr/jcgay/maven/notifier/ConfigurationParserTest.groovy index 95f4aa4..13059ee 100644 --- a/src/test/groovy/fr/jcgay/maven/notifier/ConfigurationParserTest.groovy +++ b/src/test/groovy/fr/jcgay/maven/notifier/ConfigurationParserTest.groovy @@ -55,7 +55,7 @@ class ConfigurationParserTest { Configuration result = parser.get(new Properties()); - assertThat result.isShortDescription() isFalse() + assertThat result.isShortDescription() isTrue() } @Test @@ -63,12 +63,12 @@ class ConfigurationParserTest { Properties properties = new Properties(); properties << [(IMPLEMENTATION.key()):('test')] - properties << [(Property.SHORT_DESCRIPTION.key()):('true')] + properties << [(Property.SHORT_DESCRIPTION.key()):('false')] Configuration result = parser.get(properties) assertThat result.getImplementation() isEqualTo 'test' - assertThat result.isShortDescription() isTrue() + assertThat result.isShortDescription() isFalse() } @Test