cri 2.12.0 → 2.13.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e0e63fb09fa1cd1a483f34f9f535adca0ce5f543e975163584ecf43a509e4195
-  data.tar.gz: be17d3d95165270a73992dfa8a819704750d3b73b973a176f2d5c553a1934829
+  metadata.gz: 2aa633b78bf3bb39d2ddc9c5abc1ce94f27dd57bb1d130336acc58c1ae83ffcf
+  data.tar.gz: 2cc7102d387382bdfe15a0ccdb080d92c989f77716d217c0899a3ec688090886
 SHA512:
-  metadata.gz: 55dc3514760d66bf2c4a7edf8e06ba37947dc278efaf339ac8de24afc5c6f8ecfa4d651557351433763800327907ec4c25859c335350a95adcb2decad9cd313f
-  data.tar.gz: 3bda1e11790abbe1d2187961a1868ffcb1d255cf4d82a4f514e1a966bf72280fc34a0fe9f41303d7905024b092570220615120386940c7c429bad81ac5bd29ef
+  metadata.gz: 82e32293ba6c802449eeb1731742250bb5f5503cfdd1d05d8fd2e1b113ee3704e39a684cf9d87adf02c26becb95209bb34f32bc749e09017fb31b10dc5abde72
+  data.tar.gz: 5a51c230c68d3a4612eaae905605448997133418b12a703a1b154ac6c39a7e67e94305bbd680edfd6332b8fa806d9e3b94eb7d7c54529b0bc868f081f295d246

data/NEWS.md

@@ -1,6 +1,12 @@
 Cri News
 ========
 
+## 2.13.0
+
+Features:
+
+* Added support for explicitly specifying zero parameters using `#no_params` (#71)
+
 ## 2.12.0
 
 Features:

data/README.md

@@ -257,6 +257,30 @@ end
 The command in this example has one parameter named `filename`. This means that
 the command takes a single argument, named `filename`.
 
+If no parameters are specified, Cri performs no argument parsing or validation; any number of arguments is allowed. To explicitly specify that a command has no parameters, use `#no_params`:
+
+```ruby
+name        'reset'
+usage       'reset'
+summary     'resets the site'
+description '…'
+no_params
+
+run do |opts, args, cmd|
+  puts "Resetting…"
+end
+```
+
+```
+% my-tool reset x
+reset: incorrect number of arguments given: expected 0, but got 1
+% my-tool reset
+Resetting…
+%
+```
+
+A future version of Cri will likely make `#no_params` the default behavior.
+
 (*Why the distinction between argument and parameter?* A parameter is a name, e.g. `filename`, while an argument is a value for a parameter, e.g. `kitten.jpg`.)
 
 ### The run block

data/lib/cri/argument_list.rb

@@ -17,8 +17,9 @@ module Cri
 
     include Enumerable
 
-    def initialize(raw_arguments, param_defns)
+    def initialize(raw_arguments, explicitly_no_params, param_defns)
       @raw_arguments = raw_arguments
+      @explicitly_no_params = explicitly_no_params
       @param_defns = param_defns
 
       load
@@ -56,8 +57,8 @@ module Cri
       @arguments_array = @raw_arguments.reject { |a| a == '--' }.freeze
       @arguments_hash = {}
 
-      if @param_defns.empty?
-        # For now, don’t check arguments when no parameter definitions are given.
+      if !@explicitly_no_params && @param_defns.empty?
+        # No parameters defined; ignore
         return
       end
 

data/lib/cri/command.rb

@@ -93,6 +93,10 @@ module Cri
     # @return [Array<Hash>] The list of parameter definitions
     attr_accessor :parameter_definitions
 
+    # @return [Boolean] Whether or not this command has parameters
+    attr_accessor :explicitly_no_params
+    alias explicitly_no_params? explicitly_no_params
+
     # @return [Proc] The block that should be executed when invoking this
     #   command (ignored for commands with subcommands)
     attr_accessor :block
@@ -151,6 +155,7 @@ module Cri
       @commands           = Set.new
       @option_definitions = Set.new
       @parameter_definitions = []
+      @explicitly_no_params = false
       @default_subcommand_name = nil
     end
 
@@ -319,6 +324,7 @@ module Cri
           opts_and_args,
           global_option_definitions,
           parameter_definitions,
+          explicitly_no_params?,
         )
         handle_errors_while { parser.run }
         local_opts  = parser.options
