super_diff 0.13.0 → 0.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 343438b938b014502f7af0df76fc22ff73f3eaada655081b994b8c4d77bd3618
-  data.tar.gz: 2738da41f517ecfcd1e068e03256cd37aa5436fd984856a2c958518596bd58bf
+  metadata.gz: 573a4d219dcad689aca11cffe1beaa7e3c4974958231b1cacd92c32ef0542d74
+  data.tar.gz: dfb59c966c00ec59df137eb45289c056f1dd58ec461d36913b49527294c99da9
 SHA512:
-  metadata.gz: 522481e293667b16bf0d2c752fc02ebce5c6b8c607d23d53b009d52eec34e9a0d429fa410cb8dc4802c5b9b3dffd20632f9671024e3ba009e591179a6cb155dc
-  data.tar.gz: c08edc9460956e59f391249f26b508c944797ceda4db80d36b2119415322ef00210cd57817e74f24b6d13cc3b869bf7de1ca50276cd75e01288b1733c4c48b43
+  metadata.gz: 333c9c71197fa48a68e0328ae1eed9703e96838e6afcef4bc33dcc68f34761ad8b694a8e768bffe86d11372d14c34704634176d35bab5d6584e042fb3eb99627
+  data.tar.gz: '09136f70b82bef9591e709fffb12023da3beb59d69a55d52f89171649e7df1e6bf618ba00bf6ad08ad404369da7b505cd5027a18917701e41737e07a48faa0c7'

data/README.md

@@ -134,7 +134,7 @@ Thank you to the authors of these libraries!
 [diff-lcs]: https://github.com/halostatue/diff-lcs
 [pretty-printer]: https://github.com/ruby/ruby/tree/master/lib/prettyprint.rb
 [awesome-print]: https://github.com/awesome-print/awesome_print
-[inspection-tree]: https://github.com/splitwise/super_diff/blob/main/lib/super_diff/object_inspection/inspection_tree.rb
+[inspection-tree]: https://github.com/splitwise/super_diff/blob/main/lib/super_diff/core/inspection_tree.rb
 
 ## Author/License
 

data/lib/super_diff/active_record/differs/active_record_relation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module Differs

data/lib/super_diff/active_record/differs.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module Differs
       autoload(
         :ActiveRecordRelation,
-        "super_diff/active_record/differs/active_record_relation"
+        'super_diff/active_record/differs/active_record_relation'
       )
     end
   end

data/lib/super_diff/active_record/inspection_tree_builders/active_record_model.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module InspectionTreeBuilders
@@ -19,8 +21,8 @@ module SuperDiff
 
               # stree-ignore
               t2.when_rendering_to_lines do |t3|
-                  t3.add_text "{"
-                end
+                t3.add_text '{'
+              end
             end
 
             t1.nested do |t2|
@@ -44,10 +46,10 @@ module SuperDiff
             ) do |t2|
               # stree-ignore
               t2.when_rendering_to_lines do |t3|
-                  t3.add_text "}"
-                end
+                t3.add_text '}'
+              end
 
-              t2.add_text ">"
+              t2.add_text '>'
             end
           end
         end

data/lib/super_diff/active_record/inspection_tree_builders/active_record_relation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module InspectionTreeBuilders
@@ -10,22 +12,22 @@ module SuperDiff
           Core::InspectionTree.new do |t1|
             # stree-ignore
             t1.as_lines_when_rendering_to_lines(
-                collection_bookend: :open
-              ) do |t2|
-                t2.add_text "#<ActiveRecord::Relation ["
-              end
+              collection_bookend: :open
+            ) do |t2|
+              t2.add_text '#<ActiveRecord::Relation ['
+            end
 
             # stree-ignore
             t1.nested do |t2|
-                t2.insert_array_inspection_of(object)
-              end
+              t2.insert_array_inspection_of(object)
+            end
 
             # stree-ignore
             t1.as_lines_when_rendering_to_lines(
-                collection_bookend: :close
-              ) do |t2|
-                t2.add_text "]>"
-              end
+              collection_bookend: :close
+            ) do |t2|
+              t2.add_text ']>'
+            end
           end
         end
       end

data/lib/super_diff/active_record/inspection_tree_builders.rb

