opendev/gerrit
Code Issues Proposed changes

Use values() to set StringListConfig from Annotation

Browse Source 
Change-Id: Idb7e5834bd24bfae58dc021872fd2b1d42c09af9
This commit is contained in:
Patrick Hiesel
2017-02-08 08:48:00 +01:00
parent 7dbec98c26
commit 265e0f6898
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-acceptance-framework/src/test/java/com/google/gerrit/acceptance/ConfigAnnotationParser.java
View File

@@ -55,7 +55,7 @@ class ConfigAnnotationParser {
if (!Strings.isNullOrEmpty(c.value())) { if (!Strings.isNullOrEmpty(c.value())) {
cfg.setString(l.get(0), l.get(1), l.get(2), c.value()); cfg.setString(l.get(0), l.get(1), l.get(2), c.value());
} else { } else {
cfg.setStringList(l.get(0), l.get(1), l.get(2), Arrays.asList(c.value())); cfg.setStringList(l.get(0), l.get(1), l.get(2), Arrays.asList(c.values()));
} }
} else { } else {
throw new IllegalArgumentException( throw new IllegalArgumentException(