ggem 1.8.4 → 1.9.4

data/test/unit/cli_tests.rb

@@ -1,15 +1,16 @@
-require 'assert'
-require 'ggem/cli'
+# frozen_string_literal: true
 
-require 'ggem/cli/clirb'
-require 'ggem/cli/commands'
-require 'ggem/gem'
-require 'ggem/gemspec'
-require 'ggem/git_repo'
-require 'much-plugin'
+require "assert"
+require "ggem/cli"
 
-class GGem::CLI
+require "ggem/cli/clirb"
+require "ggem/cli/commands"
+require "ggem/gem"
+require "ggem/gemspec"
+require "ggem/git_repo"
+require "much-plugin"
 
+class GGem::CLI
   class UnitTests < Assert::Context
     desc "GGem::CLI"
     setup do
@@ -33,21 +34,20 @@ class GGem::CLI
 
       assert_instance_of InvalidCommand, COMMANDS[Factory.string]
 
-      assert_instance_of GenerateCommand, COMMANDS['generate']
-      assert_instance_of BuildCommand,    COMMANDS['build']
-      assert_instance_of InstallCommand,  COMMANDS['install']
-      assert_instance_of PushCommand,     COMMANDS['push']
-      assert_instance_of TagCommand,      COMMANDS['tag']
-      assert_instance_of ReleaseCommand,  COMMANDS['release']
+      assert_instance_of GenerateCommand, COMMANDS["generate"]
+      assert_instance_of BuildCommand,    COMMANDS["build"]
+      assert_instance_of InstallCommand,  COMMANDS["install"]
+      assert_instance_of PushCommand,     COMMANDS["push"]
+      assert_instance_of TagCommand,      COMMANDS["tag"]
+      assert_instance_of ReleaseCommand,  COMMANDS["release"]
 
-      assert_same COMMANDS['generate'], COMMANDS['g']
-      assert_same COMMANDS['build'],    COMMANDS['b']
-      assert_same COMMANDS['install'],  COMMANDS['i']
-      assert_same COMMANDS['push'],     COMMANDS['p']
-      assert_same COMMANDS['tag'],      COMMANDS['t']
-      assert_same COMMANDS['release'],  COMMANDS['r']
+      assert_same COMMANDS["generate"], COMMANDS["g"]
+      assert_same COMMANDS["build"],    COMMANDS["b"]
+      assert_same COMMANDS["install"],  COMMANDS["i"]
+      assert_same COMMANDS["push"],     COMMANDS["p"]
+      assert_same COMMANDS["tag"],      COMMANDS["t"]
+      assert_same COMMANDS["release"],  COMMANDS["r"]
     end
-
   end
 
   class InitTests < UnitTests
@@ -62,7 +62,6 @@ class GGem::CLI
     subject{ @cli }
 
     should have_imeths :run
-
   end
 
   class RunSetupTests < InitTests
@@ -80,7 +79,6 @@ class GGem::CLI
     teardown do
       COMMANDS.remove(@command_name)
     end
-
   end
 
   class RunTests < RunSetupTests
@@ -96,7 +94,6 @@ class GGem::CLI
     should "have successfully exited" do
       assert_equal 0, @kernel_spy.exit_status
     end
-
   end
 
   class RunWithNoArgsTests < RunSetupTests
@@ -113,7 +110,6 @@ class GGem::CLI
     should "have successfully exited" do
       assert_equal 0, @kernel_spy.exit_status
     end
-
   end
 
   class RunWithInvalidCommandTests < RunSetupTests
@@ -125,7 +121,7 @@ class GGem::CLI
     end
 
     should "output that it is invalid and output the invalid command's help" do
-      exp = "'#{@name}' is not a command.\n\n"
+      exp = "`#{@name}` is not a command.\n\n"
       assert_equal exp, @stderr.read
       assert_equal @invalid_command.help, @stdout.read
     end
@@ -133,7 +129,6 @@ class GGem::CLI
     should "have unsuccessfully exited" do
       assert_equal 1, @kernel_spy.exit_status
     end
-
   end
 
   class RunWithCommandExitErrorTests < RunSetupTests
