Skip to content

Commit f4a28c0

Browse files
committed
separate testcases
1 parent 4f9c95b commit f4a28c0

1 file changed

Lines changed: 14 additions & 6 deletions

File tree

src/test/java/org/apache/commons/cli/PatternOptionBuilderTest.java

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -161,15 +161,23 @@ public void testURLPattern() throws Exception
161161
@Test
162162
public void testExistingFilePattern() throws Exception
163163
{
164-
final Options options = PatternOptionBuilder.parsePattern("f<g<");
164+
final Options options = PatternOptionBuilder.parsePattern("g<");
165165
final CommandLineParser parser = new PosixParser();
166-
final CommandLine line = parser.parse(options, new String[] { "-f", "non-existing.file", "-g", "src/test/resources/existing-readable.file" });
167-
168-
assertNull("option f parsed", line.getOptionObject("f"));
166+
final CommandLine line = parser.parse(options, new String[] { "-g", "src/test/resources/existing-readable.file" });
167+
168+
final Object parsedReadableFileStream = line.getOptionObject("g");
169169

170-
Object parsedReadableFileStream = line.getOptionObject("g");
171170
assertNotNull("option g not parsed", parsedReadableFileStream);
172-
assertEquals("option g not FileInputStream", FileInputStream.class, parsedReadableFileStream.getClass());
171+
assertTrue("option g not FileInputStream", parsedReadableFileStream instanceof FileInputStream);
172+
}
173+
174+
@Test
175+
public void testExistingFilePatternFileNotExist() throws Exception {
176+
final Options options = PatternOptionBuilder.parsePattern("f<");
177+
final CommandLineParser parser = new PosixParser();
178+
final CommandLine line = parser.parse(options, new String[] { "-f", "non-existing.file" });
179+
180+
assertNull("option f parsed", line.getOptionObject("f"));
173181
}
174182

175183
@Test

0 commit comments

Comments
 (0)