swaggard 1.1.1 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 57f3251f22ae437dfa4a4030dc6e1929def89936e209bd9fdef0b6d7ed92c554
-  data.tar.gz: 859c0573aee509bf0f6c376e20d451c5f97729c3b6cf03f988f450d90041ae4d
+  metadata.gz: 7a49ea847a04d150e7b42923bf6a8290950f5fa18c53deba16907c36a59a664f
+  data.tar.gz: 3986acba75f1b962c13ca5b93ceb2f1481609f3516b5438689822d0916675952
 SHA512:
-  metadata.gz: 4d7f32e8aef8e8f5edfd761982e8a9b845364de71a042f1e56ebae621a00ea7193c1d6377991a22c021d5a1316466031aeb25204fc23d388a4ba96cca9c30f2f
-  data.tar.gz: d9b0b90737efd86097f61bbcc06ac9c3b51881a83bfff8ad1f726940c2140f640adff9e549d81dec907b3f221a2fa21737cbeab76fd0c80ca9f450e91315f756
+  metadata.gz: 3fd5f67a5c03861c8e78a77088f1aa4712a75d9fb251a2c1acb2e597f5609ae81b935d5ac090f5e49ae9dba7ab0a0ceb17be40a553ea183185d3fc88cf14c13f
+  data.tar.gz: db5042eb41674f3f85f94aa860743eed9861eb49d0bc44b9ed02bc36dcb578816552ff44dc2016a61bb01c5f90d71c0b3a31b23f55fd538d8b53403a9ac3cc85

data/lib/swaggard/api_definition.rb

@@ -7,7 +7,7 @@ module Swaggard
     def initialize
       @paths        = {}
       @tags         = {}
-      @definitions  = []
+      @definitions  = {}
     end
 
     def add_tag(tag)
@@ -19,7 +19,7 @@ module Swaggard
     def add_operation(operation)
       @paths[operation.path] ||= Swagger::Path.new(operation.path)
       @paths[operation.path].add_operation(operation)
-      @definitions.concat(operation.definitions)
+      @definitions.merge!(operation.definitions)
     end
 
     def ignore_put_if_patch!
@@ -51,7 +51,7 @@ module Swaggard
         'produces'    => Swaggard.configuration.api_formats.map { |format| "application/#{format}" },
         'tags'        => @tags.map { |_, tag| tag.to_doc },
         'paths'       => Hash[@paths.values.map { |path| [format_path(path.path), path.to_doc] }],
-        'definitions' => Hash[@definitions.map { |definition| [definition.id, definition.to_doc] }]
+        'definitions' => Hash[@definitions.map { |id, definition| [id, definition.to_doc(@definitions)] }]
       }
     end
 

data/lib/swaggard/parsers/models.rb

@@ -6,24 +6,33 @@ module Swaggard
     class Models
 
       def run(yard_objects)
-        definitions = []
+        definitions = {}
 
         yard_objects.each do |yard_object|
-          next unless yard_object.type == :class
+          definition = parse_yard_object(yard_object)
 
-          definition = Swagger::Definition.new(yard_object.path)
+          definitions[definition.id] = definition if definition
+        end
+
+        definitions
+      end
 
+      def parse_yard_object(yard_object)
+        return unless yard_object.type == :class
+
+        Swagger::Definition.new(yard_object.path, ancestors: yard_object.inheritance_tree.map(&:path)).tap do |definition|
           yard_object.tags.each do |tag|
-            property = Swagger::Property.new(tag)
-            definition.add_property(property)
+            case tag.tag_name 
+            when 'attr'
+              property = Swagger::Property.new(tag)
+              definition.add_property(property)
+            when 'ignore_inherited'
+              definition.ignore_inherited = true
+            end
           end
-
-          definitions << definition
         end
-
-        definitions
       end
 
     end
   end
-end
+end

data/lib/swaggard/swagger/definition.rb

@@ -3,13 +3,15 @@ module Swaggard
     class Definition
 
       attr_reader :id
-      attr_writer :description, :title
+      attr_writer :description, :title, :ignore_inherited
 
-      def initialize(id)
+      def initialize(id, ancestors: [])
         @id = id
         @title = ''
         @properties = []
         @description = ''
+        @ancestors = ancestors
+        @ignore_inherited = false
       end
 
       def add_property(property)
@@ -20,15 +22,35 @@ module Swaggard
         @properties.empty?
       end
 
