gli 2.18.2 → 2.20.1

data/gli.gemspec

@@ -1,33 +1,30 @@
 # Make sure we get the gli that's local
-require File.join([File.dirname(__FILE__),'lib','gli','version.rb'])
+require File.join([File.dirname(__FILE__),"lib","gli","version.rb"])
 
 spec = Gem::Specification.new do |s|
-  s.name = 'gli'
+  s.name = "gli"
   s.version = GLI::VERSION
-  s.licenses = ['Apache-2.0']
-  s.author = 'David Copeland'
-  s.email = 'davidcopeland@naildrivin5.com'
-  s.homepage = 'http://davetron5000.github.com/gli'
+  s.licenses = ["Apache-2.0"]
+  s.author = "David Copeland"
+  s.email = "davidcopeland@naildrivin5.com"
+  s.homepage = "http://davetron5000.github.io/gli"
   s.platform = Gem::Platform::RUBY
-  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.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.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  s.executables   =  'gli'
   s.require_paths = ["lib"]
 
-  s.extra_rdoc_files = ['README.rdoc', 'gli.rdoc']
-  s.rdoc_options << '--title' << 'Git Like Interface' << '--main' << 'README.rdoc'
-  s.bindir = 'bin'
-  s.rubyforge_project = 'gli'
-  s.add_development_dependency('rake', '~> 0.9.2.2')
-  s.add_development_dependency('rdoc', '~> 4.2')
-  s.add_development_dependency('rainbow', '~> 1.1', '~> 1.1.1')
-  s.add_development_dependency('clean_test', '~> 1.0')
-  s.add_development_dependency('cucumber', '~> 3.1.2')
-  s.add_development_dependency('gherkin', '~> 5.1.0')
-  s.add_development_dependency('aruba', '~> 0.7.4')
-  s.add_development_dependency('sdoc', '~> 0.4')
-  s.add_development_dependency('faker','~> 1.9.1')
+  s.extra_rdoc_files = ["README.rdoc", "gli.rdoc"]
+  s.rdoc_options << "--title" << "Git Like Interface" << "--main" << "README.rdoc"
+
+  s.bindir      = "exe"
+  s.executables = "gli"
+
+  s.add_development_dependency("rake")
+  s.add_development_dependency("rdoc")
+  s.add_development_dependency("rainbow", "~> 1.1", "~> 1.1.1")
+  s.add_development_dependency("sdoc")
+  s.add_development_dependency("minitest")
 end

data/gli.rdoc

@@ -1,6 +1,6 @@
 == gli - create scaffolding for a GLI-powered application
 
-v2.17.1
+v2.19.2
 
 === Global Options
 === -r|--root arg
@@ -43,7 +43,7 @@ List commands one per line, to assist with shell completion
 
 
 
-==== Command: <tt>init|scaffold  project_name [command_name][, [command_name]]*</tt>
+==== Command: <tt>init|scaffold  project_name [command_name]...</tt>
 Create a new GLI-based project
 
 This will create a scaffold command line project that uses GLI

data/lib/gli/command_support.rb

@@ -166,12 +166,8 @@ module GLI
 
     def generate_error_action(arguments)
       lambda { |global_options,options,arguments|
-        if am_subcommand?
-          if arguments.size > 0
-            raise UnknownCommand,"Unknown command '#{arguments[0]}'"
-          else
-            raise BadCommandLine,"Command '#{name}' requires a subcommand"
-          end
+        if am_subcommand? && arguments.size > 0
+          raise UnknownCommand,"Unknown command '#{arguments[0]}'"
         elsif have_subcommands?
           raise BadCommandLine,"Command '#{name}' requires a subcommand #{self.commands.keys.join(',')}"
         else

data/lib/gli/commands/help_modules/arg_name_formatter.rb

@@ -22,7 +22,7 @@ module GLI
               arg_desc = "[#{arg_desc}]"
             end
             if arg.multiple?
-              arg_desc = "#{arg_desc}[, #{arg_desc}]*"
+              arg_desc = "#{arg_desc}..."
             end
             desc = desc + " " + arg_desc
           end
@@ -37,7 +37,7 @@ module GLI
             desc = "[#{desc}]"
           end
           if arguments_options.include? :multiple
-            desc = "#{desc}[, #{desc}]*"
+            desc = "#{desc}..."
           end
           " " + desc
         end

data/lib/gli/commands/help_modules/command_help_format.rb

@@ -53,7 +53,7 @@ COMMANDS
 
 
 <%= command_examples %>
-<% end %>),nil,'<>')
+<% end %>))
 
 
         def flags_and_switches(command,app)