@@ -147,13 +142,12 @@ class GGem::CLI
       assert_equal 1, @kernel_spy.exit_status
       assert_empty @stderr.read
     end
-
   end
 
   class RunWithHelpTests < RunSetupTests
     desc "and run with the help switch"
     setup do
-      @cli.run([ '--help' ])
+      @cli.run([ "--help" ])
     end
 
     should "output the invalid command's help" do
@@ -164,13 +158,12 @@ class GGem::CLI
     should "have successfully exited" do
       assert_equal 0, @kernel_spy.exit_status
     end
-
   end
 
   class RunWithVersionTests < RunSetupTests
     desc "and run with the version switch"
     setup do
-      @cli.run([ '--version' ])
+      @cli.run([ "--version" ])
     end
 
     should "have output its version" do
@@ -181,7 +174,6 @@ class GGem::CLI
     should "have successfully exited" do
       assert_equal 0, @kernel_spy.exit_status
     end
-
   end
 
   class RunWithErrorTests < RunSetupTests
@@ -201,7 +193,6 @@ class GGem::CLI
     should "have unsuccessfully exited" do
       assert_equal 1, @kernel_spy.exit_status
     end
-
   end
 
   class InvalidCommandTests < UnitTests
@@ -226,12 +217,12 @@ class GGem::CLI
     end
 
     should "parse its argv on run" do
-      assert_raises(CLIRB::HelpExit){ subject.new.run([ '--help' ]) }
-      assert_raises(CLIRB::VersionExit){ subject.new.run([ '--version' ]) }
+      assert_raises(CLIRB::HelpExit){ subject.new.run([ "--help" ]) }
+      assert_raises(CLIRB::VersionExit){ subject.new.run([ "--version" ]) }
     end
 
     should "raise a help exit if its name is empty" do
-      cmd = @command_class.new([nil, ''].sample)
+      cmd = @command_class.new([nil, ""].sample)
       argv = [Factory.string, Factory.string]
       assert_raises(CLIRB::HelpExit){ cmd.new.run(argv) }
     end
@@ -247,7 +238,6 @@ class GGem::CLI
             "#{COMMANDS.to_s.split("\n").map{ |l| "  #{l}" }.join("\n")}\n"
       assert_equal exp, subject.help
     end
-
   end
 
   class IOCommandTests < UnitTests
@@ -256,7 +246,6 @@ class GGem::CLI
       @stdout, @stderr = IOSpy.new, IOSpy.new
     end
     subject{ @cmd }
-
   end
 
   class ValidCommandTests < IOCommandTests
@@ -280,22 +269,21 @@ class GGem::CLI
 
     should "take custom CLIRB build procs" do
       cmd = @command_class.new do
-        option 'test', 'testing', :abbrev => 't'
+        option "test", "testing", :abbrev => "t"
       end
-      cmd.run(['-t'], @stdout, @stderr)
-      assert_true cmd.clirb.opts['test']
+      cmd.run(["-t"], @stdout, @stderr)
+      assert_true cmd.clirb.opts["test"]
     end
 
     should "default its summary" do
-      assert_equal '', subject.summary
+      assert_equal "", subject.summary
     end
-
   end
 
   class GitRepoCommandTests < IOCommandTests
     desc "GitRepoCommand"
     setup do
-      @gem1_root_path = TEST_SUPPORT_PATH.join('gem1')
+      @gem1_root_path = TEST_SUPPORT_PATH.join("gem1")
       Assert.stub(Dir, :pwd){ @gem1_root_path}
 
       @command_class = Class.new{ include GitRepoCommand }
@@ -314,7 +302,6 @@ class GGem::CLI
       @command_class.new
       assert_equal [Dir.pwd], gitrepo_new_called_with
     end
-
   end
 
   module RootPathTests
@@ -326,7 +313,6 @@ class GGem::CLI
         Assert.stub(Dir, :pwd){ @root_path }
       end
     end
-
   end
 
   module GitRepoSpyTests
@@ -339,9 +325,7 @@ class GGem::CLI
         @repo_spy = nil
         Assert.stub(GGem::GitRepo, :new){ |*args| @repo_spy = GitRepoSpy.new(*args) }
       end
