Uploaded image for project: 'Commons CLI'
  1. Commons CLI
  2. CLI-132

MissingOptionException should contain a useful error message

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: Minor
        • Resolution: Fixed
        • Affects Version/s: 1.0, 1.1
        • Fix Version/s: 1.1
        • Component/s: CLI-1.x
        • Labels:
          None

          Description

          Most exception message contain a useful message string like "Missing argument for option:". The MissingOptionException contains only the missing options. Adding a message to the exception would make it easier to catch a ParseException and display the getMessage() string.

          1. Text File
            bug132.patch
            2 kB
            Brian Egge

            Issue Links

            is a clone of

            Improvement - An improvement or enhancement to an existing feature or task. CLI-131 Options class returns options in random order

            • Minor - Minor loss of function, or other problem where easy workaround is present.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              brianegge Brian Egge added a comment -

              Test case and fix.

              Show
              brianegge Brian Egge added a comment - Test case and fix.
              Hide
              Permalink
              bayard Henri Yandell added a comment -

              Applied (r544360).

              As it's an improvement, I wrapped the unit test into OptionsTest rather than having a separate bug test for it.

              Show
              bayard Henri Yandell added a comment - Applied (r544360). As it's an improvement, I wrapped the unit test into OptionsTest rather than having a separate bug test for it.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  brianegge Brian Egge
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development