data/lib/gli/commands/help_modules/global_help_format.rb

@@ -51,7 +51,7 @@ GLOBAL OPTIONS
 
 <% end %>
 COMMANDS
-<%= commands %>),nil,'<>')
+<%= commands %>))
 
         def global_flags_and_switches
           @app.flags_declaration_order + @app.switches_declaration_order

data/lib/gli/commands/scaffold.rb

@@ -70,10 +70,10 @@ spec = Gem::Specification.new do |s|
   s.rdoc_options << '--title' << '#{project_name}' << '--main' << 'README.rdoc' << '-ri'
   s.bindir = 'bin'
   s.executables << '#{project_name}'
-  s.add_development_dependency('rake')
-  s.add_development_dependency('rdoc')
-  s.add_development_dependency('aruba')
-  s.add_runtime_dependency('gli','#{GLI::VERSION}')
+  s.add_development_dependency('rake','~> 0.9.2')
+  s.add_development_dependency('rdoc', '~> 4.3')
+  s.add_development_dependency('minitest', '~> 5.14')
+  s.add_runtime_dependency('gli','~> #{GLI::VERSION}')
 end
 EOS
       end
@@ -114,12 +114,6 @@ require 'rubygems'
 require 'rubygems/package_task'
 require 'rdoc/task'
 EOS
-        if create_test_dir
-          file.puts <<EOS
-require 'cucumber'
-require 'cucumber/rake/task'
-EOS
-        end
         file.puts <<EOS
 Rake::RDocTask.new do |rd|
   rd.main = "README.rdoc"
@@ -134,44 +128,19 @@ end
 EOS
         if create_test_dir
           file.puts <<EOS
-CUKE_RESULTS = 'results.html'
-CLEAN << CUKE_RESULTS
-desc 'Run features'
-Cucumber::Rake::Task.new(:features) do |t|
-  opts = "features --format html -o \#{CUKE_RESULTS} --format progress -x"
-  opts += " --tags \#{ENV['TAGS']}" if ENV['TAGS']
-  t.cucumber_opts =  opts
-  t.fork = false
-end
-
-desc 'Run features tagged as work-in-progress (@wip)'
-Cucumber::Rake::Task.new('features:wip') do |t|
-  tag_opts = ' --tags ~@pending'
-  tag_opts = ' --tags @wip'
-  t.cucumber_opts = "features --format html -o \#{CUKE_RESULTS} --format pretty -x -s\#{tag_opts}"
-  t.fork = false
-end
-
-task :cucumber => :features
-task 'cucumber:wip' => 'features:wip'
-task :wip => 'features:wip'
-EOS
-        end
-        if create_test_dir
-          file.puts <<EOS
 require 'rake/testtask'
 Rake::TestTask.new do |t|
   t.libs << "test"
   t.test_files = FileList['test/*_test.rb']
 end
 
-task :default => [:test,:features]
+task :default => :test
 EOS
           File.open("#{root_dir}/#{project_name}/test/default_test.rb",'w') do |test_file|
             test_file.puts <<EOS
-require 'test_helper'
+require_relative "test_helper"
 
-class DefaultTest < Test::Unit::TestCase
+class DefaultTest < Minitest::Test
 
   def setup
   end
@@ -188,15 +157,10 @@ EOS
           puts "Created #{root_dir}/#{project_name}/test/default_test.rb"
           File.open("#{root_dir}/#{project_name}/test/test_helper.rb",'w') do |test_file|
             test_file.puts <<EOS
-require 'test/unit'
+require "minitest/autorun"
 
 # Add test libraries you want to use here, e.g. mocha
-
-class Test::Unit::TestCase
-
-  # Add global extensions to the test case class here
-
-end
+# Add helper classes or methods here, too
 EOS
           end
           puts "Created #{root_dir}/#{project_name}/test/test_helper.rb"
@@ -210,54 +174,6 @@ EOS
         bundler_file.puts "gemspec"
       end
       puts "Created #{root_dir}/#{project_name}/Gemfile"
