apipie-rails 0.5.19 → 1.3.0

data/lib/apipie/extractor.rb

@@ -15,13 +15,10 @@ class Apipie::Railtie
         end
       end
     end
-    app.middleware.use ::Apipie::Extractor::Recorder::Middleware
 
-    if Gem::Version.new(Rails.version) < Gem::Version.new('5.0.0')
-      ActionController::TestCase::Behavior.instance_eval do
-        prepend Apipie::Extractor::Recorder::FunctionalTestRecording
-      end
-    else
+    if Apipie.configuration.record
+      app.middleware.use ::Apipie::Extractor::Recorder::Middleware
+
       ActionController::TestCase.send(:prepend, Apipie::Extractor::Recorder::FunctionalTestRecording)
       ActionController::TestCase::Behavior.send(:prepend, Apipie::Extractor::Recorder::FunctionalTestRecording)
     end
@@ -87,7 +84,7 @@ module Apipie
       def apis_from_routes
         return @apis_from_routes if @apis_from_routes
 
-        @api_prefix = Apipie.api_base_url.sub(/\/$/,"")
+        @api_prefix = Apipie.api_base_url.sub(%r{/$},"")
         populate_api_routes
         update_api_descriptions
 
@@ -160,10 +157,10 @@ module Apipie
       def update_api_descriptions
         apis_from_docs = all_apis_from_docs
         @apis_from_routes.each do |(controller, action), new_apis|
-          method_key = "#{Apipie.get_resource_name(controller.safe_constantize || next)}##{action}"
+          method_key = "#{Apipie.get_resource_id(controller.safe_constantize || next)}##{action}"
           old_apis = apis_from_docs[method_key] || []
           new_apis.each do |new_api|
-            new_api[:path].sub!(/\(\.:format\)$/,"") if new_api[:path]
+            new_api[:path]&.sub!(/\(\.:format\)$/,"")
             old_api = old_apis.find do |api|
               api[:path] == "#{@api_prefix}#{new_api[:path]}"
             end

data/lib/apipie/generator/config.rb

@@ -0,0 +1,12 @@
+module Apipie
+  module Generator
+    # Configuration interface for generators
+    class Config
+      include Singleton
+
+      def swagger
+        Apipie::Generator::Swagger::Config.instance
+      end
+    end
+  end
+end

data/lib/apipie/generator/generator.rb

@@ -0,0 +1,2 @@
+module Apipie::Generator
+end

data/lib/apipie/generator/swagger/computed_interface_id.rb

@@ -0,0 +1,23 @@
+# The {Apipie::Generator::Swagger::ComputedInterfaceId.id} is a number that is
+# uniquely derived from the list of operations added to the swagger definition (in an order-dependent way).
+# it can be used for regression testing, allowing some differentiation between changes that
+# result from changes to the input and those that result from changes to the generation
+# algorithms.
+#
+# @note At the moment, this only takes operation ids into account, and ignores
+# parameter definitions, so it's only partially useful.
+class Apipie::Generator::Swagger::ComputedInterfaceId
+  include Singleton
+
+  def initialize
+    @computed_interface_id = 0
+  end
+
+  def add!(operation_id)
+    @computed_interface_id = Zlib.crc32("#{@computed_interface_id} #{operation_id}")
+  end
+
+  def id
+    @computed_interface_id
+  end
+end

data/lib/apipie/generator/swagger/config.rb

