jeweler 1.8.2 → 2.3.7

data/test/jeweler/commands/version/test_bump_major.rb

@@ -4,8 +4,7 @@ class Jeweler
   module Commands
     module Version
       class TestBumpMajor < Test::Unit::TestCase
-
-        should "call bump_major on version_helper in update_version" do
+        should 'call bump_major on version_helper in update_version' do
           mock(version_helper = Object.new).bump_major
 
           command = Jeweler::Commands::Version::BumpMajor.new
@@ -14,9 +13,6 @@ class Jeweler
           command.update_version
         end
       end
-
     end
   end
 end
-
-

data/test/jeweler/commands/version/test_bump_minor.rb

@@ -4,8 +4,7 @@ class Jeweler
   module Commands
     module Version
       class TestBumpMinor < Test::Unit::TestCase
-
-        should "call bump_minor on version_helper in update_version" do
+        should 'call bump_minor on version_helper in update_version' do
           mock(version_helper = Object.new).bump_minor
 
           command = Jeweler::Commands::Version::BumpMinor.new

data/test/jeweler/commands/version/test_bump_patch.rb

@@ -4,8 +4,7 @@ class Jeweler
   module Commands
     module Version
       class TestBumpPatch < Test::Unit::TestCase
-
-        should "call bump_patch on version_helper in update_version" do
+        should 'call bump_patch on version_helper in update_version' do
           mock(version_helper = Object.new).bump_patch
 
           command = Jeweler::Commands::Version::BumpPatch.new
@@ -17,4 +16,3 @@ class Jeweler
     end
   end
 end
-

data/test/jeweler/commands/version/test_write.rb

@@ -4,8 +4,7 @@ class Jeweler
   module Commands
     module Version
       class TestWrite < Test::Unit::TestCase
-
-        should "call write_version on version_helper in update_version" do
+        should 'call write_version on version_helper in update_version' do
           mock(version_helper = Object.new).update_to 1, 2, 3, nil
 
           command = Jeweler::Commands::Version::Write.new
@@ -19,4 +18,3 @@ class Jeweler
     end
   end
 end
-

data/test/jeweler/generator/test_application.rb

@@ -37,38 +37,36 @@ class TestApplication < Test::Unit::TestCase
     end
   end
 
-
-  context "when options indicate version" do
+  context 'when options indicate version' do
     setup do
       stub(Jeweler::Generator::Application).build_opts do
-        stub_options(:show_version => true)
+        stub_options(show_version: true)
       end
 
-      stub(Jeweler::Generator).new { raise "Shouldn't have made this far"}
+      stub(Jeweler::Generator).new { raise "Shouldn't have made this far" }
 
       assert_nothing_raised do
-        @result = run_application("-v")
+        @result = run_application('-v')
       end
     end
-    
+
     should_exit_with_code 1
-    
+
     should 'should puts option version' do
       assert_match 'Version:', @stderr
     end
   end
-  
 
-  context "when options indicate help usage" do
+  context 'when options indicate help usage' do
     setup do
       stub(Jeweler::Generator::Application).build_opts do
-        stub_options(:show_help => true)
+        stub_options(show_help: true)
       end
 
-      stub(Jeweler::Generator).new { raise "Shouldn't have made this far"}
+      stub(Jeweler::Generator).new { raise "Shouldn't have made this far" }
 
       assert_nothing_raised do
-        @result = run_application("-h")
+        @result = run_application('-h')
       end
     end
 
@@ -83,16 +81,16 @@ class TestApplication < Test::Unit::TestCase
     end
   end
 
-  context "when options indicate an invalid argument" do
+  context 'when options indicate an invalid argument' do
     setup do
       stub(Jeweler::Generator::Application).build_opts do
-        stub_options(:invalid_argument => '--invalid-argument')
+        stub_options(invalid_argument: '--invalid-argument')
       end
 
-      stub(Jeweler::Generator).new { raise "Shouldn't have made this far"}
+      stub(Jeweler::Generator).new { raise "Shouldn't have made this far" }
 
       assert_nothing_raised do
-        @result = run_application("--invalid-argument")
+        @result = run_application('--invalid-argument')
       end
     end
 
@@ -109,34 +107,32 @@ class TestApplication < Test::Unit::TestCase
     should 'not display anything on stdout' do
       assert_equal '', @stdout.squeeze.strip
     end
-
   end
 
-  context "when options are good" do
+  context 'when options are good' do
     setup do
