Replace be_true with be true

sferik · sferik · commit fd0abd1fcde8 · 2013-11-09T07:53:18.000-05:00
diff --git a/spec/actions/create_file_spec.rb b/spec/actions/create_file_spec.rb
@@ -30,7 +30,7 @@ def silence!
     it "creates a file" do
       create_file("doc/config.rb")
       invoke!
-      expect(File.exists?(File.join(destination_root, "doc/config.rb"))).to be_true
+      expect(File.exists?(File.join(destination_root, "doc/config.rb"))).to be true
     end
 
     it "does not create a file if pretending" do
@@ -59,7 +59,7 @@ def silence!
     it "converts encoded instructions" do
       create_file("doc/%file_name%.rb.tt")
       invoke!
-      expect(File.exists?(File.join(destination_root, "doc/rdoc.rb.tt"))).to be_true
+      expect(File.exists?(File.join(destination_root, "doc/rdoc.rb.tt"))).to be true
     end
 
     describe "when file exists" do
@@ -155,7 +155,7 @@ def silence!
       create_file("doc/config.rb")
       expect(@action.exists?).to be false
       invoke!
-      expect(@action.exists?).to be_true
+      expect(@action.exists?).to be true
     end
   end
 
@@ -164,7 +164,7 @@ def silence!
       create_file("doc/config.rb")
       expect(@action.identical?).to be false
       invoke!
-      expect(@action.identical?).to be_true
+      expect(@action.identical?).to be true
     end
   end
 end
diff --git a/spec/actions/create_link_spec.rb b/spec/actions/create_link_spec.rb
@@ -36,24 +36,24 @@ def silence!
       create_link("doc/config.rb", :symbolic => true)
       invoke!
       destination_path = File.join(destination_root, "doc/config.rb")
-      expect(File.exists?(destination_path)).to be_true
-      expect(File.symlink?(destination_path)).to be_true
+      expect(File.exists?(destination_path)).to be true
+      expect(File.symlink?(destination_path)).to be true
     end
 
     it "creates a hard link for :symbolic => false" do
       create_link(@hardlink_to, :symbolic => false)
       invoke!
       destination_path = @hardlink_to
-      expect(File.exists?(destination_path)).to be_true
+      expect(File.exists?(destination_path)).to be true
       expect(File.symlink?(destination_path)).to be false
     end
 
     it "creates a symbolic link by default" do
       create_link("doc/config.rb")
       invoke!
       destination_path = File.join(destination_root, "doc/config.rb")
-      expect(File.exists?(destination_path)).to be_true
-      expect(File.symlink?(destination_path)).to be_true
+      expect(File.exists?(destination_path)).to be true
+      expect(File.symlink?(destination_path)).to be true
     end
 
     it "does not create a link if pretending" do
@@ -79,7 +79,7 @@ def silence!
       create_link("doc/config.rb")
       expect(@action.identical?).to be false
       invoke!
-      expect(@action.identical?).to be_true
+      expect(@action.identical?).to be true
     end
   end
 
diff --git a/spec/actions/directory_spec.rb b/spec/actions/directory_spec.rb
@@ -28,8 +28,8 @@ def exists_and_identical?(source_path, destination_path)
       source      = File.join(source_root, source_path, file)
       destination = File.join(destination_root, destination_path, file)
 
-      expect(File.exists?(destination)).to be_true
-      expect(FileUtils.identical?(source, destination)).to be_true
+      expect(File.exists?(destination)).to be true
+      expect(FileUtils.identical?(source, destination)).to be true
     end
   end
 
@@ -59,7 +59,7 @@ def exists_and_identical?(source_path, destination_path)
       invoke! ".", "commands", :recursive => false
 
       file = File.join(destination_root, "commands", "group.thor")
-      expect(File.exists?(file)).to be_true
+      expect(File.exists?(file)).to be true
 
       file = File.join(destination_root, "commands", "doc")
       expect(File.exists?(file)).to be false
@@ -77,7 +77,7 @@ def exists_and_identical?(source_path, destination_path)
     it "copies and evaluates files within excluding/ directory when no exclude_pattern is present" do
       invoke! "doc", "docs"
       file = File.join(destination_root, "docs", "excluding", "rdoc.rb")
-      expect(File.exists?(file)).to be_true
+      expect(File.exists?(file)).to be true
       expect(File.read(file)).to eq("BAR = BAR\n")
     end
 
@@ -91,7 +91,7 @@ def exists_and_identical?(source_path, destination_path)
     it "copies and evaluates templates" do
       invoke! "doc", "docs"
       file = File.join(destination_root, "docs", "rdoc.rb")
-      expect(File.exists?(file)).to be_true
+      expect(File.exists?(file)).to be true
       expect(File.read(file)).to eq("FOO = FOO\n")
     end
 
@@ -105,8 +105,8 @@ def exists_and_identical?(source_path, destination_path)
     it "copies directories" do
       invoke! "doc", "docs"
       file = File.join(destination_root, "docs", "components")
