Refactor yard definition generator about location

lib/rucoa/position.rb

@@ -47,6 +47,12 @@ def initialize(
       @line = line
     end
 
+    # @param other [Rucoa::Position]
+    # @return [Boolean]
+    def ==(other)
+      column == other.column && line == other.line
+    end
+
     # @param text [String]
     # @return [Integer]
     def to_index_of(text)

lib/rucoa/range.rb

@@ -41,6 +41,12 @@ def initialize(
       @including_ending = including_ending
     end
 
+    # @param other [Rucoa::Range]
+    # @return [Boolean]
+    def ==(other)
+      beginning == other.beginning && ending == other.ending
+    end
+
     # @param range [Rucoa::Range]
     # @return [Boolean]
     # @example returns true when the range is contained in self

lib/rucoa/yard/definition_generators/attribute_reader_definition_generator.rb

@@ -42,7 +42,7 @@ def call
             Definitions::MethodDefinition.new(
               description: description,
               kind: :instance,
-              location: Location.from_rucoa_node(@node),
+              location: location,
               method_name: argument.value.to_s,
               namespace: @node.namespace,
               types: return_types.map do |type|

lib/rucoa/yard/definition_generators/attribute_writer_definition_generator.rb

@@ -42,7 +42,7 @@ def call
             Definitions::MethodDefinition.new(
               description: description,
               kind: :instance,
-              location: Location.from_rucoa_node(@node),
+              location: location,
               method_name: "#{argument.value}=",
               namespace: @node.namespace,
               types: return_types.map do |type|

lib/rucoa/yard/definition_generators/base.rb

@@ -51,6 +51,34 @@ def docstring_parser
           end
         end
 
+        # @return [Rucoa::Location]
+        # @example returns source uri and entire range of target node
+        #   definitions = Rucoa::Source.new(
+        #     content: <<~RUBY,
+        #       class Foo
+        #       end
+        #     RUBY
+        #     uri: '/path/to/foo.rb'
+        #   ).definitions
+        #   expect(definitions[0].location).to eq(
+        #     Rucoa::Location.new(
+        #       range: Rucoa::Range.new(
+        #         Rucoa::Position.new(
+        #           column: 0,
+        #           line: 1
+        #         ),
+        #         Rucoa::Position.new(
+        #           column: 3,
+        #           line: 2
+        #         )
+        #       ),
+        #       uri: '/path/to/foo.rb'
+        #     )
+        #   )
+        def location
+          Location.from_rucoa_node(@node)
+        end
+
         # @return [Array<String>]
         # @example returns annotated return types if return tag is provided
         #   definitions = Rucoa::Source.new(

lib/rucoa/yard/definition_generators/class_definition_generator.rb

@@ -46,28 +46,33 @@ class ClassDefinitionGenerator < Base
         def call
           return [] unless @node.is_a?(Nodes::ClassNode)
 
-          included_module_chained_names = @node.body_children.flat_map do |child|
-            next [] unless child.is_a?(Nodes::SendNode)
-            next [] unless child.name == 'include'
-
-            child.arguments.filter_map do |includee|
-              next unless includee.is_a?(Nodes::ConstNode)
-
-              includee.chained_name
-            end
-          end
-
           [
             Definitions::ClassDefinition.new(
               description: description,
               fully_qualified_name: @node.fully_qualified_name,
               included_module_chained_names: included_module_chained_names,
-              location: Location.from_rucoa_node(@node),
+              location: location,
               module_nesting: @node.module_nesting,
               super_class_chained_name: @node.super_class_chained_name
             )
           ]
         end
+
+        private
+
+        # @return [Array<String>]
+        def included_module_chained_names
+          @node.body_children.flat_map do |child|
+            next [] unless child.is_a?(Nodes::SendNode)
+            next [] unless child.name == 'include'
+
+            child.arguments.filter_map do |includee|
+              next unless includee.is_a?(Nodes::ConstNode)
+
+              includee.chained_name
+            end
+          end
+        end
       end
     end
   end

lib/rucoa/yard/definition_generators/constant_assignment_definition_generator.rb

@@ -19,7 +19,7 @@ def call
             Definitions::ConstantDefinition.new(
               description: description,
               fully_qualified_name: @node.fully_qualified_name,
-              location: Location.from_rucoa_node(@node)
+              location: location
             )
           ]
         end

lib/rucoa/yard/definition_generators/method_definition_generator.rb

@@ -29,7 +29,7 @@ def call
             Definitions::MethodDefinition.new(
               description: description,
               kind: @node.singleton? ? :singleton : :instance,
-              location: Location.from_rucoa_node(@node),
+              location: location,
               method_name: @node.name,
               namespace: @node.namespace,
               types: return_types.map do |type|

lib/rucoa/yard/definition_generators/module_definition_generator.rb

@@ -20,7 +20,7 @@ def call
             Definitions::ModuleDefinition.new(
               description: description,
               fully_qualified_name: @node.fully_qualified_name,
-              location: Location.from_rucoa_node(@node)
+              location: location
             )
           ]
         end