-      @generator = "generator"
+      @generator = 'generator'
       stub(@generator).run
       stub(Jeweler::Generator).new { @generator }
 
-      @jeweler_options = stub_options :project_name => 'zomg'
+      @jeweler_options = stub_options project_name: 'zomg'
       stub(Jeweler::Generator::Application).build_options do
         @jeweler_options
       end
 
       assert_nothing_raised do
-        @result = run_application("zomg")
+        @result = run_application('zomg')
       end
     end
 
     should_exit_with_code 0
 
-    should "create generator with options" do
-      assert_received(Jeweler::Generator) {|subject| subject.new(@jeweler_options) }
+    should 'create generator with options' do
+      assert_received(Jeweler::Generator) { |subject| subject.new(@jeweler_options) }
     end
 
-    should "run generator" do
-      assert_received(@generator) {|subject| subject.run }
+    should 'run generator' do
+      assert_received(@generator, &:run)
     end
   end
-
 end

data/test/jeweler/generator/test_options.rb

@@ -1,7 +1,6 @@
 require 'test_helper'
 
 class TestOptions < Test::Unit::TestCase
-
   def setup
     set_default_git_config
     stub_git_config valid_git_config
@@ -21,17 +20,17 @@ class TestOptions < Test::Unit::TestCase
 
   def setup_options(*arguments)
     stub(Jeweler::Generator::Options).git_config { valid_git_config }
-    @options = Jeweler::Generator::Options.new(["project_name"] + arguments)
+    @options = Jeweler::Generator::Options.new(['project_name'] + arguments)
   end
 
   def self.for_options(*options)
     context options.join(' ') do
-      setup { setup_options *options }
+      setup { setup_options(*options) }
       yield
     end
   end
 
-  context "default options" do
+  context 'default options' do
     setup { setup_options }
     should_have_testing_framework :shoulda
     should_have_docmentation_framework :rdoc
@@ -40,32 +39,27 @@ class TestOptions < Test::Unit::TestCase
       assert ! @options[:create_repo]
     end
 
-    should "have project name" do
-      assert_equal "project_name", @options[:project_name]
+    should 'have project name' do
+      assert_equal 'project_name', @options[:project_name]
     end
 
-    should "use github username from git config" do
+    should 'use github username from git config' do
       assert_equal @github_user, @options[:github_username]
     end
-
-    should "use github token from git config" do
-      assert_equal @github_token, @options[:github_token]
-    end
-
-    should "use user name from git config" do
+    should 'use user name from git config' do
       assert_equal @git_name, @options[:user_name]
     end
 
-    should "use user email from git config" do
+    should 'use user email from git config' do
       assert_equal @git_email, @options[:user_email]
     end
 
-    should "use bundler" do
+    should 'use bundler' do
       assert @options[:use_bundler]
     end
   end
 
-  for_options "--bacon" do
+  for_options '--bacon' do
     should_have_testing_framework :bacon
   end
 
@@ -85,11 +79,11 @@ class TestOptions < Test::Unit::TestCase
     should_have_testing_framework :shoulda
   end
 
-  for_options "--testunit" do
+  for_options '--testunit' do
     should_have_testing_framework :testunit
   end
 
-  for_options "--testspec" do
+  for_options '--testspec' do
     should_have_testing_framework :testspec
   end
 
@@ -142,13 +136,13 @@ class TestOptions < Test::Unit::TestCase
       assert_equal 'foo', @options[:directory]
     end
   end
-  
+
   for_options '--version' do
     should 'show version' do
       assert @options[:show_version]
     end
   end
-  
+
   for_options '-v' do
     should 'show version' do
       assert @options[:show_version]
@@ -174,13 +168,13 @@ class TestOptions < Test::Unit::TestCase
   end
 
   for_options '--user-name', 'myname' do
-    should "set user name" do
+    should 'set user name' do
       assert_equal 'myname', @options[:user_name]
     end
   end
 
   for_options '--user-email', 'myname@mydomain.com' do
-    should "set user email" do
+    should 'set user email' do
       assert_equal 'myname@mydomain.com', @options[:user_email]
     end
   end
@@ -198,40 +192,34 @@ class TestOptions < Test::Unit::TestCase
   end
 
   for_options '--github-username', 'mygithub' do
-    should "set github username" do
+    should 'set github username' do
       assert_equal 'mygithub', @options[:github_username]
     end
   end
 