-
     end
-
   end
 
   class GenerateCommandTests < IOCommandTests
@@ -411,13 +395,12 @@ class GGem::CLI
       assert_equal exp, err.message
       assert_not_empty err.backtrace
     end
-
   end
 
   class GemspecCommandTests < IOCommandTests
     desc "GemspecCommand"
     setup do
-      @gem1_root_path = TEST_SUPPORT_PATH.join('gem1')
+      @gem1_root_path = TEST_SUPPORT_PATH.join("gem1")
       Assert.stub(Dir, :pwd){ @gem1_root_path}
 
       @command_class = Class.new{ include GemspecCommand }
@@ -449,7 +432,6 @@ class GGem::CLI
       exp = "There are no gemspecs at #{Dir.pwd}"
       assert_equal exp, err.message
     end
-
   end
 
   module GemspecSpyTests
@@ -463,7 +445,6 @@ class GGem::CLI
         Assert.stub(GGem::Gemspec, :new){ |*args| @spec_spy = GemspecSpy.new(*args) }
       end
     end
-
   end
 
   class BuildCommandTests < IOCommandTests
@@ -494,16 +475,16 @@ class GGem::CLI
     end
 
     should "call the spec's run build cmd when run" do
-      ENV['DEBUG'] = [nil, '1'].sample
+      ENV["DEBUG"] = [nil, "1"].sample
       subject.run([], @stdout, @stderr)
 
       assert_true @spec_spy.run_build_cmd_called
 
-      exp = ENV['DEBUG'] == '1' ? "build\nbuild cmd was run\n" : ''
+      exp = ENV["DEBUG"] == "1" ? "build\nbuild cmd was run\n" : ""
       exp += "#{@spec_spy.name} #{@spec_spy.version} built to #{@spec_spy.gem_file}\n"
       assert_equal exp, @stdout.read
 
-      ENV['DEBUG'] = nil
+      ENV["DEBUG"] = nil
     end
 
     should "handle cmd errors when run" do
