diff-lcs 1.5.1 → 1.6.1

data/spec/fixtures/ldiff/output.diff.missing_new_line1-u

@@ -0,0 +1,9 @@
+--- spec/fixtures/four_lines	2025-01-31 12:17:43.926013315 +0100
++++ spec/fixtures/four_lines_with_missing_new_line	2025-01-31 12:17:43.926013315 +0100
+@@ -1,4 +1,4 @@
+ one
+ two
+ three
+-four
++four
+

data/spec/fixtures/ldiff/output.diff.missing_new_line2

@@ -0,0 +1,5 @@
+4c4
+< four
+
+---
+> four

data/spec/fixtures/ldiff/output.diff.missing_new_line2-c

@@ -0,0 +1,14 @@
+*** spec/fixtures/four_lines_with_missing_new_line	2025-01-31 12:17:43.926013315 +0100
+--- spec/fixtures/four_lines	2025-01-31 12:17:43.926013315 +0100
+***************
+*** 1,4 ****
+  one
+  two
+  three
+! four
+
+--- 1,4 ----
+  one
+  two
+  three
+! four

data/spec/fixtures/ldiff/output.diff.missing_new_line2-u

@@ -0,0 +1,9 @@
+--- spec/fixtures/four_lines_with_missing_new_line	2025-01-31 12:17:43.926013315 +0100
++++ spec/fixtures/four_lines	2025-01-31 12:17:43.926013315 +0100
+@@ -1,4 +1,4 @@
+ one
+ two
+ three
+-four
+
++four

data/spec/hunk_spec.rb

@@ -72,7 +72,7 @@ if String.method_defined?(:encoding)
 
       it "produces a unified diff" do
         expected = <<-EXPECTED.gsub(/^\s+/, "").encode("UTF-16LE").chomp
-          @@ -1 +1,2 @@
+          @@ -0,0 +1 @@
           +Tu a un carte avec {count} items
         EXPECTED
 

data/spec/issues_spec.rb

@@ -68,38 +68,6 @@ describe "Diff::LCS Issues" do
     end
   end
 
-  describe "issue #60" do
-    it "should produce unified output with correct context" do
-      # standard:disable Layout/HeredocIndentation
-      old_data = <<-DATA_OLD.strip.split("\n").map(&:chomp)
-{
-  "name": "x",
-  "description": "hi"
-}
-      DATA_OLD
-
-      new_data = <<-DATA_NEW.strip.split("\n").map(&:chomp)
-{
-  "name": "x",
-  "description": "lo"
-}
-      DATA_NEW
-
-      diff = ::Diff::LCS.diff(old_data, new_data)
-      hunk = ::Diff::LCS::Hunk.new(old_data, new_data, diff.first, 3, 0)
-
-      expect(hunk.diff(:unified)).to eq(<<-EXPECTED.chomp)
-@@ -1,5 +1,5 @@
- {
-   "name": "x",
--  "description": "hi"
-+  "description": "lo"
- }
-      EXPECTED
-      # standard:enable Layout/HeredocIndentation
-    end
-  end
-
   describe "issue #65" do
     def diff_lines(old_lines, new_lines)
       file_length_difference = 0
@@ -157,4 +125,36 @@ describe "Diff::LCS Issues" do
       # standard:enable Layout/HeredocIndentation
     end
   end
+
+  describe "issue #107 (replaces issue #60)" do
+    it "should produce unified output with correct context" do
+      # standard:disable Layout/HeredocIndentation
+      old_data = <<-DATA_OLD.strip.split("\n").map(&:chomp)
+{
+  "name": "x",
+  "description": "hi"
+}
+      DATA_OLD
+
+      new_data = <<-DATA_NEW.strip.split("\n").map(&:chomp)
+{
+  "name": "x",
+  "description": "lo"
+}
+      DATA_NEW
+
+      diff = ::Diff::LCS.diff(old_data, new_data)
+      hunk = ::Diff::LCS::Hunk.new(old_data, new_data, diff.first, 3, 0)
+
+      expect(hunk.diff(:unified)).to eq(<<-EXPECTED.chomp)
+@@ -1,4 +1,4 @@
+ {
+   "name": "x",
+-  "description": "hi"
++  "description": "lo"
+ }
+      EXPECTED
+      # standard:enable Layout/HeredocIndentation
+    end
+  end
 end

data/spec/ldiff_spec.rb

