millisami-thor 0.14.6

data/spec/actions_spec.rb

@@ -0,0 +1,322 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe Thor::Actions do
+  def runner(options={})
+    @runner ||= MyCounter.new([1], options, { :destination_root => destination_root })
+  end
+
+  def action(*args, &block)
+    capture(:stdout){ runner.send(*args, &block) }
+  end
+
+  def file
+    File.join(destination_root, "foo")
+  end
+
+  describe "on include" do
+    it "adds runtime options to the base class" do
+      MyCounter.class_options.keys.should include(:pretend)
+      MyCounter.class_options.keys.should include(:force)
+      MyCounter.class_options.keys.should include(:quiet)
+      MyCounter.class_options.keys.should include(:skip)
+    end
+  end
+
+  describe "#initialize" do
+    it "has default behavior invoke" do
+      runner.behavior.should == :invoke
+    end
+
+    it "can have behavior revoke" do
+      MyCounter.new([1], {}, :behavior => :revoke).behavior.should == :revoke
+    end
+
+    it "when behavior is set to force, overwrite options" do
+      runner = MyCounter.new([1], { :force => false, :skip => true }, :behavior => :force)
+      runner.behavior.should == :invoke
+      runner.options.force.should be_true
+      runner.options.skip.should_not be_true
+    end
+
+    it "when behavior is set to skip, overwrite options" do
+      runner = MyCounter.new([1], ["--force"], :behavior => :skip)
+      runner.behavior.should == :invoke
+      runner.options.force.should_not be_true
+      runner.options.skip.should be_true
+    end
+  end
+
+  describe "accessors" do
+    describe "#destination_root=" do
+      it "gets the current directory and expands the path to set the root" do
+        base = MyCounter.new([1])
+        base.destination_root = "here"
+        base.destination_root.should == File.expand_path(File.join(File.dirname(__FILE__), "..", "here"))
+      end
+
+      it "does not use the current directory if one is given" do
+        root = File.expand_path("/")
+        base = MyCounter.new([1])
+        base.destination_root = root
+        base.destination_root.should == root
+      end
+
+      it "uses the current directory if none is given" do
+        base = MyCounter.new([1])
+        base.destination_root.should == File.expand_path(File.join(File.dirname(__FILE__), ".."))
+      end
+    end
+
+    describe "#relative_to_original_destination_root" do
+      it "returns the path relative to the absolute root" do
+        runner.relative_to_original_destination_root(file).should == "foo"
+      end
+
+      it "does not remove dot if required" do
+        runner.relative_to_original_destination_root(file, false).should == "./foo"
+      end
+
+      it "always use the absolute root" do
+        runner.inside("foo") do
+          runner.relative_to_original_destination_root(file).should == "foo"
+        end
+      end
+
+      describe "#source_paths_for_search" do
+        it "add source_root to source_paths_for_search" do
+          MyCounter.source_paths_for_search.should include(File.expand_path("fixtures", File.dirname(__FILE__)))
+        end
+
+        it "keeps only current source root in source paths" do
+          ClearCounter.source_paths_for_search.should include(File.expand_path("fixtures/bundle", File.dirname(__FILE__)))
+          ClearCounter.source_paths_for_search.should_not include(File.expand_path("fixtures", File.dirname(__FILE__)))
+        end
+
+        it "customized source paths should be before source roots" do
+          ClearCounter.source_paths_for_search[0].should == File.expand_path("fixtures/doc", File.dirname(__FILE__))
+          ClearCounter.source_paths_for_search[1].should == File.expand_path("fixtures/bundle", File.dirname(__FILE__))
+        end
+
+        it "keeps inherited source paths at the end" do
+          ClearCounter.source_paths_for_search.last.should == File.expand_path("fixtures/broken", File.dirname(__FILE__))
+        end
+      end
+    end
+
+    describe "#find_in_source_paths" do
+      it "raises an error if source path is empty" do
+        lambda {
+          A.new.find_in_source_paths("foo")
+        }.should raise_error(Thor::Error, /Currently you have no source paths/)
+      end
+
+      it "finds a template inside the source path" do
+        runner.find_in_source_paths("doc").should == File.expand_path("doc", source_root)
+        lambda { runner.find_in_source_paths("README") }.should raise_error
+
+        new_path = File.join(source_root, "doc")
+        runner.instance_variable_set(:@source_paths, nil)
+        runner.source_paths.unshift(new_path)
+        runner.find_in_source_paths("README").should == File.expand_path("README", new_path)
+      end
+    end
+  end
+
+  describe "#inside" do
+    it "executes the block inside the given folder" do
+      runner.inside("foo") do
+        Dir.pwd.should == file
+      end
+    end
+
+    it "changes the base root" do
+      runner.inside("foo") do
+        runner.destination_root.should == file
+      end
+    end
+
+    it "creates the directory if it does not exist" do
+      runner.inside("foo") do
+        File.exists?(file).should be_true
+      end
+    end
+
+    describe "when pretending" do
+      it "no directories should be created" do
+        runner.inside("bar", :pretend => true) {}
+        File.exists?("bar").should be_false
+      end
+    end
+
+    describe "when verbose" do
+      it "logs status" do
+        capture(:stdout) do
+          runner.inside("foo", :verbose => true) {}
+        end.should =~ /inside  foo/
+      end
+
+      it "uses padding in next status" do
+        capture(:stdout) do
+          runner.inside("foo", :verbose => true) do
+            runner.say_status :cool, :padding
+          end
+        end.should =~ /cool    padding/
+      end
+
+      it "removes padding after block" do
+        capture(:stdout) do
+          runner.inside("foo", :verbose => true) {}
+          runner.say_status :no, :padding
+        end.should =~ /no  padding/
+      end
+    end
+  end
+
+  describe "#in_root" do
+    it "executes the block in the root folder" do
+      runner.inside("foo") do
+        runner.in_root { Dir.pwd.should == destination_root }
+      end
+    end
+
+    it "changes the base root" do
+      runner.inside("foo") do
+        runner.in_root { runner.destination_root.should == destination_root }
+      end
+    end
+
+    it "returns to the previous state" do
+      runner.inside("foo") do
+        runner.in_root { }
+        runner.destination_root.should == file
+      end
+    end
+  end
+
+  describe "#apply" do
+    before(:each) do
+      @template = <<-TEMPLATE
+        @foo = "FOO"
+        say_status :cool, :padding
+      TEMPLATE
+      @template.stub(:read).and_return(@template)
+
+      @file = '/'
+      runner.stub(:open).and_return(@template)
+    end
+
+    it "accepts a URL as the path" do
+      @file = "http://gist.github.com/103208.txt"
+      runner.should_receive(:open).with(@file, "Accept" => "application/x-thor-template").and_return(@template)
+      action(:apply, @file)
+    end
+
+    it "accepts a secure URL as the path" do
+      @file = "https://gist.github.com/103208.txt"
+      runner.should_receive(:open).with(@file, "Accept" => "application/x-thor-template").and_return(@template)
+      action(:apply, @file)
+    end
+
+    it "accepts a local file path with spaces" do
+      @file = File.expand_path("fixtures/path with spaces", File.dirname(__FILE__))
+      runner.should_receive(:open).with(@file).and_return(@template)
+      action(:apply, @file)
+    end
+
+    it "opens a file and executes its content in the instance binding" do
+      action :apply, @file
+      runner.instance_variable_get("@foo").should == "FOO"
+    end
+
+    it "applies padding to the content inside the file" do
+      action(:apply, @file).should =~ /cool    padding/
+    end
+
+    it "logs its status" do
+      action(:apply, @file).should =~ /       apply  #{@file}\n/
+    end
+
+    it "does not log status" do
+      content = action(:apply, @file, :verbose => false)
+      content.should =~ /cool  padding/
+      content.should_not =~ /apply http/
+    end
+  end
+
+  describe "#run_cmd" do
+    before(:each) do
+      runner.should_receive(:system).with("ls")
+    end
+
+    it "executes the command given" do
+      action :run_cmd, "ls"
+    end
+
+    it "logs status" do
+      action(:run_cmd, "ls").should == "         run_cmd  ls from \".\"\n"
+    end
+
+    it "does not log status if required" do
+      action(:run_cmd, "ls", :verbose => false).should be_empty
+    end
+
+    it "accepts a color as status" do
+      runner.shell.should_receive(:say_status).with(:run_cmd, 'ls from "."', :yellow)
+      action :run_cmd, "ls", :verbose => :yellow
+    end
+  end
+
+  describe "#run_ruby_script" do
+    before(:each) do
+      Thor::Util.stub!(:ruby_command).and_return("/opt/jruby")
+      runner.should_receive(:system).with("/opt/jruby script.rb")
+    end
+
+    it "executes the ruby script" do
+      action :run_ruby_script, "script.rb"
+    end
+
+    it "logs status" do
+      action(:run_ruby_script, "script.rb").should == "         run_cmd  jruby script.rb from \".\"\n"
+    end
+
+    it "does not log status if required" do
+      action(:run_ruby_script, "script.rb", :verbose => false).should be_empty
+    end
+  end
+
+  describe "#thor" do
+    it "executes the thor command" do
+      runner.should_receive(:system).with("thor list")
+      action :thor, :list, :verbose => true
+    end
+
+    it "converts extra arguments to command arguments" do
+      runner.should_receive(:system).with("thor list foo bar")
+      action :thor, :list, "foo", "bar"
+    end
+
+    it "converts options hash to switches" do
+      runner.should_receive(:system).with("thor list foo bar --foo")
+      action :thor, :list, "foo", "bar", :foo => true
+
+      runner.should_receive(:system).with("thor list --foo 1 2 3")
+      action :thor, :list, :foo => [1,2,3]
+    end
+
+    it "logs status" do
+      runner.should_receive(:system).with("thor list")
+      action(:thor, :list).should == "         run_cmd  thor list from \".\"\n"
+    end
+
+    it "does not log status if required" do
+      runner.should_receive(:system).with("thor list --foo 1 2 3")
+      action(:thor, :list, :foo => [1,2,3], :verbose => false).should be_empty
+    end
+
+    it "captures the output when :capture is given" do
+      runner.should_receive(:`).with("thor foo bar")
+      action(:thor, "foo", "bar", :capture => true)
+    end
+  end
+end

data/spec/base_spec.rb

@@ -0,0 +1,274 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'thor/base'
+
+class Amazing
+  desc "hello", "say hello"
+  def hello
+    puts "Hello"
+  end
+end
+
+describe Thor::Base do
+  describe "#initialize" do
+    it "sets arguments array" do
+      base = MyCounter.new [1, 2]
+      base.first.should == 1
+      base.second.should == 2
+    end
+
+    it "sets arguments default values" do
+      base = MyCounter.new [1]
+      base.second.should == 2
+    end
+
+    it "sets options default values" do
+      base = MyCounter.new [1, 2]
+      base.options[:third].should == 3
+    end
+
+    it "allows options to be given as symbols or strings" do
+      base = MyCounter.new [1, 2], :third => 4
+      base.options[:third].should == 4
+
+      base = MyCounter.new [1, 2], "third" => 4
+      base.options[:third].should == 4
+    end
+
+    it "creates options with indifferent access" do
+      base = MyCounter.new [1, 2], :third => 3
+      base.options['third'].should == 3
+    end
+
+    it "creates options with magic predicates" do
+      base = MyCounter.new [1, 2], :third => 3
+      base.options.third.should == 3
+    end
+  end
+
+  describe "#no_tasks" do
+    it "avoids methods being added as tasks" do
+      MyScript.tasks.keys.should include("animal")
+      MyScript.tasks.keys.should_not include("this_is_not_a_task")
+    end
+  end
+
+  describe "#argument" do
+    it "sets a value as required and creates an accessor for it" do
+      MyCounter.start(["1", "2", "--third", "3"])[0].should == 1
+      Scripts::MyScript.start(["zoo", "my_special_param", "--param=normal_param"]).should == "my_special_param"
+    end
+
+    it "does not set a value in the options hash" do
+      BrokenCounter.start(["1", "2", "--third", "3"])[0].should be_nil
+    end
+  end
+
+  describe "#arguments" do
+    it "returns the arguments for the class" do
+      MyCounter.arguments.should have(2).items
+    end
+  end
+
+  describe "#class_option" do
+    it "sets options class wise" do
+      MyCounter.start(["1", "2", "--third", "3"])[2].should == 3
+    end
+
+    it "does not create an accessor for it" do
+      BrokenCounter.start(["1", "2", "--third", "3"])[3].should be_false
+    end
+  end
+
+  describe "#class_options" do
+    it "sets default options overwriting superclass definitions" do
+      options = Scripts::MyScript.class_options
+      options[:force].should_not be_required
+    end
+  end
+
+  describe "#remove_argument" do
+    it "removes previous defined arguments from class" do
+      ClearCounter.arguments.should be_empty
+    end
+
+    it "undefine accessors if required" do
+      ClearCounter.new.should_not respond_to(:first)
+      ClearCounter.new.should_not respond_to(:second)
+    end
+  end
+
+  describe "#remove_class_option" do
+    it "removes previous defined class option" do
+      ClearCounter.class_options[:third].should be_nil
+    end
+  end
+
+  describe "#class_options_help" do
+    before(:each) do
+      @content = capture(:stdout) { MyCounter.help(Thor::Base.shell.new) }
+    end
+
+    it "shows options description" do
+      @content.should =~ /# The third argument/
+    end
+
+    it "shows usage with banner content" do
+      @content.should =~ /\[\-\-third=THREE\]/
+    end
+
+    it "shows default values below description" do
+      @content.should =~ /# Default: 3/
+    end
+
+    it "shows options in different groups" do
+      @content.should =~ /Options\:/
+      @content.should =~ /Runtime options\:/
+      @content.should =~ /\-p, \[\-\-pretend\]/
+    end
+
+    it "use padding in options that does not have aliases" do
+      @content.should =~ /^  -t, \[--third/
+      @content.should =~ /^      \[--fourth/
+    end
+
+    it "allows extra options to be given" do
+      hash = { "Foo" => B.class_options.values }
+
+      content = capture(:stdout) { MyCounter.send(:class_options_help, Thor::Base.shell.new, hash) }
+      content.should =~ /Foo options\:/
+      content.should =~ /--last-name=LAST_NAME/
+    end
+  end
+
+  describe "#namespace" do
+    it "returns the default class namespace" do
+      Scripts::MyScript.namespace.should == "scripts:my_script"
+    end
+
+    it "sets a namespace to the class" do
+      Scripts::MyDefaults.namespace.should == "default"
+    end
+  end
+
+  describe "#group" do
+    it "sets a group" do
+      MyScript.group.should == "script"
+    end
+
+    it "inherits the group from parent" do
+      MyChildScript.group.should == "script"
+    end
+
+    it "defaults to standard if no group is given" do
+      Amazing.group.should == "standard"
+    end
+  end
+
+  describe "#subclasses" do
+    it "tracks its subclasses in an Array" do
+      Thor::Base.subclasses.should include(MyScript)
+      Thor::Base.subclasses.should include(MyChildScript)
+      Thor::Base.subclasses.should include(Scripts::MyScript)
+    end
+  end
+
+  describe "#subclass_files" do
+    it "returns tracked subclasses, grouped by the files they come from" do
+      thorfile = File.join(File.dirname(__FILE__), "fixtures", "script.thor")
+      Thor::Base.subclass_files[File.expand_path(thorfile)].should == [
+        MyScript, MyScript::AnotherScript, MyChildScript, Barn,
+        Scripts::MyScript, Scripts::MyDefaults, Scripts::ChildDefault
+      ]
+    end
+
+    it "tracks a single subclass across multiple files" do
+      thorfile = File.join(File.dirname(__FILE__), "fixtures", "task.thor")
+      Thor::Base.subclass_files[File.expand_path(thorfile)].should include(Amazing)
+      Thor::Base.subclass_files[File.expand_path(__FILE__)].should include(Amazing)
+    end
+  end
+
+  describe "#tasks" do
+    it "returns a list with all tasks defined in this class" do
+      MyChildScript.new.should respond_to("animal")
+      MyChildScript.tasks.keys.should include("animal")
+    end
+
+    it "raises an error if a task with reserved word is defined" do
+      lambda {
+        klass = Class.new(Thor::Group)
+        klass.class_eval "def shell; end"
+      }.should raise_error(RuntimeError, /"shell" is a Thor reserved word and cannot be defined as task/)
+    end
+  end
+
+  describe "#all_tasks" do
+    it "returns a list with all tasks defined in this class plus superclasses" do
+      MyChildScript.new.should respond_to("foo")
+      MyChildScript.all_tasks.keys.should include("foo")
+    end
+  end
+
+  describe "#remove_task" do
+    it "removes the task from its tasks hash" do
+      MyChildScript.tasks.keys.should_not include("bar")
+      MyChildScript.tasks.keys.should_not include("boom")
+    end
+
+    it "undefines the method if desired" do
+      MyChildScript.new.should_not respond_to("boom")
+    end
+  end
+
+  describe "#from_superclass" do
+    it "does not send a method to the superclass if the superclass does not respond to it" do
+      MyCounter.get_from_super.should == 13
+    end
+  end
+
+  describe "#start" do
+    it "raises an error instead of rescueing if THOR_DEBUG=1 is given" do
+      begin
+        ENV["THOR_DEBUG"] = 1
+        lambda {
+          MyScript.start ["what", "--debug"]
+        }.should raise_error(Thor::UndefinedTaskError, 'Could not find task "what" in "my_script" namespace.')
+      rescue
+        ENV["THOR_DEBUG"] = nil
+      end
+    end
+
+    it "does not steal args" do
+      args = ["foo", "bar", "--force", "true"]
+      MyScript.start(args)
+      args.should == ["foo", "bar", "--force", "true"]
+    end
+
+    it "checks unknown options" do
+      capture(:stderr) {
+        MyScript.start(["foo", "bar", "--force", "true", "--unknown", "baz"])
+      }.strip.should == "Unknown switches '--unknown'"
+    end
+
+    it "checks unknown options except specified" do
+      capture(:stderr) {
+        MyScript.start(["with_optional", "NAME", "--omg", "--invalid"]).should == ["NAME", {}, ["--omg", "--invalid"]]
+      }.strip.should be_empty
+    end
+  end
+
+  describe "attr_*" do
+    it "should not add attr_reader as a task" do
+      capture(:stderr){ MyScript.start(["another_attribute"]) }.should =~ /Could not find/
+    end
+
+    it "should not add attr_writer as a task" do
+      capture(:stderr){ MyScript.start(["another_attribute=", "foo"]) }.should =~ /Could not find/
+    end
+
+    it "should not add attr_accessor as a task" do
+      capture(:stderr){ MyScript.start(["some_attribute"]) }.should =~ /Could not find/
+      capture(:stderr){ MyScript.start(["some_attribute=", "foo"]) }.should =~ /Could not find/
+    end
+  end
+end

data/spec/core_ext/hash_with_indifferent_access_spec.rb

@@ -0,0 +1,43 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require 'thor/core_ext/hash_with_indifferent_access'
+
+describe Thor::CoreExt::HashWithIndifferentAccess do
+  before(:each) do
+    @hash = Thor::CoreExt::HashWithIndifferentAccess.new :foo => 'bar', 'baz' => 'bee', :force => true
+  end
+
+  it "has values accessible by either strings or symbols" do
+    @hash['foo'].should == 'bar'
+    @hash[:foo].should  == 'bar'
+
+    @hash.values_at(:foo, :baz).should == ['bar', 'bee']
+    @hash.delete(:foo).should == 'bar'
+  end
+
+  it "handles magic boolean predicates" do
+    @hash.force?.should be_true
+    @hash.foo?.should be_true
+    @hash.nothing?.should be_false
+  end
+
+  it "handles magic comparisions" do
+    @hash.foo?('bar').should be_true
+    @hash.foo?('bee').should be_false
+  end
+
+  it "maps methods to keys" do
+    @hash.foo.should == @hash['foo']
+  end
+
+  it "merges keys independent if they are symbols or strings" do
+    @hash.merge!('force' => false, :baz => "boom")
+    @hash[:force].should == false
+    @hash[:baz].should == "boom"
+  end
+
+  it "creates a new hash by merging keys independent if they are symbols or strings" do
+    other = @hash.merge('force' => false, :baz => "boom")
+    other[:force].should == false
+    other[:baz].should == "boom"
+  end
+end