-      expect(File.exists?(file)).to be_true
-      expect(File.directory?(file)).to be_true
+      expect(File.exists?(file)).to be true
+      expect(File.directory?(file)).to be true
     end
 
     it "does not copy .empty_directory files" do
@@ -118,7 +118,7 @@ def exists_and_identical?(source_path, destination_path)
     it "copies directories even if they are empty" do
       invoke! "doc/components", "docs/components"
       file = File.join(destination_root, "docs", "components")
-      expect(File.exists?(file)).to be_true
+      expect(File.exists?(file)).to be true
     end
 
     it "does not copy empty directories twice" do
@@ -139,7 +139,7 @@ def exists_and_identical?(source_path, destination_path)
       invoke!("doc") do |content|
         checked ||= !!(content =~ /FOO/)
       end
-      expect(checked).to be_true
+      expect(checked).to be true
     end
 
     it "works with glob characters in the path" do
@@ -160,7 +160,7 @@ def exists_and_identical?(source_path, destination_path)
 
     it "works with glob characters in the path" do
       invoke! "app{1}"
-      expect(File.exists?(File.join(destination_root, "app{1}", "README"))).to be_true
+      expect(File.exists?(File.join(destination_root, "app{1}", "README"))).to be true
 
       revoke! "app{1}"
       expect(File.exists?(File.join(destination_root, "app{1}", "README"))).to be false
diff --git a/spec/actions/empty_directory_spec.rb b/spec/actions/empty_directory_spec.rb
@@ -54,7 +54,7 @@ def base
     it "copies the file to the specified destination" do
       empty_directory("doc")
       invoke!
-      expect(File.exists?(File.join(destination_root, "doc"))).to be_true
+      expect(File.exists?(File.join(destination_root, "doc"))).to be true
     end
 
     it "shows created status to the user" do
@@ -92,7 +92,7 @@ def base
       empty_directory("doc")
       expect(@action.exists?).to be false
       invoke!
-      expect(@action.exists?).to be_true
+      expect(@action.exists?).to be true
     end
   end
 
diff --git a/spec/actions/file_manipulation_spec.rb b/spec/actions/file_manipulation_spec.rb
@@ -13,7 +13,7 @@ def action(*args, &block)
 
   def exists_and_identical?(source, destination)
    destination = File.join(destination_root, destination)
-   expect(File.exists?(destination)).to be_true
+   expect(File.exists?(destination)).to be true
 
    source = File.join(source_root, source)
    expect(FileUtils).to be_identical(source, destination)
@@ -174,14 +174,14 @@ def file
     it "copies the template to the specified destination" do
       action :template, "doc/config.rb", "doc/configuration.rb"
       file = File.join(destination_root, "doc/configuration.rb")
-      expect(File.exists?(file)).to be_true
+      expect(File.exists?(file)).to be true
     end
 
     it "converts encoded instructions" do
       expect(runner).to receive(:file_name).and_return("rdoc")
       action :template, "doc/%file_name%.rb.tt"
       file = File.join(destination_root, "doc/rdoc.rb")
-      expect(File.exists?(file)).to be_true
+      expect(File.exists?(file)).to be true
     end
 
     it "logs status" do
@@ -199,7 +199,7 @@ def file
       action :template, "doc/config.yaml.tt"
 
       file = File.join(destination_root, "doc/config.yaml")
-      expect(File.exists?(file)).to be_true
+      expect(File.exists?(file)).to be true
     end
   end
 
@@ -226,7 +226,7 @@ def file
       it "does not remove if pretending" do
         runner(:pretend => true)
         action :remove_file, "doc/README"
-        expect(File.exists?(file)).to be_true
+        expect(File.exists?(file)).to be true
       end
 
       it "logs status" do
diff --git a/spec/actions_spec.rb b/spec/actions_spec.rb
@@ -34,15 +34,15 @@ def file
     it "when behavior is set to force, overwrite options" do
       runner = MyCounter.new([1], { :force => false, :skip => true }, :behavior => :force)
       expect(runner.behavior).to eq(:invoke)
-      expect(runner.options.force).to be_true
-      expect(runner.options.skip).not_to be_true
+      expect(runner.options.force).to be true
+      expect(runner.options.skip).not_to be true
     end
 
     it "when behavior is set to skip, overwrite options" do
       runner = MyCounter.new([1], ["--force"], :behavior => :skip)
       expect(runner.behavior).to eq(:invoke)
-      expect(runner.options.force).not_to be_true
-      expect(runner.options.skip).to be_true
+      expect(runner.options.force).not_to be true
+      expect(runner.options.skip).to be true
     end
   end
 
@@ -146,7 +146,7 @@ def file
 
     it "creates the directory if it does not exist" do
       runner.inside("foo") do
-        expect(File.exists?(file)).to be_true
+        expect(File.exists?(file)).to be true
       end
     end
 
diff --git a/spec/core_ext/hash_with_indifferent_access_spec.rb b/spec/core_ext/hash_with_indifferent_access_spec.rb
@@ -15,13 +15,13 @@
   end
 
   it "handles magic boolean predicates" do