@@ -0,0 +1,80 @@
+require 'singleton'
+
+module Apipie
+  module Generator
+    module Swagger
+      class Config
+        include Singleton
+
+        CONFIG_ATTRIBUTES = [:include_warning_tags, :content_type_input,
+                             :json_input_uses_refs, :suppress_warnings, :api_host,
+                             :generate_x_computed_id_field, :allow_additional_properties_in_response,
+                             :responses_use_refs, :schemes, :security_definitions,
+                             :global_security, :skip_default_tags].freeze
+
+        attr_accessor(*CONFIG_ATTRIBUTES)
+
+        CONFIG_ATTRIBUTES.each do |attribute|
+          old_setter_method = "swagger_#{attribute}="
+          define_method(old_setter_method) do |value|
+            ActiveSupport::Deprecation.warn(
+              <<~HEREDOC
+                config.#{old_setter_method}#{value} is deprecated.
+                config.generator.swagger.#{attribute} instead.
+              HEREDOC
+            )
+
+            send(:"#{attribute}=", value)
+          end
+
+          old_setter_method = "swagger_#{attribute}"
+          define_method(old_setter_method) do
+            ActiveSupport::Deprecation.warn(
+              <<~HEREDOC
+                config.#{old_setter_method} is deprecated.
+                Use config.generator.swagger.#{attribute} instead.
+              HEREDOC
+            )
+
+            send(attribute)
+          end
+        end
+
+        alias include_warning_tags? include_warning_tags
+        alias json_input_uses_refs? json_input_uses_refs
+        alias responses_use_refs? responses_use_refs
+        alias skip_default_tags? skip_default_tags
+        alias generate_x_computed_id_field? generate_x_computed_id_field
+        alias swagger_include_warning_tags? swagger_include_warning_tags
+        alias swagger_json_input_uses_refs? swagger_json_input_uses_refs
+        alias swagger_responses_use_refs? swagger_responses_use_refs
+        alias swagger_generate_x_computed_id_field? swagger_generate_x_computed_id_field
+
+        def initialize
+          @content_type_input = :form_data # this can be :json or :form_data
+          @json_input_uses_refs = false
+          @include_warning_tags = false
+          @suppress_warnings = false # [105,100,102]
+          @api_host = 'localhost:3000'
+          @generate_x_computed_id_field = false
+          @allow_additional_properties_in_response = false
+          @responses_use_refs = true
+          @schemes = [:https]
+          @security_definitions = {}
+          @global_security = []
+          @skip_default_tags = false
+        end
+
+        def self.deprecated_methods
+          CONFIG_ATTRIBUTES.map do |attribute|
+            [
+              :"swagger_#{attribute}=",
+              :"swagger_#{attribute}?",
+              :"swagger_#{attribute}"
+            ]
+          end.flatten
+        end
+      end
+    end
+  end
+end

data/lib/apipie/generator/swagger/context.rb

@@ -0,0 +1,38 @@
+class Apipie::Generator::Swagger::Context
+  attr_reader :default_in_value, :language, :http_method, :controller_method,
+              :prefix
+
+  def initialize(
+    allow_null:,
+    http_method:,
+    controller_method:,
+    prefix: nil,
+    default_in_value: nil,
+    language: nil,
+    in_schema: true
+  )
+    @default_in_value = default_in_value
+    @allow_null = allow_null
+    @language = language
+    @in_schema = in_schema
+    @http_method = http_method
+    @controller_method = controller_method
+    @prefix = prefix
+  end
+
+  def allow_null?
+    @allow_null == true
+  end
+
+  def in_schema?
+    @in_schema == true
+  end
+
+  def add_to_prefix!(prefix)
+    @prefix = if @prefix.present?
+                "#{@prefix}[#{prefix}]"
+              else
+                prefix
+              end
+  end
+end

data/lib/apipie/generator/swagger/method_description/api_decorator.rb

@@ -0,0 +1,20 @@
+class Apipie::Generator::Swagger::MethodDescription::ApiDecorator < SimpleDelegator
+  def normalized_http_method
+    @normalized_http_method ||= http_method.downcase
+  end
+
+  def summary(method_description:, language:)
+    s = Apipie.app.translate(short_description, language)
+
+    if s.blank?
+      Apipie::Generator::Swagger::Warning.for_code(
+        Apipie::Generator::Swagger::Warning::MISSING_METHOD_SUMMARY_CODE,
+        method_description.ruby_name
+      ).warn_through_writer
+
+      nil
+    else
+      s
+    end
+  end
+end

data/lib/apipie/generator/swagger/method_description/api_schema_service.rb

