technicalpickles-jeweler 0.8.1 → 0.9.0

data/test/test_application.rb

@@ -0,0 +1,109 @@
+require 'test_helper'
+
+class TestApplication < Test::Unit::TestCase
+  def run_application(*arguments)
+    original_stdout = $stdout
+    original_stderr = $stderr
+
+    fake_stdout = StringIO.new
+    fake_stderr = StringIO.new
+
+    $stdout = fake_stdout
+    $stderr = fake_stderr
+
+    result = nil
+    begin
+      result = Jeweler::Generator::Application.run!(*arguments)
+    ensure
+      $stdout = original_stdout
+      $stderr = original_stderr
+    end
+
+    @stdout = fake_stdout.string
+    @stderr = fake_stderr.string
+
+    result
+  end
+
+  def self.should_exit_with_code(code)
+    should "exit with code #{code}" do
+      assert_equal code, @result
+    end
+  end
+
+  context "called without any args" do
+    setup do
+      @result = run_application
+    end
+
+    should_exit_with_code 1
+
+    should 'display usage on stderr' do
+      assert_match 'Usage:', @stderr
+    end
+
+    should 'not display anything on stdout' do
+      assert_equal '', @stdout.squeeze.strip
+    end
+  end
+
+  def build_generator(name = 'zomg', options = {:testing_framework => :shoulda})
+    Jeweler::Generator.new(name, options)
+  end
+
+  context "called with -h" do
+    setup do
+      @generator = build_generator
+      stub(@generator).run
+      stub(Jeweler::Generator).new { raise "Shouldn't have made this far"}
+
+      assert_nothing_raised do
+        @result = run_application("-h")
+      end
+    end
+
+    should_exit_with_code 1
+
+    should 'display usage on stderr' do
+      assert_match 'Usage:', @stderr
+    end
+
+    should 'not display anything on stdout' do
+      assert_equal '', @stdout.squeeze.strip
+    end
+  end
+
+  context "when called with repo name" do
+    setup do
+      @options = {:testing_framework => :shoulda}
+      @generator = build_generator('zomg', @options)
+
+      stub(@generator).run
+      stub(Jeweler::Generator).new { @generator }
+    end
+
+    should 'return exit code 0' do
+      result = run_application("zomg")
+      assert_equal 0, result
+    end
+    
+    should 'create generator with repo name and no options' do
+      run_application("zomg")
+
+      assert_received Jeweler::Generator do |subject|
+        subject.new('zomg', @options)
+      end
+    end
+
+    should 'run generator' do
+      run_application("zomg")
+
+      assert_received(@generator) {|subject| subject.run }
+    end
+
+    should 'not display usage on stderr' do
+      assert_no_match /Usage:/, @stderr
+    end
+  end
+
+end

data/test/{test_gemspec.rb → test_gemspec_helper.rb}

@@ -1,7 +1,7 @@
-require File.dirname(__FILE__) + '/test_helper'
+require 'test_helper'
 
-class TestGemspec < Test::Unit::TestCase
-  context "A Jeweler::GemSpec, given a gemspec" do
+class TestGemspecHelper < Test::Unit::TestCase
+  context "given a gemspec" do
     setup do
       @spec = build_spec
       @helper = Jeweler::GemSpecHelper.new(@spec, File.dirname(__FILE__))
@@ -12,15 +12,19 @@ class TestGemspec < Test::Unit::TestCase
     end
   end
 
-  context "Jeweler::GemSpec#write" do
+  context "#write" do
     setup do
       @spec = build_spec
-      @helper = Jeweler::GemSpecHelper.new(@spec)
+      @helper = Jeweler::GemSpecHelper.new(@spec, File.dirname(__FILE__))
       FileUtils.rm_f(@helper.path)
 
       @helper.write
     end
 
+    teardown do
+      FileUtils.rm_f(@helper.path)
+    end
+
     should "create gemspec file" do
       assert File.exists?(@helper.path)
     end

data/test/test_generator.rb