@@ -7,9 +7,16 @@ RSpec.describe "bin/ldiff" do
 
   # standard:disable Style/HashSyntax
   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"}
+    {:name => "diff", :left => "aX", :right => "bXaX", :diff => 1},
+    {:name => "diff.missing_new_line1", :left => "four_lines", :right => "four_lines_with_missing_new_line", :diff => 1},
+    {:name => "diff.missing_new_line2", :left => "four_lines_with_missing_new_line", :right => "four_lines", :diff => 1},
+    {:name => "diff.issue95_trailing_context", :left => "123_x", :right => "456_x", :diff => 1},
+    {:name => "diff.four_lines.vs.empty", :left => "four_lines", :right => "empty", :diff => 1},
+    {:name => "diff.empty.vs.four_lines", :left => "empty", :right => "four_lines", :diff => 1},
+    {:name => "diff.bin1", :left => "file1.bin", :right => "file1.bin", :diff => 0},
+    {:name => "diff.bin2", :left => "file1.bin", :right => "file2.bin", :diff => 1},
+    {:name => "diff.chef", :left => "old-chef", :right => "new-chef", :diff => 1},
+    {:name => "diff.chef2", :left => "old-chef2", :right => "new-chef2", :diff => 1}
   ].product([nil, "-e", "-f", "-c", "-u"]).map { |(fixture, flag)|
     fixture = fixture.dup
     fixture[:flag] = flag
@@ -24,11 +31,14 @@ RSpec.describe "bin/ldiff" do
       "spec/fixtures/#{fixture[:right]}",
       "#",
       "=>",
-      "spec/fixtures/ldiff/#{fixture[:name]}#{fixture[:flag]}"
+      "spec/fixtures/ldiff/output.#{fixture[:name]}#{fixture[:flag]}"
     ].join(" ")
 
     it desc do
-      expect(run_ldiff(fixture)).to eq(read_fixture(fixture))
+      stdout, stderr, status = run_ldiff(fixture)
+      expect(status).to eq(fixture[:diff])
+      expect(stderr).to eq(read_fixture(fixture, mode: "error", allow_missing: true))
+      expect(stdout).to eq(read_fixture(fixture, mode: "output", allow_missing: false))
     end
   end
 
@@ -36,10 +46,13 @@ RSpec.describe "bin/ldiff" do
     test_ldiff(fixture)
   end
 
-  def read_fixture(options)
+  def read_fixture(options, mode: "output", allow_missing: false)
     fixture = options.fetch(:name)
     flag = options.fetch(:flag)
-    name = "spec/fixtures/ldiff/#{fixture}#{flag}"
+    name = "spec/fixtures/ldiff/#{mode}.#{fixture}#{flag}"
+
+    return "" if !::File.exist?(name) && allow_missing
+
     data = IO.__send__(IO.respond_to?(:binread) ? :binread : :read, name)
     clean_data(data, flag)
   end
@@ -82,8 +95,6 @@ RSpec.describe "bin/ldiff" 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)
+    [clean_data(stdout, flag), stderr, $?.exitstatus]
   end
 end

data/spec/patch_spec.rb

@@ -181,7 +181,7 @@ describe "Diff::LCS.patch" do
   # above.
   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] # rubocop:disable Layout/SpaceInsideArrayPercentLiteral
+      @s1 = %w[a b c d   e f g h i j k] # standard:disable Layout/SpaceInsideArrayPercentLiteral
       @s2 = %w[a b c d D e f g h i j k]
     end
 

data/spec/spec_helper.rb

@@ -7,28 +7,21 @@ require "psych" if RUBY_VERSION >= "1.9"
 
 if ENV["COVERAGE"]
   require "simplecov"
+  require "simplecov-lcov"
 
-  def require_do(resource)
-    require resource
-    yield if block_given?
-  rescue LoadError
-    nil
+  SimpleCov::Formatter::LcovFormatter.config do |config|
+    config.report_with_single_file = true
+    config.lcov_file_name = "lcov.info"
   end
 