@@ -0,0 +1,89 @@
+class Apipie::Generator::Swagger::MethodDescription::ApiSchemaService
+  # @param [Apipie::Generator::Swagger::MethodDescription::Decorator] method_description
+  def initialize(method_description, language: nil)
+    @method_description = method_description
+    @language = language
+  end
+
+  # @return [Hash]
+  def call
+    @method_description.apis.each_with_object({}) do |api, paths|
+      api = Apipie::Generator::Swagger::MethodDescription::ApiDecorator.new(api)
+      path = Apipie::Generator::Swagger::PathDecorator.new(api.path)
+      op_id = Apipie::Generator::Swagger::OperationId.from(api).to_s
+
+      if Apipie.configuration.generator.swagger.generate_x_computed_id_field?
+        Apipie::Generator::Swagger::ComputedInterfaceId.instance.add!(op_id)
+      end
+
+      parameters = Apipie::Generator::Swagger::MethodDescription::ParametersService
+                   .new(@method_description, path: path, http_method: api.normalized_http_method)
+                   .call
+
+      paths[path.swagger_path(@method_description)] ||= {}
+      paths[path.swagger_path(@method_description)][api.normalized_http_method] = {
+        tags: tags,
+        consumes: consumes,
+        operationId: op_id,
+        summary: api.summary(method_description: @method_description, language: @language),
+        parameters: parameters,
+        responses: responses(api),
+        description: Apipie.app.translate(@method_description.full_description, @language)
+      }.compact
+    end
+  end
+
+  private
+
+  def summary(api)
+    translated_description = Apipie.app.translate(api.short_description, @language)
+
+    return translated_description if translated_description.present?
+
+    Apipie::Generator::Swagger::Warning.for_code(
+      Apipie::Generator::Swagger::Warning::MISSING_METHOD_SUMMARY_CODE,
+      @method_description.ruby_name
+    ).warn_through_writer
+  end
+
+  def tags
+    tags = if Apipie.configuration.generator.swagger.skip_default_tags?
+             []
+           else
+             [@method_description.resource._id]
+           end
+    tags + warning_tags + @method_description.tag_list.tags
+  end
+
+  def warning_tags
+    if Apipie.configuration.generator.swagger.include_warning_tags? &&
+       Apipie::Generator::Swagger::WarningWriter.instance.issued_warnings?
+      ['warnings issued']
+    else
+      []
+    end
+  end
+
+  def consumes
+    if params_in_body?
+      ['application/json']
+    else
+      ['application/x-www-form-urlencoded', 'multipart/form-data']
+    end
+  end
+
+  def params_in_body?
+    Apipie.configuration.generator.swagger.content_type_input == :json
+  end
+
+  # @param [Apipie::Generator::Swagger::MethodDescription::ApiDecorator] api
+  def responses(api)
+    Apipie::Generator::Swagger::MethodDescription::ResponseService
+      .new(
+        @method_description,
+        language: @language,
+        http_method: api.normalized_http_method
+      )
+      .call
+  end
+end

data/lib/apipie/generator/swagger/method_description/decorator.rb

@@ -0,0 +1,22 @@
+class Apipie::Generator::Swagger::MethodDescription::Decorator < SimpleDelegator
+  # @return [String]
+  def operation_id
+    "#{object.resource.controller.name}__#{object.method_name}"
+  end
+
+  # @return [String]
+  def ruby_name
+    if object.blank?
+      '<no method>'
+    else
+      "#{object.resource.controller.name}##{object.method_name}"
+    end
+  end
+
+  private
+
+  # @return [Apipie::MethodDescription, nil]
+  def object
+    __getobj__
+  end
+end

data/lib/apipie/generator/swagger/method_description/parameters_service.rb