@@ -378,6 +384,7 @@ module Cri
         opts_and_args,
         global_option_definitions,
         parameter_definitions,
+        explicitly_no_params?,
       )
       parser.delegate = delegate
       handle_errors_while { parser.run }

data/lib/cri/command_dsl.rb

@@ -4,6 +4,31 @@ module Cri
   # The command DSL is a class that is used for building and modifying
   # commands.
   class CommandDSL
+    # Error that will be raised when specifying a parameter after the command is
+    # already declared as taken no params.
+    class AlreadySpecifiedAsNoParams < Cri::Error
+      def initialize(param, command)
+        @param = param
+        @command = command
+      end
+
+      def message
+        "Attempted to specify a parameter #{@param.inspect} to the command #{@command.name.inspect}, which is already specified as taking no params. Suggestion: remove the #no_params call."
+      end
+    end
+
+    # Error that will be raised when declaring the command as taking no
+    # parameters, when the command is already declared with parameters.
+    class AlreadySpecifiedWithParams < Cri::Error
+      def initialize(command)
+        @command = command
+      end
+
+      def message
+        "Attempted to declare the command #{@command.name.inspect} as taking no parameters, but some parameters are already declared for this command. Suggestion: remove the #no_params call."
+      end
+    end
+
     # @return [Cri::Command] The built command
     attr_reader :command
 
@@ -146,9 +171,21 @@ module Cri
     #
     # @param [Symbol] name The name of the parameter
     def param(name)
+      if @command.explicitly_no_params?
+        raise AlreadySpecifiedAsNoParams.new(name, @command)
+      end
+
       @command.parameter_definitions << Cri::ParamDefinition.new(name: name)
     end
 
+    def no_params
+      if @command.parameter_definitions.any?
+        raise AlreadySpecifiedWithParams.new(@command)
+      end
+
+      @command.explicitly_no_params = true
+    end
+
     # Adds a new option with a required argument to the command. If a block is
     # given, it will be executed when the option is successfully parsed.
     #

data/lib/cri/option_parser.rb

@@ -114,10 +114,11 @@ module Cri
     #
     # @param [Array<Cri::ParamDefinition>] param_defns An array of parameter
     #   definitions
-    def initialize(arguments_and_options, option_defns, param_defns)
+    def initialize(arguments_and_options, option_defns, param_defns, explicitly_no_params)
       @unprocessed_arguments_and_options = arguments_and_options.dup
       @option_defns = option_defns
       @param_defns = param_defns
+      @explicitly_no_params = explicitly_no_params
 
       @options       = {}
       @raw_arguments = []
@@ -179,7 +180,7 @@ module Cri
     # @return [Cri::ArgumentList] The list of arguments that have already been
     #   parsed, excluding the -- separator.
     def arguments
-      ArgumentList.new(@raw_arguments, @param_defns)
+      ArgumentList.new(@raw_arguments, @explicitly_no_params, @param_defns)
     end
 
     private

data/lib/cri/version.rb

@@ -2,5 +2,5 @@
 
 module Cri
   # The current Cri version.
-  VERSION = '2.12.0'
+  VERSION = '2.13.0'
 end

data/test/test_argument_list.rb

@@ -5,7 +5,7 @@ require 'helper'
 module Cri
   class ArgumentListTestCase < Cri::TestCase
     def test_empty
-      args = Cri::ArgumentList.new([], [])
+      args = Cri::ArgumentList.new([], false, [])
 
       assert_equal([], args.to_a)
       assert(args.empty?)
