nanoc 4.4.5 → 4.4.6

data/spec/regression_filenames_spec.rb

@@ -0,0 +1,16 @@
+describe 'regression tests', chdir: false do
+  let(:regression_test_filenames) do
+    Dir['spec/nanoc/regressions/*']
+  end
+
+  let(:regression_test_numbers) do
+    regression_test_filenames
+      .map { |fn| File.readlines(fn).first.match(/GH-(\d+)/)[1] }
+  end
+
+  it 'should have the proper filenames' do
+    regression_test_filenames.zip(regression_test_numbers) do |fn, num|
+      expect(fn).to match(/gh_#{num}[a-z]*_spec/), "#{fn} has the wrong name in its #define block"
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,173 @@
+require 'simplecov'
+SimpleCov.start
+
+require 'nanoc'
+require 'nanoc/cli'
+require 'nanoc/spec'
+
+require 'timecop'
+require 'rspec/its'
+
+Nanoc::CLI.setup
+
+RSpec.configure do |c|
+  c.around(:each) do |example|
+    Nanoc::CLI::ErrorHandler.disable
+    example.run
+    Nanoc::CLI::ErrorHandler.enable
+  end
+
+  c.around(:each) do |example|
+    Dir.mktmpdir('nanoc-test') do |dir|
+      FileUtils.cd(dir) do
+        example.run
+      end
+    end
+  end
+
+  c.around(:each, chdir: false) do |example|
+    FileUtils.cd(File.dirname(__FILE__) + '/..') do
+      example.run
+    end
+  end
+
+  c.before(:each) do
+    Nanoc::Int::NotificationCenter.reset
+  end
+
+  c.before(:each, v8: true) do
+    if ENV.key?('DISABLE_V8')
+      skip 'V8 specs are disabled (broken on Ruby 2.4)'
+    end
+  end
+
+  c.around(:each, stdio: true) do |example|
+    orig_stdout = $stdout
+    orig_stderr = $stderr
+
+    unless ENV['QUIET'] == 'false'
+      $stdout = StringIO.new
+      $stderr = StringIO.new
+    end
+
+    example.run
+
+    $stdout = orig_stdout
+    $stderr = orig_stderr
+  end
+
+  c.before(:each, site: true) do
+    FileUtils.mkdir_p('content')
+    FileUtils.mkdir_p('layouts')
+    FileUtils.mkdir_p('lib')
+    FileUtils.mkdir_p('output')
+
+    File.write('nanoc.yaml', '{}')
+
+    File.write('Rules', 'passthrough "/**/*"')
+  end
+
+  c.include(Nanoc::Spec::HelperHelper, helper: true)
+
+  # Set focus if any
+  if ENV.fetch('FOCUS', false)
+    $stdout.puts "Focusing spec on '#{ENV['FOCUS']}'"
+    c.filter_run_including ENV['FOCUS'].to_sym => true
+  end
+end
+
+RSpec::Matchers.define :raise_frozen_error do |_expected|
+  match do |actual|
+    begin
+      actual.call
+      false
+    rescue => e
+      if e.is_a?(RuntimeError) || e.is_a?(TypeError)
+        e.message =~ /(^can't modify frozen |^unable to modify frozen object$)/
+      else
+        false
+      end
+    end
+  end
+
+  supports_block_expectations
+
+  failure_message do |_actual|
+    'expected that proc would raise a frozen error'
+  end
+
+  failure_message_when_negated do |_actual|
+    'expected that proc would not raise a frozen error'
+  end
+end
+
+RSpec::Matchers.define :be_humanly_sorted do
+  match do |actual|
+    actual == sort(actual)
+  end
+
+  description do
+    'be humanly sorted'
+  end
+
+  failure_message do |actual|
+    expected_order = []
+    actual.zip(sort(actual)).each do |a, b|
+      if a != b
+        expected_order << b
+      end
+    end
+
+    "expected collection to be sorted (incorrect order: #{expected_order.join(' < ')})"
+  end
+
+  def sort(x)
+    x.sort_by { |n| n.dup.unicode_normalize(:nfd).encode('ASCII', fallback: ->(_) { '' }).downcase }
+  end
+end
+
+RSpec::Matchers.define :finish_in_under do |expected|
+  supports_block_expectations
+
+  match do |actual|
+    before = Time.now
+    actual.call
+    after = Time.now
+    @actual_duration = after - before
+    @actual_duration < expected
+  end
+
+  chain :seconds do
+  end
+
+  failure_message do |_actual|
+    "expected that proc would finish in under #{expected}s, but took #{format '%0.1fs', @actual_duration}"
+  end
+
+  failure_message_when_negated do |_actual|
+    "expected that proc would not finish in under #{expected}s, but took #{format '%0.1fs', @actual_duration}"
+  end
+end
+
+RSpec::Matchers.define :yield_from_fiber do |expected|
+  supports_block_expectations
+
+  include RSpec::Matchers::Composable
+
+  match do |actual|
+    @res = Fiber.new { actual.call }.resume
+    values_match?(expected, @res)
+  end
+
+  description do
+    "yield #{expected.inspect} from fiber"
+  end
+
+  failure_message do |_actual|
+    "expected that proc would yield #{expected.inspect} from fiber, but was #{@res.inspect}"
+  end
+
+  failure_message_when_negated do |_actual|
+    "expected that proc would not yield #{expected.inspect} from fiber, but was #{@res.inspect}"
+  end
+end

data/test/base/core_ext/array_spec.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 describe 'Array#__nanoc_symbolize_keys_recursively' do
   it 'should convert keys to symbols' do
     array_old = [:abc, 'xyz', { 'foo' => 'bar', :baz => :qux }]

data/test/base/core_ext/hash_spec.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 describe 'Hash#__nanoc_symbolize_keys_recursively' do
   it 'should convert keys to symbols' do
     hash_old = { 'foo' => 'bar' }

data/test/base/core_ext/string_spec.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 describe 'String#__nanoc_cleaned_identifier' do
   it 'should not convert already clean paths' do
     '/foo/bar/'.__nanoc_cleaned_identifier.must_equal '/foo/bar/'

data/test/base/test_code_snippet.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::CodeSnippetTest < Nanoc::TestCase
   def test_load
     # Initialize

data/test/base/test_compiler.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::CompilerTest < Nanoc::TestCase
   def new_compiler(site = nil)
     site ||= Nanoc::Int::Site.new(

data/test/base/test_context.rb

@@ -1,7 +1,9 @@
+require 'helper'
+
 class Nanoc::Int::ContextTest < Nanoc::TestCase
   def test_context_with_instance_variable
     # Create context
-    context = Nanoc::Int::Context.new({ foo: 'bar', baz: 'quux' })
+    context = Nanoc::Int::Context.new(foo: 'bar', baz: 'quux')
 
     # Ensure correct evaluation
     assert_equal('bar', eval('@foo', context.get_binding))
@@ -9,7 +11,7 @@ class Nanoc::Int::ContextTest < Nanoc::TestCase
 
   def test_context_with_instance_method
     # Create context
-    context = Nanoc::Int::Context.new({ foo: 'bar', baz: 'quux' })
+    context = Nanoc::Int::Context.new(foo: 'bar', baz: 'quux')
 
     # Ensure correct evaluation
     assert_equal('bar', eval('foo', context.get_binding))

data/test/base/test_data_source.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::DataSourceTest < Nanoc::TestCase
   def test_loading
     # Create data source

data/test/base/test_dependency_tracker.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::DependencyTrackerTest < Nanoc::TestCase
   def test_initialize
     # Mock items

data/test/base/test_directed_graph.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::DirectedGraphTest < Nanoc::TestCase
   def test_direct_predecessors
     graph = Nanoc::Int::DirectedGraph.new([1, 2, 3])

data/test/base/test_filter.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::FilterTest < Nanoc::TestCase
   def test_initialize
     # Create filter
@@ -9,7 +11,7 @@ class Nanoc::FilterTest < Nanoc::TestCase
 
   def test_assigns
     # Create filter
-    filter = Nanoc::Filter.new({ foo: 'bar' })
+    filter = Nanoc::Filter.new(foo: 'bar')
 
     # Check assigns
     assert_equal('bar', filter.assigns[:foo])
@@ -17,7 +19,7 @@ class Nanoc::FilterTest < Nanoc::TestCase
 
   def test_assigns_with_instance_variables
     # Create filter
-    filter = Nanoc::Filter.new({ foo: 'bar' })
+    filter = Nanoc::Filter.new(foo: 'bar')
 
     # Check assigns
     assert_equal('bar', filter.instance_eval { @foo })
@@ -25,7 +27,7 @@ class Nanoc::FilterTest < Nanoc::TestCase
 
   def test_assigns_with_instance_methods
     # Create filter
-    filter = Nanoc::Filter.new({ foo: 'bar' })
+    filter = Nanoc::Filter.new(foo: 'bar')
 
     # Check assigns
     assert_equal('bar', filter.instance_eval { foo })
@@ -49,7 +51,7 @@ class Nanoc::FilterTest < Nanoc::TestCase
     item_rep.expects(:name).returns(:quux)
 
     # Create filter
-    filter = Nanoc::Filter.new({ item: item, item_rep: item_rep })
+    filter = Nanoc::Filter.new(item: item, item_rep: item_rep)
 
     # Check filename
     assert_equal('item /foo/bar/baz/ (rep quux)', filter.filename)
@@ -61,7 +63,7 @@ class Nanoc::FilterTest < Nanoc::TestCase
     layout.expects(:identifier).returns('/wohba/')
 
     # Create filter
-    filter = Nanoc::Filter.new({ item: mock, item_rep: mock, layout: layout })
+    filter = Nanoc::Filter.new(item: mock, item_rep: mock, layout: layout)
 
     # Check filename
     assert_equal('layout /wohba/', filter.filename)

data/test/base/test_item.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::ItemTest < Nanoc::TestCase
   def test_initialize_with_attributes_with_string_keys
     item = Nanoc::Int::Item.new('foo', { 'abc' => 'xyz' }, '/foo/')

data/test/base/test_item_array.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::IdentifiableCollectionTest < Nanoc::TestCase
   def setup
     super
@@ -25,7 +27,7 @@ class Nanoc::Int::IdentifiableCollectionTest < Nanoc::TestCase
   end
 
   def test_brackets_with_glob
-    @items = Nanoc::Int::IdentifiableCollection.new({ string_pattern_type: 'glob' })
+    @items = Nanoc::Int::IdentifiableCollection.new(string_pattern_type: 'glob')
     @items << @one
     @items << @two
 

data/test/base/test_layout.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::LayoutTest < Nanoc::TestCase
   def test_initialize
     # Make sure attributes are cleaned

data/test/base/test_memoization.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::MemoizationTest < Nanoc::TestCase
   class Sample1
     extend Nanoc::Int::Memoization

data/test/base/test_notification_center.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::NotificationCenterTest < Nanoc::TestCase
   def test_post
     # Set up notification

data/test/base/test_outdatedness_checker.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::OutdatednessCheckerTest < Nanoc::TestCase
   def test_not_outdated
     # Compile once

data/test/base/test_plugin.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::PluginTest < Nanoc::TestCase
   class SampleFilter < Nanoc::Filter
     identifier :_plugin_test_sample_filter

data/test/base/test_site.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::SiteTest < Nanoc::TestCase
   def test_initialize_with_dir_without_config_yaml
     assert_raises(Nanoc::Int::ConfigLoader::NoConfigFileFoundError) do

data/test/base/test_store.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Int::StoreTest < Nanoc::TestCase
   class TestStore < Nanoc::Int::Store
     def data

data/test/checking/checks/test_css.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::Checks::CSSTest < Nanoc::TestCase
   def test_run_ok
     VCR.use_cassette('css_run_ok') do

data/test/checking/checks/test_external_links.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::Checks::ExternalLinksTest < Nanoc::TestCase
   def test_run
     with_site do |site|
@@ -77,7 +79,7 @@ class Nanoc::Checking::Checks::ExternalLinksTest < Nanoc::TestCase
     with_site do |site|
       # Create check
       check = Nanoc::Checking::Checks::ExternalLinks.create(site)
-      site.config.update({ checks: { external_links: { exclude: ['^http://excluded.com$'] } } })
+      site.config.update(checks: { external_links: { exclude: ['^http://excluded.com$'] } })
 
       # Test
       assert check.send(:excluded?, 'http://excluded.com')
@@ -90,7 +92,7 @@ class Nanoc::Checking::Checks::ExternalLinksTest < Nanoc::TestCase
     with_site do |site|
       # Create check
       check = Nanoc::Checking::Checks::ExternalLinks.create(site)
-      site.config.update({ checks: { external_links: { exclude_files: ['blog/page'] } } })
+      site.config.update(checks: { external_links: { exclude_files: ['blog/page'] } })
 
       # Test
       assert check.send(:excluded_file?, 'output/blog/page1/index.html')

data/test/checking/checks/test_html.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::Checks::HTMLTest < Nanoc::TestCase
   def test_run_ok
     require 'w3c_validators'

data/test/checking/checks/test_internal_links.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::Checks::InternalLinksTest < Nanoc::TestCase
   def test_run
     with_site do |site|
@@ -69,7 +71,7 @@ class Nanoc::Checking::Checks::InternalLinksTest < Nanoc::TestCase
     with_site do |site|
       # Create check
       check = Nanoc::Checking::Checks::InternalLinks.create(site)
-      site.config.update({ checks: { internal_links: { exclude: ['^/excluded\d+'] } } })
+      site.config.update(checks: { internal_links: { exclude: ['^/excluded\d+'] } })
 
       # Test
       assert check.send(:valid?, '/excluded1', 'output/origin')
@@ -82,7 +84,7 @@ class Nanoc::Checking::Checks::InternalLinksTest < Nanoc::TestCase
     with_site do |site|
       # Create check
       check = Nanoc::Checking::Checks::InternalLinks.create(site)
-      site.config.update({ checks: { internal_links: { exclude_targets: ['^/excluded\d+'] } } })
+      site.config.update(checks: { internal_links: { exclude_targets: ['^/excluded\d+'] } })
 
       # Test
       assert check.send(:valid?, '/excluded1', 'output/origin')
@@ -95,7 +97,7 @@ class Nanoc::Checking::Checks::InternalLinksTest < Nanoc::TestCase
     with_site do |site|
       # Create check
       check = Nanoc::Checking::Checks::InternalLinks.create(site)
-      site.config.update({ checks: { internal_links: { exclude_origins: ['^/excluded'] } } })
+      site.config.update(checks: { internal_links: { exclude_origins: ['^/excluded'] } })
 
       # Test
       assert check.send(:valid?, '/foo', 'output/excluded')

data/test/checking/checks/test_mixed_content.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::Checks::MixedContentTest < Nanoc::TestCase
   def create_output_file(name, lines)
     FileUtils.mkdir_p('output')

data/test/checking/checks/test_stale.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::Checks::StaleTest < Nanoc::TestCase
   def check_class
     Nanoc::Checking::Checks::Stale

data/test/checking/test_check.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::CheckTest < Nanoc::TestCase
   def test_output_filenames
     with_site do |site|

data/test/checking/test_dsl.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::DSLTest < Nanoc::TestCase
   def test_from_file
     with_site do |_site|

data/test/checking/test_runner.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::Checking::RunnerTest < Nanoc::TestCase
   def test_run_specific
     with_site do |site|

data/test/cli/commands/test_check.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::Commands::CheckTest < Nanoc::TestCase
   def test_check_stale
     with_site do |_site|

data/test/cli/commands/test_compile.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::Commands::CompileTest < Nanoc::TestCase
   def test_profiling_information
     with_site do |_site|

data/test/cli/commands/test_create_site.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::Commands::CreateSiteTest < Nanoc::TestCase
   def test_create_site_with_existing_name
     Nanoc::CLI.run %w(create_site foo)

data/test/cli/commands/test_help.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::Commands::HelpTest < Nanoc::TestCase
   def test_run
     Nanoc::CLI.run %w(help)

data/test/cli/commands/test_info.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::Commands::InfoTest < Nanoc::TestCase
   def test_run
     Nanoc::CLI.run %w(info)

data/test/cli/commands/test_prune.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::Commands::PruneTest < Nanoc::TestCase
   def test_run_without_yes
     with_site do |_site|

data/test/cli/test_cleaning_stream.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::CleaningStreamTest < Nanoc::TestCase
   class Stream
     attr_accessor :called_methods

data/test/cli/test_cli.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLITest < Nanoc::TestCase
   COMMAND_CODE = <<EOS.freeze
 usage       '_test [options]'
@@ -176,17 +178,17 @@ EOS
     with_env_vars(new_env_diff) do
       refute Nanoc::CLI.enable_utf8?(io)
 
-      with_env_vars({ 'LC_ALL'   => 'en_US.UTF-8' }) { assert Nanoc::CLI.enable_utf8?(io) }
-      with_env_vars({ 'LC_CTYPE' => 'en_US.UTF-8' }) { assert Nanoc::CLI.enable_utf8?(io) }
-      with_env_vars({ 'LANG'     => 'en_US.UTF-8' }) { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LC_ALL'   => 'en_US.UTF-8') { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LC_CTYPE' => 'en_US.UTF-8') { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LANG'     => 'en_US.UTF-8') { assert Nanoc::CLI.enable_utf8?(io) }
 
-      with_env_vars({ 'LC_ALL'   => 'en_US.utf-8' }) { assert Nanoc::CLI.enable_utf8?(io) }
-      with_env_vars({ 'LC_CTYPE' => 'en_US.utf-8' }) { assert Nanoc::CLI.enable_utf8?(io) }
-      with_env_vars({ 'LANG'     => 'en_US.utf-8' }) { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LC_ALL'   => 'en_US.utf-8') { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LC_CTYPE' => 'en_US.utf-8') { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LANG'     => 'en_US.utf-8') { assert Nanoc::CLI.enable_utf8?(io) }
 
-      with_env_vars({ 'LC_ALL'   => 'en_US.utf8'  }) { assert Nanoc::CLI.enable_utf8?(io) }
-      with_env_vars({ 'LC_CTYPE' => 'en_US.utf8'  }) { assert Nanoc::CLI.enable_utf8?(io) }
-      with_env_vars({ 'LANG'     => 'en_US.utf8'  }) { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LC_ALL'   => 'en_US.utf8') { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LC_CTYPE' => 'en_US.utf8') { assert Nanoc::CLI.enable_utf8?(io) }
+      with_env_vars('LANG'     => 'en_US.utf8') { assert Nanoc::CLI.enable_utf8?(io) }
     end
   end
 end

data/test/cli/test_error_handler.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::ErrorHandlerTest < Nanoc::TestCase
   def setup
     super

data/test/cli/test_logger.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::CLI::LoggerTest < Nanoc::TestCase
   def test_stub; end
 end

data/test/data_sources/test_filesystem.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::DataSources::FilesystemTest < Nanoc::TestCase
   def new_data_source(params = nil)
     # Mock site
@@ -69,7 +71,7 @@ class Nanoc::DataSources::FilesystemTest < Nanoc::TestCase
 
   def test_load_objects_with_same_extensions
     # Create data source
-    data_source = new_data_source({ identifier_type: 'full' })
+    data_source = new_data_source(identifier_type: 'full')
 
     # Create a fake class
     klass = Class.new do
@@ -142,7 +144,7 @@ class Nanoc::DataSources::FilesystemTest < Nanoc::TestCase
 
   def test_identifier_for_filename_with_full_style_identifier
     # Create data source
-    data_source = new_data_source({ identifier_type: 'full' })
+    data_source = new_data_source(identifier_type: 'full')
 
     # Get input and expected output
     expected = {
@@ -463,7 +465,7 @@ class Nanoc::DataSources::FilesystemTest < Nanoc::TestCase
   end
 
   def test_load_objects_correct_identifier_with_separate_yaml_file
-    data_source = new_data_source({ identifier_type: 'full' })
+    data_source = new_data_source(identifier_type: 'full')
 
     FileUtils.mkdir_p('foo')
     File.write('foo/donkey.jpeg', 'data')
@@ -541,7 +543,7 @@ class Nanoc::DataSources::FilesystemTest < Nanoc::TestCase
 
   def test_all_split_files_in_allowing_periods_in_identifiers
     # Create data source
-    data_source = Nanoc::DataSources::Filesystem.new(nil, nil, nil, { allow_periods_in_identifiers: true })
+    data_source = Nanoc::DataSources::Filesystem.new(nil, nil, nil, allow_periods_in_identifiers: true)
 
     # Write sample files
     FileUtils.mkdir_p('foo')
@@ -649,7 +651,7 @@ class Nanoc::DataSources::FilesystemTest < Nanoc::TestCase
 
   def test_basename_of_allowing_periods_in_identifiers
     # Create data source
-    data_source = Nanoc::DataSources::Filesystem.new(nil, nil, nil, { allow_periods_in_identifiers: true })
+    data_source = Nanoc::DataSources::Filesystem.new(nil, nil, nil, allow_periods_in_identifiers: true)
 
     # Get input and expected output
     expected = {
@@ -709,7 +711,7 @@ class Nanoc::DataSources::FilesystemTest < Nanoc::TestCase
 
   def test_ext_of_allowing_periods_in_identifiers
     # Create data source
-    data_source = Nanoc::DataSources::Filesystem.new(nil, nil, nil, { allow_periods_in_identifiers: true })
+    data_source = Nanoc::DataSources::Filesystem.new(nil, nil, nil, allow_periods_in_identifiers: true)
 
     # Get input and expected output
     expected = {

data/test/data_sources/test_filesystem_tools.rb

@@ -1,3 +1,5 @@
+require 'helper'
+
 class Nanoc::DataSources::FilesystemToolsTest < Nanoc::TestCase
   def setup
     super