nanoc 3.7.3 → 3.7.4

data/lib/nanoc/helpers/capturing.rb

@@ -101,7 +101,7 @@ module Nanoc::Helpers
       if block_given? # Set content
         # Get args
         if args.size != 1
-          raise ArgumentError, 'expected 1 argument (the name ' +
+          raise ArgumentError, 'expected 1 argument (the name ' \
             "of the capture) but got #{args.size} instead"
         end
         name = args[0]
@@ -112,7 +112,7 @@ module Nanoc::Helpers
       else # Get content
         # Get args
         if args.size != 2
-          raise ArgumentError, 'expected 2 arguments (the item ' +
+          raise ArgumentError, 'expected 2 arguments (the item ' \
             "and the name of the capture) but got #{args.size} instead"
         end
         item = args[0]
@@ -128,7 +128,7 @@ module Nanoc::Helpers
           # item from which we use content. For this, we need to manually edit
           # the content attribute to reset it. :(
           # FIXME clean this up
-          if !@site.captures_store_compiled_items.include? item
+          unless @site.captures_store_compiled_items.include? item
             @site.captures_store_compiled_items << item
             item.forced_outdated = true
             item.reps.each do |r|

data/lib/nanoc/helpers/html_escape.rb

@@ -38,13 +38,13 @@ module Nanoc::Helpers
         buffer = eval('_erbout', block.binding)
         buffer << escaped_data
       elsif string
-        string.gsub('&', '&amp;').
-               gsub('<', '&lt;').
-               gsub('>', '&gt;').
-               gsub('"', '&quot;')
+        string.gsub('&', '&amp;')
+               .gsub('<', '&lt;')
+               .gsub('>', '&gt;')
+               .gsub('"', '&quot;')
       else
-        raise "The #html_escape or #h function needs either a " \
-          "string or a block to HTML-escape, but neither a string nor a block was given"
+        raise 'The #html_escape or #h function needs either a ' \
+          'string or a block to HTML-escape, but neither a string nor a block was given'
       end
     end
 

data/lib/nanoc/helpers/tagging.rb

@@ -63,7 +63,7 @@ module Nanoc::Helpers
     #
     # @return [String] A link for the given tag and the given base URL
     def link_for_tag(tag, base_url)