@@ -0,0 +1,139 @@
+class Apipie::Generator::Swagger::MethodDescription::ParametersService
+  # @param [Apipie::Generator::Swagger::MethodDescription::Decorator] method_description
+  # @param [Apipie::Generator::Swagger::PathDecorator] path
+  # @param [Symbol] http_method
+  def initialize(method_description, path:, http_method:)
+    @method_description = method_description
+    @path = path
+    @http_method = http_method
+  end
+
+  # @return [Array]
+  def call
+    path_params_schema + body_params_schema + header_params_schema
+  end
+
+  private
+
+  def path_params_schema
+    Apipie::Generator::Swagger::ParamDescription::PathParamsComposite.new(
+      path_param_descriptions,
+      Apipie::Generator::Swagger::Context.new(
+        allow_null: false,
+        default_in_value: 'path',
+        http_method: @http_method,
+        controller_method: @method_description
+      )
+    ).to_swagger
+  end
+
+  def body_params_schema
+    if params_in_body? && body_allowed_for_current_method?
+      composite = Apipie::Generator::Swagger::ParamDescription::Composite.new(
+        body_param_descriptions,
+        Apipie::Generator::Swagger::Context.new(
+          allow_null: false,
+          http_method: @http_method,
+          controller_method: @method_description
+        )
+      )
+
+      if Apipie.configuration.generator.swagger.json_input_uses_refs?
+        composite = composite
+                    .referenced("#{@method_description.operation_id}_input")
+      end
+
+      swagger_schema_for_body = composite.to_swagger
+
+      swagger_body_param = {
+        name: 'body',
+        in: 'body',
+        schema: swagger_schema_for_body
+      }
+
+      if swagger_schema_for_body.present?
+        [swagger_body_param]
+      else
+        []
+      end
+    else
+      Apipie::Generator::Swagger::ParamDescription::PathParamsComposite.new(
+        body_param_descriptions,
+        Apipie::Generator::Swagger::Context.new(
+          allow_null: false,
+          http_method: @http_method,
+          controller_method: @method_description
+        )
+      ).to_swagger
+    end
+  end
+
+  def all_params
+    @all_params ||=
+      begin
+        param_names_from_method = @method_description.params.keys
+        missing = @path.param_names - param_names_from_method
+
+        result = @method_description.params
+
+        missing.each do |name|
+          warn_path_parameter_not_described(name, @path)
+
+          result[name.to_sym] = Apipie::Generator::Swagger::ParamDescription
+                                .create_for_missing_param(@method_description, name)
+        end
+
+        result
+      end
+  end
+
+  def body_param_descriptions
+    @body_param_descriptions ||= all_params
+                                 .reject { |k, _| @path.param?(k) }
+                                 .values
+  end
+
+  def path_param_descriptions
+    @path_param_descriptions ||= all_params
+                                 .select { |k, _| @path.param?(k) }
+                                 .each_value { |desc| desc.required = true }
+                                 .values
+  end
+
+  # @return [Array]
+  def header_params_schema
+    return [] if @method_description.headers.blank?
+
+    @method_description.headers.map do |header|
+      header_hash = {
+        name: header[:name],
+        in: 'header',
+        required: header[:options][:required],
+        description: header[:description],
+        type: header[:options][:type] || 'string'
+
+      }
+      if header[:options][:default]
+        header_hash[:default] = header[:options][:default]
+      end
+
+      header_hash
+    end
+  end
+
+  def params_in_body?
+    Apipie.configuration.generator.swagger.content_type_input == :json
+  end
+
+  def body_allowed_for_current_method?
+    %w[get head].exclude?(@http_method)
+  end
+
+  def warn_path_parameter_not_described(name, path)
+    Apipie::Generator::Swagger::Warning.for_code(
+      Apipie::Generator::Swagger::Warning::PATH_PARAM_NOT_DESCRIBED_CODE,
+      @method_description.ruby_name,
+      { name: name, path: path }
+    ).warn_through_writer
+  end
+end

data/lib/apipie/generator/swagger/method_description/response_schema_service.rb

@@ -0,0 +1,46 @@
+class Apipie::Generator::Swagger::MethodDescription::ResponseSchemaService
+  # @param [Apipie::ResponseDescription, Apipie::ResponseDescriptionAdapter] response_description
+  def initialize(response_description, allow_null:, http_method:, controller_method:)
+    @response_description = response_description
+    @allow_null = allow_null
+    @http_method = http_method
+    @controller_method = controller_method
+  end
+
+  def to_swagger
+    composite = Apipie::Generator::Swagger::ParamDescription::Composite.new(
+      @response_description.params_ordered,
+      Apipie::Generator::Swagger::Context.new(
+        allow_null: @allow_null,
+        http_method: @http_method,
+        controller_method: @controller_method
+      )
+    )
+
+    if Apipie.configuration.generator.swagger.responses_use_refs? && @response_description.typename.present?
+      composite = composite.referenced(@response_description.typename)
+    end
+
+    schema = composite.to_swagger
+
+    if @response_description.is_array? && schema
+      schema = { type: type_for_array, items: schema }
+    end
+
+    if @response_description.allow_additional_properties
+      schema[:additionalProperties] = true
+    end
+
+    schema
+  end
+
+  private
+
+  def type_for_array
+    if @allow_null == true
+      %w[array null]
+    else
+      'array'
+    end
+  end
+end

