Merge branch 'main' into rubocop

Author: jas14

CHANGELOG.md

@@ -6,6 +6,7 @@
 
 - Improve inspection of Module. [#263](https://github.com/splitwise/super_diff/pull/263) by [@phorsuedzie](https://github.com/phorsuedzie)
 - Fix multiline string diff with blank lines. [#266](https://github.com/splitwise/super_diff/pull/263)
+- Improve inspection of Range objects. [#267](https://github.com/splitwise/super_diff/pull/267)
 
 ### Other changes
 

lib/super_diff/basic.rb

@@ -29,6 +29,7 @@ module Basic
         InspectionTreeBuilders::TimeLike,
         InspectionTreeBuilders::DateLike,
         InspectionTreeBuilders::DataObject,
+        InspectionTreeBuilders::RangeObject,
         InspectionTreeBuilders::DefaultObject
       )
 

lib/super_diff/basic/inspection_tree_builders.rb

@@ -12,15 +12,19 @@ module InspectionTreeBuilders
         :DataObject,
         'super_diff/basic/inspection_tree_builders/data_object'
       )
+      autoload :DateLike, 'super_diff/basic/inspection_tree_builders/date_like'
       autoload(
         :DefaultObject,
         'super_diff/basic/inspection_tree_builders/default_object'
       )
       autoload :Hash, 'super_diff/basic/inspection_tree_builders/hash'
       autoload :Primitive, 'super_diff/basic/inspection_tree_builders/primitive'
+      autoload(
+        :RangeObject,
+        'super_diff/basic/inspection_tree_builders/range_object'
+      )
       autoload :String, 'super_diff/basic/inspection_tree_builders/string'
       autoload :TimeLike, 'super_diff/basic/inspection_tree_builders/time_like'
-      autoload :DateLike, 'super_diff/basic/inspection_tree_builders/date_like'
     end
   end
 end

lib/super_diff/basic/inspection_tree_builders/range_object.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module SuperDiff
+  module Basic
+    module InspectionTreeBuilders
+      class RangeObject < Core::AbstractInspectionTreeBuilder
+        def self.applies_to?(value)
+          value.is_a?(Range)
+        end
+
+        def call
+          Core::InspectionTree.new do |t1|
+            t1.as_lines_when_rendering_to_lines { |t2| t2.add_text object.to_s }
+          end
+        end
+      end
+    end
+  end
+end

spec/integration/rspec/eq_matcher_spec.rb

@@ -1058,6 +1058,37 @@
     end
   end
 
+  context 'when comparing ranges' do
+    it 'produces the correct failure message when used in the positive' do
+      as_both_colored_and_uncolored do |color_enabled|
+        snippet = 'expect(1..5).to eq(5..6)'
+        program = make_plain_test_program(snippet, color_enabled: color_enabled)
+
+        expected_output =
+          build_expected_output(
+            color_enabled: color_enabled,
+            snippet: snippet,
+            newline_before_expectation: true,
+            expectation:
+              proc do
+                line do
+                  plain 'Expected '
+                  actual '1..5'
+                  plain ' to eq '
+                  expected '5..6'
+                  plain '.'
+                end
+              end,
+            diff: nil
+          )
+
+        expect(program).to produce_output_when_run(expected_output).in_color(
+          color_enabled
+        )
+      end
+    end
+  end
+
   it_behaves_like 'a matcher that supports elided diffs' do
     let(:matcher) { :eq }
   end

spec/unit/basic/inspection_tree_builders/range_object_spec.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe SuperDiff, type: :unit do
+  describe '.inspect_object' do
+    context 'given as_lines: false' do
+      subject(:output) do
+        described_class.inspect_object(object, as_lines: false)
+      end
+
+      context 'given a simple range' do
+        let(:object) { 1..5 }
+
+        it 'shows the data' do
+          expect(output).to eq('1..5')
+        end
+      end
+    end
+  end
+end