diff-lcs 1.3 → 1.5.0

data/spec/ldiff_spec.rb

@@ -1,47 +1,87 @@
-# -*- ruby encoding: utf-8 -*-
+# frozen_string_literal: true
 
 require 'spec_helper'
 
-describe "Diff::LCS.diff" do
-  include Diff::LCS::SpecHelper::Matchers
+RSpec.describe 'bin/ldiff' do
+  include CaptureSubprocessIO
 
-  it 'correctly diffs seq1 to seq2' do
-    diff_s1_s2 = Diff::LCS.diff(seq1, seq2)
-    expect(change_diff(correct_forward_diff)).to eq(diff_s1_s2)
-  end
+  fixtures = [
+    { :name => 'output.diff', :left => 'aX', :right => 'bXaX' },
+    { :name => 'output.diff.chef', :left => 'old-chef', :right => 'new-chef' },
+    { :name => 'output.diff.chef2', :left => 'old-chef2', :right => 'new-chef2' }
+  ].product([nil, '-e', '-f', '-c', '-u']).map { |(fixture, flag)|
+    fixture = fixture.dup
+    fixture[:flag] = flag
+    fixture
+  }
 
-  it 'correctly diffs seq2 to seq1' do
-    diff_s2_s1 = Diff::LCS.diff(seq2, seq1)
-    expect(change_diff(correct_backward_diff)).to eq(diff_s2_s1)
-  end
+  def self.test_ldiff(fixture)
+    desc = [
+      fixture[:flag],
+      "spec/fixtures/#{fixture[:left]}",
+      "spec/fixtures/#{fixture[:right]}",
+      '#',
+      '=>',
+      "spec/fixtures/ldiff/#{fixture[:name]}#{fixture[:flag]}"
+    ].join(' ')
 
-  it 'correctly diffs against an empty sequence' do
-    diff = Diff::LCS.diff(word_sequence, [])
-    correct_diff = [
-      [ [ '-', 0, 'abcd'           ],
-        [ '-', 1, 'efgh'           ],
-        [ '-', 2, 'ijkl'           ],
-        [ '-', 3, 'mnopqrstuvwxyz' ] ]
-    ]
+    it desc do
+      expect(run_ldiff(fixture)).to eq(read_fixture(fixture))
+    end
+  end
 
-    expect(change_diff(correct_diff)).to eq(diff)
+  fixtures.each do |fixture|
+    test_ldiff(fixture)
+  end
 
-    diff = Diff::LCS.diff([], word_sequence)
-    correct_diff.each { |hunk| hunk.each { |change| change[0] = '+' } }
-    expect(change_diff(correct_diff)).to eq(diff)
+  def read_fixture(options)
+    fixture = options.fetch(:name)
+    flag = options.fetch(:flag)
+    name = "spec/fixtures/ldiff/#{fixture}#{flag}"
+    data = IO.__send__(IO.respond_to?(:binread) ? :binread : :read, name)
+    clean_data(data, flag)
   end
 
-  it "correctly diffs 'xx' and 'xaxb'" do
-    left = 'xx'
-    right = 'xaxb'
-    expect(Diff::LCS.patch(left, Diff::LCS.diff(left, right))).to eq(right)
+  def clean_data(data, flag)
+    data =
+      case flag
+      when '-c', '-u'
+        clean_output_timestamp(data)
+      else
+        data
+      end
+    data.gsub(/\r\n?/, "\n")
   end
 
-  it "returns an empty diff with (hello, hello)" do
-    expect(Diff::LCS.diff(hello, hello)).to eq([])
+  def clean_output_timestamp(data)
+    data.gsub(
+      %r{
+        ^
+        [-+*]{3}
+        \s*
+        spec/fixtures/(\S+)
+        \s*
+        \d{4}-\d\d-\d\d
+        \s*
+        \d\d:\d\d:\d\d(?:\.\d+)
+        \s*
+        (?:[-+]\d{4}|Z)
+      }x,
+      '*** spec/fixtures/\1	0000-00-00 :00 =>:00 =>00.000000000 -0000'
+    )
   end
 
