ggem 1.9.2 → 1.10.1

data/lib/ggem/gemspec.rb

@@ -1,11 +1,14 @@
+# frozen_string_literal: true
+
 require "pathname"
 require "scmd"
 
 module GGem; end
+
 class GGem::Gemspec
   PUSH_HOST_META_KEY = "allowed_push_host"
-  DEFAULT_PUSH_HOST  = "https://rubygems.org".freeze
-  BUILD_TO_DIRNAME   = "pkg".freeze
+  DEFAULT_PUSH_HOST  = "https://rubygems.org"
+  BUILD_TO_DIRNAME   = "pkg"
 
   NotFoundError = Class.new(ArgumentError)
   LoadError     = Class.new(ArgumentError)
@@ -53,25 +56,34 @@ class GGem::Gemspec
   def run_cmd(cmd_string)
     cmd = Scmd.new(cmd_string)
     cmd.run
-    if !cmd.success?
-      raise CmdError, "#{cmd_string}\n" \
-                      "#{cmd.stderr.empty? ? cmd.stdout : cmd.stderr}"
+    unless cmd.success?
+      raise(
+        CmdError,
+        "#{cmd_string}\n#{cmd.stderr.empty? ? cmd.stdout : cmd.stderr}",
+      )
     end
     [cmd_string, cmd.exitstatus, cmd.stdout]
   end
 
   def load_gemspec(path)
-    eval(path.read, TOPLEVEL_BINDING, path.expand_path.to_s)
-  rescue ScriptError, StandardError => e
-    original_line = e.backtrace.find{ |line| line.include?(path.to_s) }
-    msg = "There was a #{e.class} while loading #{path.basename}: \n#{e.message}"
+    eval( # rubocop:disable Security/Eval
+      path.read,
+      TOPLEVEL_BINDING,
+      path.expand_path.to_s,
+    )
+  rescue ScriptError, StandardError => ex
+    original_line = ex.backtrace.find{ |line| line.include?(path.to_s) }
+    msg =
+      "There was a #{ex.class} while loading #{path.basename}: \n#{ex.message}"
     msg << " from\n  #{original_line}" if original_line
     msg << "\n"
     raise LoadError, msg
   end
 
   def get_push_host(spec)
-    ENV["GGEM_PUSH_HOST"] || get_meta(spec)[PUSH_HOST_META_KEY] || DEFAULT_PUSH_HOST
+    ENV["GGEM_PUSH_HOST"] ||
+    get_meta(spec)[PUSH_HOST_META_KEY] ||
+    DEFAULT_PUSH_HOST
   end
 
   def get_meta(spec)

data/lib/ggem/git_repo.rb

@@ -1,7 +1,10 @@
+# frozen_string_literal: true
+
 require "pathname"
 require "scmd"
 
 module GGem; end
+
 class GGem::GitRepo
   NotFoundError = Class.new(ArgumentError)
   CmdError      = Class.new(RuntimeError)
@@ -46,7 +49,7 @@ class GGem::GitRepo
     cmd_string = "cd #{@path} && #{cmd_string}"
     cmd = Scmd.new(cmd_string)
     cmd.run
-    if !cmd.success?
+    unless cmd.success?
       raise CmdError, "#{cmd_string}\n" \
                       "#{cmd.stderr.empty? ? cmd.stdout : cmd.stderr}"
     end

data/lib/ggem/template.rb

@@ -1,7 +1,10 @@
+# frozen_string_literal: true
+
 require "erb"
 require "fileutils"
 
 module GGem; end
+
 class GGem::Template
   def initialize(ggem)
     @ggem = ggem
@@ -16,12 +19,15 @@ class GGem::Template
     save_folder "log"
     save_folder "tmp"
 
+    save_file("l.yml.erb",        ".l.yml")
+    save_file("t.yml.erb",        ".t.yml")
+    save_file("rubocop.yml.erb",  ".rubocop.yml")
     save_file("ruby-version.erb", ".ruby-version")
-    save_file("gitignore.erb", ".gitignore")
-    save_file("Gemfile.erb",   "Gemfile")
-    save_file("gemspec.erb",   "#{@ggem.name}.gemspec")
-    save_file("README.md.erb", "README.md")
-    save_file("LICENSE.erb",   "LICENSE")
+    save_file("gitignore.erb",    ".gitignore")
+    save_file("Gemfile.erb",      "Gemfile")
+    save_file("gemspec.erb",      "#{@ggem.name}.gemspec")
+    save_file("README.md.erb",    "README.md")
+    save_file("LICENSE.erb",      "LICENSE")
 
     save_file("lib.rb.erb",         "lib/#{@ggem.ruby_name}.rb")
     save_file("lib_version.rb.erb", "lib/#{@ggem.ruby_name}/version.rb")
