multilist: ensure min <= max

Author: sveinn

lib/tty/prompt/multi_list.rb

@@ -18,6 +18,7 @@ class MultiList < List
       # @api public
       def initialize(prompt, **options)
         super
+        check_minmax(options[:min], options[:max])
         @selected = SelectedChoices.new
         @help = options[:help]
         @echo = options.fetch(:echo, true)
@@ -29,13 +30,15 @@ def initialize(prompt, **options)
       #
       # @api public
       def min(value)
+        check_minmax(value, @max)
         @min = value
       end
 
       # Set a maximum number of choices
       #
       # @api public
       def max(value)
+        check_minmax(@min, value)
         @max = value
       end
 
@@ -136,6 +139,17 @@ def minmax_help
         "(%s) " % [help.join(", ")]
       end
 
+      # Check that min is not greater than max
+      #
+      # @raise [ConfigurationError]
+      #
+      # @api_private
+      def check_minmax(min, max)
+        return unless min && max && min > max
+
+        raise ConfigurationError, "min must not be greater than max"
+      end
+
       # Build a default help text
       #
       # @return [String]

spec/unit/multi_select_spec.rb

@@ -933,6 +933,7 @@ def exit_message(prompt, choices)
 
       expect(prompt.output.string).to eq(expected_output)
     end
+    
     it "raises an error message when the min is greater than max" do
       choices = %w[A B C]
       expect {