-  for_options '--github-token', 'mygithubtoken' do
-    should "set github token" do
-      assert_equal 'mygithubtoken', @options[:github_token]
-    end
-  end
-
   for_options '--bundler' do
-    should "use bundler" do
+    should 'use bundler' do
       assert @options[:use_bundler]
     end
   end
 
   for_options '--no-bundler' do
-    should "not use bundler" do
+    should 'not use bundler' do
       assert !@options[:use_bundler]
     end
   end
 
-  context "merging options" do
-    should "take options from each" do
-      options = Jeweler::Generator::Options.new(["--rspec"]).
-        merge Jeweler::Generator::Options.new(["--create-repo"])
+  context 'merging options' do
+    should 'take options from each' do
+      options = Jeweler::Generator::Options.new(['--rspec'])
+                                           .merge Jeweler::Generator::Options.new(['--create-repo'])
       assert_equal :rspec, options[:testing_framework]
       assert options[:create_repo]
     end
 
-    should "shadow options" do
-      options = Jeweler::Generator::Options.new(["--bacon"]).
-        merge Jeweler::Generator::Options.new(["--rspec"])
+    should 'shadow options' do
+      options = Jeweler::Generator::Options.new(['--bacon'])
+                                           .merge Jeweler::Generator::Options.new(['--rspec'])
       assert_equal :rspec, options[:testing_framework]
     end
   end

data/test/jeweler/test_gemspec_helper.rb

@@ -2,10 +2,10 @@ require 'test_helper'
 
 class TestGemspecHelper < Test::Unit::TestCase
   def setup
-    Rake.application.instance_variable_set(:@rakefile, "Rakefile")
+    Rake.application.instance_variable_set(:@rakefile, 'Rakefile')
   end
 
-  context "given a gemspec" do
+  context 'given a gemspec' do
     setup do
       @spec = build_spec
       @helper = Jeweler::GemSpecHelper.new(@spec, File.dirname(__FILE__))
@@ -16,7 +16,7 @@ class TestGemspecHelper < Test::Unit::TestCase
     end
   end
 
-  context "#write" do
+  context '#write' do
     setup do
       @spec = build_spec
       @helper = Jeweler::GemSpecHelper.new(@spec, File.dirname(__FILE__))
@@ -29,15 +29,15 @@ class TestGemspecHelper < Test::Unit::TestCase
       FileUtils.rm_f(@helper.path)
     end
 
-    should "create gemspec file" do
-      assert File.exists?(@helper.path)
+    should 'create gemspec file' do
+      assert File.exist?(@helper.path)
     end
 
-    should "make valid spec" do
+    should 'make valid spec' do
       assert @helper.valid?
     end
 
-    should "parse" do
+    should 'parse' do
       @helper.parse
     end
   end

data/test/jeweler/test_generator.rb

@@ -3,49 +3,48 @@ require 'test_helper'
 class TestGenerator < Test::Unit::TestCase
   def build_generator(testing_framework = :shoulda, options = {})
     options = {
-      :project_name => 'the-perfect-gem',
-      :user_name => 'John Doe',
-      :user_email => 'john@example.com',
-      :github_username => 'johndoe',
-      :github_token => 'yyz',
-      :documentation_framework => :rdoc
+      project_name: 'the-perfect-gem',
+      user_name: 'John Doe',
+      user_email: 'john@example.com',
+      github_username: 'johndoe',
+      documentation_framework: :rdoc
     }.merge(options)
 
     options[:testing_framework] = testing_framework
     Jeweler::Generator.new(options)
   end
 
-  should "have the correct constant name" do
-    assert_equal "ThePerfectGem", build_generator.constant_name
+  should 'have the correct constant name' do
+    assert_equal 'ThePerfectGem', build_generator.constant_name
   end
 
-  should "have the correct file name prefix" do
-    assert_equal "the_perfect_gem", build_generator.file_name_prefix
+  should 'have the correct file name prefix' do
+    assert_equal 'the_perfect_gem', build_generator.file_name_prefix
   end
 
-  should "have the correct require name" do
-    assert_equal "the-perfect-gem", build_generator.require_name
+  should 'have the correct require name' do
+    assert_equal 'the-perfect-gem', build_generator.require_name
   end
 
-  should "have the correct lib file name" do
-    assert_equal "the-perfect-gem.rb", build_generator.lib_filename
+  should 'have the correct lib file name' do
+    assert_equal 'the-perfect-gem.rb', build_generator.lib_filename
   end
