optional values not working

[fovea1959]

boolean isName() does not seem to work as expected: trying to make an optional switch that takes an optional value. Leaving the value off throws an ArgumentValidationException. See testOptionalPresentNoValue in this Junit test.

import static org.junit.Assert.*;
import org.junit.Test;
import com.lexicalscope.jewel.cli.*;

public class JewelParseTest {
  public interface TestOption {
    @Option
    String getName();
    boolean isName();
  }

  @Test
  public void testOptionalMissing() throws ArgumentValidationException {
    final TestOption rv = CliFactory.parseArguments(TestOption.class);
    System.out.println(rv.toString());
    assertFalse(rv.isName());
    assertNull(rv.getName());
  }

  @Test
  public void testOptionalPresentNoValue()
      throws ArgumentValidationException {
    final TestOption rv = CliFactory.parseArguments(TestOption.class, "--name");
    System.out.println(rv.toString());
    assertTrue(rv.isName());
    assertNull(rv.getName());
  }

  @Test
  public void testOptionalPresentWithValue() throws ArgumentValidationException {
    final TestOption rv = CliFactory.parseArguments(TestOption.class, "--name",
        "value");
    System.out.println(rv.toString());
    assertTrue(rv.isName());
    assertEquals(rv.getName(), "value");
  }
}

[lexicalscope]

If I understand you correctly, then this behaviour is by design. I think
the easiest way to create a option with an optional value to use:

@option(minimum=0, maximum=1) List getName();

boolean isName();

Which should give an optional argument that takes 0 or 1 values. You will
get an empty list if the option is specified but no value is given, and a
single element list if 1 value is given, and a validation exception if more
than 1 value is given.

On Wed, Nov 6, 2013 at 8:47 PM, fovea1959 notifications@github.com wrote:

boolean isName() does not seem to work as expected: trying to make an
optional switch that takes an optional value. Leaving the value off throws
an ArgumentValidationException. See testOptionalPresentNoValue in this
Junit test.

import static org.junit.Assert.;
import org.junit.Test;
import com.lexicalscope.jewel.cli.;

public class JewelParseTest {
public interface TestOption {
@option
String getName();
boolean isName();
}

@test
public void testOptionalMissing() throws ArgumentValidationException {
final TestOption rv = CliFactory.parseArguments(TestOption.class);
System.out.println(rv.toString());
assertFalse(rv.isName());
assertNull(rv.getName());
}

@test
public void testOptionalPresentNoValue()
throws ArgumentValidationException {
final TestOption rv = CliFactory.parseArguments(TestOption.class, "--name");
System.out.println(rv.toString());
assertTrue(rv.isName());
assertNull(rv.getName());
}

@test
public void testOptionalPresentWithValue() throws ArgumentValidationException {
final TestOption rv = CliFactory.parseArguments(TestOption.class, "--name",
"value");
System.out.println(rv.toString());
assertTrue(rv.isName());
assertEquals(rv.getName(), "value");
}
}

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/25
.

[fovea1959]

that works. I thought the other seemed reasonable for a single value use case, but can work with it.

Thanks.

---

From: Tim Wood notifications@github.com
To: lexicalscope/jewelcli jewelcli@noreply.github.com
Cc: fovea1959 dwegscheid@sbcglobal.net
Sent: Wednesday, November 6, 2013 4:14 PM
Subject: Re: [jewelcli] optional values not working (#25)

If I understand you correctly, then this behaviour is by design. I think
the easiest way to create a option with an optional value to use:

@option(minimum=0, maximum=1) List getName();

boolean isName();

Which should give an optional argument that takes 0 or 1 values. You will
get an empty list if the option is specified but no value is given, and a
single element list if 1 value is given, and a validation exception if more
than 1 value is given.

On Wed, Nov 6, 2013 at 8:47 PM, fovea1959 notifications@github.com wrote:

boolean isName() does not seem to work as expected: trying to make an
optional switch that takes an optional value. Leaving the value off throws
an ArgumentValidationException. See testOptionalPresentNoValue in this
Junit test.

import static org.junit.Assert.;
import org.junit.Test;
import com.lexicalscope.jewel.cli.;

public class JewelParseTest {
public interface TestOption {
@option
String getName();
boolean isName();
}

@test
public void testOptionalMissing() throws ArgumentValidationException {
final TestOption rv = CliFactory.parseArguments(TestOption.class);
System.out.println(rv.toString());
assertFalse(rv.isName());
assertNull(rv.getName());
}

@test
public void testOptionalPresentNoValue()
throws ArgumentValidationException {
final TestOption rv = CliFactory.parseArguments(TestOption.class, "--name");
System.out.println(rv.toString());
assertTrue(rv.isName());
assertNull(rv.getName());
}

@test
public void testOptionalPresentWithValue() throws ArgumentValidationException {
final TestOption rv = CliFactory.parseArguments(TestOption.class, "--name",
"value");
System.out.println(rv.toString());
assertTrue(rv.isName());
assertEquals(rv.getName(), "value");
}
}

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/25
.

—
Reply to this email directly or view it on GitHub.