@@ -37,7 +43,7 @@ class GGem::Template
 
   private
 
-  def save_folder(relative_path=nil)
+  def save_folder(relative_path = nil)
     path = File.join([@ggem.path, relative_path].compact)
     FileUtils.mkdir_p(path)
   end
@@ -51,10 +57,10 @@ class GGem::Template
     source_file = File.join(File.dirname(__FILE__), "template_file", source)
     output_file = File.join(@ggem.root_path, @ggem.name, output)
 
-    if File.exists?(source_file)
+    if File.exist?(source_file)
       FileUtils.mkdir_p(File.dirname(output_file))
       erb = ERB.new(File.read(source_file))
-      File.open(output_file, "w") {|f| f << erb.result(binding) }
+      File.open(output_file, "w"){ |f| f << erb.result(binding) }
     else
       raise ArgumentError, "the source file `#{source_file}` does not exist"
     end

data/lib/ggem/template_file/Gemfile.erb

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 source "https://rubygems.org"
 
 ruby "~> 2.5"
 
 gemspec
 
-gem "pry", "~> 0.12.2"
+gem "pry"

data/lib/ggem/template_file/gemspec.erb

@@ -1,4 +1,6 @@
 # -*- encoding: utf-8 -*-
+# frozen_string_literal: true
+
 lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require "<%= @ggem.ruby_name %>/version"
@@ -13,14 +15,16 @@ Gem::Specification.new do |gem|
   gem.homepage    = "TODO: homepage"
   gem.license     = "MIT"
 
-  gem.files         = `git ls-files | grep "^[^.]"`.split($/)
+  gem.files = `git ls-files | grep "^[^.]"`.split($INPUT_RECORD_SEPARATOR)
+
   gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
   gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
   gem.require_paths = ["lib"]
 
   gem.required_ruby_version = "~> 2.5"
 
-  gem.add_development_dependency("assert", ["~> 2.18.2"])
+  gem.add_development_dependency("much-style-guide", ["~> 0.6.0"])
+  gem.add_development_dependency("assert",           ["~> 2.19.3"])
 
   # TODO: gem.add_dependency("gem-name", ["~> 0.0.0"])
 end

data/lib/ggem/template_file/gitignore.erb

@@ -4,6 +4,7 @@
 .rbx/
 .bundle
 .config
+.rubocop-*-yml
 .yardoc
 Gemfile.lock
 InstalledFiles

data/lib/ggem/template_file/l.yml.erb

@@ -0,0 +1,8 @@
+# https://github.com/redding/l.rb
+
+linters:
+  - name: "Rubocop"
+    cmd: "bundle exec rubocop"
+    extensions:
+      - ".rb"
+    cli_abbrev: "u"

data/lib/ggem/template_file/lib.rb.erb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "<%= @ggem.ruby_name %>/version"
 
 module <%= @ggem.module_name %>

data/lib/ggem/template_file/lib_version.rb.erb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module <%= @ggem.module_name %>
   VERSION = "0.0.1"
 end

data/lib/ggem/template_file/rubocop.yml.erb

@@ -0,0 +1,3 @@
+inherit_gem:
+  much-style-guide:
+    - "lib/much-style-guide/rubocop.yml"

data/lib/ggem/template_file/ruby-version.erb

@@ -1 +1 @@
-2.5.5
+2.5.8

data/lib/ggem/template_file/t.yml.erb

@@ -0,0 +1,6 @@
+# https://github.com/redding/t.rb
+
+default_cmd: "bundle exec assert"
+test_dir: "test"
+test_file_suffixes:
+  - "_tests.rb"

data/lib/ggem/template_file/test_helper.rb.erb

@@ -1,10 +1,12 @@
-# this file is automatically required when you run `assert`
-# put any test helpers here
+# frozen_string_literal: true
 
-# add the root dir to the load path
+# This file is automatically required when you run `assert`; put any test
+# helpers here.
+
+# Add the root dir to the load path.
 $LOAD_PATH.unshift(File.expand_path("../..", __FILE__))
 