@@ -15,7 +15,7 @@ module Cri
     end
 
     def test_no_param_defns
-      args = Cri::ArgumentList.new(%w[a b c], [])
+      args = Cri::ArgumentList.new(%w[a b c], false, [])
 
       assert_equal(%w[a b c], args.to_a)
       refute(args.empty?)
@@ -28,13 +28,13 @@ module Cri
     end
 
     def test_enum
-      args = Cri::ArgumentList.new(%w[a b c], [])
+      args = Cri::ArgumentList.new(%w[a b c], false, [])
 
       assert_equal(%w[A B C], args.map(&:upcase))
     end
 
     def test_no_method_error
-      args = Cri::ArgumentList.new(%w[a b c], [])
+      args = Cri::ArgumentList.new(%w[a b c], false, [])
 
       refute args.respond_to?(:oink)
       assert_raises(NoMethodError, 'x') do
@@ -43,14 +43,14 @@ module Cri
     end
 
     def test_dash_dash
-      args = Cri::ArgumentList.new(%w[a -- b -- c], [])
+      args = Cri::ArgumentList.new(%w[a -- b -- c], false, [])
 
       assert_equal(%w[a b c], args.to_a)
     end
 
     def test_one_param_defn_matched
       param_defns = [Cri::ParamDefinition.new(name: 'filename')]
-      args = Cri::ArgumentList.new(%w[notbad.jpg], param_defns)
+      args = Cri::ArgumentList.new(%w[notbad.jpg], false, param_defns)
 
       assert_equal(['notbad.jpg'], args.to_a)
       assert_equal(1, args.size)
@@ -62,7 +62,7 @@ module Cri
       param_defns = [Cri::ParamDefinition.new(name: 'filename')]
 
       exception = assert_raises(Cri::ArgumentList::ArgumentCountMismatchError) do
-        Cri::ArgumentList.new(%w[notbad.jpg verybad.jpg], param_defns)
+        Cri::ArgumentList.new(%w[notbad.jpg verybad.jpg], false, param_defns)
       end
       assert_equal('incorrect number of arguments given: expected 1, but got 2', exception.message)
     end
@@ -71,9 +71,24 @@ module Cri
       param_defns = [Cri::ParamDefinition.new(name: 'filename')]
 
       exception = assert_raises(Cri::ArgumentList::ArgumentCountMismatchError) do
-        Cri::ArgumentList.new(%w[], param_defns)
+        Cri::ArgumentList.new(%w[], false, param_defns)
       end
       assert_equal('incorrect number of arguments given: expected 1, but got 0', exception.message)
     end
+
+    def test_zero_params_zero_args
+      args = Cri::ArgumentList.new(%w[], false, [])
+
+      assert_equal([], args.to_a)
+      assert args.empty?
+      assert_equal(0, args.size)
+    end
+
+    def test_zero_params_one_arg
+      exception = assert_raises(Cri::ArgumentList::ArgumentCountMismatchError) do
+        Cri::ArgumentList.new(%w[a], true, [])
+      end
+      assert_equal('incorrect number of arguments given: expected 0, but got 1', exception.message)
+    end
   end
 end

data/test/test_command.rb

@@ -714,5 +714,47 @@ module Cri
       assert_equal [], lines(out)
       assert_equal ['publish: incorrect number of arguments given: expected 1, but got 0'], lines(err)
     end
+
+    def test_no_params_zero_args
+      dsl = Cri::CommandDSL.new
+      dsl.instance_eval do
+        name        'moo'
+        usage       'dunno whatever'
+        summary     'does stuff'
+        description 'This command does a lot of stuff.'
+        no_params
+
+        run do |_opts, args|
+        end
+      end
+      command = dsl.command
+
+      command.run([])
+    end
+
+    def test_no_params_one_arg
+      dsl = Cri::CommandDSL.new
+      dsl.instance_eval do
+        name        'moo'
+        usage       'dunno whatever'
+        summary     'does stuff'
+        description 'This command does a lot of stuff.'
+        no_params
+
+        run do |_opts, args|
+        end
+      end
+      command = dsl.command
+
+      out, err = capture_io_while do
+        err = assert_raises SystemExit do
+          command.run(['a'])
+        end
+        assert_equal 1, err.status
+      end
+
+      assert_equal [], lines(out)
+      assert_equal ['moo: incorrect number of arguments given: expected 0, but got 1'], lines(err)
+    end
   end
 end