-      %[<a href="#{h base_url}#{h tag}" rel="tag">#{h tag}</a>]
+      %(<a href="#{h base_url}#{h tag}" rel="tag">#{h tag}</a>)
     end
 
   end

data/lib/nanoc/helpers/xml_sitemap.rb

@@ -64,10 +64,10 @@ module Nanoc::Helpers
           reps.reject! { |r| !select_proc[r] } if select_proc
           reps.sort_by { |r| r.name.to_s }.each do |rep|
             xml.url do
-              xml.loc         @site.config[:base_url] + rep.path
-              xml.lastmod     item[:mtime].to_iso8601_date unless item[:mtime].nil?
-              xml.changefreq  item[:changefreq] unless item[:changefreq].nil?
-              xml.priority    item[:priority] unless item[:priority].nil?
+              xml.loc @site.config[:base_url] + rep.path
+              xml.lastmod item[:mtime].to_iso8601_date unless item[:mtime].nil?
+              xml.changefreq item[:changefreq] unless item[:changefreq].nil?
+              xml.priority item[:priority] unless item[:priority].nil?
             end
           end
         end

data/lib/nanoc/tasks/clean.rake

@@ -5,7 +5,7 @@ task :clean do
   # Load site
   site = Nanoc::Site.new('.')
   if site.nil?
-    $stderr.puts 'The current working directory does not seem to be a ' +
+    $stderr.puts 'The current working directory does not seem to be a ' \
                  'valid/complete nanoc site directory; aborting.'
     exit 1
   end

data/lib/nanoc/tasks/clean.rb

@@ -14,7 +14,7 @@ module Nanoc::Tasks
       end
     end
 
-  private
+    private
 
     def filenames
       @site.items.map do |item|

data/lib/nanoc/tasks/deploy/rsync.rake

@@ -6,10 +6,10 @@ namespace :deploy do
 
   desc 'Upload the compiled site using rsync'
   task :rsync do
-    dry_run     = !!ENV['dry_run']
+    dry_run     = ENV['dry_run']
     config_name = ENV['config'] || :default
 
-    cmd = [ 'deploy', '-t', config_name ]
+    cmd = ['deploy', '-t', config_name]
     cmd << '-n' if dry_run
 
     Nanoc::CLI.run cmd

data/lib/nanoc/version.rb

@@ -3,6 +3,6 @@
 module Nanoc
 
   # The current nanoc version.
-  VERSION = '3.7.3'
+  VERSION = '3.7.4'
 
 end

data/tasks/rubocop.rake

@@ -2,6 +2,7 @@ begin
   require 'rubocop/rake_task'
 
   RuboCop::RakeTask.new(:rubocop) do |task|
+    task.options  = %w( --display-cop-names --format simple )
     task.patterns = ['lib/**/*.rb']
   end
 rescue LoadError

data/test/base/core_ext/array_spec.rb

@@ -3,8 +3,8 @@
 describe 'Array#symbolize_keys_recursively' do
 
   it 'should convert keys to symbols' do
-    array_old = [ :abc, 'xyz', { 'foo' => 'bar', :baz => :qux } ]
-    array_new = [ :abc, 'xyz', { :foo  => 'bar', :baz => :qux } ]
+    array_old = [:abc, 'xyz', { 'foo' => 'bar', :baz => :qux }]
+    array_new = [:abc, 'xyz', { :foo  => 'bar', :baz => :qux }]
     array_old.symbolize_keys_recursively.must_equal array_new
   end
 
@@ -13,8 +13,8 @@ end
 describe 'Array#stringify_keys_recursively' do
 
   it 'should convert keys to strings' do
-    array_old = [ :abc, 'xyz', { :foo  => 'bar', 'baz' => :qux } ]
-    array_new = [ :abc, 'xyz', { 'foo' => 'bar', 'baz' => :qux } ]
+    array_old = [:abc, 'xyz', { :foo  => 'bar', 'baz' => :qux }]
+    array_new = [:abc, 'xyz', { 'foo' => 'bar', 'baz' => :qux }]
     array_old.stringify_keys_recursively.must_equal array_new
   end
 
@@ -25,7 +25,7 @@ describe 'Array#freeze_recursively' do
   include Nanoc::TestHelpers
 
   it 'should prevent first-level elements from being modified' do
-    array = [ :a, [ :b, :c ], :d ]
+    array = [:a, [:b, :c], :d]
     array.freeze_recursively
 
     assert_raises_frozen_error do
@@ -34,7 +34,7 @@ describe 'Array#freeze_recursively' do
   end
 
   it 'should prevent second-level elements from being modified' do
-    array = [ :a, [ :b, :c ], :d ]
+    array = [:a, [:b, :c], :d]
     array.freeze_recursively
 
     assert_raises_frozen_error do
@@ -59,7 +59,7 @@ describe 'Array#checksum' do
 
   it 'should work' do
     expectation = 'CEUlNvu/3DUmlbtpFRiLHU8oHA0='
-    [ [ :foo, 123 ] ].checksum.must_equal expectation
+    [[:foo, 123]].checksum.must_equal expectation
   end
 
 end

data/test/base/core_ext/hash_spec.rb

@@ -88,7 +88,7 @@ describe 'Hash#checksum' do
 
   it 'should not sort keys' do
     if RUBY_VERSION =~ /\A1\.8./
-      skip "Ruby 1.8.x does not have ordered hashes"
+      skip 'Ruby 1.8.x does not have ordered hashes'
     end
 
     a = { :a => 1, :c => 2, :b => 3 }.checksum

data/test/base/test_checksum_store.rb

@@ -9,7 +9,7 @@ class Nanoc::ChecksumStoreTest < Nanoc::TestCase
     FileUtils.mkdir_p('tmp')
     pstore = PStore.new('tmp/checksums')
     pstore.transaction do
-      pstore[:data] = { [ :item, '/moo/' ] => 'zomg' }
+      pstore[:data] = { [:item, '/moo/'] => 'zomg' }
       pstore[:version] = 1
     end
 

data/test/base/test_compiler.rb

@@ -4,7 +4,7 @@ class Nanoc::CompilerTest < Nanoc::TestCase
 
   def test_compilation_rule_for
     # Mock rules
-    rules = [ mock, mock, mock ]
+    rules = [mock, mock, mock]
     rules[0].expects(:applicable_to?).returns(false)
     rules[1].expects(:applicable_to?).returns(true)
     rules[1].expects(:rep_name).returns('wrong')
@@ -27,7 +27,7 @@ class Nanoc::CompilerTest < Nanoc::TestCase
 
   def test_routing_rule_for
     # Mock rules
-    rules = [ mock, mock, mock ]
+    rules = [mock, mock, mock]
     rules[0].expects(:applicable_to?).returns(false)
     rules[1].expects(:applicable_to?).returns(true)
     rules[1].expects(:rep_name).returns('wrong')
@@ -54,14 +54,14 @@ class Nanoc::CompilerTest < Nanoc::TestCase
 
     # Create compiler
     compiler = Nanoc::Compiler.new(site)
-    compiler.rules_collection.layout_filter_mapping[/.*/] = [ :erb, { :foo => 'bar' } ]
+    compiler.rules_collection.layout_filter_mapping[/.*/] = [:erb, { :foo => 'bar' }]
 
     # Mock layout
     layout = MiniTest::Mock.new
     layout.expect(:identifier, '/some_layout/')
 
     # Check
-    assert_equal([ :erb, { :foo => 'bar' } ], compiler.rules_collection.filter_for_layout(layout))
+    assert_equal([:erb, { :foo => 'bar' }], compiler.rules_collection.filter_for_layout(layout))
   end
 
   def test_filter_for_layout_with_existant_layout_and_unknown_filter
@@ -70,14 +70,14 @@ class Nanoc::CompilerTest < Nanoc::TestCase
 
     # Create compiler
     compiler = Nanoc::Compiler.new(site)
-    compiler.rules_collection.layout_filter_mapping[/.*/] = [ :some_unknown_filter, { :foo => 'bar' } ]
+    compiler.rules_collection.layout_filter_mapping[/.*/] = [:some_unknown_filter, { :foo => 'bar' }]
 
     # Mock layout
     layout = MiniTest::Mock.new
     layout.expect(:identifier, '/some_layout/')
 
     # Check
-    assert_equal([ :some_unknown_filter, { :foo => 'bar' } ], compiler.rules_collection.filter_for_layout(layout))
+    assert_equal([:some_unknown_filter, { :foo => 'bar' }], compiler.rules_collection.filter_for_layout(layout))
   end
 
   def test_filter_for_layout_with_nonexistant_layout
@@ -86,7 +86,7 @@ class Nanoc::CompilerTest < Nanoc::TestCase
 
     # Create compiler
     compiler = Nanoc::Compiler.new(site)
-    compiler.rules_collection.layout_filter_mapping[%r{^/foo/$}] = [ :erb, { :foo => 'bar' } ]
+    compiler.rules_collection.layout_filter_mapping[%r{^/foo/$}] = [:erb, { :foo => 'bar' }]
 
     # Mock layout
     layout = MiniTest::Mock.new
@@ -102,13 +102,13 @@ class Nanoc::CompilerTest < Nanoc::TestCase
 
     # Create compiler
     compiler = Nanoc::Compiler.new(site)
-    compiler.rules_collection.layout_filter_mapping[%r{^/a/b/c/.*/$}] = [ :erb, { :char => 'd' } ]
-    compiler.rules_collection.layout_filter_mapping[%r{^/a/.*/$}]     = [ :erb, { :char => 'b' } ]
-    compiler.rules_collection.layout_filter_mapping[%r{^/a/b/.*/$}]   = [ :erb, { :char => 'c' } ] # never used!
-    compiler.rules_collection.layout_filter_mapping[%r{^/.*/$}]       = [ :erb, { :char => 'a' } ]
+    compiler.rules_collection.layout_filter_mapping[%r{^/a/b/c/.*/$}] = [:erb, { :char => 'd' }]
+    compiler.rules_collection.layout_filter_mapping[%r{^/a/.*/$}]     = [:erb, { :char => 'b' }]
+    compiler.rules_collection.layout_filter_mapping[%r{^/a/b/.*/$}]   = [:erb, { :char => 'c' }] # never used!
+    compiler.rules_collection.layout_filter_mapping[%r{^/.*/$}]       = [:erb, { :char => 'a' }]
 
     # Mock layout
-    layouts = [ mock, mock, mock, mock ]
+    layouts = [mock, mock, mock, mock]
     layouts[0].stubs(:identifier).returns('/a/b/c/d/')
     layouts[1].stubs(:identifier).returns('/a/b/c/')
     layouts[2].stubs(:identifier).returns('/a/b/')
@@ -159,12 +159,12 @@ class Nanoc::CompilerTest < Nanoc::TestCase
     site = mock
     site.stubs(:config).returns({})
     site.stubs(:items).returns([])
-    site.stubs(:layouts).returns([ layout ])
+    site.stubs(:layouts).returns([layout])
 
     # Create compiler
     compiler = Nanoc::Compiler.new(site)
     compiler.rules_collection.expects(:compilation_rule_for).times(2).with(rep).returns(rule)
-    compiler.rules_collection.layout_filter_mapping[%r{^/blah/$}] = [ :erb, {} ]
+    compiler.rules_collection.layout_filter_mapping[%r{^/blah/$}] = [:erb, {}]
     site.stubs(:compiler).returns(compiler)
 
     # Compile

data/test/base/test_compiler_dsl.rb

@@ -106,7 +106,7 @@ EOS
       # Create items
       assert Dir['content/*'].empty?
       File.open('content/robots.txt', 'w') do |io|
-        io.write "Hello I am robots"
+        io.write 'Hello I am robots'
       end
 
       # Compile
@@ -114,7 +114,7 @@ EOS
       site.compile
 
       # Check paths
-      assert_equal [ 'output/robots.txt' ], Dir['output/*']
+      assert_equal ['output/robots.txt'], Dir['output/*']
     end
   end
 
@@ -130,7 +130,7 @@ EOS
       # Create items
       assert Dir['content/*'].empty?
       File.open('content/foo', 'w') do |io|
-        io.write "Hello I am foo"
+        io.write 'Hello I am foo'
       end
 
       # Compile
@@ -138,7 +138,7 @@ EOS
       site.compile
 
       # Check paths
-      assert_equal [ 'output/foo' ], Dir['output/*']
+      assert_equal ['output/foo'], Dir['output/*']
     end
   end
 
@@ -160,7 +160,7 @@ EOS
       # Create items
       FileUtils.mkdir_p('static')
       File.open('static/foo.txt', 'w') do |io|
-        io.write "Hello I am foo"
+        io.write 'Hello I am foo'
       end
 
       # Compile
@@ -168,7 +168,7 @@ EOS
       site.compile
 
       # Check paths
-      assert_equal [ 'output/foo.txt' ], Dir['output/*']
+      assert_equal ['output/foo.txt'], Dir['output/*']
     end
   end
 
@@ -200,8 +200,8 @@ EOS
       site.compile
 
       # Check paths
-      assert_equal [ 'output/foo' ],            Dir['output/*']
-      assert_equal [ 'output/foo/index.html' ], Dir['output/foo/*']
+      assert_equal ['output/foo'],            Dir['output/*']
+      assert_equal ['output/foo/index.html'], Dir['output/foo/*']
     end
   end
 
@@ -219,11 +219,11 @@ EOS
       # Create items
       assert Dir['content/*'].empty?
       File.open('content/lame.txt', 'w') do |io|
-        io.write "Hello I am lame"
+        io.write 'Hello I am lame'
       end
 
       File.open('content/notlame.txt', 'w') do |io|
-        io.write "Hello I am not lame"
+        io.write 'Hello I am not lame'
       end
 
       # Compile
@@ -231,7 +231,7 @@ EOS
       site.compile
 
       # Check paths
-      assert_equal [ 'output/notlame.txt'], Dir['output/*']
+      assert_equal ['output/notlame.txt'], Dir['output/*']
     end
   end
 
@@ -263,8 +263,8 @@ EOS
       site.compile
 
       # Check paths
-      assert_equal [ 'output/foo' ],            Dir['output/*']
-      assert_equal [ 'output/foo/index.html' ], Dir['output/foo/*']
+      assert_equal ['output/foo'],            Dir['output/*']
+      assert_equal ['output/foo/index.html'], Dir['output/foo/*']
     end
   end
 

data/test/base/test_context.rb

@@ -7,7 +7,7 @@ class Nanoc::ContextTest < Nanoc::TestCase
     context = Nanoc::Context.new({ :foo => 'bar', :baz => 'quux' })
 
     # Ensure correct evaluation
-    assert_equal('bar', eval("@foo", context.get_binding))
+    assert_equal('bar', eval('@foo', context.get_binding))
   end
 
   def test_context_with_instance_method
@@ -15,7 +15,7 @@ class Nanoc::ContextTest < Nanoc::TestCase
     context = Nanoc::Context.new({ :foo => 'bar', :baz => 'quux' })
 
     # Ensure correct evaluation
-    assert_equal('bar', eval("foo", context.get_binding))
+    assert_equal('bar', eval('foo', context.get_binding))
   end
 
   def test_example

data/test/base/test_dependency_tracker.rb

@@ -4,7 +4,7 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
 
   def test_initialize
     # Mock items
-    items = [ mock, mock ]
+    items = [mock, mock]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -16,7 +16,7 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
 
   def test_record_dependency
     # Mock items
-    items = [ mock, mock ]
+    items = [mock, mock]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -25,12 +25,12 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.record_dependency(items[0], items[1])
 
     # Verify dependencies
-    assert_contains_exactly [ items[1] ], tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[1]], tracker.objects_causing_outdatedness_of(items[0])
   end
 
   def test_record_dependency_no_self
     # Mock items
-    items = [ mock, mock ]
+    items = [mock, mock]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -40,12 +40,12 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.record_dependency(items[0], items[1])
 
     # Verify dependencies
-    assert_contains_exactly [ items[1] ], tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[1]], tracker.objects_causing_outdatedness_of(items[0])
   end
 
   def test_record_dependency_no_doubles
     # Mock items
