diff options
author | Simon Richard Clarkstone <s.r.clarkstone@dunelm.org.uk> | 2015-08-09 14:55:59 +0100 |
---|---|---|
committer | Simon Richard Clarkstone <s.r.clarkstone@dunelm.org.uk> | 2015-08-09 14:55:59 +0100 |
commit | 079dadd33129c4c8e4b422b791e6feb69585b5e7 (patch) | |
tree | ea760fb077ea376e879267cf9ea20b758e6732c9 /src | |
parent | db2567a9f4464faf7dd898cf3f69f3066944a325 (diff) | |
download | jcommander-079dadd33129c4c8e4b422b791e6feb69585b5e7.tar.gz |
Tweaked tests to work on Java 7 and 8.
Diffstat (limited to 'src')
-rw-r--r-- | src/test/java/com/beust/jcommander/JCommanderTest.java | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/src/test/java/com/beust/jcommander/JCommanderTest.java b/src/test/java/com/beust/jcommander/JCommanderTest.java index 7cf66eb..488fcb5 100644 --- a/src/test/java/com/beust/jcommander/JCommanderTest.java +++ b/src/test/java/com/beust/jcommander/JCommanderTest.java @@ -348,6 +348,9 @@ public class JCommanderTest { @Parameter(names = "--extensions", splitter = HiddenParameterSplitter.class) public List<String> extensions; } + if (HiddenParameterSplitter.class.getConstructors().length == 0) { + return; // Compiler has optimised away the private constructor + } Args args = new Args(); new JCommander(args, "--extensions", ".txt;.md"); @@ -641,9 +644,16 @@ public class JCommanderTest { List<ChoiceType> expected = Arrays.asList(ChoiceType.ONE, ChoiceType.Two); Assert.assertEquals(expected, args.choices); - Assert.assertEquals(jc.getParameters().get(0).getDescription(), - "Options: " + EnumSet.allOf((Class<? extends Enum>) ArgsEnum.ChoiceType.class)); + for (ParameterDescription param : jc.getParameters()) { + // order can vary depending on JDK version + if (param.getLongestName().equals("-choice")) { + Assert.assertEquals(param.getDescription(), + "Options: " + EnumSet.allOf((Class<? extends Enum>) ArgsEnum.ChoiceType.class)); + return; + } + } + Assert.fail("Could not find -choice parameter."); } public void enumArgsCaseInsensitive() { |