data/lib/apipie/generator/swagger/method_description/response_service.rb

@@ -0,0 +1,58 @@
+class Apipie::Generator::Swagger::MethodDescription::ResponseService
+  # @param [Apipie::Generator::Swagger::MethodDescription::Decorator] method_description
+  # @param [Symbol] http_method
+  def initialize(method_description, http_method:, language:)
+    @method_description = method_description
+    @http_method = http_method
+    @language = language
+  end
+
+  # @return [Hash]
+  def call
+    result = {}
+    result.merge!(errors)
+    result.merge!(responses)
+    result.merge!(empty_returns)
+
+    result
+  end
+
+  private
+
+  # @return [Hash]
+  def errors
+    @errors ||= @method_description.errors.each_with_object({}) do |error, errors|
+      errors[error.code] = {
+        description: Apipie.app.translate(error.description, @language)
+      }
+    end
+  end
+
+  # @return [Hash]
+  def responses
+    @responses ||=
+      @method_description.returns.each_with_object({}) do |response, responses_schema|
+        responses_schema[response.code] = {
+          description: Apipie.app.translate(response.description, @language),
+          schema: Apipie::Generator::Swagger::MethodDescription::ResponseSchemaService.new(
+            response,
+            allow_null: false,
+            http_method: @http_method,
+            controller_method: @method_description
+          ).to_swagger
+        }.compact
+      end
+  end
+
+  # @return [Hash]
+  def empty_returns
+    return {} if errors.present? || responses.present?
+
+    Apipie::Generator::Swagger::Warning.for_code(
+      Apipie::Generator::Swagger::Warning::NO_RETURN_CODES_SPECIFIED_CODE,
+      @method_description.ruby_name
+    ).warn_through_writer
+
+    { 200 => { description: 'ok' } }
+  end
+end

data/lib/apipie/generator/swagger/method_description.rb

@@ -0,0 +1,2 @@
+module Apipie::Generator::Swagger::MethodDescription
+end

data/lib/apipie/generator/swagger/operation_id.rb

@@ -0,0 +1,51 @@
+class Apipie::Generator::Swagger::OperationId
+  def initialize(path:, http_method:, param: nil)
+    @path = path
+    @http_method = http_method
+    @param = param
+  end
+
+  # @return [String]
+  def to_s
+    base = normalized_http_method + path
+
+    if @param.present?
+      "#{base}_param_#{@param}"
+    else
+      base
+    end
+  end
+
+  # @param [Apipie::MethodDescription::Api, Apipie::MethodDescription] describable
+  # @param [String, Symbol, nil] param
+  #
+  # @return [Apipie::Generator::Swagger::OperationId]
+  def self.from(describable, param: nil)
+    path, http_method =
+      if describable.respond_to?(:path) && describable.respond_to?(:http_method)
+        [describable.path, describable.http_method]
+      elsif describable.is_a?(Apipie::MethodDescription)
+        [describable.apis.first.path, describable.apis.first.http_method]
+      end
+
+    new(path: path, http_method: http_method, param: param)
+  end
+
+  private
+
+  # Converts an http path for example `/api/concerns/:id` to `_api_concerns_id`
+  #
+  # @return [String]
+  def path
+    @path.gsub(%r{/}, '_').gsub(/:(\w+)/, '\1').gsub(/_$/, '')
+  end
+
+  # Converts an http method like `GET` to `get` Using lowercase http method,
+  #   because the 'swagger-codegen' tool outputs strange method names if the
+  #   http method is in uppercase
+  #
+  # @return [String]
+  def normalized_http_method
+    @http_method.to_s.downcase
+  end
+end