-    expect(@hash.force?).to be_true
-    expect(@hash.foo?).to be_true
+    expect(@hash.force?).to be true
+    expect(@hash.foo?).to be true
     expect(@hash.nothing?).to be false
   end
 
   it "handles magic comparisons" do
-    expect(@hash.foo?('bar')).to be_true
+    expect(@hash.foo?('bar')).to be true
     expect(@hash.foo?('bee')).to be false
   end
 
diff --git a/spec/invocation_spec.rb b/spec/invocation_spec.rb
@@ -16,7 +16,7 @@
     end
 
     it "invokes a command with arguments" do
-      expect(A.new.invoke(:five, [5])).to be_true
+      expect(A.new.invoke(:five, [5])).to be true
       expect(A.new.invoke(:five, [7])).to be false
     end
 
diff --git a/spec/parser/options_spec.rb b/spec/parser/options_spec.rb
@@ -79,16 +79,16 @@ def remaining
     it "accepts conjoined short switches" do
       create ["--foo", "-f"] => true, ["--bar", "-b"] => true, ["--app", "-a"] => true
       opts = parse("-fba")
-      expect(opts["foo"]).to be_true
-      expect(opts["bar"]).to be_true
-      expect(opts["app"]).to be_true
+      expect(opts["foo"]).to be true
+      expect(opts["bar"]).to be true
+      expect(opts["app"]).to be true
     end
 
     it "accepts conjoined short switches with input" do
       create ["--foo", "-f"] => true, ["--bar", "-b"] => true, ["--app", "-a"] => :required
       opts = parse "-fba", "12"
-      expect(opts["foo"]).to be_true
-      expect(opts["bar"]).to be_true
+      expect(opts["foo"]).to be true
+      expect(opts["bar"]).to be true
       expect(opts["app"]).to eq("12")
     end
 
diff --git a/spec/runner_spec.rb b/spec/runner_spec.rb
@@ -61,7 +61,7 @@ def when_no_thorfiles_exist
 
     it "invokes a command from a specific Thor class" do
       ARGV.replace ["my_script:zoo"]
-      expect(Thor::Runner.start).to be_true
+      expect(Thor::Runner.start).to be true
     end
 
     it "invokes the default command from a specific Thor class if none is specified" do
diff --git a/spec/shell/basic_spec.rb b/spec/shell/basic_spec.rb
@@ -67,19 +67,19 @@ def shell
     it "asks the user and returns true if the user replies yes" do
       expect($stdout).to receive(:print).with("Should I overwrite it? ")
         expect($stdin).to receive(:gets).and_return('y')
-      expect(shell.yes?("Should I overwrite it?")).to be_true
+      expect(shell.yes?("Should I overwrite it?")).to be true
 
       expect($stdout).to receive(:print).with("Should I overwrite it? ")
         expect($stdin).to receive(:gets).and_return('n')
-      expect(shell.yes?("Should I overwrite it?")).not_to be_true
+      expect(shell.yes?("Should I overwrite it?")).not_to be true
     end
   end
 
   describe "#no?" do
     it "asks the user and returns true if the user replies no" do
       expect($stdout).to receive(:print).with("Should I overwrite it? ")
         expect($stdin).to receive(:gets).and_return('n')
-      expect(shell.no?("Should I overwrite it?")).to be_true
+      expect(shell.no?("Should I overwrite it?")).to be true
 
       expect($stdout).to receive(:print).with("Should I overwrite it? ")
         expect($stdin).to receive(:gets).and_return('Yes')
@@ -268,7 +268,7 @@ def  #456  Lanç...
     it "returns true if the user chooses default option" do
       allow($stdout).to receive(:print)
       expect($stdin).to receive(:gets).and_return('')
-      expect(shell.file_collision('foo')).to be_true
+      expect(shell.file_collision('foo')).to be true
     end
 
     it "returns false if the user chooses no" do
@@ -280,7 +280,7 @@ def  #456  Lanç...
     it "returns true if the user chooses yes" do
       allow($stdout).to receive(:print)
       expect($stdin).to receive(:gets).and_return('y')
-      expect(shell.file_collision('foo')).to be_true
+      expect(shell.file_collision('foo')).to be true
     end
 
     it "shows help usage if the user chooses help" do
@@ -305,10 +305,10 @@ def  #456  Lanç...
       expect($stdout).to receive(:print).with('Overwrite foo? (enter "h" for help) [Ynaqh] ')
       expect($stdin).to receive(:gets).and_return('a')
 
-      expect(shell.file_collision('foo')).to be_true
+      expect(shell.file_collision('foo')).to be true
 
       expect($stdout).not_to receive(:print)
-      expect(shell.file_collision('foo')).to be_true
+      expect(shell.file_collision('foo')).to be true
     end
 
     describe "when a block is given" do
diff --git a/spec/thor_spec.rb b/spec/thor_spec.rb
@@ -136,8 +136,8 @@ def boring(*args)
         stop_on_unknown_option! :foo, :bar
       end
       it "affects all specified commands" do
-        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 => "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,8 +148,8 @@ def boring(*args)
         stop_on_unknown_option! :bar
       end
       it "affects all specified commands" do
-        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 => "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