@@ -0,0 +1,179 @@
+require 'test_helper'
+
+class TestGenerator < Test::Unit::TestCase
+  def build_generator(testing_framework = nil, options = {})
+    options[:testing_framework] = testing_framework
+    Jeweler::Generator.new('the-perfect-gem', options)
+  end
+
+  context "initialize" do
+    should "raise error if nil repo name given" do
+      assert_raise Jeweler::NoGitHubRepoNameGiven do
+        Jeweler::Generator.new(nil)
+      end
+    end
+
+    should "raise error if blank repo name given" do
+      assert_raise Jeweler::NoGitHubRepoNameGiven do
+        Jeweler::Generator.new("")
+      end
+    end
+
+    should "have shoulda as default framework" do
+      assert_equal :shoulda, build_generator.testing_framework
+    end
+
+    should "have repository name as default target dir" do
+      assert_equal 'the-perfect-gem', build_generator.target_dir
+    end
+
+    should "have TODO as default summary" do
+      assert_equal "TODO", build_generator.summary
+    end
+
+    should "not create repo by default" do
+      assert ! build_generator.should_create_repo
+    end
+
+    should "not use cucumber by default" do
+      assert ! build_generator.should_use_cucumber
+    end
+
+    should "raise error for invalid testing frameworks" do
+      assert_raise ArgumentError do
+        build_generator(:zomg_invalid)
+      end
+    end
+  end
+
+  context "test_or_spec" do
+    should "be test for shoulda" do
+      assert_equal 'test', build_generator(:shoulda).test_or_spec
+    end
+
+    should "be test for testunit" do
+      assert_equal 'test', build_generator(:testunit).test_or_spec
+    end
+
+    should "be test for minitest" do
+      assert_equal 'test', build_generator(:minitest).test_or_spec
+    end
+
+    should "be spec for bacon" do
+      assert_equal 'spec', build_generator(:bacon).test_or_spec
+    end
+
+    should "be spec for rspec" do
+      assert_equal 'spec', build_generator(:rspec).test_or_spec
+    end
+
+    should "be example for micronaut" do
+      assert_equal 'example', build_generator(:micronaut).test_or_spec
+    end
+  end
+
+  context "test_dir" do
+    should "be test for shoulda" do
+      assert_equal 'test', build_generator(:shoulda).test_dir
+    end
+
+    should "be test for testunit" do
+      assert_equal 'test', build_generator(:testunit).test_dir
+    end
+
+    should "be test for minitest" do
+      assert_equal 'test', build_generator(:minitest).test_dir
+    end
+
+    should "be spec for bacon" do
+      assert_equal 'spec', build_generator(:bacon).test_dir
+    end
+
+    should "be spec for rspec" do
+      assert_equal 'spec', build_generator(:rspec).test_dir
+    end
+
+    should "be examples for micronaut" do
+      assert_equal 'examples', build_generator(:micronaut).test_dir
+    end
+  end
+
+  context "default_task" do
+    should "be test for shoulda" do
+      assert_equal 'test', build_generator(:shoulda).default_task
+    end
+
+    should "be test for testunit" do
+      assert_equal 'test', build_generator(:testunit).default_task
+    end
+
+    should "be test for minitest" do
+      assert_equal 'test', build_generator(:minitest).default_task
+    end
+
+    should "be spec for bacon" do
+      assert_equal 'spec', build_generator(:bacon).default_task
+    end
+
+    should "be spec for rspec" do
+      assert_equal 'spec', build_generator(:rspec).default_task
+    end
+
+    should "be examples for micronaut" do
+      assert_equal 'examples', build_generator(:micronaut).default_task
+    end
+  end
+
+  context "feature_support_require" do
+    should "be test/unit/assertions for shoulda" do
+      assert_equal 'test/unit/assertions', build_generator(:shoulda).feature_support_require
+    end
+
+    should "be test/unit/assertions for testunit" do
+      assert_equal 'test/unit/assertions', build_generator(:testunit).feature_support_require
+    end
+
+    should "be mini/test for minitest" do
+      assert_equal 'mini/test', build_generator(:minitest).feature_support_require
+    end
+
+    should "be test/unit/assertions for bacon" do
+      assert_equal 'test/unit/assertions', build_generator(:bacon).feature_support_require
+    end
+
+    should "be spec/expectations for rspec" do
+      assert_equal 'spec/expectations', build_generator(:rspec).feature_support_require
+    end
+
+    should "be micronaut/expectations for micronaut" do
+      assert_equal 'micronaut/expectations', build_generator(:micronaut).feature_support_require
+    end
+  end
+
+  context "feature_support_extend" do
+    should "be Test::Unit::Assertions for shoulda" do
+      assert_equal 'Test::Unit::Assertions', build_generator(:shoulda).feature_support_extend
+    end
+
+    should "be Test::Unit::Assertions for testunit" do
+      assert_equal 'Test::Unit::Assertions', build_generator(:testunit).feature_support_extend
+    end
+
+    should "be Mini::Test::Assertions for minitest" do
+      assert_equal 'Mini::Test::Assertions', build_generator(:minitest).feature_support_extend
+    end
+
+    should "be Test::Unit::Assertions for bacon" do
+      assert_equal 'Test::Unit::Assertions', build_generator(:bacon).feature_support_extend
+    end
+
+    should "be nil for rspec" do
+      assert_equal nil, build_generator(:rspec).feature_support_extend
+    end
+
+    should "be Micronaut::Matchers for micronaut" do
+      assert_equal 'Micronaut::Matchers', build_generator(:micronaut).feature_support_extend
+    end
+  end
+
+end