-  it "returns an empty diff with (hello_ary, hello_ary)" do
-    expect(Diff::LCS.diff(hello_ary, hello_ary)).to eq([])
+  def run_ldiff(options)
+    flag = options.fetch(:flag)
+    left = options.fetch(:left)
+    right = options.fetch(:right)
+
+    stdout, stderr = capture_subprocess_io do
+      system("ruby -Ilib bin/ldiff #{flag} spec/fixtures/#{left} spec/fixtures/#{right}")
+    end
+
+    expect(stderr).to be_empty if RUBY_VERSION >= '1.9'
+    expect(stdout).not_to be_empty
+    clean_data(stdout, flag)
   end
 end

data/spec/patch_spec.rb

@@ -1,54 +1,54 @@
-# -*- ruby encoding: utf-8 -*-
+# frozen_string_literal: true
 
 require 'spec_helper'
 
-describe "Diff::LCS.patch" do
+describe 'Diff::LCS.patch' do
   include Diff::LCS::SpecHelper::Matchers
 
-  shared_examples "patch sequences correctly" do
-    it "correctly patches left-to-right (patch autodiscovery)" do
+  shared_examples 'patch sequences correctly' do
+    it 'correctly patches left-to-right (patch autodiscovery)' do
       expect(Diff::LCS.patch(s1, patch_set)).to eq(s2)
     end
 
-    it "correctly patches left-to-right (explicit patch)" do
+    it 'correctly patches left-to-right (explicit patch)' do
       expect(Diff::LCS.patch(s1, patch_set, :patch)).to eq(s2)
       expect(Diff::LCS.patch!(s1, patch_set)).to eq(s2)
     end
 
-    it "correctly patches right-to-left (unpatch autodiscovery)" do
+    it 'correctly patches right-to-left (unpatch autodiscovery)' do
       expect(Diff::LCS.patch(s2, patch_set)).to eq(s1)
     end
 
-    it "correctly patches right-to-left (explicit unpatch)" do
+    it 'correctly patches right-to-left (explicit unpatch)' do
       expect(Diff::LCS.patch(s2, patch_set, :unpatch)).to eq(s1)
       expect(Diff::LCS.unpatch!(s2, patch_set)).to eq(s1)
     end
   end
 
-  describe "using a Diff::LCS.diff patchset" do
-    describe "an empty patchset returns the source" do
-      it "works on a string (hello)" do
+  describe 'using a Diff::LCS.diff patchset' do
+    describe 'an empty patchset returns the source' do
+      it 'works on a string (hello)' do
         diff = Diff::LCS.diff(hello, hello)
-        expect(Diff::LCS::patch(hello, diff)).to eq(hello)
+        expect(Diff::LCS.patch(hello, diff)).to eq(hello)
       end
 
-      it "works on an array %W(h e l l o)" do
+      it 'works on an array %W(h e l l o)' do
         diff = Diff::LCS.diff(hello_ary, hello_ary)
-        expect(Diff::LCS::patch(hello_ary, diff)).to eq(hello_ary)
+        expect(Diff::LCS.patch(hello_ary, diff)).to eq(hello_ary)
       end
     end
 
-    describe "with default diff callbacks (DiffCallbacks)" do
-      describe "forward (s1 -> s2)" do
-        it_has_behavior "patch sequences correctly" do
+    describe 'with default diff callbacks (DiffCallbacks)' do
+      describe 'forward (s1 -> s2)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq1 }
           let(:s2) { seq2 }
           let(:patch_set) { Diff::LCS.diff(seq1, seq2) }
         end
       end
 
-      describe "reverse (s2 -> s1)" do
-        it_has_behavior "patch sequences correctly" do
+      describe 'reverse (s2 -> s1)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq2 }
           let(:s2) { seq1 }
           let(:patch_set) { Diff::LCS.diff(seq2, seq1) }