-      if create_test_dir
-        features_dir = File.join(root_dir,project_name,'features')
-        FileUtils.mkdir features_dir
-        FileUtils.mkdir File.join(features_dir,"step_definitions")
-        FileUtils.mkdir File.join(features_dir,"support")
-        File.open(File.join(features_dir,"#{project_name}.feature"),'w') do |file|
-          file.puts <<EOS
-Feature: My bootstrapped app kinda works
-  In order to get going on coding my awesome app
-  I want to have aruba and cucumber setup
-  So I don't have to do it myself
-
-  Scenario: App just runs
-    When I get help for "#{project_name}"
-    Then the exit status should be 0
-EOS
-        end
-        File.open(File.join(features_dir,"step_definitions","#{project_name}_steps.rb"),'w') do |file|
-          file.puts <<EOS
-When /^I get help for "([^"]*)"$/ do |app_name|
-  @app_name = app_name
-  step %(I run `\#{app_name} help`)
-end
-
-# Add more step definitions here
-EOS
-        end
-        File.open(File.join(features_dir,"support","env.rb"),'w') do |file|
-          file.puts <<EOS
-require 'aruba/cucumber'
-
-ENV['PATH'] = "\#{File.expand_path(File.dirname(__FILE__) + '/../../bin')}\#{File::PATH_SEPARATOR}\#{ENV['PATH']}"
-LIB_DIR = File.join(File.expand_path(File.dirname(__FILE__)),'..','..','lib')
-
-Before do
-  # Using "announce" causes massive warnings on 1.9.2
-  @puts = true
-  @original_rubylib = ENV['RUBYLIB']
-  ENV['RUBYLIB'] = LIB_DIR + File::PATH_SEPARATOR + ENV['RUBYLIB'].to_s
-end
-
-After do
-  ENV['RUBYLIB'] = @original_rubylib
-end
-EOS
-        end
-        puts "Created #{features_dir}"
-      end
     end
 
     def self.mk_binfile(root_dir,create_ext_dir,force,dry_run,project_name,commands)

data/lib/gli/dsl.rb

@@ -50,7 +50,7 @@ module GLI
     #     command :pack do ...
     #
     # Produces the synopsis:
-    #     app.rb [global options] pack output input[, input]*
+    #     app.rb [global options] pack output input...
     def arg(name, options=[])
       @next_arguments ||= []
       @next_arguments << Argument.new(name, Array(options).flatten)

data/lib/gli/options.rb

@@ -7,12 +7,12 @@ module GLI
 
     # Return the value of an attribute
     def[](k)
-      @table[k.to_sym]
+      self.send(k.to_sym)
     end
 
     # Set the value of an attribute
     def[]=(k, v)
-      @table[k.to_sym] = v
+      self.send("#{k.to_sym}=",v)
     end
 
     def map(&block)

data/lib/gli/version.rb

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

data/object-model.dot

@@ -0,0 +1,29 @@
+digraph G {
+
+  rankdir="BT"
+  nodesep=0.5
+
+  node[shape=record fontname=courier fontsize=18]
+  edge[fontname=avenir fontsize=12]
+
+  CommandLineToken  [ label="{ CommandLineToken | #name\l | #description\l | #long_description\l | #aliases\l}"]
+  CommandLineOption [ label="{ CommandLineOption | #default_value \l }"]
+  DSL
+  Command
+  Flag              [ label="{ Flag | #argument_name\l }"]
+  Switch
+  App
+  TopLevel          [ label="top level?" shape=diamond fontname=avenir fontsize=12]
+
+  Command           -> DSL               [ arrowhead=oarrow   label="  includes" minlen=3]
+  Command           -> CommandLineToken  [ arrowhead=oarrow   label="inherits"]
+  CommandLineOption -> CommandLineToken  [ arrowhead=oarrow   label="inherits"]
+  Flag              -> CommandLineOption [ arrowhead=oarrow   label="inherits"]
+  Switch            -> CommandLineOption [ arrowhead=oarrow   label="inherits"]
+  Command           -> TopLevel          [ arrowhead=none     label="parent" style=dotted]
+  TopLevel          -> App               [ arrowhead=odiamond label="YES"    style=dotted ]
+  TopLevel          -> Command           [ arrowhead=odiamond label="NO"     style=dotted ]
+  CommandLineOption -> Command           [ arrowhead=odiamond style=dotted label="associated_command"]
+
+  { rank=same; DSL; App }
+}

data/test/apps/todo/Gemfile

@@ -1,2 +1,2 @@
-source :rubygems
+source 'https://rubygems.org'
 gemspec