data/test/test_command_dsl.rb

@@ -268,5 +268,35 @@ module Cri
       assert_equal({ foo: 'a', bar: 'b', qux: 'c' }, $args_num)
       assert_equal({ foo: 'a', bar: 'b', qux: 'c' }, $args_sym)
     end
+
+    def test_no_params_with_one_param_specified
+      dsl = Cri::CommandDSL.new
+      err = assert_raises Cri::CommandDSL::AlreadySpecifiedWithParams do
+        dsl.instance_eval do
+          name        'moo'
+          usage       'dunno whatever'
+          summary     'does stuff'
+          description 'This command does a lot of stuff.'
+          param :oink
+          no_params
+        end
+      end
+      assert_equal('Attempted to declare the command "moo" as taking no parameters, but some parameters are already declared for this command. Suggestion: remove the #no_params call.', err.message)
+    end
+
+    def test_one_param_with_no_params_specified
+      dsl = Cri::CommandDSL.new
+      err = assert_raises Cri::CommandDSL::AlreadySpecifiedAsNoParams do
+        dsl.instance_eval do
+          name        'moo'
+          usage       'dunno whatever'
+          summary     'does stuff'
+          description 'This command does a lot of stuff.'
+          no_params
+          param :oink
+        end
+      end
+      assert_equal('Attempted to specify a parameter :oink to the command "moo", which is already specified as taking no params. Suggestion: remove the #no_params call.', err.message)
+    end
   end
 end

data/test/test_option_parser.rb

@@ -8,7 +8,7 @@ module Cri
       input = %w[foo bar baz]
       opt_defns = []
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({}, parser.options)
       assert_equal(%w[foo bar baz], parser.arguments.to_a)
@@ -19,7 +19,7 @@ module Cri
       opt_defns = []
 
       assert_raises(Cri::OptionParser::IllegalOptionError) do
-        Cri::OptionParser.new(input, opt_defns, []).run
+        Cri::OptionParser.new(input, opt_defns, [], false).run
       end
     end
 
@@ -29,7 +29,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :forbidden },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert(!parser.options[:aaa])
     end
@@ -40,7 +40,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :forbidden },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert(parser.options[:aaa])
       assert_equal(%w[foo bar], parser.arguments.to_a)
@@ -52,7 +52,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :required },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ aaa: 'xxx' }, parser.options)
       assert_equal(%w[foo bar], parser.arguments.to_a)
@@ -64,7 +64,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :required },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ aaa: 'xxx' }, parser.options)
       assert_equal(%w[foo bar], parser.arguments.to_a)
@@ -77,7 +77,7 @@ module Cri
       ].map { |hash| make_opt_defn(hash) }
 
       assert_raises(Cri::OptionParser::OptionRequiresAnArgumentError) do
-        Cri::OptionParser.new(input, opt_defns, []).run
+        Cri::OptionParser.new(input, opt_defns, [], false).run
       end
     end
 
@@ -89,7 +89,7 @@ module Cri
       ].map { |hash| make_opt_defn(hash) }
 
       assert_raises(Cri::OptionParser::OptionRequiresAnArgumentError) do
-        Cri::OptionParser.new(input, opt_defns, []).run
+        Cri::OptionParser.new(input, opt_defns, [], false).run
       end
     end
 
@@ -99,7 +99,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :optional },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert(parser.options[:aaa])
       assert_equal(['foo'], parser.arguments.to_a)