@@ -56,9 +56,9 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "with context diff callbacks (ContextDiffCallbacks)" do
-      describe "forward (s1 -> s2)" do
-        it_has_behavior "patch sequences correctly" do
+    describe 'with context diff callbacks (ContextDiffCallbacks)' do
+      describe 'forward (s1 -> s2)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq1 }
           let(:s2) { seq2 }
           let(:patch_set) {
@@ -67,8 +67,8 @@ describe "Diff::LCS.patch" do
         end
       end
 
-      describe "reverse (s2 -> s1)" do
-        it_has_behavior "patch sequences correctly" do
+      describe 'reverse (s2 -> s1)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq2 }
           let(:s2) { seq1 }
           let(:patch_set) {
@@ -78,9 +78,9 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "with sdiff callbacks (SDiffCallbacks)" do
-      describe "forward (s1 -> s2)" do
-        it_has_behavior "patch sequences correctly" do
+    describe 'with sdiff callbacks (SDiffCallbacks)' do
+      describe 'forward (s1 -> s2)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq1 }
           let(:s2) { seq2 }
           let(:patch_set) {
@@ -89,8 +89,8 @@ describe "Diff::LCS.patch" do
         end
       end
 
-      describe "reverse (s2 -> s1)" do
-        it_has_behavior "patch sequences correctly" do
+      describe 'reverse (s2 -> s1)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq2 }
           let(:s2) { seq1 }
           let(:patch_set) {
@@ -101,20 +101,20 @@ describe "Diff::LCS.patch" do
     end
   end
 
-  describe "using a Diff::LCS.sdiff patchset" do
-    describe "an empty patchset returns the source" do
-      it "works on a string (hello)" do
-        expect(Diff::LCS::patch(hello, Diff::LCS.sdiff(hello, hello))).to eq(hello)
+  describe 'using a Diff::LCS.sdiff patchset' do
+    describe 'an empty patchset returns the source' do
+      it 'works on a string (hello)' do
+        expect(Diff::LCS.patch(hello, Diff::LCS.sdiff(hello, hello))).to eq(hello)
       end
 
-      it "works on an array %W(h e l l o)" do
-        expect(Diff::LCS::patch(hello_ary, Diff::LCS.sdiff(hello_ary, hello_ary))).to eq(hello_ary)
+      it 'works on an array %W(h e l l o)' do
+        expect(Diff::LCS.patch(hello_ary, Diff::LCS.sdiff(hello_ary, hello_ary))).to eq(hello_ary)
       end
     end
 
-    describe "with default diff callbacks (DiffCallbacks)" do
-      describe "forward (s1 -> s2)" do
-        it_has_behavior "patch sequences correctly" do
+    describe 'with default diff callbacks (DiffCallbacks)' do
+      describe 'forward (s1 -> s2)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq1 }
           let(:s2) { seq2 }
           let(:patch_set) {
@@ -123,8 +123,8 @@ describe "Diff::LCS.patch" do
         end
       end
 
-      describe "reverse (s2 -> s1)" do
-        it_has_behavior "patch sequences correctly" do
+      describe 'reverse (s2 -> s1)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq2 }
           let(:s2) { seq1 }
           let(:patch_set) {
@@ -134,9 +134,9 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "with context diff callbacks (DiffCallbacks)" do
-      describe "forward (s1 -> s2)" do
-        it_has_behavior "patch sequences correctly" do
+    describe 'with context diff callbacks (DiffCallbacks)' do
+      describe 'forward (s1 -> s2)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq1 }
           let(:s2) { seq2 }
           let(:patch_set) {
@@ -145,8 +145,8 @@ describe "Diff::LCS.patch" do
         end
       end
 
-      describe "reverse (s2 -> s1)" do
-        it_has_behavior "patch sequences correctly" do
+      describe 'reverse (s2 -> s1)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq2 }
           let(:s2) { seq1 }
           let(:patch_set) {
@@ -156,17 +156,17 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "with sdiff callbacks (SDiffCallbacks)" do
-      describe "forward (s1 -> s2)" do
-        it_has_behavior "patch sequences correctly" do
+    describe 'with sdiff callbacks (SDiffCallbacks)' do
+      describe 'forward (s1 -> s2)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq1 }
           let(:s2) { seq2 }
           let(:patch_set) { Diff::LCS.sdiff(seq1, seq2) }
         end
       end
 
-      describe "reverse (s2 -> s1)" do
-        it_has_behavior "patch sequences correctly" do
+      describe 'reverse (s2 -> s1)' do
+        it_has_behavior 'patch sequences correctly' do
           let(:s1) { seq2 }
           let(:s2) { seq1 }
           let(:patch_set) { Diff::LCS.sdiff(seq2, seq1) }
@@ -179,43 +179,43 @@ describe "Diff::LCS.patch" do
   # to s2 patches"), this cannot use the "patch sequences correctly" shared
   # set. Once the bug in autodiscovery is fixed, this can be converted as
   # above.
-  describe "fix bug 891: patchsets do not contain the last equal part" do
+  describe 'fix bug 891: patchsets do not contain the last equal part' do
     before :each do
-      @s1 = %w(a b c d   e f g h i j k)
+      @s1 = %w(a b c d   e f g h i j k) # rubocop:disable Layout/SpaceInsideArrayPercentLiteral
       @s2 = %w(a b c d D e f g h i j k)
     end
 
-    describe "using Diff::LCS.diff with default diff callbacks" do
+    describe 'using Diff::LCS.diff with default diff callbacks' do
       before :each do
         @patch_set_s1_s2 = Diff::LCS.diff(@s1, @s2)
         @patch_set_s2_s1 = Diff::LCS.diff(@s2, @s1)
       end
 
-      it "autodiscovers s1 to s2 patches" do
+      it 'autodiscovers s1 to s2 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s1_s2)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 patches" do
+      it 'autodiscovers s2 to s1 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s2_s1)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 the left-to-right patches" do
+      it 'autodiscovers s2 to s1 the left-to-right patches' do
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1)).to eq(@s1)
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2)).to eq(@s1)
       end
 
-      it "correctly patches left-to-right (explicit patch)" do
+      it 'correctly patches left-to-right (explicit patch)' do
         expect(Diff::LCS.patch(@s1, @patch_set_s1_s2, :patch)).to eq(@s2)
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1, :patch)).to eq(@s1)
         expect(Diff::LCS.patch!(@s1, @patch_set_s1_s2)).to eq(@s2)
         expect(Diff::LCS.patch!(@s2, @patch_set_s2_s1)).to eq(@s1)
       end
 
-      it "correctly patches right-to-left (explicit unpatch)" do
+      it 'correctly patches right-to-left (explicit unpatch)' do
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2, :unpatch)).to eq(@s1)
         expect(Diff::LCS.patch(@s1, @patch_set_s2_s1, :unpatch)).to eq(@s2)
         expect(Diff::LCS.unpatch!(@s2, @patch_set_s1_s2)).to eq(@s1)