-    items = [ mock, mock ]
+    items = [mock, mock]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -56,12 +56,12 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.record_dependency(items[0], items[1])
 
     # Verify dependencies
-    assert_contains_exactly [ items[1] ], tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[1]], tracker.objects_causing_outdatedness_of(items[0])
   end
 
   def test_objects_causing_outdatedness_of
     # Mock items
-    items = [ mock, mock, mock ]
+    items = [mock, mock, mock]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -71,12 +71,12 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.record_dependency(items[1], items[2])
 
     # Verify dependencies
-    assert_contains_exactly [ items[1] ], tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[1]], tracker.objects_causing_outdatedness_of(items[0])
   end
 
   def test_objects_outdated_due_to
     # Mock items
-    items = [ mock, mock, mock ]
+    items = [mock, mock, mock]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -86,12 +86,12 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.record_dependency(items[1], items[2])
 
     # Verify dependencies
-    assert_contains_exactly [ items[0] ], tracker.objects_outdated_due_to(items[1])
+    assert_contains_exactly [items[0]], tracker.objects_outdated_due_to(items[1])
   end
 
   def test_start_and_stop
     # Mock items
-    items = [ mock, mock ]
+    items = [mock, mock]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -105,18 +105,18 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.stop
 
     # Verify dependencies