@@ -111,7 +111,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :optional },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ aaa: 'xxx' }, parser.options)
       assert_equal(['foo'], parser.arguments.to_a)
@@ -125,7 +125,7 @@ module Cri
         { long: 'ccc', short: 'c', argument: :forbidden },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert(parser.options[:aaa])
       assert(parser.options[:bbb])
@@ -139,7 +139,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :forbidden },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert(parser.options[:aaa])
       assert_equal(%w[foo bar], parser.arguments.to_a)
@@ -152,7 +152,7 @@ module Cri
       ].map { |hash| make_opt_defn(hash) }
 
       assert_raises(Cri::OptionParser::OptionRequiresAnArgumentError) do
-        Cri::OptionParser.new(input, opt_defns, []).run
+        Cri::OptionParser.new(input, opt_defns, [], false).run
       end
     end
 
@@ -164,7 +164,7 @@ module Cri
         { long: 'ccc', short: 'c', argument: :forbidden },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert(parser.options[:aaa])
       assert(parser.options[:bbb])
@@ -180,7 +180,7 @@ module Cri
         { long: 'ccc', short: 'c', argument: :forbidden },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal('bar', parser.options[:aaa])
       assert(parser.options[:bbb])
@@ -196,7 +196,7 @@ module Cri
       ].map { |hash| make_opt_defn(hash) }
 
       assert_raises(Cri::OptionParser::OptionRequiresAnArgumentError) do
-        Cri::OptionParser.new(input, opt_defns, []).run
+        Cri::OptionParser.new(input, opt_defns, [], false).run
       end
     end
 
@@ -206,7 +206,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :optional },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert(parser.options[:aaa])
       assert_equal(['foo'], parser.arguments.to_a)
@@ -218,7 +218,7 @@ module Cri
         { long: 'aaa', short: 'a', argument: :optional },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ aaa: 'xxx' }, parser.options)
       assert_equal(['foo'], parser.arguments.to_a)
@@ -232,7 +232,7 @@ module Cri
         { long: 'ccc', short: 'c', argument: :forbidden },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert(parser.options[:aaa])
       assert(parser.options[:bbb])
@@ -244,7 +244,7 @@ module Cri
       input = %w[foo - bar]
       opt_defns = []
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({}, parser.options)
       assert_equal(['foo', '-', 'bar'], parser.arguments.to_a)
@@ -254,7 +254,7 @@ module Cri
       input = %w[foo bar -- -x --yyy -abc]
       opt_defns = []
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({}, parser.options)
       assert_equal(['foo', 'bar', '-x', '--yyy', '-abc'], parser.arguments.to_a)
@@ -267,7 +267,7 @@ module Cri
       ].map { |hash| make_opt_defn(hash) }
 
       assert_raises(Cri::OptionParser::OptionRequiresAnArgumentError) do
-        Cri::OptionParser.new(input, opt_defns, []).run
+        Cri::OptionParser.new(input, opt_defns, [], false).run
       end
     end
 
@@ -278,7 +278,7 @@ module Cri
         { long: 'verbose', short: 'v', multiple: true },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal(%w[test test2], parser.options[:long])
       assert_equal(3, parser.options[:verbose].size)
@@ -290,7 +290,7 @@ module Cri
         { long: 'animal', short: 'a', argument: :required, default: 'donkey' },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ animal: 'donkey' }, parser.options)
       assert_equal(['foo'], parser.arguments.to_a)
@@ -303,7 +303,7 @@ module Cri
       ].map { |hash| make_opt_defn(hash) }
 
       assert_raises(Cri::OptionParser::OptionRequiresAnArgumentError) do
-        Cri::OptionParser.new(input, opt_defns, []).run
+        Cri::OptionParser.new(input, opt_defns, [], false).run
       end
     end
 
@@ -313,7 +313,7 @@ module Cri
         { long: 'animal', short: 'a', argument: :required, default: 'donkey' },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ animal: 'giraffe' }, parser.options)
       assert_equal(['foo'], parser.arguments.to_a)