-  
-  should "have the correct git-remote" do
-    assert_equal 'user@host:/path/to/repo', build_generator(:shoulda, {:git_remote => "user@host:/path/to/repo"}).git_remote
-    assert_equal 'git@github.com:johndoe/the-perfect-gem.git', build_generator.git_remote 
+
+  should 'have the correct git-remote' do
+    assert_equal 'user@host:/path/to/repo', build_generator(:shoulda, git_remote: 'user@host:/path/to/repo').git_remote
+    assert_equal 'git@github.com:johndoe/the-perfect-gem.git', build_generator.git_remote
   end
 
-  should "extract project name from absolut path" do
-    assert_equal "my-project", build_generator(:shoulda, {:project_name => "/tmp/my-project"}).project_name
+  should 'extract project name from absolut path' do
+    assert_equal 'my-project', build_generator(:shoulda, project_name: '/tmp/my-project').project_name
   end
 
-  should "extract project name from relative path" do
-    assert_equal "my-project", build_generator(:shoulda, {:project_name => "../my-project"}).project_name
+  should 'extract project name from relative path' do
+    assert_equal 'my-project', build_generator(:shoulda, project_name: '../my-project').project_name
   end
 
-  should "extract project name from direct path" do
-    assert_equal "my-project", build_generator(:shoulda, {:project_name => "my-project"}).project_name
+  should 'extract project name from direct path' do
+    assert_equal 'my-project', build_generator(:shoulda, project_name: 'my-project').project_name
   end
 
   def self.should_have_generator_attribute(attribute, value)
@@ -54,7 +53,7 @@ class TestGenerator < Test::Unit::TestCase
     end
   end
 
-  context "shoulda" do
+  context 'shoulda' do
     setup { @framework = :shoulda }
     should_have_generator_attribute :test_task, 'test'
     should_have_generator_attribute :test_dir, 'test'
@@ -66,7 +65,7 @@ class TestGenerator < Test::Unit::TestCase
     should_have_generator_attribute :test_helper_filename, 'helper.rb'
   end
 
-  context "testunit" do
+  context 'testunit' do
     setup { @framework = :testunit }
     should_have_generator_attribute :test_task, 'test'
     should_have_generator_attribute :test_dir, 'test'
@@ -78,7 +77,7 @@ class TestGenerator < Test::Unit::TestCase
     should_have_generator_attribute :test_helper_filename, 'helper.rb'
   end
 
-  context "minitest" do
+  context 'minitest' do
     setup { @framework = :minitest }
     should_have_generator_attribute :test_task, 'test'
     should_have_generator_attribute :test_dir, 'test'
@@ -90,7 +89,7 @@ class TestGenerator < Test::Unit::TestCase
     should_have_generator_attribute :test_helper_filename, 'helper.rb'
   end
 
-  context "bacon" do
+  context 'bacon' do
     setup { @framework = :bacon }
     should_have_generator_attribute :test_task, 'spec'
     should_have_generator_attribute :test_dir, 'spec'
@@ -102,7 +101,7 @@ class TestGenerator < Test::Unit::TestCase
     should_have_generator_attribute :test_helper_filename, 'spec_helper.rb'
   end
 
-  context "rspec" do
+  context 'rspec' do
     setup { @framework = :rspec }
     should_have_generator_attribute :test_task, 'spec'
     should_have_generator_attribute :test_dir, 'spec'
@@ -114,7 +113,7 @@ class TestGenerator < Test::Unit::TestCase
     should_have_generator_attribute :test_helper_filename, 'spec_helper.rb'
   end
 
-  context "micronaut" do
+  context 'micronaut' do
     setup { @framework = :micronaut }
     should_have_generator_attribute :test_task, 'examples'
     should_have_generator_attribute :test_dir, 'examples'
@@ -125,8 +124,8 @@ class TestGenerator < Test::Unit::TestCase
     should_have_generator_attribute :test_filename, 'the-perfect-gem_example.rb'
     should_have_generator_attribute :test_helper_filename, 'example_helper.rb'
   end
-  
-  context "testspec" do
+
+  context 'testspec' do
     setup { @framework = :testspec }
     should_have_generator_attribute :test_task, 'test'
     should_have_generator_attribute :test_dir, 'test'
@@ -137,5 +136,4 @@ class TestGenerator < Test::Unit::TestCase
     should_have_generator_attribute :test_filename, 'the-perfect-gem_test.rb'
     should_have_generator_attribute :test_helper_filename, 'test_helper.rb'
   end
-  
 end