data/test/test_helper.rb

@@ -1,14 +1,17 @@
 require 'test/unit'
 
 require 'rubygems'
-gem 'thoughtbot-shoulda'
 require 'shoulda'
-gem 'ruby-debug'
 require 'ruby-debug'
-gem 'mocha'
-require 'mocha'
+require 'rr'
+require 'output_catcher'
+require 'time'
 
-require File.dirname(__FILE__) + '/shoulda_macros/jeweler_macros'
+$LOAD_PATH.unshift(File.dirname(__FILE__) + '/../lib')
+require 'jeweler'
+
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+require 'shoulda_macros/jeweler_macros'
 
 # Use vendored gem because of limited gem availability on runcoderun
 # This is loosely based on 'vendor everything'.
@@ -17,24 +20,9 @@ Dir[File.join(File.dirname(__FILE__), '..', 'vendor', 'gems', '**')].each do |di
   $LOAD_PATH.unshift(lib) if File.directory?(lib)
 end
 
-require 'output_catcher'
-require 'time'
-
-$LOAD_PATH.unshift(File.dirname(__FILE__) + '/../lib')
-$LOAD_PATH.unshift(File.dirname(__FILE__))
-require 'jeweler'
-
-# Fake out FileList from Rake
-class FileList
-  def self.[](*args)
-    TMP_DIR.entries - ['.','..','.DS_STORE']
-  end
-end
-
-TMP_DIR = File.join(File.dirname(__FILE__), 'tmp')
-FileUtils.rm_f(TMP_DIR) # GAH, dirty hax. Somewhere isn't tearing up correctly, so do some cleanup first
-
 class Test::Unit::TestCase
+  include RR::Adapters::TestUnit unless include?(RR::Adapters::TestUnit)
+
   def catch_out(&block)
      OutputCatcher.catch_out do
        block.call
@@ -56,7 +44,7 @@ class Test::Unit::TestCase
       s.email = "josh@technicalpickles.com"
       s.homepage = "http://github.com/technicalpickles/jeweler"
       s.description = "Simple and opinionated helper for creating Rubygem projects on GitHub"
-      s.authors = ["Josh Nichols", "Dan Croak"]
+      s.authors = ["Josh Nichols"]
       s.files = FileList[*files] unless files.empty?
     end
   end

data/test/test_jeweler.rb

@@ -1,10 +1,11 @@
-require File.dirname(__FILE__) + '/test_helper'
+require 'test_helper'
 
 class TestJeweler < Test::Unit::TestCase
 
   def setup
     @now = Time.now
-    Time.stubs(:now).returns(@now)
+    stub(Time).now { @now }
+
     FileUtils.rm_rf("#{File.dirname(__FILE__)}/tmp")
   end
 
@@ -12,13 +13,13 @@ class TestJeweler < Test::Unit::TestCase
     FileUtils.rm_rf("#{File.dirname(__FILE__)}/tmp")
   end
 
-  context "A jeweler without a VERSION.yml" do
+  context "Initializing jewewler in a blank directory" do
     setup do
       FileUtils.mkdir_p(tmp_dir)
       @jeweler = Jeweler.new(build_spec, tmp_dir)
     end
 