@@ -325,7 +325,7 @@ module Cri
         { long: 'animal', short: 'a', argument: :optional, default: 'donkey' },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ animal: 'donkey' }, parser.options)
       assert_equal(['foo'], parser.arguments.to_a)
@@ -337,7 +337,7 @@ module Cri
         { long: 'animal', short: 'a', argument: :optional, default: 'donkey' },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ animal: 'donkey' }, parser.options)
       assert_equal(['foo'], parser.arguments.to_a)
@@ -349,7 +349,7 @@ module Cri
         { long: 'animal', short: 'a', argument: :optional, default: 'donkey' },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ animal: 'giraffe' }, parser.options)
       assert_equal(['foo'], parser.arguments.to_a)
@@ -361,7 +361,7 @@ module Cri
         { long: 'animal', short: 'a', argument: :optional, default: 'donkey' },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ animal: 'gi' }, parser.options)
       assert_equal(%w[foo raffe], parser.arguments.to_a)
@@ -375,7 +375,7 @@ module Cri
         { long: 'ccc', short: 'c', argument: :required },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ aaa: true, bbb: 'xxx', ccc: 'yyy' }, parser.options)
       assert_equal(%w[foo zzz], parser.arguments.to_a)
@@ -389,7 +389,7 @@ module Cri
         { long: 'ccc', short: 'c', argument: :required },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ aaa: true, bbb: 'xxx', ccc: 'yyy' }, parser.options)
       assert_equal(%w[foo zzz], parser.arguments.to_a)
@@ -403,7 +403,7 @@ module Cri
         { long: 'ccc', short: 'c', argument: :optional, default: 'c default' },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ aaa: true, bbb: 'xxx', ccc: 'c default' }, parser.options)
       assert_equal(%w[foo], parser.arguments.to_a)
@@ -415,7 +415,7 @@ module Cri
         { long: 'port', short: 'p', argument: :required, transform: ->(x) { Integer(x) } },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ port: 123 }, parser.options)
       assert_equal([], parser.arguments.to_a)
@@ -427,7 +427,7 @@ module Cri
         { long: 'port', short: 'p', argument: :required, transform: method(:Integer) },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ port: 123 }, parser.options)
       assert_equal([], parser.arguments.to_a)
@@ -445,7 +445,7 @@ module Cri
         { long: 'port', short: 'p', argument: :required, transform: port },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ port: 123 }, parser.options)
       assert_equal([], parser.arguments.to_a)
@@ -464,7 +464,7 @@ module Cri
         { long: 'port', short: 'p', argument: :required, default: 8080, transform: port },
       ].map { |hash| make_opt_defn(hash) }
 
-      parser = Cri::OptionParser.new(input, opt_defns, []).run
+      parser = Cri::OptionParser.new(input, opt_defns, [], false).run
 
       assert_equal({ port: 8080 }, parser.options)
       assert_equal([], parser.arguments.to_a)
@@ -477,7 +477,7 @@ module Cri
       ].map { |hash| make_opt_defn(hash) }
 
       exception = assert_raises(Cri::OptionParser::IllegalOptionValueError) do
-        Cri::OptionParser.new(input, opt_defns, []).run
+        Cri::OptionParser.new(input, opt_defns, [], false).run
       end
       assert_equal('invalid value "one_hundred_and_twenty_three" for --port option', exception.message)
     end
@@ -488,7 +488,7 @@ module Cri
         { name: 'host' },
       ].map { |hash| Cri::ParamDefinition.new(hash) }
 
-      parser = Cri::OptionParser.new(input, [], param_defns).run
+      parser = Cri::OptionParser.new(input, [], param_defns, false).run
       assert_equal({}, parser.options)
       assert_equal('localhost', parser.arguments[0])
       assert_equal('localhost', parser.arguments[:host])
@@ -500,7 +500,7 @@ module Cri
         { name: 'host' },
       ].map { |hash| Cri::ParamDefinition.new(hash) }
 