@@ -1,13 +1,15 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module InspectionTreeBuilders
       autoload(
         :ActiveRecordModel,
-        "super_diff/active_record/inspection_tree_builders/active_record_model"
+        'super_diff/active_record/inspection_tree_builders/active_record_model'
       )
       autoload(
         :ActiveRecordRelation,
-        "super_diff/active_record/inspection_tree_builders/active_record_relation"
+        'super_diff/active_record/inspection_tree_builders/active_record_relation'
       )
     end
   end

data/lib/super_diff/active_record/monkey_patches.rb

@@ -1,4 +1,6 @@
-# rubocop:disable Style/BracesAroundHashParameters, Style/ClassAndModuleChildren
+# frozen_string_literal: true
+
+# rubocop:disable Style/ClassAndModuleChildren
 class ActiveRecord::Base
   # TODO: Remove this monkey patch if possible
   def attributes_for_super_diff
@@ -9,4 +11,4 @@ class ActiveRecord::Base
     ) { |hash, key| hash.merge(key.to_sym => attributes[key]) }
   end
 end
-# rubocop:enable Style/BracesAroundHashParameters, Style/ClassAndModuleChildren
+# rubocop:enable Style/ClassAndModuleChildren

data/lib/super_diff/active_record/object_inspection.rb

@@ -1,16 +1,18 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module ObjectInspection
       module InspectionTreeBuilders
         def self.const_missing(missing_const_name)
           if ActiveRecord::InspectionTreeBuilders.const_defined?(
-               missing_const_name
-             )
-            warn <<~EOT
+            missing_const_name
+          )
+            warn <<~WARNING
               WARNING: SuperDiff::ActiveRecord::ObjectInspection::InspectionTreeBuilders::#{missing_const_name} is deprecated and will be removed in the next major release.
               Please use SuperDiff::ActiveRecord::InspectionTreeBuilders::#{missing_const_name} instead.
               #{caller_locations.join("\n")}
-            EOT
+            WARNING
             ActiveRecord::InspectionTreeBuilders.const_get(missing_const_name)
           else
             super

data/lib/super_diff/active_record/operation_tree_builders/active_record_model.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module OperationTreeBuilders
       class ActiveRecordModel < Basic::OperationTreeBuilders::CustomObject
         def self.applies_to?(expected, actual)
           expected.is_a?(::ActiveRecord::Base) &&
-            actual.is_a?(::ActiveRecord::Base) && expected.class == actual.class
+            actual.is_a?(::ActiveRecord::Base) && expected.instance_of?(actual.class)
         end
 
         protected

data/lib/super_diff/active_record/operation_tree_builders/active_record_relation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module OperationTreeBuilders
@@ -15,7 +17,7 @@ module SuperDiff
         private
 
         def operation_tree
-          @_operation_tree ||= OperationTrees::ActiveRecordRelation.new([])
+          @operation_tree ||= OperationTrees::ActiveRecordRelation.new([])
         end
       end
     end

data/lib/super_diff/active_record/operation_tree_builders.rb

@@ -1,13 +1,15 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module OperationTreeBuilders
       autoload(
         :ActiveRecordModel,
-        "super_diff/active_record/operation_tree_builders/active_record_model"
+        'super_diff/active_record/operation_tree_builders/active_record_model'
       )
       autoload(
         :ActiveRecordRelation,
-        "super_diff/active_record/operation_tree_builders/active_record_relation"
+        'super_diff/active_record/operation_tree_builders/active_record_relation'
       )
     end
   end

data/lib/super_diff/active_record/operation_tree_flatteners/active_record_relation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module OperationTreeFlatteners
@@ -5,11 +7,11 @@ module SuperDiff
         protected
 
         def open_token
-          "#<ActiveRecord::Relation ["
+          '#<ActiveRecord::Relation ['
         end
 
         def close_token
-          "]>"
+          ']>'
         end
       end
     end

data/lib/super_diff/active_record/operation_tree_flatteners.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module OperationTreeFlatteners
       autoload(
         :ActiveRecordRelation,
-        "super_diff/active_record/operation_tree_flatteners/active_record_relation"
+        'super_diff/active_record/operation_tree_flatteners/active_record_relation'
       )
     end
   end

data/lib/super_diff/active_record/operation_trees/active_record_relation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module OperationTrees

data/lib/super_diff/active_record/operation_trees.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveRecord
     module OperationTrees
       autoload(
         :ActiveRecordRelation,
-        "super_diff/active_record/operation_trees/active_record_relation"
+        'super_diff/active_record/operation_trees/active_record_relation'
       )
     end
   end

data/lib/super_diff/active_record.rb