-    assert_contains_exactly [ items[1] ], tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[1]], tracker.objects_causing_outdatedness_of(items[0])
     assert_empty tracker.objects_causing_outdatedness_of(items[1])
   end
 
   def test_store_graph_and_load_graph_simple
     # Mock items
-    items = [ mock('0'), mock('1'), mock('2'), mock('3') ]
+    items = [mock('0'), mock('1'), mock('2'), mock('3')]
     items.each { |i| i.stubs(:type).returns(:item) }
-    items[0].stubs(:reference).returns([ :item, '/aaa/' ])
-    items[1].stubs(:reference).returns([ :item, '/bbb/' ])
-    items[2].stubs(:reference).returns([ :item, '/ccc/' ])
-    items[3].stubs(:reference).returns([ :item, '/ddd/' ])
+    items[0].stubs(:reference).returns([:item, '/aaa/'])
+    items[1].stubs(:reference).returns([:item, '/bbb/'])
+    items[2].stubs(:reference).returns([:item, '/ccc/'])
+    items[3].stubs(:reference).returns([:item, '/ddd/'])
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -137,24 +137,24 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.load_graph
 
     # Check loaded graph
-    assert_contains_exactly [ items[1] ],           tracker.objects_causing_outdatedness_of(items[0])
-    assert_contains_exactly [ items[2], items[3] ], tracker.objects_causing_outdatedness_of(items[1])
+    assert_contains_exactly [items[1]],           tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[2], items[3]], tracker.objects_causing_outdatedness_of(items[1])
     assert_empty tracker.objects_causing_outdatedness_of(items[2])
     assert_empty tracker.objects_causing_outdatedness_of(items[3])
   end
 
   def test_store_graph_and_load_graph_with_removed_items
     # Mock items