@@ -513,7 +494,6 @@ class GGem::CLI
       assert_raises(CommandExitError){ subject.run([], @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
     end
-
   end
 
   class InstallCommandTests < IOCommandTests
@@ -550,18 +530,18 @@ class GGem::CLI
     end
 
     should "run the build command and call the spec's run install cmds when run" do
-      ENV['DEBUG'] = [nil, '1'].sample
+      ENV["DEBUG"] = [nil, "1"].sample
       subject.run(@argv, @stdout, @stderr)
 
       assert_true @build_spy.run_called
       assert_equal [], @build_spy.argv
       assert_true @spec_spy.run_install_cmd_called
 
-      exp = ENV['DEBUG'] == '1' ? "install\ninstall cmd was run\n" : ''
+      exp = ENV["DEBUG"] == "1" ? "install\ninstall cmd was run\n" : ""
       exp += "#{@spec_spy.name} #{@spec_spy.version} installed to system gems\n"
       assert_includes exp, @stdout.read
 
-      ENV['DEBUG'] = nil
+      ENV["DEBUG"] = nil
     end
 
     should "handle cmd errors when run" do
@@ -571,7 +551,6 @@ class GGem::CLI
       assert_raises(CommandExitError){ subject.run(@argv, @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
     end
-
   end
 
   class PushCommandTests < IOCommandTests
@@ -608,7 +587,7 @@ class GGem::CLI
     end
 
     should "run the build command and call the spec's run push cmds when run" do
-      ENV['DEBUG'] = [nil, '1'].sample
+      ENV["DEBUG"] = [nil, "1"].sample
       subject.run(@argv, @stdout, @stderr)
 
       assert_true @build_spy.run_called
@@ -616,11 +595,11 @@ class GGem::CLI
       assert_true @spec_spy.run_push_cmd_called
 
       exp = "Pushing #{@spec_spy.gem_file_name} to #{@spec_spy.push_host}...\n"
-      exp += ENV['DEBUG'] == '1' ? "push\npush cmd was run\n" : ''
+      exp += ENV["DEBUG"] == "1" ? "push\npush cmd was run\n" : ""
       exp += "#{@spec_spy.gem_file_name} received.\n"
       assert_equal exp, @stdout.read
 
-      ENV['DEBUG'] = nil
+      ENV["DEBUG"] = nil
     end
 
     should "handle cmd errors when run" do
@@ -630,7 +609,6 @@ class GGem::CLI
       assert_raises(CommandExitError){ subject.run(@argv, @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
     end
-
   end
 
   class ForceTagOptionCommandTests < IOCommandTests
@@ -645,10 +623,9 @@ class GGem::CLI
     end
 
     should "add a force-tag CLIRB option" do
-      subject.run(['-f'], @stdout, @stderr)
-      assert_true subject.clirb.opts['force-tag']
+      subject.run(["-f"], @stdout, @stderr)
+      assert_true subject.clirb.opts["force-tag"]
     end
-
   end
 
   class TagCommandTests < IOCommandTests
@@ -681,7 +658,7 @@ class GGem::CLI
     end
 
     should "call the repo's run build/push cmds when run" do
-      ENV['DEBUG'] = [nil, '1'].sample
+      ENV["DEBUG"] = [nil, "1"].sample
       subject.run([], @stdout, @stderr)
 
       assert_true @repo_spy.run_validate_clean_cmd_called
@@ -693,19 +670,19 @@ class GGem::CLI
       assert_true @repo_spy.run_push_cmd_called
       assert_nil @repo_spy.run_rm_tag_cmd_called_with
 
-      exp = if ENV['DEBUG'] == '1'
+      exp = if ENV["DEBUG"] == "1"
         "validate clean\nvalidate clean cmd was run\n" \
         "validate committed\nvalidate committed cmd was run\n" \
         "add tag\nadd tag cmd was run\n"
       else
-        ''
+        ""
       end
       exp += "Tagged #{@spec_spy.version_tag}.\n"
-      exp += ENV['DEBUG'] == '1' ? "push\npush cmd was run\n" : ''
+      exp += ENV["DEBUG"] == "1" ? "push\npush cmd was run\n" : ""
       exp += "Pushed git commits and tags.\n"
       assert_equal exp, @stdout.read
 
-      ENV['DEBUG'] = nil
+      ENV["DEBUG"] = nil
     end
 
     should "handle validation cmd errors when run" do
@@ -723,7 +700,7 @@ class GGem::CLI
       err_on = [:run_validate_clean_cmd, :run_validate_committed_cmd].sample
       Assert.stub(@repo_spy, err_on){ raise GGem::GitRepo::CmdError, err_msg }
 
-      subject.run([['--force-tag', '-f'].sample], @stdout, @stderr)
+      subject.run([["--force-tag", "-f"].sample], @stdout, @stderr)
       exp = "There are files that need to be committed first.\n" \
             "Forcing tag anyway...\n"
       assert_equal exp, @stderr.read
@@ -757,7 +734,6 @@ class GGem::CLI
       assert_raises(CommandExitError){ subject.run([], @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
     end
-
   end
 
   class ReleaseCommandTests < IOCommandTests
@@ -812,16 +788,15 @@ class GGem::CLI
     end
 
     should "pass any force-tag option to the tag cmd but not the release cmd" do
-      force_tag_argv = [['--force-tag', '-f'].sample]
+      force_tag_argv = [["--force-tag", "-f"].sample]
       subject.run(force_tag_argv, @stdout, @stderr)
 
       assert_true @tag_spy.run_called
-      assert_equal ['--force-tag'], @tag_spy.argv
+      assert_equal ["--force-tag"], @tag_spy.argv
 
       assert_true @push_spy.run_called
       assert_equal [], @push_spy.argv
     end
-
   end
 
   class CommandSetTests < UnitTests
@@ -836,41 +811,41 @@ class GGem::CLI
 
     should "add/rm commands, be able to look them up and know its size" do
       assert_equal 0,  subject.size
-      assert_equal '', subject.to_s
+      assert_equal "", subject.to_s
 
-      subject.add(CommandSpy, 'test', 't', 'tst')
+      subject.add(CommandSpy, "test", "t", "tst")
       assert_equal 1, subject.size
 
-      assert_instance_of CommandSpy, subject['test']
-      assert_same subject['test'], subject['t']
-      assert_same subject['test'], subject['tst']
+      assert_instance_of CommandSpy, subject["test"]
+      assert_same subject["test"], subject["t"]
+      assert_same subject["test"], subject["tst"]
 
-      exp_strs = ["test (t, tst) # #{subject['test'].summary}"]
+      exp_strs = ["test (t, tst) # #{subject["test"].summary}"]
       assert_equal exp_strs.join("\n"), subject.to_s
 
-      subject.add(CommandSpy, 'add1')
-      exp_strs << "add1          # #{subject['add1'].summary}"
+      subject.add(CommandSpy, "add1")
+      exp_strs << "add1          # #{subject["add1"].summary}"
 
       @cmd_spy = CommandSpy.new
-      Assert.stub(@cmd_spy, :summary){ [nil, ''].sample }
+      Assert.stub(@cmd_spy, :summary){ [nil, ""].sample }
       Assert.stub(CommandSpy, :new){ @cmd_spy }
 
-      subject.add(CommandSpy, 'add2', 'add')
+      subject.add(CommandSpy, "add2", "add")
       exp_strs << "add2 (add)    "
 
-      subject.add(CommandSpy, 'add3')
-      Assert.stub(subject['add3'], :summary){ [nil, ''].sample }
+      subject.add(CommandSpy, "add3")
+      Assert.stub(subject["add3"], :summary){ [nil, ""].sample }
       exp_strs << "add3          "
 
       assert_equal exp_strs.join("\n"), subject.to_s
 
-      subject.remove('test')
-      subject.remove('add1')
-      subject.remove('add2')
-      subject.remove('add3')
+      subject.remove("test")
+      subject.remove("add1")
+      subject.remove("add2")
+      subject.remove("add3")
 
       assert_equal 0,  subject.size
-      assert_equal '', subject.to_s
+      assert_equal "", subject.to_s
     end
 
     should "call the given block when looking up unknown command names" do
@@ -878,7 +853,6 @@ class GGem::CLI
       subject[unknown_cmd_name]
       assert_equal [unknown_cmd_name], @unknown_cmd_block_called_with
     end
-
   end
 
   class CLISpy
@@ -991,19 +965,18 @@ class GGem::CLI
 
     def run_build_cmd
       @run_build_cmd_called = true
-      ['build', 0, 'build cmd was run']
+      ["build", 0, "build cmd was run"]
     end
 
     def run_install_cmd
       @run_install_cmd_called = true
-      ['install', 0, 'install cmd was run']
+      ["install", 0, "install cmd was run"]
     end
 
     def run_push_cmd
       @run_push_cmd_called = true
-      ['push', 0, 'push cmd was run']
+      ["push", 0, "push cmd was run"]
     end
-
   end
 
   class GitRepoSpy
@@ -1029,34 +1002,32 @@ class GGem::CLI
 
     def run_init_cmd
       @run_init_cmd_called = true
-      ['init', 0, 'init cmd was run']
+      ["init", 0, "init cmd was run"]
     end
 
     def run_validate_clean_cmd
       @run_validate_clean_cmd_called = true
-      ['validate clean', 0, 'validate clean cmd was run']
+      ["validate clean", 0, "validate clean cmd was run"]
     end
 
     def run_validate_committed_cmd
       @run_validate_committed_cmd_called = true
-      ['validate committed', 0, 'validate committed cmd was run']
+      ["validate committed", 0, "validate committed cmd was run"]
     end
 
     def run_add_version_tag_cmd(*args)
       @run_add_version_tag_cmd_called_with = args
-      ['add tag', 0, 'add tag cmd was run']
+      ["add tag", 0, "add tag cmd was run"]
     end
 
     def run_rm_tag_cmd(*args)
       @run_rm_tag_cmd_called_with = args
-      ['rm tag', 0, 'rm tag cmd was run']
+      ["rm tag", 0, "rm tag cmd was run"]
     end
 
     def run_push_cmd
       @run_push_cmd_called = true
-      ['push', 0, 'push cmd was run']
+      ["push", 0, "push cmd was run"]
     end
-
   end
-
 end