data/test/apps/todo/bin/todo

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 
 # These would not be in a real GLI app; we do this so we can easily run this on the command line
-$: << File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','lib'))
+$: << File.expand_path(File.join(File.dirname(__FILE__),'..','..','..','..','lib'))
 $: << File.expand_path(File.join(File.dirname(__FILE__),'..','lib'))
 
 require 'gli'

data/test/integration/gli_cli_test.rb

@@ -0,0 +1,69 @@
+require_relative "test_helper"
+require "open3"
+
+class GLICLITest < MiniTest::Test
+  include TestHelper
+
+  class AppHelp < GLICLITest
+    def test_running_with_no_options_produces_help
+      out = run_gli
+      assert_output_looks_like_help out
+    end
+
+    def test_running_with_help_command_produces_help
+      out = run_gli("help")
+      assert_output_looks_like_help out
+    end
+
+    def test_running_with_help_switch_produces_help
+      out = run_gli("--help")
+      assert_output_looks_like_help out
+    end
+
+  private
+
+    def assert_output_looks_like_help(out)
+      assert_match /gli - create scaffolding for a GLI-powered application/,out
+      assert_match /SYNOPSIS/,out
+      assert_match /GLOBAL OPTIONS/,out
+      assert_match /COMMANDS/,out
+    end
+
+  end
+
+  class Scaffolding < GLICLITest
+    def test_help_on_scaffold_command
+      out = run_gli("help scaffold")
+      assert_output_looks_like_help(out)
+    end
+    def test_help_on_scaffold_command_as_init
+      out = run_gli("help init")
+      assert_output_looks_like_help(out)
+    end
+
+  private
+
+    def assert_output_looks_like_help(out)
+      assert_match /init - Create a new GLI-based project/,out
+      assert_match /SYNOPSIS/,out
+      assert_match /COMMAND OPTIONS/,out
+    end
+  end
+
+private
+
+  def run_gli(args="", return_err_and_status: false, expect_failure: false)
+    command_line_invocation = "bin/gli #{args}"
+    stdout_string, stderr_string, status = Open3.capture3(command_line_invocation)
+    if expect_failure
+      refute_equal 0,status.exitstatus,"Expected failure for '#{command_line_invocation}' but it succeeded"
+    else
+      assert_equal 0,status.exitstatus,"Expected success for '#{command_line_invocation}' but it failed"
+    end
+    if return_err_and_status
+      [ stdout_string, stderr_string, status ]
+    else
+      stdout_string
+    end
+  end
+end

data/test/integration/gli_powered_app_test.rb

@@ -0,0 +1,52 @@
+require_relative "test_helper"
+require "open3"
+
+class GLIPoweredAppTest < MiniTest::Test
+  include TestHelper
+
+  def teardown
+    FileUtils.rm_f("todo.rdoc")
+  end
+
+  def test_help_works
+    out = run_app("help")
+    assert_top_level_help(out)
+  end
+
+  def test_unknown_command_exits_nonzero
+    out, err, status = run_app("asdfasdfasdf", expect_failure: true, return_err_and_status: true)
+    assert_match /Unknown command 'asdfasdfasdf'/,err
+    assert_equal 64, status.exitstatus
+    assert_top_level_help(out)
+  end
+
+  def test_unknown_switch_exits_nonzero
+    out, err, status = run_app("list --foo", expect_failure: true, return_err_and_status: true)
+    assert_match /Unknown option \-\-foo/,err
+    assert_equal 64, status.exitstatus
+    assert_match /COMMAND OPTIONS/, out
+  end
+
+  def test_missing_args_exits_nonzero
+    out, err, status = run_app("list", expect_failure: true, return_err_and_status: true)
+    assert_match /required_flag is required, required_flag2 is required/,err
+    assert_equal 64, status.exitstatus
+    assert_match /COMMAND OPTIONS/, out
+  end
+
+  def test_doc_generation
+    out, err, status = run_app("_doc", return_err_and_status: true)
+    assert File.exists?("todo.rdoc")
+  end
+
+private
+  def assert_top_level_help(out)
+    assert_match /SYNOPSIS/, out
+    assert_match /GLOBAL OPTIONS/, out
+    assert_match /COMMANDS/, out
+  end
+
+  def run_app(args="", return_err_and_status: false, expect_failure: false)
+    run_command("test/apps/todo/bin/todo",args,return_err_and_status:return_err_and_status,expect_failure:expect_failure)
+  end
+end