nanoc 4.8.12 → 4.8.13

data/spec/nanoc/cli/error_handler_spec.rb

@@ -1,43 +0,0 @@
-# frozen_string_literal: true
-
-describe Nanoc::CLI::ErrorHandler do
-  subject(:error_handler) { described_class.new }
-
-  describe '#forwards_stack_trace?' do
-    subject { error_handler.forwards_stack_trace? }
-
-    context 'feature enabled' do
-      around do |ex|
-        Nanoc::Feature.enable(Nanoc::Feature::SENSIBLE_STACK_TRACES) do
-          ex.run
-        end
-      end
-
-      context 'Ruby 2.4' do
-        it { is_expected.to be(true) }
-      end
-
-      context 'Ruby 2.5' do
-        it { is_expected.to be(true) }
-      end
-    end
-
-    context 'feature not enabled' do
-      context 'Ruby 2.4' do
-        before do
-          expect(error_handler).to receive(:ruby_version).and_return('2.4.2')
-        end
-
-        it { is_expected.to be(false) }
-      end
-
-      context 'Ruby 2.5' do
-        before do
-          expect(error_handler).to receive(:ruby_version).and_return('2.5.0')
-        end
-
-        it { is_expected.to be(true) }
-      end
-    end
-  end
-end

data/spec/nanoc/cli/stack_trace_writer_spec.rb

@@ -1,156 +0,0 @@
-# frozen_string_literal: true
-
-describe Nanoc::CLI::StackTraceWriter do
-  subject(:writer) do
-    described_class.new(io, forwards: forwards)
-  end
-
-  let(:io) { StringIO.new }
-  let(:forwards) { true }
-
-  describe '#write' do
-    let(:exception) do
-      backtrace_generator = lambda do |af|
-        if af.zero?
-          raise 'finally!'
-        else
-          backtrace_generator.call(af - 1)
-        end
-      end
-
-      begin
-        backtrace_generator.call(3)
-      rescue => e
-        return e
-      end
-    end
-
-    subject { writer.write(exception, verbose: verbose) }
-
-    let(:verbose) { false }
-
-    context 'backwards' do
-      let(:forwards) { false }
-
-      context 'verbose' do
-        let(:verbose) { true }
-
-        it 'starts with zero' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(start_with('  0. '))
-        end
-
-        it 'has more recent stack frames at the top' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(match(%r{^  0\. /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d+.*$\n  1\. /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d}m))
-        end
-
-        it 'has more than 10 stack frames' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(match(%r{^  11\. }))
-        end
-
-        it 'does not contain a see-more explanation' do
-          subject
-          expect(io.string).not_to match(/crash\.log/)
-        end
-      end
-
-      context 'not verbose' do
-        let(:verbose) { false }
-
-        it 'starts with zero' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(start_with('  0. '))
-        end
-
-        it 'has more recent stack frames at the top' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(match(%r{^  0\. /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d+.*$\n  1\. /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d}m))
-        end
-
-        it 'has not more than 10 stack frames' do
-          subject
-          expect(io.string).not_to match(/^  11\. /)
-        end
-
-        it 'does not contain a see-more explanation' do
-          subject
-          expect(io.string).to include(" lines omitted (see crash.log for details)\n")
-        end
-      end
-    end
-
-    context 'forwards' do
-      let(:forwards) { true }
-
-      context 'verbose' do
-        let(:verbose) { true }
-
-        it 'ends with most recent line' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(match(%r{^  1\. from /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d+.*$\n  /.+/spec/nanoc/cli}m))
-        end
-
-        it 'has more recent stack frames at the bottom' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(match(%r{^  2\. from /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d+.*$\n  1\. from /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d}m))
-        end
-
-        it 'has more than 10 stack frames' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(match(%r{^  11\. from }))
-        end
-
-        it 'does not contain a see-more explanation' do
-          subject
-          expect(io.string).not_to match(/crash\.log/)
-        end
-      end
-
-      context 'not verbose' do
-        let(:verbose) { false }
-
-        it 'ends with most recent line' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(match(%r{^  1\. from /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d+.*$\n  /.+/spec/nanoc/cli}m))
-        end
-
-        it 'has more recent stack frames at the top' do
-          expect { subject }
-            .to change { io.string }
-            .from('')
-            .to(match(%r{^  2\. from /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d+.*$\n  1\. from /.+/spec/nanoc/cli/stack_trace_writer_spec\.rb:\d}m))
-        end
-
-        it 'has not more than 10 stack frames' do
-          subject
-          expect(io.string).not_to match(/^  11\. from /)
-        end
-
-        it 'does not contain a see-more explanation' do
-          subject
-          expect(io.string).to include(" lines omitted (see crash.log for details)\n")
-        end
-      end
-    end
-  end
-end

