nanoc 4.8.10 → 4.8.11

data/spec/nanoc/base/repos/in_mem_data_source_spec.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+describe Nanoc::Int::InMemDataSource, stdio: true do
+  let(:klass) do
+    Class.new(Nanoc::DataSource) do
+      def item_changes
+        %i[one_foo one_bar]
+      end
+
+      def layout_changes
+        %i[one_foo one_bar]
+      end
+    end
+  end
+
+  let(:original_data_source) do
+    klass.new({}, nil, nil, {})
+  end
+
+  subject(:data_source) do
+    described_class.new([], [], original_data_source)
+  end
+
+  describe '#item_changes' do
+    subject { data_source.item_changes }
+
+    it 'yields changes from the original' do
+      expect(subject).to eq(original_data_source.item_changes)
+    end
+  end
+
+  describe '#layout_changes' do
+    subject { data_source.layout_changes }
+
+    it 'yields changes from the original' do
+      expect(subject).to eq(original_data_source.layout_changes)
+    end
+  end
+end

data/spec/nanoc/base/repos/prefixed_data_source_spec.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+describe Nanoc::Int::PrefixedDataSource, stdio: true do
+  let(:klass) do
+    Class.new(Nanoc::DataSource) do
+      def item_changes
+        %i[one_foo one_bar]
+      end
+
+      def layout_changes
+        %i[one_foo one_bar]
+      end
+    end
+  end
+
+  let(:original_data_source) do
+    klass.new({}, nil, nil, {})
+  end
+
+  subject(:data_source) do
+    described_class.new(original_data_source, '/itemz', '/layoutz')
+  end
+
+  describe '#item_changes' do
+    subject { data_source.item_changes }
+
+    it 'yields changes from the original' do
+      expect(subject).to eq(original_data_source.item_changes)
+    end
+  end
+
+  describe '#layout_changes' do
+    subject { data_source.layout_changes }
+
+    it 'yields changes from the original' do
+      expect(subject).to eq(original_data_source.layout_changes)
+    end
+  end
+end

data/spec/nanoc/cli/error_handler_spec.rb

@@ -0,0 +1,43 @@
+# 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

@@ -0,0 +1,156 @@
+# 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/data_sources/filesystem_spec.rb

@@ -79,4 +79,50 @@ describe Nanoc::DataSources::Filesystem do
       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/extra/live_recompiler_spec.rb

@@ -0,0 +1,129 @@
+# frozen_string_literal: true
+
+describe Nanoc::Extra::LiveRecompiler, site: true, stdio: true do
+  it 'detects content changes' do
+    command = nil
+    command_runner = Nanoc::CLI::CommandRunner.new({}, [], command)
+    live_recompiler = described_class.new(command_runner: command_runner)
+
+    pid = fork do
+      trap(:INT) { exit(0) }
+      live_recompiler.run
+    end
+
+    # FIXME: wait is ugly
+    sleep 0.5
+
+    File.write('content/lol.html', 'hej')
+    sleep 0.1 until File.file?('output/lol.html')
+    expect(File.read('output/lol.html')).to eq('hej')
+
+    sleep 1.0 # HFS+ mtime resolution is 1s
+    File.write('content/lol.html', 'bye')
+    sleep 0.1 until File.read('output/lol.html') == 'bye'
+
+    # Stop
+    Process.kill('INT', pid)
+    Process.waitpid(pid)
+  end
+
+  it 'detects rules changes' do
+    command = nil
+    command_runner = Nanoc::CLI::CommandRunner.new({}, [], command)
+    live_recompiler = described_class.new(command_runner: command_runner)
+
+    pid = fork do
+      trap(:INT) { exit(0) }
+      live_recompiler.run
+    end
+
+    # FIXME: wait is ugly
+    sleep 0.5
+
+    File.write('content/lol.html', '<%= "hej" %>')
+    sleep 0.1 until File.file?('output/lol.html')
+    expect(File.read('output/lol.html')).to eq('<%= "hej" %>')
+
+    sleep 1.0 # HFS+ mtime resolution is 1s
+    File.write('Rules', <<~RULES)
+      compile '/**/*' do
+        filter :erb
+        write item.identifier
+      end
+    RULES
+    sleep 0.1 until File.read('output/lol.html') == 'hej'
+
+    # Stop
+    Process.kill('INT', pid)
+    Process.waitpid(pid)
+  end
+
+  it 'detects config changes' do
+    command = nil
+    command_runner = Nanoc::CLI::CommandRunner.new({}, [], command)
+    live_recompiler = described_class.new(command_runner: command_runner)
+
+    File.write('nanoc.yaml', 'site_name: Oldz')
+    File.write('content/lol.html', '<%= @config[:site_name] %>')
+    File.write('Rules', <<~RULES)
+      compile '/**/*' do
+        filter :erb
+        write item.identifier
+      end
+    RULES
+
+    pid = fork do
+      trap(:INT) { exit(0) }
+      live_recompiler.run
+    end
+
+    # FIXME: wait is ugly
+    sleep 0.5
+
+    sleep 0.1 until File.file?('output/lol.html')
+    expect(File.read('output/lol.html')).to eq('Oldz')
+
+    sleep 1.0 # HFS+ mtime resolution is 1s
+    File.write('nanoc.yaml', 'site_name: Newz')
+    sleep 0.1 until File.read('output/lol.html') == 'Newz'
+
+    # Stop
+    Process.kill('INT', pid)
+    Process.waitpid(pid)
+  end
+
+  it 'detects config changes' do
+    command = nil
+    command_runner = Nanoc::CLI::CommandRunner.new({}, [], command)
+    live_recompiler = described_class.new(command_runner: command_runner)
+
+    FileUtils.mkdir_p('lib')
+    File.write('lib/lol.rb', 'def greeting; "hi"; end')
+    File.write('content/lol.html', '<%= greeting %>')
+    File.write('Rules', <<~RULES)
+      compile '/**/*' do
+        filter :erb
+        write item.identifier
+      end
+    RULES
+
+    pid = fork do
+      trap(:INT) { exit(0) }
+      live_recompiler.run
+    end
+
+    # FIXME: wait is ugly
+    sleep 0.5
+
+    sleep 0.1 until File.file?('output/lol.html')
+    expect(File.read('output/lol.html')).to eq('hi')
+
+    sleep 1.0 # HFS+ mtime resolution is 1s
+    File.write('lib/lol.rb', 'def greeting; "yo"; end')
+    sleep 0.1 until File.read('output/lol.html') == 'yo'
+
+    # Stop
+    Process.kill('INT', pid)
+    Process.waitpid(pid)
+  end
+end