-  formatters = [SimpleCov::Formatter::HTMLFormatter]
-
-  require_do("simplecov-rcov") {
-    formatters << SimpleCov::Formatter::RcovFormatter
-  }
-  require_do("simplecov-vim/formatter") {
-    formatters << SimpleCov::Formatter::VimFormatter
-  }
-  require_do("simplecov-sublime-ruby-coverage") {
-    formatters << SimpleCov::Formatter::SublimeRubyCoverageFormatter
-  }
-
-  SimpleCov.start do
-    formatter SimpleCov::Formatter::MultiFormatter.new(formatters)
+  SimpleCov.start "test_frameworks" do
+    enable_coverage :branch
+    primary_coverage :branch
+    formatter SimpleCov::Formatter::MultiFormatter.new([
+      SimpleCov::Formatter::HTMLFormatter,
+      SimpleCov::Formatter::LcovFormatter,
+      SimpleCov::Formatter::SimpleFormatter
+    ])
   end
 end
 
@@ -250,100 +243,107 @@ module Diff::LCS::SpecHelper
     new_result
   end
 
-  def simple_callback
-    callbacks = Object.new
-    class << callbacks
-      attr_reader :matched_a
-      attr_reader :matched_b
-      attr_reader :discards_a
-      attr_reader :discards_b
-      attr_reader :done_a
-      attr_reader :done_b
-
-      def reset
-        @matched_a = []
-        @matched_b = []
-        @discards_a = []
-        @discards_b = []
-        @done_a = []
-        @done_b = []
-      end
+  class SimpleCallback
+    def initialize
+      reset
+    end
 
-      def match(event)
-        @matched_a << event.old_element
-        @matched_b << event.new_element
-      end
+    attr_reader :matched_a
+    attr_reader :matched_b
+    attr_reader :discards_a
+    attr_reader :discards_b
+    attr_reader :done_a
+    attr_reader :done_b
+
+    def reset
+      @matched_a = []
+      @matched_b = []
+      @discards_a = []
+      @discards_b = []
+      @done_a = []
+      @done_b = []
+      self
+    end
 
-      def discard_b(event)
-        @discards_b << event.new_element
-      end
+    def match(event)
+      @matched_a << event.old_element
+      @matched_b << event.new_element
+    end
 
-      def discard_a(event)
-        @discards_a << event.old_element
-      end
+    def discard_b(event)
+      @discards_b << event.new_element
+    end
 
-      def finished_a(event)
-        @done_a << [
-          event.old_element, event.old_position,
-          event.new_element, event.new_position
-        ]
-      end
+    def discard_a(event)
+      @discards_a << event.old_element
+    end
 
-      def finished_b(event)
-        @done_b << [
-          event.old_element, event.old_position,
-          event.new_element, event.new_position
-        ]
-      end
+    def finished_a(event)
+      @done_a << [
+        event.old_element, event.old_position,
+        event.new_element, event.new_position
+      ]
+    end
+
+    def finished_b(event)
+      @done_b << [
+        event.old_element, event.old_position,
+        event.new_element, event.new_position
+      ]
     end
-    callbacks.reset
-    callbacks
+  end
+
+  def simple_callback
+    SimpleCallback.new
+  end
+
+  class SimpleCallbackNoFinishers < SimpleCallback
+    undef :finished_a
+    undef :finished_b
   end
 
   def simple_callback_no_finishers
-    simple = simple_callback
-    class << simple
-      undef :finished_a
-      undef :finished_b
-    end
-    simple
+    SimpleCallbackNoFinishers.new
   end
 
-  def balanced_callback
-    cb = Object.new
-    class << cb
-      attr_reader :result
+  class BalancedCallback
+    def initialize
+      reset
+    end
 
-      def reset
-        @result = []
-      end
+    attr_reader :result
 
-      def match(event)
-        @result << ["=", event.old_position, event.new_position]
-      end
+    def reset
+      @result = []
+    end
 
-      def discard_a(event)
-        @result << ["<", event.old_position, event.new_position]
-      end
+    def match(event)
+      @result << ["=", event.old_position, event.new_position]
+    end
 
-      def discard_b(event)
-        @result << [">", event.old_position, event.new_position]
-      end
+    def discard_a(event)
+      @result << ["<", event.old_position, event.new_position]
+    end
 
-      def change(event)
-        @result << ["!", event.old_position, event.new_position]
-      end
+    def discard_b(event)
+      @result << [">", event.old_position, event.new_position]
     end
-    cb.reset
-    cb
+
+    def change(event)
+      @result << ["!", event.old_position, event.new_position]
+    end
+  end
+
+  def balanced_callback
+    BalancedCallback.new
+  end
+
+  class BalancedCallbackNoChange < BalancedCallback
+    undef :change
   end
 
   def balanced_callback_no_change