data/spec/nanoc/cli/stream_cleaners/utf8_spec.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-describe Nanoc::CLI::StreamCleaners::UTF8 do
-  subject { described_class.new }
-
-  it 'handles all cases' do
-    expect(subject.clean('┼─ “© Denis” ‘and others…’ ─┼')).to eq('+- "(c) Denis" \'and others...\' -+')
-  end
-end

data/spec/nanoc/cli_spec.rb

@@ -1,44 +0,0 @@
-# frozen_string_literal: true
-
-describe Nanoc::CLI do
-  let(:all_commands) do
-    ObjectSpace.each_object(Cri::Command)
-  end
-
-  let(:exceptions) do
-    # FIXME: [Nanoc 5] Get rid of these exceptions
-    [
-      ['deploy', ['C']],
-      ['help', ['v']],
-      ['check', ['d']],
-    ]
-  end
-
-  def ancestors_of_command(command)
-    if command.is_a?(Cri::Command)
-      [command] + ancestors_of_command(command.supercommand)
-    else
-      []
-    end
-  end
-
-  def short_options_for_command(command)
-    ancestors = ancestors_of_command(command)
-    ancestors.flat_map { |a| a.option_definitions.to_a.map { |od| od[:short] } }.compact
-  end
-
-  it 'has no commands that have conflicting options' do
-    all_commands.each do |command|
-      short_options = short_options_for_command(command)
-
-      duplicate_options = short_options.select { |o| short_options.count(o) > 1 }.uniq
-
-      next if exceptions.include?([command.name, duplicate_options])
-
-      expect(duplicate_options).to(
-        be_empty,
-        "The #{command.name} command’s option shorthands #{duplicate_options.uniq} are used by multiple options",
-      )
-    end
-  end
-end

data/spec/nanoc/data_sources/filesystem_spec.rb

