1 files changed, 44 insertions, 36 deletions

diff --git a/Library/Homebrew/test/test_software_spec.rb b/Library/Homebrew/test/test_software_spec.rb
index eb828f95c..71eb3762f 100644
--- a/Library/Homebrew/test/test_software_spec.rb
+++ b/Library/Homebrew/test/test_software_spec.rb
@@ -1,5 +1,5 @@
-require 'testing_env'
-require 'software_spec'
+require "testing_env"
+require "software_spec"
 
 class SoftwareSpecTests < Homebrew::TestCase
   def setup
@@ -7,54 +7,62 @@ class SoftwareSpecTests < Homebrew::TestCase
   end
 
   def test_resource
-    @spec.resource('foo') { url 'foo-1.0' }
+    @spec.resource("foo") { url "foo-1.0" }
     assert @spec.resource_defined?("foo")
   end
 
   def test_raises_when_duplicate_resources_are_defined
-    @spec.resource('foo') { url 'foo-1.0' }
+    @spec.resource("foo") { url "foo-1.0" }
     assert_raises(DuplicateResourceError) do
-      @spec.resource('foo') { url 'foo-1.0' }
+      @spec.resource("foo") { url "foo-1.0" }
     end
   end
 
   def test_raises_when_accessing_missing_resources
     @spec.owner = Class.new do
-      def name; "test"; end
-      def full_name; "test"; end
-      def tap; "Homebrew/homebrew"; end
+      def name
+        "test"
+      end
+
+      def full_name
+        "test"
+      end
+
+      def tap
+        "Homebrew/homebrew"
+      end
     end.new
-    assert_raises(ResourceMissingError) { @spec.resource('foo') }
+    assert_raises(ResourceMissingError) { @spec.resource("foo") }
   end
 
   def test_set_owner
-    owner = stub :name => 'some_name',
-                 :full_name => 'some_name',
+    owner = stub :name => "some_name",
+                 :full_name => "some_name",
                  :tap => "Homebrew/homebrew"
     @spec.owner = owner
     assert_equal owner, @spec.owner
   end
 
   def test_resource_owner
-    @spec.resource('foo') { url 'foo-1.0' }
-    @spec.owner = stub :name => 'some_name',
-                       :full_name => 'some_name',
+    @spec.resource("foo") { url "foo-1.0" }
+    @spec.owner = stub :name => "some_name",
+                       :full_name => "some_name",
                        :tap => "Homebrew/homebrew"
-    assert_equal 'some_name', @spec.name
+    assert_equal "some_name", @spec.name
     @spec.resources.each_value { |r| assert_equal @spec, r.owner }
   end
 
   def test_resource_without_version_receives_owners_version
-    @spec.url('foo-42')
-    @spec.resource('bar') { url 'bar' }
-    @spec.owner = stub :name => 'some_name',
-                       :full_name => 'some_name',
+    @spec.url("foo-42")
+    @spec.resource("bar") { url "bar" }
+    @spec.owner = stub :name => "some_name",
+                       :full_name => "some_name",
                        :tap => "Homebrew/homebrew"
-    assert_version_equal '42', @spec.resource('bar').version
+    assert_version_equal "42", @spec.resource("bar").version
   end
 
   def test_option
-    @spec.option('foo')
+    @spec.option("foo")
     assert @spec.option_defined?("foo")
   end
 
@@ -83,14 +91,14 @@ class SoftwareSpecTests < Homebrew::TestCase
   end
 
   def test_deprecated_option
-    @spec.deprecated_option('foo' => 'bar')
+    @spec.deprecated_option("foo" => "bar")
     refute_empty @spec.deprecated_options
     assert_equal "foo", @spec.deprecated_options.first.old
     assert_equal "bar", @spec.deprecated_options.first.current
   end
 
   def test_deprecated_options
-    @spec.deprecated_option(['foo1', 'foo2'] => 'bar1', 'foo3' => ['bar2', 'bar3'])
+    @spec.deprecated_option(["foo1", "foo2"] => "bar1", "foo3" => ["bar2", "bar3"])
     assert_includes @spec.deprecated_options, DeprecatedOption.new("foo1", "bar1")
     assert_includes @spec.deprecated_options, DeprecatedOption.new("foo2", "bar1")
     assert_includes @spec.deprecated_options, DeprecatedOption.new("foo3", "bar2")
@@ -102,20 +110,20 @@ class SoftwareSpecTests < Homebrew::TestCase
   end
 
   def test_depends_on
-    @spec.depends_on('foo')
-    assert_equal 'foo', @spec.deps.first.name
+    @spec.depends_on("foo")
+    assert_equal "foo", @spec.deps.first.name
   end
 
   def test_dependency_option_integration
-    @spec.depends_on 'foo' => :optional
-    @spec.depends_on 'bar' => :recommended
+    @spec.depends_on "foo" => :optional
+    @spec.depends_on "bar" => :recommended
     assert @spec.option_defined?("with-foo")
     assert @spec.option_defined?("without-bar")
   end
 
   def test_explicit_options_override_default_dep_option_description
-    @spec.option('with-foo', 'blah')
-    @spec.depends_on('foo' => :optional)
+    @spec.option("with-foo", "blah")
+    @spec.depends_on("foo" => :optional)
     assert_equal "blah", @spec.options.first.description
   end
 
@@ -132,7 +140,7 @@ class HeadSoftwareSpecTests < Homebrew::TestCase
   end
 
   def test_version
-    assert_version_equal 'HEAD', @spec.version
+    assert_version_equal "HEAD", @spec.version
   end
 
   def test_verify_download_integrity
@@ -147,10 +155,10 @@ class BottleSpecificationTests < Homebrew::TestCase
 
   def test_checksum_setters
     checksums = {
-      :snow_leopard_32 => 'deadbeef'*5,
-      :snow_leopard    => 'faceb00c'*5,
-      :lion            => 'baadf00d'*5,
-      :mountain_lion   => '8badf00d'*5,
+      :snow_leopard_32 => "deadbeef"*5,
+      :snow_leopard    => "faceb00c"*5,
+      :lion            => "baadf00d"*5,
+      :mountain_lion   => "8badf00d"*5
     }
 
     checksums.each_pair do |cat, sha1|
@@ -158,14 +166,14 @@ class BottleSpecificationTests < Homebrew::TestCase
     end
 
     checksums.each_pair do |cat, sha1|
-      checksum, _ = @spec.checksum_for(cat)
+      checksum, = @spec.checksum_for(cat)
       assert_equal Checksum.new(:sha1, sha1), checksum
     end
   end
 
   def test_other_setters
     double = Object.new
-    %w{root_url prefix cellar revision}.each do |method|
+    %w[root_url prefix cellar revision].each do |method|
       @spec.send(method, double)
       assert_equal double, @spec.send(method)
     end