@@ -223,39 +223,37 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "using Diff::LCS.diff with context diff callbacks" do
+    describe 'using Diff::LCS.diff with context diff callbacks' do
       before :each do
-        @patch_set_s1_s2 = Diff::LCS.diff(@s1, @s2,
-                                          Diff::LCS::ContextDiffCallbacks)
-        @patch_set_s2_s1 = Diff::LCS.diff(@s2, @s1,
-                                          Diff::LCS::ContextDiffCallbacks)
+        @patch_set_s1_s2 = Diff::LCS.diff(@s1, @s2, Diff::LCS::ContextDiffCallbacks)
+        @patch_set_s2_s1 = Diff::LCS.diff(@s2, @s1, Diff::LCS::ContextDiffCallbacks)
       end
 
-      it "autodiscovers s1 to s2 patches" do
+      it 'autodiscovers s1 to s2 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s1_s2)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 patches" do
+      it 'autodiscovers s2 to s1 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s2_s1)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 the left-to-right patches" do
+      it 'autodiscovers s2 to s1 the left-to-right patches' do
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1)).to eq(@s1)
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2)).to eq(@s1)
       end
 
-      it "correctly patches left-to-right (explicit patch)" do
+      it 'correctly patches left-to-right (explicit patch)' do
         expect(Diff::LCS.patch(@s1, @patch_set_s1_s2, :patch)).to eq(@s2)
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1, :patch)).to eq(@s1)
         expect(Diff::LCS.patch!(@s1, @patch_set_s1_s2)).to eq(@s2)
         expect(Diff::LCS.patch!(@s2, @patch_set_s2_s1)).to eq(@s1)
       end
 