-# require pry for debugging (`binding.pry`)
+# Require pry for debugging (`binding.pry`).
 require "pry"
 
 require "test/support/factory"

data/lib/ggem/template_file/test_support_factory.rb.erb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert/factory"
 
 module Factory

data/lib/ggem/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module GGem
-  VERSION = "1.9.2"
+  VERSION = "1.10.1"
 end

data/test/helper.rb

@@ -1,14 +1,16 @@
-# this file is automatically required when you run `assert`
-# put any test helpers here
+# frozen_string_literal: true
 
-# add the root dir to the load path
+# This file is automatically required when you run `assert`; put any test
+# helpers here.
+
+# Add the root dir to the load path.
 require "pathname"
 ROOT_PATH = Pathname.new(File.expand_path("../..", __FILE__))
 $LOAD_PATH.unshift(ROOT_PATH.to_s)
 TMP_PATH = ROOT_PATH.join("tmp")
 TEST_SUPPORT_PATH = ROOT_PATH.join("test/support")
 
-# require pry for debugging (`binding.pry`)
+# Require pry for debugging (`binding.pry`).
 require "pry"
 
 require "test/support/factory"

data/test/support/cmd_tests_helpers.rb

@@ -1,11 +1,14 @@
-require "much-plugin"
+# frozen_string_literal: true
+
+require "much-mixin"
 require "scmd"
 
 module GGem; end
+
 module GGem::CmdTestsHelpers
-  include MuchPlugin
+  include MuchMixin
 
-  plugin_included do
+  mixin_included do
     setup do
       ENV["SCMD_TEST_MODE"] = "1"
 
@@ -37,12 +40,11 @@ module GGem::CmdTestsHelpers
         cmd.stderr     = Factory.string
         @cmd_spy       = cmd
       end
-      err = nil
-      begin; run_cmd_block.call; rescue StandardError => err; end
 
-      assert_kind_of cmd_error_class, err
+      ex =
+        assert_that{ run_cmd_block.call }.raises(cmd_error_class)
       exp = "#{@cmd_spy.cmd_str}\n#{@cmd_spy.stderr}"
-      assert_equal exp, err.message
+      assert_equal exp, ex.message
     end
   end
 end

data/test/support/factory.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert/factory"
 
 module Factory

data/test/support/gem1/gem1.gemspec

@@ -1,16 +1,17 @@
 # -*- encoding: utf-8 -*-
+# frozen_string_literal: true
+
 Gem::Specification.new do |gem|
   gem.name        = "gem1"
   gem.version     = "0.1.0"
   gem.authors     = []
   gem.email       = []
-  gem.summary     = %q{Gem 1 summary}
-  gem.description = %q{Gem 1 description}
+  gem.summary     = "Gem 1 summary"
+  gem.description = "Gem 1 description"
   gem.license     = "MIT"
 
   gem.files         = []
   gem.executables   = []
   gem.test_files    = []
   gem.require_paths = []
-
 end

data/test/support/gem2/gem2.gemspec

@@ -1,11 +1,13 @@
 # -*- encoding: utf-8 -*-
+# frozen_string_literal: true
+
 Gem::Specification.new do |gem|
   gem.name        = "gem2"
   gem.version     = "0.2.0"
   gem.authors     = []
   gem.email       = []
-  gem.summary     = %q{Gem 2 summary}
-  gem.description = %q{Gem 2 description}
+  gem.summary     = "Gem 2 summary"
+  gem.description = "Gem 2 description"
   gem.license     = "MIT"
 
   gem.metadata["allowed_push_host"] = "http://gems.example.com"
@@ -14,5 +16,4 @@ Gem::Specification.new do |gem|
   gem.executables   = []
   gem.test_files    = []
   gem.require_paths = []
-
 end

data/test/support/name_set.rb

@@ -1,10 +1,14 @@
+# frozen_string_literal: true
+
 module GGem; end
+
 module GGem::NameSet
   class Base
     attr_reader :variations, :name, :module_name, :ruby_name
 
     def expected_folders