-    items = [ mock('0'), mock('1'), mock('2'), mock('3') ]
+    items = [mock('0'), mock('1'), mock('2'), mock('3')]
     items.each { |i| i.stubs(:type).returns(:item) }
-    items[0].stubs(:reference).returns([ :item, '/aaa/' ])
-    items[1].stubs(:reference).returns([ :item, '/bbb/' ])
-    items[2].stubs(:reference).returns([ :item, '/ccc/' ])
-    items[3].stubs(:reference).returns([ :item, '/ddd/' ])
+    items[0].stubs(:reference).returns([:item, '/aaa/'])
+    items[1].stubs(:reference).returns([:item, '/bbb/'])
+    items[2].stubs(:reference).returns([:item, '/ccc/'])
+    items[3].stubs(:reference).returns([:item, '/ddd/'])
 
     # Create new and old lists
-    old_items = [ items[0], items[1], items[2], items[3] ]
-    new_items = [ items[0], items[1], items[2]           ]
+    old_items = [items[0], items[1], items[2], items[3]]
+    new_items = [items[0], items[1], items[2]]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(old_items)
@@ -175,18 +175,18 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.load_graph
 
     # Check loaded graph
-    assert_contains_exactly [ items[1] ],       tracker.objects_causing_outdatedness_of(items[0])
-    assert_contains_exactly [ items[2], nil ],  tracker.objects_causing_outdatedness_of(items[1])
+    assert_contains_exactly [items[1]],       tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[2], nil],  tracker.objects_causing_outdatedness_of(items[1])
     assert_empty tracker.objects_causing_outdatedness_of(items[2])
   end
 
   def test_store_graph_with_nils_in_dst
     # Mock items