-      it "correctly patches right-to-left (explicit unpatch)" do
+      it 'correctly patches right-to-left (explicit unpatch)' do
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2, :unpatch)).to eq(@s1)
         expect(Diff::LCS.patch(@s1, @patch_set_s2_s1, :unpatch)).to eq(@s2)
         expect(Diff::LCS.unpatch!(@s2, @patch_set_s1_s2)).to eq(@s1)
@@ -263,39 +261,37 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "using Diff::LCS.diff with sdiff callbacks" do
+    describe 'using Diff::LCS.diff with sdiff callbacks' do
       before(:each) do
-        @patch_set_s1_s2 = Diff::LCS.diff(@s1, @s2,
-                                          Diff::LCS::SDiffCallbacks)
-        @patch_set_s2_s1 = Diff::LCS.diff(@s2, @s1,
-                                          Diff::LCS::SDiffCallbacks)
+        @patch_set_s1_s2 = Diff::LCS.diff(@s1, @s2, Diff::LCS::SDiffCallbacks)
+        @patch_set_s2_s1 = Diff::LCS.diff(@s2, @s1, Diff::LCS::SDiffCallbacks)
       end
 
-      it "autodiscovers s1 to s2 patches" do
+      it 'autodiscovers s1 to s2 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s1_s2)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 patches" do
+      it 'autodiscovers s2 to s1 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s2_s1)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 the left-to-right patches" do
+      it 'autodiscovers s2 to s1 the left-to-right patches' do
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1)).to eq(@s1)
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2)).to eq(@s1)
       end
 
-      it "correctly patches left-to-right (explicit patch)" do
+      it 'correctly patches left-to-right (explicit patch)' do
         expect(Diff::LCS.patch(@s1, @patch_set_s1_s2, :patch)).to eq(@s2)
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1, :patch)).to eq(@s1)
         expect(Diff::LCS.patch!(@s1, @patch_set_s1_s2)).to eq(@s2)
         expect(Diff::LCS.patch!(@s2, @patch_set_s2_s1)).to eq(@s1)
       end
 
-      it "correctly patches right-to-left (explicit unpatch)" do
+      it 'correctly patches right-to-left (explicit unpatch)' do
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2, :unpatch)).to eq(@s1)
         expect(Diff::LCS.patch(@s1, @patch_set_s2_s1, :unpatch)).to eq(@s2)
         expect(Diff::LCS.unpatch!(@s2, @patch_set_s1_s2)).to eq(@s1)
@@ -303,37 +299,37 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "using Diff::LCS.sdiff with default sdiff callbacks" do
+    describe 'using Diff::LCS.sdiff with default sdiff callbacks' do
       before(:each) do
         @patch_set_s1_s2 = Diff::LCS.sdiff(@s1, @s2)
         @patch_set_s2_s1 = Diff::LCS.sdiff(@s2, @s1)
       end
 
-      it "autodiscovers s1 to s2 patches" do
+      it 'autodiscovers s1 to s2 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s1_s2)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 patches" do
+      it 'autodiscovers s2 to s1 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s2_s1)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 the left-to-right patches" do
+      it 'autodiscovers s2 to s1 the left-to-right patches' do
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1)).to eq(@s1)
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2)).to eq(@s1)
       end
 
-      it "correctly patches left-to-right (explicit patch)" do
+      it 'correctly patches left-to-right (explicit patch)' do
         expect(Diff::LCS.patch(@s1, @patch_set_s1_s2, :patch)).to eq(@s2)
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1, :patch)).to eq(@s1)
         expect(Diff::LCS.patch!(@s1, @patch_set_s1_s2)).to eq(@s2)
         expect(Diff::LCS.patch!(@s2, @patch_set_s2_s1)).to eq(@s1)
       end
 
-      it "correctly patches right-to-left (explicit unpatch)" do
+      it 'correctly patches right-to-left (explicit unpatch)' do
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2, :unpatch)).to eq(@s1)
         expect(Diff::LCS.patch(@s1, @patch_set_s2_s1, :unpatch)).to eq(@s2)
         expect(Diff::LCS.unpatch!(@s2, @patch_set_s1_s2)).to eq(@s1)
@@ -341,39 +337,37 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "using Diff::LCS.sdiff with context diff callbacks" do
+    describe 'using Diff::LCS.sdiff with context diff callbacks' do
       before(:each) do