@@ -1,14 +1,16 @@
-require "super_diff/active_support"
+# frozen_string_literal: true
 
-require "super_diff/active_record/differs"
-require "super_diff/active_record/inspection_tree_builders"
-require "super_diff/active_record/operation_trees"
-require "super_diff/active_record/operation_tree_builders"
-require "super_diff/active_record/operation_tree_flatteners"
+require 'super_diff/active_support'
+
+require 'super_diff/active_record/differs'
+require 'super_diff/active_record/inspection_tree_builders'
+require 'super_diff/active_record/operation_trees'
+require 'super_diff/active_record/operation_tree_builders'
+require 'super_diff/active_record/operation_tree_flatteners'
 
 module SuperDiff
   module ActiveRecord
-    autoload :ObjectInspection, "super_diff/active_record/object_inspection"
+    autoload :ObjectInspection, 'super_diff/active_record/object_inspection'
 
     SuperDiff.configure do |config|
       config.prepend_extra_differ_classes(Differs::ActiveRecordRelation)
@@ -24,4 +26,4 @@ module SuperDiff
   end
 end
 
-require "super_diff/active_record/monkey_patches"
+require 'super_diff/active_record/monkey_patches'

data/lib/super_diff/active_support/differs/hash_with_indifferent_access.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module Differs

data/lib/super_diff/active_support/differs.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module Differs
       autoload(
         :HashWithIndifferentAccess,
-        "super_diff/active_support/differs/hash_with_indifferent_access"
+        'super_diff/active_support/differs/hash_with_indifferent_access'
       )
     end
   end

data/lib/super_diff/active_support/inspection_tree_builders/hash_with_indifferent_access.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module InspectionTreeBuilders
@@ -10,32 +12,32 @@ module SuperDiff
           Core::InspectionTree.new do |t1|
             # stree-ignore
             t1.as_lines_when_rendering_to_lines(
-                collection_bookend: :open
-              ) do |t2|
-                t2.add_text "#<HashWithIndifferentAccess {"
-              end
+              collection_bookend: :open
+            ) do |t2|
+              t2.add_text '#<HashWithIndifferentAccess {'
+            end
 
             # stree-ignore
             t1.when_rendering_to_string do |t2|
-                t2.add_text " "
-              end
+              t2.add_text ' '
+            end
 
             # stree-ignore
             t1.nested do |t2|
-                t2.insert_hash_inspection_of(object)
-              end
+              t2.insert_hash_inspection_of(object)
+            end
 
             # stree-ignore
             t1.when_rendering_to_string do |t2|
-                t2.add_text " "
-              end
+              t2.add_text ' '
+            end
 
             # stree-ignore
             t1.as_lines_when_rendering_to_lines(
-                collection_bookend: :close
-              ) do |t2|
-                t2.add_text "}>"
-              end
+              collection_bookend: :close
+            ) do |t2|
+              t2.add_text '}>'
+            end
           end
         end
       end

data/lib/super_diff/active_support/inspection_tree_builders/ordered_options.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module InspectionTreeBuilders
@@ -10,32 +12,32 @@ module SuperDiff
           Core::InspectionTree.new do |t1|
             # stree-ignore
             t1.as_lines_when_rendering_to_lines(
-                collection_bookend: :open
-              ) do |t2|
-                t2.add_text "#<OrderedOptions {"
-              end
+              collection_bookend: :open
+            ) do |t2|
+              t2.add_text '#<OrderedOptions {'
+            end
 
             # stree-ignore
             t1.when_rendering_to_string do |t2|
-                t2.add_text " "
-              end
+              t2.add_text ' '
+            end
 
             # stree-ignore
             t1.nested do |t2|
-                t2.insert_hash_inspection_of(object.to_hash)
-              end
+              t2.insert_hash_inspection_of(object.to_hash)
+            end
 
             # stree-ignore
             t1.when_rendering_to_string do |t2|
-                t2.add_text " "
-              end
+              t2.add_text ' '
+            end
 
             # stree-ignore
             t1.as_lines_when_rendering_to_lines(
-                collection_bookend: :close
-              ) do |t2|
-                t2.add_text "}>"
-              end
+              collection_bookend: :close
+            ) do |t2|
+              t2.add_text '}>'
+            end
           end
         end
       end

data/lib/super_diff/active_support/inspection_tree_builders.rb