-      parser = Cri::OptionParser.new(input, [], param_defns).run
+      parser = Cri::OptionParser.new(input, [], param_defns, false).run
       exception = assert_raises(Cri::ArgumentList::ArgumentCountMismatchError) do
         parser.arguments
       end
@@ -513,7 +513,7 @@ module Cri
         { name: 'host' },
       ].map { |hash| Cri::ParamDefinition.new(hash) }
 
-      parser = Cri::OptionParser.new(input, [], param_defns).run
+      parser = Cri::OptionParser.new(input, [], param_defns, false).run
       exception = assert_raises(Cri::ArgumentList::ArgumentCountMismatchError) do
         parser.arguments
       end
@@ -526,7 +526,7 @@ module Cri
         { name: 'host' },
       ].map { |hash| Cri::ParamDefinition.new(hash) }
 
-      parser = Cri::OptionParser.new(input, [], param_defns).run
+      parser = Cri::OptionParser.new(input, [], param_defns, false).run
 
       exception = assert_raises(ArgumentError) do
         parser.arguments['oink']
@@ -541,7 +541,7 @@ module Cri
         { name: 'target' },
       ].map { |hash| Cri::ParamDefinition.new(hash) }
 
-      parser = Cri::OptionParser.new(input, [], param_defns).run
+      parser = Cri::OptionParser.new(input, [], param_defns, false).run
       assert_equal({}, parser.options)
       assert_equal('localhost', parser.arguments[0])
       assert_equal('localhost', parser.arguments[:source])

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: cri
 version: !ruby/object:Gem::Version
-  version: 2.12.0
+  version: 2.13.0
 platform: ruby
 authors:
 - Denis Defreyne
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-08-12 00:00:00.000000000 Z
+date: 2018-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colored
@@ -50,7 +50,6 @@ extra_rdoc_files:
 files:
 - CODE_OF_CONDUCT.md
 - Gemfile
-- Gemfile.lock
 - LICENSE
 - NEWS.md
 - README.md

data/Gemfile.lock

@@ -1,64 +0,0 @@
-PATH
-  remote: .
-  specs:
-    cri (2.12.0)
-      colored (~> 1.2)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    asciidoctor (1.5.7.1)
-    ast (2.4.0)
-    colored (1.2)
-    coveralls (0.8.22)
-      json (>= 1.8, < 3)
-      simplecov (~> 0.16.1)
-      term-ansicolor (~> 1.3)
-      thor (~> 0.19.4)
-      tins (~> 1.6)
-    docile (1.3.1)
-    jaro_winkler (1.5.1)
-    json (2.1.0)
-    minitest (5.11.3)
-    parallel (1.12.1)
-    parser (2.5.1.2)
-      ast (~> 2.4.0)
-    powerpack (0.1.2)
-    rainbow (3.0.0)
-    rake (12.3.1)
-    rubocop (0.58.2)
-      jaro_winkler (~> 1.5.1)
-      parallel (~> 1.10)
-      parser (>= 2.5, != 2.5.1.1)
-      powerpack (~> 0.1)
-      rainbow (>= 2.2.2, < 4.0)
-      ruby-progressbar (~> 1.7)
-      unicode-display_width (~> 1.0, >= 1.0.1)
-    ruby-progressbar (1.10.0)
-    simplecov (0.16.1)
-      docile (~> 1.1)
-      json (>= 1.8, < 3)
-      simplecov-html (~> 0.10.0)
-    simplecov-html (0.10.2)
-    term-ansicolor (1.6.0)
-      tins (~> 1.0)
-    thor (0.19.4)
-    tins (1.16.3)
-    unicode-display_width (1.4.0)
-    yard (0.9.16)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  asciidoctor
-  bundler (~> 1.6)
-  coveralls
-  cri!
-  minitest
-  rake
-  rubocop
-  yard
-
-BUNDLED WITH
-   1.16.3