-        @patch_set_s1_s2 = Diff::LCS.sdiff(@s1, @s2,
-                                           Diff::LCS::ContextDiffCallbacks)
-        @patch_set_s2_s1 = Diff::LCS.sdiff(@s2, @s1,
-                                           Diff::LCS::ContextDiffCallbacks)
+        @patch_set_s1_s2 = Diff::LCS.sdiff(@s1, @s2, Diff::LCS::ContextDiffCallbacks)
+        @patch_set_s2_s1 = Diff::LCS.sdiff(@s2, @s1, Diff::LCS::ContextDiffCallbacks)
       end
 
-      it "autodiscovers s1 to s2 patches" do
+      it 'autodiscovers s1 to s2 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s1_s2)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 patches" do
+      it 'autodiscovers s2 to s1 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s2_s1)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 the left-to-right patches" do
+      it 'autodiscovers s2 to s1 the left-to-right patches' do
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1)).to eq(@s1)
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2)).to eq(@s1)
       end
 
-      it "correctly patches left-to-right (explicit patch)" do
+      it 'correctly patches left-to-right (explicit patch)' do
         expect(Diff::LCS.patch(@s1, @patch_set_s1_s2, :patch)).to eq(@s2)
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1, :patch)).to eq(@s1)
         expect(Diff::LCS.patch!(@s1, @patch_set_s1_s2)).to eq(@s2)
         expect(Diff::LCS.patch!(@s2, @patch_set_s2_s1)).to eq(@s1)
       end
 
-      it "correctly patches right-to-left (explicit unpatch)" do
+      it 'correctly patches right-to-left (explicit unpatch)' do
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2, :unpatch)).to eq(@s1)
         expect(Diff::LCS.patch(@s1, @patch_set_s2_s1, :unpatch)).to eq(@s2)
         expect(Diff::LCS.unpatch!(@s2, @patch_set_s1_s2)).to eq(@s1)
@@ -381,37 +375,37 @@ describe "Diff::LCS.patch" do
       end
     end
 
-    describe "using Diff::LCS.sdiff with default diff callbacks" do
+    describe 'using Diff::LCS.sdiff with default diff callbacks' do
       before(:each) do
         @patch_set_s1_s2 = Diff::LCS.sdiff(@s1, @s2, Diff::LCS::DiffCallbacks)
         @patch_set_s2_s1 = Diff::LCS.sdiff(@s2, @s1, Diff::LCS::DiffCallbacks)
       end
 
-      it "autodiscovers s1 to s2 patches" do
+      it 'autodiscovers s1 to s2 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s1_s2)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 patches" do
+      it 'autodiscovers s2 to s1 patches' do
         expect do
           expect(Diff::LCS.patch(@s1, @patch_set_s2_s1)).to eq(@s2)
         end.to_not raise_error
       end
 
-      it "autodiscovers s2 to s1 the left-to-right patches" do
+      it 'autodiscovers s2 to s1 the left-to-right patches' do
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1)).to eq(@s1)
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2)).to eq(@s1)
       end
 
-      it "correctly patches left-to-right (explicit patch)" do
+      it 'correctly patches left-to-right (explicit patch)' do
         expect(Diff::LCS.patch(@s1, @patch_set_s1_s2, :patch)).to eq(@s2)
         expect(Diff::LCS.patch(@s2, @patch_set_s2_s1, :patch)).to eq(@s1)
         expect(Diff::LCS.patch!(@s1, @patch_set_s1_s2)).to eq(@s2)
         expect(Diff::LCS.patch!(@s2, @patch_set_s2_s1)).to eq(@s1)
       end
 
-      it "correctly patches right-to-left (explicit unpatch)" do
+      it 'correctly patches right-to-left (explicit unpatch)' do
         expect(Diff::LCS.patch(@s2, @patch_set_s1_s2, :unpatch)).to eq(@s1)
         expect(Diff::LCS.patch(@s1, @patch_set_s2_s1, :unpatch)).to eq(@s2)
         expect(Diff::LCS.unpatch!(@s2, @patch_set_s1_s2)).to eq(@s1)