Replace mock with double

Author: sferik

.rspec

@@ -1,3 +1,2 @@
 --color
---fail-fast
 --order random

Thorfile

@@ -25,6 +25,6 @@ class Default < Thor
 
   desc "spec", "Run RSpec code examples"
   def spec
-    exec "rspec --color --format=documentation spec"
+    exec "rspec spec"
   end
 end

spec/command_spec.rb

@@ -41,9 +41,9 @@ def command(options={})
     end
 
     it "does not invoke an existing method" do
-      mock = mock()
-      expect(mock.class).to receive(:handle_no_command_error).with("to_s")
-      Thor::DynamicCommand.new('to_s').run(mock)
+      double = double()
+      expect(double.class).to receive(:handle_no_command_error).with("to_s")
+      Thor::DynamicCommand.new('to_s').run(double)
     end
   end
 
@@ -57,9 +57,9 @@ def command(options={})
 
   describe "#run" do
     it "runs a command by calling a method in the given instance" do
-      mock = mock()
-      expect(mock).to receive(:can_has).and_return {|*args| args }
-      expect(command.run(mock, [1, 2, 3])).to eq([1, 2, 3])
+      double = double()
+      expect(double).to receive(:can_has).and_return {|*args| args }
+      expect(command.run(double, [1, 2, 3])).to eq([1, 2, 3])
     end
 
     it "raises an error if the method to be invoked is private" do

spec/thor_spec.rb

@@ -136,9 +136,9 @@ def boring(*args)
         stop_on_unknown_option! :foo, :bar
       end
       it "affects all specified commands" do
-        expect(klass.stop_on_unknown_option?(mock :name => "foo")).to be_true
-        expect(klass.stop_on_unknown_option?(mock :name => "bar")).to be_true
-        expect(klass.stop_on_unknown_option?(mock :name => "baz")).to be_false
+        expect(klass.stop_on_unknown_option?(double(:name => "foo"))).to be_true
+        expect(klass.stop_on_unknown_option?(double(:name => "bar"))).to be_true
+        expect(klass.stop_on_unknown_option?(double(:name => "baz"))).to be_false
       end
     end
 
@@ -148,9 +148,9 @@ def boring(*args)
         stop_on_unknown_option! :bar
       end
       it "affects all specified commands" do
-        expect(klass.stop_on_unknown_option?(mock :name => "foo")).to be_true
-        expect(klass.stop_on_unknown_option?(mock :name => "bar")).to be_true
-        expect(klass.stop_on_unknown_option?(mock :name => "baz")).to be_false
+        expect(klass.stop_on_unknown_option?(double(:name => "foo"))).to be_true
+        expect(klass.stop_on_unknown_option?(double(:name => "bar"))).to be_true
+        expect(klass.stop_on_unknown_option?(double(:name => "baz"))).to be_false
       end
     end