-    should "not have VERSION.yml" do
+    should "not create a VERSION.yml" do
       assert ! File.exists?(File.join(tmp_dir, 'VERSION.yml'))
     end
   end
@@ -29,6 +30,7 @@ class TestJeweler < Test::Unit::TestCase
       FileUtils.cp_r(fixture_dir, tmp_dir)
       
       @jeweler = Jeweler.new(build_spec, tmp_dir)
+      @jeweler.output = StringIO.new
     end
 
     should_have_major_version 1
@@ -38,14 +40,14 @@ class TestJeweler < Test::Unit::TestCase
 
     context "bumping the patch version" do
       setup do
-        @output = catch_out { @jeweler.bump_patch_version }
+        @jeweler.bump_patch_version 
       end
       should_bump_version 1, 5, 3
     end
 
     context "bumping the minor version" do
       setup do
-        @output = catch_out { @jeweler.bump_minor_version }
+        @jeweler.bump_minor_version
       end
 
       should_bump_version 1, 6, 0
@@ -53,13 +55,13 @@ class TestJeweler < Test::Unit::TestCase
 
     context "bumping the major version" do
       setup do
-        @output = catch_out { @jeweler.bump_major_version}
+        @jeweler.bump_major_version
       end
 
       should_bump_version 2, 0, 0
     end
 
-    should "should find files" do
+    should "should populate gemspec's files" do
       assert ! @jeweler.gemspec.files.empty?
     end
 
@@ -90,7 +92,8 @@ class TestJeweler < Test::Unit::TestCase
 
     context "writing the gemspec" do
       setup do
-        @output = catch_out { @jeweler.write_gemspec }
+        @jeweler.write_gemspec
+        @output = @jeweler.output.string
       end
 
       should "create bar.gemspec" do

data/test/test_options.rb

@@ -0,0 +1,90 @@
+require 'test_helper'
+
+class TestOptions < Test::Unit::TestCase
+
+  def self.should_have_testing_framework(testing_framework)
+    should "use #{testing_framework} for testing" do
+      assert_equal testing_framework.to_sym, @options[:testing_framework]
+    end
+  end
+
+  def setup_options(*arguments)
+    @options = Jeweler::Generator::Options.new(arguments)
+  end
+
+  def self.for_options(*options)
+    context options.join(' ') do
+      setup { setup_options *options }
+      yield
+    end
+  end
+
+  context "default options" do
+    setup { setup_options }
+    should_have_testing_framework :shoulda
+    should 'not create repository' do
+      assert ! @options[:create_repo]
+    end
+  end
+
+  for_options '--shoulda' do
+    should_have_testing_framework :shoulda
+  end
+
+  for_options "--bacon" do
+    should_have_testing_framework :bacon
+  end
+
+  for_options "--testunit" do
+    should_have_testing_framework :testunit
+  end
+
+  for_options '--minitest' do
+    should_have_testing_framework :minitest
+  end
+
+  for_options '--rspec' do
+    should_have_testing_framework :rspec
+  end
+
+  for_options '--micronaut' do
+    should_have_testing_framework :micronaut
+  end
+
+  for_options '--cucumber' do
+    should 'enable cucumber' do
+      assert_equal true, @options[:use_cucumber]
+    end
+  end
+
+  for_options '--create-repo' do
+    should 'create repository' do
+      assert @options[:create_repo]
+    end
+  end
+
+  for_options '--summary', 'zomg so awesome' do
+    should 'have summary zomg so awesome' do
+      assert_equal 'zomg so awesome', @options[:summary]
+    end
+  end
+
+  for_options '--directory', 'foo' do
+    should 'have directory foo' do
+      assert_equal 'foo', @options[:directory]
+    end
+  end
+
+  for_options '--help' do
+    should 'show help' do
+      assert @options[:show_help]
+    end
+  end
+
+  for_options '-h' do
+    should 'show help' do
+      assert @options[:show_help]
+    end
+  end
+
+end

data/test/test_tasks.rb

@@ -1,4 +1,4 @@
-require File.dirname(__FILE__) + '/test_helper'
+require 'test_helper'
 
 require 'rake'
 class TestTasks < Test::Unit::TestCase
@@ -9,7 +9,6 @@ class TestTasks < Test::Unit::TestCase
       Task.clear
 
       @jt = Jeweler::Tasks.new do |s|
-
       end
     end
 