@@ -1,13 +1,15 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module InspectionTreeBuilders
       autoload(
         :HashWithIndifferentAccess,
-        "super_diff/active_support/inspection_tree_builders/hash_with_indifferent_access"
+        'super_diff/active_support/inspection_tree_builders/hash_with_indifferent_access'
       )
       autoload(
         :OrderedOptions,
-        "super_diff/active_support/inspection_tree_builders/ordered_options"
+        'super_diff/active_support/inspection_tree_builders/ordered_options'
       )
     end
   end

data/lib/super_diff/active_support/object_inspection.rb

@@ -1,16 +1,18 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module ObjectInspection
       module InspectionTreeBuilders
         def self.const_missing(missing_const_name)
           if ActiveSupport::InspectionTreeBuilders.const_defined?(
-               missing_const_name
-             )
-            warn <<~EOT
+            missing_const_name
+          )
+            warn <<~WARNING
               WARNING: SuperDiff::ActiveSupport::ObjectInspection::InspectionTreeBuilders::#{missing_const_name} is deprecated and will be removed in the next major release.
               Please use SuperDiff::ActiveSupport::InspectionTreeBuilders::#{missing_const_name} instead.
               #{caller_locations.join("\n")}
-            EOT
+            WARNING
             ActiveSupport::InspectionTreeBuilders.const_get(missing_const_name)
           else
             super

data/lib/super_diff/active_support/operation_tree_builders/hash_with_indifferent_access.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module OperationTreeBuilders
@@ -20,10 +22,10 @@ module SuperDiff
             @actual = actual.transform_keys(&:to_s)
           end
 
-          if actual.is_a?(::HashWithIndifferentAccess)
-            @expected = expected.transform_keys(&:to_s)
-            @actual = actual.to_h
-          end
+          return unless actual.is_a?(::HashWithIndifferentAccess)
+
+          @expected = expected.transform_keys(&:to_s)
+          @actual = actual.to_h
         end
 
         protected

data/lib/super_diff/active_support/operation_tree_builders.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module OperationTreeBuilders
       autoload(
         :HashWithIndifferentAccess,
-        "super_diff/active_support/operation_tree_builders/hash_with_indifferent_access"
+        'super_diff/active_support/operation_tree_builders/hash_with_indifferent_access'
       )
     end
   end

data/lib/super_diff/active_support/operation_tree_flatteners/hash_with_indifferent_access.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module OperationTreeFlatteners
@@ -5,11 +7,11 @@ module SuperDiff
         protected
 
         def open_token
-          "#<HashWithIndifferentAccess {"
+          '#<HashWithIndifferentAccess {'
         end
 
         def close_token
-          "}>"
+          '}>'
         end
       end
     end

data/lib/super_diff/active_support/operation_tree_flatteners.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module OperationTreeFlatteners
       autoload(
         :HashWithIndifferentAccess,
-        "super_diff/active_support/operation_tree_flatteners/hash_with_indifferent_access"
+        'super_diff/active_support/operation_tree_flatteners/hash_with_indifferent_access'
       )
     end
   end

data/lib/super_diff/active_support/operation_trees/hash_with_indifferent_access.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module OperationTrees

data/lib/super_diff/active_support/operation_trees.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module SuperDiff
   module ActiveSupport
     module OperationTrees
       autoload(
         :HashWithIndifferentAccess,
-        "super_diff/active_support/operation_trees/hash_with_indifferent_access"
+        'super_diff/active_support/operation_trees/hash_with_indifferent_access'
       )
     end
   end

data/lib/super_diff/active_support.rb

@@ -1,12 +1,14 @@
-require "super_diff/active_support/differs"
-require "super_diff/active_support/inspection_tree_builders"
-require "super_diff/active_support/operation_trees"
-require "super_diff/active_support/operation_tree_builders"
-require "super_diff/active_support/operation_tree_flatteners"
+# frozen_string_literal: true
+
+require 'super_diff/active_support/differs'
+require 'super_diff/active_support/inspection_tree_builders'
+require 'super_diff/active_support/operation_trees'
+require 'super_diff/active_support/operation_tree_builders'
+require 'super_diff/active_support/operation_tree_flatteners'
 
 module SuperDiff
   module ActiveSupport
-    autoload :ObjectInspection, "super_diff/active_support/object_inspection"
+    autoload :ObjectInspection, 'super_diff/active_support/object_inspection'
 
     SuperDiff.configure do |config|
       config.prepend_extra_differ_classes(Differs::HashWithIndifferentAccess)