-      def to_doc
+      def properties(definitions)
+        inherited_properties(definitions)
+          .concat(@properties)
+          .uniq { |property| property.id }
+      end
+
+      def inherited_properties(definitions)
+        return [] if @ignore_inherited
+
+        @ancestors.flat_map do |ancestor|
+          definition = definitions[ancestor]
+
+          next unless definition && definition.id != id
+
+          definition.properties(definitions)
+        end.compact
+      end
+
+      def to_doc(definitions)
         {}.tap do |doc|
           doc['title'] = @title if @title.present?
           doc['type']  = 'object'
 
           doc['description'] = @description if @description.present?
 
-          doc['properties'] =  Hash[@properties.map { |property| [property.id, property.to_doc] }]
-          required_properties = @properties.select(&:required?).map(&:id)
+          all_properties = properties(definitions)
+
+          doc['properties'] =  Hash[all_properties.map { |property| [property.id, property.to_doc] }]
+          required_properties = all_properties.select(&:required?).map(&:id)
           doc['required'] = required_properties if required_properties.any?
         end
 

data/lib/swaggard/swagger/operation.rb

@@ -92,8 +92,12 @@ module Swaggard
       end
 
       def definitions
-        @responses.map(&:definition).compact.tap do |definitions|
-          definitions << @body_parameter.definition if @body_parameter
+        @responses.map(&:definition).compact.inject({}) do |definitions, definition|
+          definitions[definition.id] = definition
+        end.tap do |definitions|
+          next unless @body_parameter
+
+          definitions[@body_parameter.definition.id] = @body_parameter.definition
         end
       end
 

data/lib/swaggard/version.rb

@@ -1,3 +1,3 @@
 module Swaggard
-  VERSION = '1.1.1'
+  VERSION = '1.2.0'
 end

data/lib/swaggard.rb

@@ -38,6 +38,7 @@ module Swaggard
       ::YARD::Tags::Library.define_tag('Response description', :response_description)
       ::YARD::Tags::Library.define_tag('Response example', :response_example)
       ::YARD::Tags::Library.define_tag('Response header', :response_header)
+      ::YARD::Tags::Library.define_tag('Ignore inherited attributes', :ignore_inherited)
     end
 
     def get_doc(host = nil)
@@ -119,7 +120,6 @@ module Swaggard
 
         tags.each do |tag|
           @tags << [tag, operations]
-          # [tag.controller_class.controller_path] ||= { tag: tag, operations: operations }
         end
       end
 
@@ -136,12 +136,12 @@ module Swaggard
     def parse_models
       parser = Parsers::Models.new
 
-      definitions =[]
+      definitions = {}
       configuration.models_paths.each do |path|
         Dir[path].each do |file|
           yard_objects = get_yard_objects(file)
 
-          definitions.concat(parser.run(yard_objects))
+          definitions.merge!(parser.run(yard_objects))
         end
 
         @api.definitions = definitions

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: swaggard
 version: !ruby/object:Gem::Version
-  version: 1.1.1
+  version: 1.2.0
 platform: ruby
 authors:
 - Adrian Gomez
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-02-06 00:00:00.000000000 Z
+date: 2021-12-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -192,7 +192,6 @@ files:
 - spec/fixtures/dummy/config/application.rb
 - spec/fixtures/dummy/config/environments/development.rb
 - spec/fixtures/dummy/config/routes.rb
-- spec/fixtures/dummy/log/development.log
 - spec/fixtures/swagger_schema.json
 - spec/integration/swaggard_spec.rb
 - spec/spec_helper.rb
@@ -200,7 +199,7 @@ homepage: https://github.com/adrian-gomez/swaggard
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -215,19 +214,18 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.15
+signing_key:
 specification_version: 4
 summary: 'Swaggard: Swagger Rails REST API doc using yard YARD'
 test_files:
-- spec/spec_helper.rb
-- spec/integration/swaggard_spec.rb
-- spec/fixtures/swagger_schema.json
-- spec/fixtures/dummy/app/controllers/pets_controller.rb
-- spec/fixtures/dummy/app/controllers/application_controller.rb
+- spec/fixtures/api.json
 - spec/fixtures/dummy/app/controllers/admin/pets_controller.rb
-- spec/fixtures/dummy/config/routes.rb
-- spec/fixtures/dummy/config/environments/development.rb
+- spec/fixtures/dummy/app/controllers/application_controller.rb
+- spec/fixtures/dummy/app/controllers/pets_controller.rb
 - spec/fixtures/dummy/config/application.rb
-- spec/fixtures/dummy/log/development.log
-- spec/fixtures/api.json
+- spec/fixtures/dummy/config/environments/development.rb
+- spec/fixtures/dummy/config/routes.rb
+- spec/fixtures/swagger_schema.json
+- spec/integration/swaggard_spec.rb
+- spec/spec_helper.rb