@@ -25,12 +24,12 @@ class TestTasks < Test::Unit::TestCase
       assert Task.task_defined?(:build)
       assert Task.task_defined?(:install)
       assert Task.task_defined?(:gemspec)
+      assert Task.task_defined?(:build)
+      assert Task.task_defined?(:install)
       assert Task.task_defined?(:'gemspec:validate')
       assert Task.task_defined?(:'gemspec:generate')
       assert Task.task_defined?(:version)
       assert Task.task_defined?(:'version:write')
-      assert Task.task_defined?(:'version:display')
-      assert Task.task_defined?(:'version:bump')
       assert Task.task_defined?(:'version:bump:major')
       assert Task.task_defined?(:'version:bump:minor')
       assert Task.task_defined?(:'version:bump:patch')

data/test/{test_version.rb → test_version_helper.rb}

@@ -1,30 +1,30 @@
-require File.dirname(__FILE__) + '/test_helper'
+require 'test_helper'
 
-class TestVersion < Test::Unit::TestCase
+class TestVersionHelper < Test::Unit::TestCase
 
   VERSION_TMP_DIR = File.dirname(__FILE__) + '/version_tmp'
 
   def self.should_have_version(major, minor, patch)
     should "have major version #{major}" do
-      assert_equal major, @version.major
+      assert_equal major, @version_helper.major
     end
 
     should "have minor version #{minor}" do
-      assert_equal minor, @version.minor
+      assert_equal minor, @version_helper.minor
     end
 
     should "have patch version #{patch}" do
-      assert_equal patch, @version.patch
+      assert_equal patch, @version_helper.patch
     end
 
     version_s = "#{major}.#{minor}.#{patch}"
     should "render string as #{version_s.inspect}" do
-      assert_equal version_s, @version.to_s
+      assert_equal version_s, @version_helper.to_s
     end
 
     version_hash = {:major => major, :minor => minor, :patch => patch}
     should "render hash as #{version_hash.inspect}" do
-      assert_equal version_hash, @version.to_hash
+      assert_equal version_hash, @version_helper.to_hash
     end
     
   end
@@ -36,23 +36,23 @@ class TestVersion < Test::Unit::TestCase
 
       build_version_yml VERSION_TMP_DIR, 3, 5, 4
 
-      @version = Jeweler::Version.new VERSION_TMP_DIR
+      @version_helper = Jeweler::VersionHelper.new VERSION_TMP_DIR
     end
 
     should_have_version 3, 5, 4
 
     context "bumping major version" do
-      setup { @version.bump_major }
+      setup { @version_helper.bump_major }
       should_have_version 4, 0, 0
     end
 
     context "bumping the minor version" do
-      setup { @version.bump_minor }
+      setup { @version_helper.bump_minor }
       should_have_version 3, 6, 0
     end
 
     context "bumping the patch version" do
-      setup { @version.bump_patch }
+      setup { @version_helper.bump_patch }
       should_have_version 3, 5, 5
     end
   end
@@ -65,14 +65,14 @@ class TestVersion < Test::Unit::TestCase
 
     should "not raise error if the VERSION.yml doesn't exist" do
       assert_nothing_raised Jeweler::VersionYmlError do
-        Jeweler::Version.new(VERSION_TMP_DIR)
+        Jeweler::VersionHelper.new(VERSION_TMP_DIR)
       end
     end
 
     context "setting an initial version" do
       setup do
-        @version = Jeweler::Version.new(VERSION_TMP_DIR)
-        @version.update_to 0, 0, 1
+        @version_helper = Jeweler::VersionHelper.new(VERSION_TMP_DIR)
+        @version_helper.update_to 0, 0, 1
       end
 
       should_have_version 0, 0, 1
@@ -82,7 +82,7 @@ class TestVersion < Test::Unit::TestCase
 
       context "outputting" do
         setup do
-          @version.write
+          @version_helper.write
         end
 
         should "create VERSION.yml" do
@@ -91,7 +91,7 @@ class TestVersion < Test::Unit::TestCase
 
         context "re-reading VERSION.yml" do
           setup do
-            @version = Jeweler::Version.new(VERSION_TMP_DIR)
+            @version_helper = Jeweler::VersionHelper.new(VERSION_TMP_DIR)
           end
 
           should_have_version 0, 0, 1