-    balanced = balanced_callback
-    class << balanced
-      undef :change
-    end
-    balanced
+    BalancedCallbackNoChange.new
   end
 
   module Matchers

metadata

@@ -1,77 +1,42 @@
 --- !ruby/object:Gem::Specification
 name: diff-lcs
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.6.1
 platform: ruby
 authors:
 - Austin Ziegler
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-31 00:00:00.000000000 Z
+date: 2025-03-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hoe
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '3.0'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '5'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '3.0'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '5'
-- !ruby/object:Gem::Dependency
-  name: hoe-doofus
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '4.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '4.0'
 - !ruby/object:Gem::Dependency
-  name: hoe-gemspec2
+  name: hoe-halostatue
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.1'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.1'
-- !ruby/object:Gem::Dependency
-  name: hoe-git2
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '2.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.7'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: hoe-rubygems
   requirement: !ruby/object:Gem::Requirement
@@ -152,14 +117,14 @@ description: |-
   to create a simple HTML diff output format and a standard diff-like tool.
 
   This is release 1.4.3, providing a simple extension that allows for
-  Diff::LCS::Change objects to be treated implicitly as arrays and fixes a
-  number of formatting issues.
+  Diff::LCS::Change objects to be treated implicitly as arrays and fixes a number
+  of formatting issues.
 
-  Ruby versions below 2.5 are soft-deprecated, which means that older versions
-  are no longer part of the CI test suite. If any changes have been introduced
-  that break those versions, bug reports and patches will be accepted, but it
-  will be up to the reporter to verify any fixes prior to release. The next
-  major release will completely break compatibility.
+  Ruby versions below 2.5 are soft-deprecated, which means that older versions are
+  no longer part of the CI test suite. If any changes have been introduced that
+  break those versions, bug reports and patches will be accepted, but it will be
+  up to the reporter to verify any fixes prior to release. The next major release
+  will completely break compatibility.
 email:
 - halostatue@gmail.com
 executables:
@@ -167,23 +132,27 @@ executables:
 - ldiff
 extensions: []
 extra_rdoc_files:
-- Code-of-Conduct.md
-- Contributing.md
-- History.md
-- License.md
+- CHANGELOG.md
+- CODE_OF_CONDUCT.md
+- CONTRIBUTING.md
+- CONTRIBUTORS.md
+- LICENCE.md
 - Manifest.txt
-- README.rdoc
+- README.md
+- SECURITY.md
 - docs/COPYING.txt
 - docs/artistic.txt
 files:
 - ".rspec"
-- Code-of-Conduct.md
-- Contributing.md
-- History.md
-- License.md
+- CHANGELOG.md
+- CODE_OF_CONDUCT.md
+- CONTRIBUTING.md
+- CONTRIBUTORS.md
+- LICENCE.md
 - Manifest.txt
-- README.rdoc
+- README.md
 - Rakefile
+- SECURITY.md
 - bin/htmldiff
 - bin/ldiff
 - docs/COPYING.txt
@@ -200,17 +169,46 @@ files:
 - lib/diff/lcs/internals.rb
 - lib/diff/lcs/ldiff.rb
 - lib/diff/lcs/string.rb
+- lib/diff/lcs/version.rb
+- mise.toml
 - spec/change_spec.rb
 - spec/diff_spec.rb
+- spec/fixtures/123_x
+- spec/fixtures/456_x
 - spec/fixtures/aX
 - spec/fixtures/bXaX
 - spec/fixtures/ds1.csv
 - spec/fixtures/ds2.csv
+- spec/fixtures/empty
+- spec/fixtures/file1.bin
+- spec/fixtures/file2.bin
+- spec/fixtures/four_lines
+- spec/fixtures/four_lines_with_missing_new_line
+- spec/fixtures/ldiff/diff.missing_new_line1-e
+- spec/fixtures/ldiff/diff.missing_new_line1-f
+- spec/fixtures/ldiff/diff.missing_new_line2-e
+- spec/fixtures/ldiff/diff.missing_new_line2-f
+- spec/fixtures/ldiff/error.diff.chef-e
+- spec/fixtures/ldiff/error.diff.chef-f
+- spec/fixtures/ldiff/error.diff.missing_new_line1-e
+- spec/fixtures/ldiff/error.diff.missing_new_line1-f
+- spec/fixtures/ldiff/error.diff.missing_new_line2-e
+- spec/fixtures/ldiff/error.diff.missing_new_line2-f
 - spec/fixtures/ldiff/output.diff
 - spec/fixtures/ldiff/output.diff-c
 - spec/fixtures/ldiff/output.diff-e
 - spec/fixtures/ldiff/output.diff-f
 - spec/fixtures/ldiff/output.diff-u
