gli 2.21.2 → 2.21.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 33feccb199126e4e39e227aaddd88d3d0a270bc3954a2e47a20f3613049eec3b
-  data.tar.gz: f1530371f4226510065a4acdc3d884bedee3e877979428c06579747863e02c8d
+  metadata.gz: 2f66f71d97f5bb59503b88b120369d123015ae612088de9a6987ad34686bcafa
+  data.tar.gz: cf4bf46c072cae3e57997f0a309e7addebe288fbef32317aee374b1eaaf9859c
 SHA512:
-  metadata.gz: ca8f6b6130a39c4074b90633fdb4ae6261cfeff78f41928d8afae1e2fe1a7e6bb7429536569b891e44084a3ae719a204e1ebea8dea7052d5b257c9f50bff85b8
-  data.tar.gz: 1ef6e27dbf75f34e06735c0f1fda33d0ee09989f5f0d165d5d79efbbd148c04d1507f8a580f7b177d0c9b4140a55035db5ed1210f79ef717d60d62f6754221a2
+  metadata.gz: 26b9cd9219f73b06ecd20b547b5e3cad441c7ff22099213b916780286815e6c57f8d7d1d50a0e1ff0d516ce27cf1513da36a80c52fa8d25c35e67fa5685f9a6d
+  data.tar.gz: 71e9d691d506a3b07a9a29d7e06988df3e798529d0a4ae4cad1ff08070537df6506d3306ed94095c4109e3740ce9b7c7aceb7a92c0540a493b847d1348418874

data/dx/build

@@ -35,7 +35,7 @@ log "✨" "Creating docker-compose.dx.yml"
 compose_file="docker-compose.dx.yml"
 log "🗑️" "Deleting previous ${compose_file}"
 
-rm "${compose_file}"
+rm -f "${compose_file}"
 echo "# THIS IS GENERATED - DO NOT EDIT" > "${compose_file}"
 echo "" >> "${compose_file}"
 echo "services:" >> "${compose_file}"
@@ -54,7 +54,7 @@ for ruby_version in ${RUBY_VERSIONS[@]}; do
   echo "    entrypoint: /root/show-help-in-app-container-then-wait.sh" >> "${compose_file}"
   echo "    working_dir: /root/work" >> "${compose_file}"
 done
-log "🎼" "${compose_file} is noq created"
+log "🎼" "${compose_file} is now created"
 log "🔄" "You can run dx/start to start it up, though you may need to stop it first with Ctrl-C"
 
 # vim: ft=bash

data/lib/gli/gli_option_parser.rb

@@ -109,9 +109,10 @@ module GLI
       def verify_required_options!(flags, command, options)
         missing_required_options = flags.values.
           select(&:required?).
-          reject { |option|
-            options[option.name] != nil
-        }
+          select { |option|
+            options[option.name] == nil ||
+            ( options[option.name].kind_of?(Array) && options[option.name].empty? )
+          }
         unless missing_required_options.empty?
           missing_required_options.sort!
           raise MissingRequiredArgumentsException.new(missing_required_options.map { |option|

data/lib/gli/version.rb

@@ -1,5 +1,5 @@
 module GLI
   unless const_defined? :VERSION
-    VERSION = '2.21.2'
+    VERSION = '2.21.3'
   end
 end

data/test/unit/gli_test.rb

@@ -81,6 +81,25 @@ class GLITest < Minitest::Test
 
   end
 
+  def test_command_options_accepting_multiple_values_can_be_required
+    @app.reset
+    @called = false
+    @app.command :foo do |c|
+      c.flag :flag, :required => true, :multiple => true
+      c.action do |global, options, arguments|
+        @called = true
+      end
+    end
+    assert_equal 64, @app.run(['foo']), "Expected exit status to be 64"
+    assert  @fake_stderr.contained?(/flag is required/), @fake_stderr.strings.inspect
+    assert  @fake_stderr.contained?(/flag is required/), @fake_stderr.strings.inspect
+    assert !@called
+
+    assert_equal 0, @app.run(['foo','--flag=bar']), "Expected exit status to be 0 #{@fake_stderr.strings.join(',')}"
+    assert @called
+
+  end
+
   def test_global_options_can_be_required
     @app.reset
     @called = false
@@ -102,6 +121,25 @@ class GLITest < Minitest::Test
 
   end
 
+  def test_global_options_accepting_multiple_can_be_required
+    @app.reset
+    @called = false
+    @app.flag :flag, :required => true, :multiple => true
+    @app.command :foo do |c|
+      c.action do |global, options, arguments|
+        @called = true
+      end
+    end
+    assert_equal 64, @app.run(['foo']), "Expected exit status to be 64"
+    assert  @fake_stderr.contained?(/flag is required/), @fake_stderr.strings.inspect
+    assert  @fake_stderr.contained?(/flag is required/), @fake_stderr.strings.inspect
+    assert !@called
+
+    assert_equal 0, @app.run(['--flag=bar','foo']), "Expected exit status to be 0 #{@fake_stderr.strings.join(',')}"
+    assert @called
+
+  end
+
   def test_global_required_options_are_ignored_on_help
     @app.reset
     @called = false
@@ -161,6 +199,7 @@ class GLITest < Minitest::Test
     raise failure if !failure.nil?
   end
 
+
   def test_command_line_overrides_config
     failure = nil
     @app.reset

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gli
 version: !ruby/object:Gem::Version
-  version: 2.21.2
+  version: 2.21.3
 platform: ruby
 authors:
 - David Copeland