gli 2.21.2 → 2.21.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 33feccb199126e4e39e227aaddd88d3d0a270bc3954a2e47a20f3613049eec3b
-  data.tar.gz: f1530371f4226510065a4acdc3d884bedee3e877979428c06579747863e02c8d
+  metadata.gz: 60f96a57b4abf16e79016ea09350797584e9a26882630c54340209c8db4ff527
+  data.tar.gz: 67e19189b4f770c374609de68d67a5ae41fb58ae92faaa3f735a4507db14d44b
 SHA512:
-  metadata.gz: ca8f6b6130a39c4074b90633fdb4ae6261cfeff78f41928d8afae1e2fe1a7e6bb7429536569b891e44084a3ae719a204e1ebea8dea7052d5b257c9f50bff85b8
-  data.tar.gz: 1ef6e27dbf75f34e06735c0f1fda33d0ee09989f5f0d165d5d79efbbd148c04d1507f8a580f7b177d0c9b4140a55035db5ed1210f79ef717d60d62f6754221a2
+  metadata.gz: 07e228e867c65b76c31929e6f2680a884c4f37841dc43a3ca778e0b272583ca8e9e194d63be45bd843def5ba4fceeb496f20c9b0392448f4acc6bf8fa62a3b4d
+  data.tar.gz: 54151c1d669e25abeb7f1061003acf9230c4906dcfdf3766aefb90eb86f901416edb657d718bceda944817fb7d6f923530a892ec3c2aba4ba6db3e61167bb53f

data/CONTRIBUTING.md

@@ -53,6 +53,9 @@ checked-out files, the other containers will have access as well.
 2. `inside-docker-container> bin/setup # installs all gems`
 3. `inside-docker-container> bin/rake  # runs all tests`
 
+You can also try using `bin/ci` on your computer (not inside a Docker container), which will run tests across the entire build
+matrix.
+
 #### If You Want To Use Your Local Computer
 
 In theory, GLI can be worked on using RVM, RBEnv, asdf, or whatever other way you want to manage Ruby.  You can create a

data/bin/ci

@@ -1,29 +1,17 @@
 #!/bin/bash
 
 set -e
-if [ -z $1 ]; then
-  echo "[bin/ci] Running with default warnings"
-  export RUBYOPT=
-else
-  if [ $1 == 'warnings' ]; then
-    echo "[bin/ci] Running with all warnings on"
-    export RUBYOPT=-w
-  else
-    if [ $1 == 'none' ]; then
-      echo "[bin/ci] Running with all warnings off"
-      export RUBYOPT='-W0'
-    else
-      echo "[bin/ci] '$1' is not a supported option"
-      echo "[bin/ci] usage: $0          # run with default warnings"
-      echo "[bin/ci] usage: $0 warnings # run with all warnings"
-      echo "[bin/ci] usage: $0 none     # run with warnings disabled"
-      exit 1
-    fi
-  fi
-fi
 
-echo "[bin/ci] Running unit tests"
-bin/rake test:unit
+SCRIPT_DIR=$( cd -- "$( dirname -- "${0}" )" > /dev/null 2>&1 && pwd )
+DX_DIR="${SCRIPT_DIR}/../dx"
+
+. "${DX_DIR}/docker-compose.env"
+. "${DX_DIR}/setupkit.sh.lib"
+
+for ruby_version in ${RUBY_VERSIONS[@]}; do
+  log "Setting up for Ruby version '${ruby_version}'"
+  dx/exec -v ${ruby_version} bin/setup
+  log "Running tests for Ruby version '${ruby_version}'"
+  dx/exec -v ${ruby_version} bin/rake
+done
 
-echo "[bin/ci] Running integration tests"
-bin/rake test:integration

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/gli.gemspec

@@ -12,6 +12,15 @@ spec = Gem::Specification.new do |s|
   s.summary = "Build command-suite CLI apps that are awesome."
   s.description = "Build command-suite CLI apps that are awesome.  Bootstrap your app, add commands, options and documentation while maintaining a well-tested idiomatic command-line app"
 
+  s.metadata = {
+    'bug_tracker_uri'   => 'https://github.com/davetron5000/gli/issues',
+    'changelog_uri'     => 'https://github.com/davetron5000/gli/releases',
+    'documentation_uri' => 'http://davetron5000.github.io/gli/rdoc/index.html',
+    'homepage_uri'      => 'https://davetron5000.github.com/gli/',
+    'source_code_uri'   => 'https://github.com/davetron5000/gli/',
+    'wiki_url'          => 'https://github.com/davetron5000/gli/wiki',
+  }
+
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.require_paths = ["lib"]

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.4'
   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,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gli
 version: !ruby/object:Gem::Version
-  version: 2.21.2
+  version: 2.21.4
 platform: ruby
 authors:
 - David Copeland
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-07-16 00:00:00.000000000 Z
+date: 2024-07-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -212,7 +212,13 @@ files:
 homepage: http://davetron5000.github.io/gli
 licenses:
 - Apache-2.0
-metadata: {}
+metadata:
+  bug_tracker_uri: https://github.com/davetron5000/gli/issues
+  changelog_uri: https://github.com/davetron5000/gli/releases
+  documentation_uri: http://davetron5000.github.io/gli/rdoc/index.html
+  homepage_uri: https://davetron5000.github.com/gli/
+  source_code_uri: https://github.com/davetron5000/gli/
+  wiki_url: https://github.com/davetron5000/gli/wiki
 post_install_message:
 rdoc_options:
 - "--title"