+- spec/fixtures/ldiff/output.diff.bin1
+- spec/fixtures/ldiff/output.diff.bin1-c
+- spec/fixtures/ldiff/output.diff.bin1-e
+- spec/fixtures/ldiff/output.diff.bin1-f
+- spec/fixtures/ldiff/output.diff.bin1-u
+- spec/fixtures/ldiff/output.diff.bin2
+- spec/fixtures/ldiff/output.diff.bin2-c
+- spec/fixtures/ldiff/output.diff.bin2-e
+- spec/fixtures/ldiff/output.diff.bin2-f
+- spec/fixtures/ldiff/output.diff.bin2-u
 - spec/fixtures/ldiff/output.diff.chef
 - spec/fixtures/ldiff/output.diff.chef-c
 - spec/fixtures/ldiff/output.diff.chef-e
@@ -222,6 +220,31 @@ files:
 - spec/fixtures/ldiff/output.diff.chef2-e
 - spec/fixtures/ldiff/output.diff.chef2-f
 - spec/fixtures/ldiff/output.diff.chef2-u
+- spec/fixtures/ldiff/output.diff.empty.vs.four_lines
+- spec/fixtures/ldiff/output.diff.empty.vs.four_lines-c
+- spec/fixtures/ldiff/output.diff.empty.vs.four_lines-e
+- spec/fixtures/ldiff/output.diff.empty.vs.four_lines-f
+- spec/fixtures/ldiff/output.diff.empty.vs.four_lines-u
+- spec/fixtures/ldiff/output.diff.four_lines.vs.empty
+- spec/fixtures/ldiff/output.diff.four_lines.vs.empty-c
+- spec/fixtures/ldiff/output.diff.four_lines.vs.empty-e
+- spec/fixtures/ldiff/output.diff.four_lines.vs.empty-f
+- spec/fixtures/ldiff/output.diff.four_lines.vs.empty-u
+- spec/fixtures/ldiff/output.diff.issue95_trailing_context
+- spec/fixtures/ldiff/output.diff.issue95_trailing_context-c
+- spec/fixtures/ldiff/output.diff.issue95_trailing_context-e
+- spec/fixtures/ldiff/output.diff.issue95_trailing_context-f
+- spec/fixtures/ldiff/output.diff.issue95_trailing_context-u
+- spec/fixtures/ldiff/output.diff.missing_new_line1
+- spec/fixtures/ldiff/output.diff.missing_new_line1-c
+- spec/fixtures/ldiff/output.diff.missing_new_line1-e
+- spec/fixtures/ldiff/output.diff.missing_new_line1-f
+- spec/fixtures/ldiff/output.diff.missing_new_line1-u
+- spec/fixtures/ldiff/output.diff.missing_new_line2
+- spec/fixtures/ldiff/output.diff.missing_new_line2-c
+- spec/fixtures/ldiff/output.diff.missing_new_line2-e
+- spec/fixtures/ldiff/output.diff.missing_new_line2-f
+- spec/fixtures/ldiff/output.diff.missing_new_line2-u
 - spec/fixtures/new-chef
 - spec/fixtures/new-chef2
 - spec/fixtures/old-chef
@@ -238,17 +261,17 @@ files:
 homepage: https://github.com/halostatue/diff-lcs
 licenses:
 - MIT
-- Artistic-2.0
+- Artistic-1.0-Perl
 - GPL-2.0-or-later
 metadata:
   homepage_uri: https://github.com/halostatue/diff-lcs
+  changelog_uri: https://github.com/halostatue/diff-lcs/blob/main/CHANGELOG.md
   source_code_uri: https://github.com/halostatue/diff-lcs
   bug_tracker_uri: https://github.com/halostatue/diff-lcs/issues
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options:
 - "--main"
-- README.rdoc
+- README.md
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
@@ -262,8 +285,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.3
-signing_key:
+rubygems_version: 3.6.6
 specification_version: 4
 summary: Diff::LCS computes the difference between two Enumerable sequences using
   the McIlroy-Hunt longest common subsequence (LCS) algorithm