@@ -1,128 +0,0 @@
-# frozen_string_literal: true
-
-describe Nanoc::DataSources::Filesystem do
-  let(:data_source) { Nanoc::DataSources::Filesystem.new(site.config, nil, nil, params) }
-  let(:params) { {} }
-  let(:site) { Nanoc::Int::SiteLoader.new.new_empty }
-
-  before { Timecop.freeze(now) }
-  after { Timecop.return }
-
-  let(:now) { Time.local(2008, 1, 2, 14, 5, 0) }
-
-  describe '#load_objects' do
-    subject { data_source.send(:load_objects, 'foo', klass) }
-
-    let(:klass) { raise 'override me' }
-
-    context 'items' do
-      let(:klass) { Nanoc::Int::Item }
-
-      context 'no files' do
-        it 'loads nothing' do
-          expect(subject).to be_empty
-        end
-      end
-
-      context 'one regular file' do
-        before do
-          FileUtils.mkdir_p('foo')
-          File.write('foo/bar.html', "---\nnum: 1\n---\ntest 1")
-          FileUtils.touch('foo/bar.html', mtime: now)
-        end
-
-        let(:expected_attributes) do
-          {
-            content_filename: 'foo/bar.html',
-            extension: 'html',
-            filename: 'foo/bar.html',
-            meta_filename: nil,
-            mtime: now,
-            num: 1,
-          }
-        end
-
-        it 'loads that file' do
-          expect(subject.size).to eq(1)
-
-          expect(subject[0].content.string).to eq('test 1')
-          expect(subject[0].attributes).to eq(expected_attributes)
-          expect(subject[0].identifier).to eq(Nanoc::Identifier.new('/bar/', type: :legacy))
-          expect(subject[0].checksum_data).to be_nil
-          expect(subject[0].attributes_checksum_data).to be_a(String)
-          expect(subject[0].attributes_checksum_data.size).to eq(20)
-          expect(subject[0].content_checksum_data).to be_a(String)
-          expect(subject[0].content_checksum_data.size).to eq(20)
-        end
-
-        context 'split files' do
-          let(:block) do
-            lambda do
-              FileUtils.mkdir_p('foo')
-
-              File.write('foo/bar.html', 'test 1')
-              FileUtils.touch('foo/bar.html', mtime: now)
-
-              File.write('foo/bar.yaml', "---\nnum: 1\n")
-              FileUtils.touch('foo/bar.yaml', mtime: now)
-            end
-          end
-
-          it 'has a different attributes checksum' do
-            expect(block).to change { data_source.send(:load_objects, 'foo', klass)[0].attributes_checksum_data }
-          end
-
-          it 'has the same content checksum' do
-            expect(block).not_to change { data_source.send(:load_objects, 'foo', klass)[0].content_checksum_data }
-          end
-        end
-      end
-    end
-  end
-
-  describe '#item_changes' do
-    subject { data_source.item_changes }
-
-    it 'returns a stream' do
-      expect(subject).to be_a(Nanoc::ChangesStream)
-    end
-
-    it 'contains one element after changing' do
-      FileUtils.mkdir_p('content')
-
-      enum = DDBuffer.new(1).call(subject.to_enum)
-      q = SizedQueue.new(1)
-      Thread.new { q << enum.take(1).first }
-
-      # FIXME: sleep is ugly
-      sleep 0.3
-      File.write('content/wat.md', 'stuff')
-
-      expect(q.pop).to eq(:unknown)
-      subject.stop
-    end
-  end
-
-  describe '#layout_changes' do
-    subject { data_source.layout_changes }
-
-    it 'returns a stream' do
-      expect(subject).to be_a(Nanoc::ChangesStream)
-    end
-
-    it 'contains one element after changing' do
-      FileUtils.mkdir_p('layouts')
-
-      enum = DDBuffer.new(1).call(subject.to_enum)
-      q = SizedQueue.new(1)
-      Thread.new { q << enum.take(1).first }
-
-      # FIXME: sleep is ugly
-      sleep 0.3
-      File.write('layouts/wat.md', 'stuff')
-
-      expect(q.pop).to eq(:unknown)
-      subject.stop
-    end
-  end
-end

data/spec/nanoc/deploying/fog_spec.rb