-      [ "",
+      [
+        "",
         "lib",
         "lib/#{@ruby_name}",
         "test",
@@ -12,12 +16,13 @@ module GGem::NameSet
         "test/system",
         "test/unit",
         "log",
-        "tmp"
+        "tmp",
       ]
     end
 
     def expected_files
-      [ ".ruby-version",
+      [
+        ".ruby-version",
         ".gitignore",
         "Gemfile",
         "#{@name}.gemspec",

data/test/system/ggem_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "ggem"
 
@@ -5,7 +7,6 @@ require "test/support/name_set"
 
 module GGem
   class SystemTests < Assert::Context
-
     NS_SIMPLE = GGem::NameSet::Simple
     NS_UNDER  = GGem::NameSet::Underscored
     NS_HYPHEN = GGem::NameSet::HyphenatedOther
@@ -16,7 +17,8 @@ module GGem
   class GemTests < SystemTests
     desc "Gem"
 
-    should "know its name attrs for various name styles (simple/underscored/hyphenated)" do
+    should "know its name attrs for various name styles "\
+           "(simple/underscored/hyphenated)" do
       [NS_SIMPLE, NS_UNDER, NS_HYPHEN].each do |ns|
         assert_gem_name_set(ns.new)
       end
@@ -43,7 +45,8 @@ module GGem
       FileUtils.rm_rf(TMP_PATH)
     end
 
-    should "save gems with various name styles (simple/underscored/hyphenated)" do
+    should "save gems with various name styles "\
+           "(simple/underscored/hyphenated)" do
       [NS_SIMPLE, NS_UNDER, NS_HYPHEN].each do |ns|
         init_gem = GGem::Gem.new(TMP_PATH, ns.new.variations.first)
         gem_from_save = init_gem.save!
@@ -58,10 +61,12 @@ module GGem
     def assert_gem_created(name_set)
       folders = name_set.expected_folders
       files   = name_set.expected_files
-      paths   = (folders + files).collect{ |p| File.join(TMP_PATH, name_set.name, p) }
+      paths   = (folders + files).collect do |p|
+        File.join(TMP_PATH, name_set.name, p)
+      end
 
       paths.flatten.each do |path|
-        assert File.exists?(path), "`#{path}` does not exist"
+        assert File.exist?(path), "`#{path}` does not exist"
       end
     end
   end

data/test/unit/cli_tests.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "assert"
 require "ggem/cli"
 
@@ -6,7 +8,7 @@ require "ggem/cli/commands"
 require "ggem/gem"
 require "ggem/gemspec"
 require "ggem/git_repo"
-require "much-plugin"
+require "much-mixin"
 
 class GGem::CLI
   class UnitTests < Assert::Context
@@ -145,7 +147,7 @@ class GGem::CLI
   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
@@ -161,7 +163,7 @@ class GGem::CLI
   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
@@ -215,8 +217,8 @@ 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
@@ -267,7 +269,7 @@ 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"]
@@ -282,7 +284,7 @@ class GGem::CLI
     desc "GitRepoCommand"
     setup do
       @gem1_root_path = TEST_SUPPORT_PATH.join("gem1")
-      Assert.stub(Dir, :pwd){ @gem1_root_path}
+      Assert.stub(Dir, :pwd){ @gem1_root_path }
 
       @command_class = Class.new{ include GitRepoCommand }
       @cmd = @command_class.new
@@ -303,9 +305,9 @@ class GGem::CLI
   end
 
   module RootPathTests
-    include MuchPlugin
+    include MuchMixin
 
-    plugin_included do
+    mixin_included do
       setup do
         @root_path = Factory.path
         Assert.stub(Dir, :pwd){ @root_path }
@@ -314,14 +316,16 @@ class GGem::CLI
   end
 
   module GitRepoSpyTests
-    include MuchPlugin
+    include MuchMixin
 
-    plugin_included do
+    mixin_included do
       include RootPathTests
 
       setup do
         @repo_spy = nil
-        Assert.stub(GGem::GitRepo, :new){ |*args| @repo_spy = GitRepoSpy.new(*args) }
+        Assert.stub(GGem::GitRepo, :new) do |*args|
+          @repo_spy = GitRepoSpy.new(*args)
+        end
       end
     end
   end
@@ -381,17 +385,15 @@ class GGem::CLI
 
     should "re-raise a specific argument error on gem 'no name' errors" do
       Assert.stub(@gem_class, :new){ raise GGem::Gem::NoNameError }
-      err = nil
-      begin
-        cmd = @command_class.new
-        cmd.run([])
-      rescue ArgumentError => err
-      end
 
-      assert_not_nil err
+      ex =
+        assert_that{
+          cmd = @command_class.new
+          cmd.run([])
+        }.raises(ArgumentError)
       exp = "GEM-NAME must be provided"
-      assert_equal exp, err.message
-      assert_not_empty err.backtrace
+      assert_equal exp, ex.message
+      assert_not_empty ex.backtrace
     end
   end
 
@@ -399,7 +401,7 @@ class GGem::CLI
     desc "GemspecCommand"
     setup do
       @gem1_root_path = TEST_SUPPORT_PATH.join("gem1")
-      Assert.stub(Dir, :pwd){ @gem1_root_path}
+      Assert.stub(Dir, :pwd){ @gem1_root_path }
 
       @command_class = Class.new{ include GemspecCommand }
       @cmd = @command_class.new
@@ -422,25 +424,23 @@ class GGem::CLI
       root = Factory.path
       Assert.stub(Dir, :pwd){ root }
 
-      begin
-        cmd = @command_class.new
-      rescue ArgumentError => err
-      end
-      assert_not_nil err
+      ex = assert_that{ @command_class.new }.raises(ArgumentError)
       exp = "There are no gemspecs at #{Dir.pwd}"
-      assert_equal exp, err.message
+      assert_equal exp, ex.message
     end
   end
 
   module GemspecSpyTests
-    include MuchPlugin
+    include MuchMixin
 
-    plugin_included do
+    mixin_included do
       include RootPathTests
 
       setup do
         @spec_spy = nil
-        Assert.stub(GGem::Gemspec, :new){ |*args| @spec_spy = GemspecSpy.new(*args) }
+        Assert.stub(GGem::Gemspec, :new) do |*args|
+          @spec_spy = GemspecSpy.new(*args)
+        end
       end
     end
   end
@@ -479,7 +479,9 @@ class GGem::CLI
       assert_true @spec_spy.run_build_cmd_called
 
       exp = ENV["DEBUG"] == "1" ? "build\nbuild cmd was run\n" : ""
-      exp += "#{@spec_spy.name} #{@spec_spy.version} built to #{@spec_spy.gem_file}\n"
+      exp +=
+        "#{@spec_spy.name} #{@spec_spy.version} built to "\
+        "#{@spec_spy.gem_file}\n"
       assert_equal exp, @stdout.read
 
       ENV["DEBUG"] = nil
@@ -487,7 +489,9 @@ class GGem::CLI
 
     should "handle cmd errors when run" do
       err_msg = Factory.string
-      Assert.stub(@spec_spy, :run_build_cmd){ raise GGem::Gemspec::CmdError, err_msg }
+      Assert.stub(@spec_spy, :run_build_cmd) do
+        raise GGem::Gemspec::CmdError, err_msg
+      end
 
       assert_raises(CommandExitError){ subject.run([], @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
@@ -500,7 +504,9 @@ class GGem::CLI
     desc "InstallCommand"
     setup do
       @build_spy = nil
-      Assert.stub(BuildCommand, :new){ |*args| @build_spy = CommandSpy.new(*args) }
+      Assert.stub(BuildCommand, :new) do |*args|
+        @build_spy = CommandSpy.new(*args)
+      end
 
       @command_class = InstallCommand
       @cmd = @command_class.new
@@ -527,7 +533,8 @@ class GGem::CLI
       assert @build_spy
     end
 
-    should "run the build command and call the spec's run install cmds when run" do
+    should "run the build command and call the spec's run install cmds when "\
+           "run" do
       ENV["DEBUG"] = [nil, "1"].sample
       subject.run(@argv, @stdout, @stderr)
 
@@ -544,7 +551,9 @@ class GGem::CLI
 
     should "handle cmd errors when run" do
       err_msg = Factory.string
-      Assert.stub(@spec_spy, :run_install_cmd){ raise GGem::Gemspec::CmdError, err_msg }
+      Assert.stub(@spec_spy, :run_install_cmd) do
+        raise GGem::Gemspec::CmdError, err_msg
+      end
 
       assert_raises(CommandExitError){ subject.run(@argv, @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
@@ -557,7 +566,9 @@ class GGem::CLI
     desc "PushCommand"
     setup do
       @build_spy = nil
-      Assert.stub(BuildCommand, :new){ |*args| @build_spy = CommandSpy.new(*args) }
+      Assert.stub(BuildCommand, :new) do |*args|
+        @build_spy = CommandSpy.new(*args)
+      end
 
       @command_class = PushCommand
       @cmd = @command_class.new
@@ -602,7 +613,9 @@ class GGem::CLI
 
     should "handle cmd errors when run" do
       err_msg = Factory.string
-      Assert.stub(@spec_spy, :run_push_cmd){ raise GGem::Gemspec::CmdError, err_msg }
+      Assert.stub(@spec_spy, :run_push_cmd) do
+        raise GGem::Gemspec::CmdError, err_msg
+      end
 
       assert_raises(CommandExitError){ subject.run(@argv, @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
@@ -715,7 +728,9 @@ class GGem::CLI
 
     should "remove the version tag on push errors" do
       err_msg = Factory.string
-      Assert.stub(@repo_spy, :run_push_cmd){ raise GGem::GitRepo::CmdError, err_msg }
+      Assert.stub(@repo_spy, :run_push_cmd) do
+        raise GGem::GitRepo::CmdError, err_msg
+      end
 
       assert_raises(CommandExitError){ subject.run([], @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
@@ -725,9 +740,13 @@ class GGem::CLI
     end
 
     should "handle tag removal cmd errors when run" do
-      Assert.stub(@repo_spy, :run_push_cmd){ raise GGem::GitRepo::CmdError, Factory.string }
+      Assert.stub(@repo_spy, :run_push_cmd) do
+        raise GGem::GitRepo::CmdError, Factory.string
+      end
       err_msg = Factory.string
-      Assert.stub(@repo_spy, :run_rm_tag_cmd){ raise GGem::GitRepo::CmdError, err_msg }
+      Assert.stub(@repo_spy, :run_rm_tag_cmd) do
+        raise GGem::GitRepo::CmdError, err_msg
+      end
 
       assert_raises(CommandExitError){ subject.run([], @stdout, @stderr) }
       assert_equal "#{err_msg}\n", @stderr.read
@@ -743,7 +762,9 @@ class GGem::CLI
       Assert.stub(TagCommand, :new){ |*args| @tag_spy = CommandSpy.new(*args) }
 
       @push_spy = nil
-      Assert.stub(PushCommand, :new){ |*args| @push_spy = CommandSpy.new(*args) }
+      Assert.stub(PushCommand, :new) do |*args|
+        @push_spy = CommandSpy.new(*args)
+      end
 
       @command_class = ReleaseCommand
       @cmd = @command_class.new
@@ -755,8 +776,9 @@ class GGem::CLI
     end
 
     should "know its summary" do
-      exp = "Tag #{@spec_spy.version_tag} and push built #{@spec_spy.gem_file_name} to " \
-               "#{@spec_spy.push_host}"
+      exp =
+        "Tag #{@spec_spy.version_tag} and push built "\
+        "#{@spec_spy.gem_file_name} to #{@spec_spy.push_host}"
       assert_equal exp, subject.summary
     end
 
@@ -935,7 +957,8 @@ class GGem::CLI
 
   class GemspecSpy
     attr_reader :name, :version, :version_tag, :push_host
-    attr_reader :run_build_cmd_called, :run_install_cmd_called, :run_push_cmd_called
+    attr_reader :run_build_cmd_called, :run_install_cmd_called
+    attr_reader :run_push_cmd_called
 
     def initialize(root_path)
       @root        = Pathname.new(File.expand_path(root_path))
@@ -950,15 +973,15 @@ class GGem::CLI
     end
 
     def path
-      @root.join("#{self.name}.gemspec")
+      @root.join("#{name}.gemspec")
     end
 
     def gem_file_name
-      "#{self.name}-#{self.version}.gem"
+      "#{name}-#{version}.gem"
     end
 
     def gem_file
-      File.join(GGem::Gemspec::BUILD_TO_DIRNAME, self.gem_file_name)
+      File.join(GGem::Gemspec::BUILD_TO_DIRNAME, gem_file_name)
     end
 
     def run_build_cmd
@@ -980,8 +1003,10 @@ class GGem::CLI
   class GitRepoSpy
     attr_reader :path
     attr_reader :run_init_cmd_called
-    attr_reader :run_validate_clean_cmd_called, :run_validate_committed_cmd_called
-    attr_reader :run_add_version_tag_cmd_called_with, :run_rm_tag_cmd_called_with
+    attr_reader :run_validate_clean_cmd_called
+    attr_reader :run_validate_committed_cmd_called
+    attr_reader :run_add_version_tag_cmd_called_with
+    attr_reader :run_rm_tag_cmd_called_with
     attr_reader :run_push_cmd_called
 
     def initialize(path)