-    items = [ mock('0'), mock('1'), mock('2') ]
+    items = [mock('0'), mock('1'), mock('2')]
     items.each { |i| i.stubs(:type).returns(:item) }
-    items[0].stubs(:reference).returns([ :item, '/aaa/' ])
-    items[1].stubs(:reference).returns([ :item, '/bbb/' ])
-    items[2].stubs(:reference).returns([ :item, '/ccc/' ])
+    items[0].stubs(:reference).returns([:item, '/aaa/'])
+    items[1].stubs(:reference).returns([:item, '/bbb/'])
+    items[2].stubs(:reference).returns([:item, '/ccc/'])
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -206,17 +206,17 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.load_graph
 
     # Check loaded graph
-    assert_contains_exactly [ items[1] ], tracker.objects_causing_outdatedness_of(items[0])
-    assert_contains_exactly [ nil ],      tracker.objects_causing_outdatedness_of(items[1])
+    assert_contains_exactly [items[1]], tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [nil],      tracker.objects_causing_outdatedness_of(items[1])
   end
 
   def test_store_graph_with_nils_in_src
     # Mock items
-    items = [ mock('0'), mock('1'), mock('2') ]
+    items = [mock('0'), mock('1'), mock('2')]
     items.each { |i| i.stubs(:type).returns(:item) }
-    items[0].stubs(:reference).returns([ :item, '/aaa/' ])
-    items[1].stubs(:reference).returns([ :item, '/bbb/' ])
-    items[2].stubs(:reference).returns([ :item, '/ccc/' ])
+    items[0].stubs(:reference).returns([:item, '/aaa/'])
+    items[1].stubs(:reference).returns([:item, '/bbb/'])
+    items[2].stubs(:reference).returns([:item, '/ccc/'])
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -236,13 +236,13 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     tracker.load_graph
 
     # Check loaded graph
-    assert_contains_exactly [ items[1] ], tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[1]], tracker.objects_causing_outdatedness_of(items[0])
     assert_empty tracker.objects_causing_outdatedness_of(items[1])
   end
 
   def test_forget_dependencies_for
     # Mock items
-    items = [ mock, mock, mock ]
+    items = [mock, mock, mock]
 
     # Create
     tracker = Nanoc::DependencyTracker.new(items)
@@ -250,7 +250,7 @@ class Nanoc::DependencyTrackerTest < Nanoc::TestCase
     # Record some dependencies
     tracker.record_dependency(items[0], items[1])
     tracker.record_dependency(items[1], items[2])
-    assert_contains_exactly [ items[1] ], tracker.objects_causing_outdatedness_of(items[0])
+    assert_contains_exactly [items[1]], tracker.objects_causing_outdatedness_of(items[0])
 
     # Forget dependencies
     tracker.forget_dependencies_for(items[0])