@@ -1,199 +0,0 @@
-# frozen_string_literal: true
-
-describe Nanoc::Deploying::Deployers::Fog, stdio: true do
-  let(:deployer) do
-    Nanoc::Deploying::Deployers::Fog.new(
-      'output/',
-      config,
-      dry_run: is_dry_run,
-    )
-  end
-
-  let(:is_dry_run) { false }
-
-  let(:config) do
-    {
-      bucket: 'bucky',
-      provider: 'local',
-      local_root: 'remote',
-    }
-  end
-
-  before do
-    # create output
-    FileUtils.mkdir_p('output')
-    FileUtils.mkdir_p('output/etc')
-    File.open('output/woof', 'w') { |io| io.write 'I am a dog!' }
-    File.open('output/etc/meow', 'w') { |io| io.write 'I am a cat!' }
-
-    # create local cloud
-    FileUtils.mkdir_p('remote')
-  end
-
-  subject { deployer.run }
-
-  shared_examples 'no effective deploy' do
-    it 'does not modify remote' do
-      expect { subject }.not_to change { Dir['remote/**/*'].sort }
-    end
-  end
-
-  shared_examples 'effective deploy' do
-    it 'modifies remote' do
-      expect { subject }.to change { Dir['remote/**/*'].sort }
-        .to([
-          'remote/bucky',
-          'remote/bucky/etc',
-          'remote/bucky/etc/meow',
-          'remote/bucky/woof',
-        ])
-    end
-
-    it 'does not leave lingering open files' do
-      expect { subject }.not_to leak_open_files
-    end
-  end
-
-  context 'dry run' do
-    let(:is_dry_run) { true }
-
-    before do
-      FileUtils.mkdir_p('remote/bucky')
-      FileUtils.mkdir_p('remote/bucky/tiny')
-      File.write('remote/bucky/pig', 'oink?')
-      File.write('remote/bucky/tiny/piglet', 'little oink?')
-    end
-
-    include_examples 'no effective deploy'
-
-    context 'with CDN ID' do
-      let(:config) { super().merge(cdn_id: 'donkey-cdn') }
-
-      let(:cdn) { Object.new }
-      let(:distribution) { Object.new }
-
-      it 'does not actually invalidate' do
-        expect(::Fog::CDN).to receive(:new).with(provider: 'local', local_root: 'remote').and_return(cdn)
-        expect(cdn).to receive(:get_distribution).with('donkey-cdn').and_return(distribution)
-
-        subject
-      end
-    end
-  end
-
-  context 'effective run' do
-    include_examples 'effective deploy'
-
-    context 'custom path' do
-      context 'custom path ends with /' do
-        let(:config) do
-          super().merge(path: 'foo/')
-        end
-
-        it 'raises' do
-          expect { subject }.to raise_error('The path `foo/` is not supposed to have a trailing slash')
-        end
-      end
-
-      context 'custom path does not end with /' do
-        let(:config) do
-          super().merge(path: 'foo')
-        end
-
-        it 'modifies remote' do
-          expect { subject }.to change { Dir['remote/**/*'].sort }
-            .to([
-              'remote/bucky',
-              'remote/bucky/foo',
-              'remote/bucky/foo/etc',
-              'remote/bucky/foo/etc/meow',
-              'remote/bucky/foo/woof',
-            ])
-        end
-      end
-    end
-
-    context 'bucket already exists' do
-      before do
-        FileUtils.mkdir_p('remote/bucky')
-      end
-
-      include_examples 'effective deploy'
-    end
-
-    context 'remote contains stale file at root' do
-      before do
-        FileUtils.mkdir_p('remote/bucky')
-        File.write('remote/bucky/pig', 'oink?')
-      end
-
-      include_examples 'effective deploy'
-
-      it 'does not contain stale files' do
-        subject
-        expect(Dir['remote/**/*'].sort).not_to include('remote/bucky/pig')
-      end
-    end
-
-    context 'remote contains stale file in subdirectory' do
-      before do
-        FileUtils.mkdir_p('remote/bucky/secret')
-        File.write('remote/bucky/secret/pig', 'oink?')
-      end
-
-      include_examples 'effective deploy'
-
-      it 'does not contain stale files' do
-        subject
-        expect(Dir['remote/**/*'].sort).not_to include('remote/bucky/secret/pig')
-      end
-    end
-
-    context 'with CDN ID' do
-      let(:config) { super().merge(cdn_id: 'donkey-cdn') }
-
-      let(:cdn) { Object.new }
-      let(:distribution) { Object.new }
-
-      it 'invalidates' do
-        expect(::Fog::CDN).to receive(:new).with(provider: 'local', local_root: 'remote').and_return(cdn)
-        expect(cdn).to receive(:get_distribution).with('donkey-cdn').and_return(distribution)
-        expect(cdn).to receive(:post_invalidation).with(distribution, contain_exactly('etc/meow', 'woof'))
-
-        subject
-      end
-    end
-
-    context 'remote list consists of truncated sets' do
-      before do
-        expect(::Fog::Storage).to receive(:new).and_return(fog_storage)
-        expect(fog_storage).to receive(:directories).and_return(directories)
-        expect(directories).to receive(:get).and_return(directory)
-        allow(directory).to receive(:files).and_return(files)
-        expect(files).to receive(:get).with('stray').and_return(file_stray).ordered
-        expect(files).to receive(:each)
-          .and_yield(double(:woof, key: 'woof'))
-          .and_yield(double(:meow, key: 'etc/meow'))
-          .and_yield(double(:stray, key: 'stray'))
-        expect(file_stray).to receive(:destroy)
-
-        expect(files).to receive(:create).with(key: 'woof', body: anything, public: true) do
-          FileUtils.mkdir_p('remote/bucky')
-          File.write('remote/bucky/woof', 'hi')
-        end
-        expect(files).to receive(:create).with(key: 'etc/meow', body: anything, public: true) do
-          FileUtils.mkdir_p('remote/bucky/etc')
-          File.write('remote/bucky/etc/meow', 'hi')
-        end
-      end
-
-      let(:fog_storage) { double(:fog_storage) }
-      let(:directories) { double(:directories) }
-      let(:directory) { double(:directory) }
-      let(:files) { double(:files) }
-      let(:file_stray) { double(:file_stray) }
-
-      include_examples 'effective deploy'
-    end
-  end
-end