openapi_first 1.3.2 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d004ac9e317cce0a7d9df95e49b3abbe435e9dab2f23d91880b29cb670ece06f
-  data.tar.gz: c8a1b0ee8238adbf6869aa4a81f642fa0bae0091eabab63417ee142df50ded47
+  metadata.gz: 5505b1339baa8f9dcfdcbedb2270eb67436f0661ebd04115bc6d3346da9d09c6
+  data.tar.gz: 4bea9a7977b95fbb05499d2c1a01eaef54dbabcdccab85c0ab327059e4450db4
 SHA512:
-  metadata.gz: 2e937d066ee559653a60c28b910da5f02def4b73cbb653673c89b2402f96f2dbe79aad482e4c92c23270d0cf9b9e8b35b2423770abc89590056bcd9b5d16332d
-  data.tar.gz: edab0f813f6e98f2db2e245d6d306b0218e678a5ff0e8b22330d6bb69354011d4c254a9e38b2bf15869294b7f9e5d2b19eb7511079665f414b15f2f3da8b2357
+  metadata.gz: 78597035bfd86554fe437f8c26d285572bdc7eb5c19a8e7fa70cb55942e3df3f57bb7851666787b111fbf29e59ac03eb1a1db2bbf8d81b1f92d50298c3559e74
+  data.tar.gz: ff9b1d5871a855187aef2de7c6eec5324b8ba32c95560d1478f5b61566f7d53d7138d5a52dc4a13a78c5a518f6de83fec6f56ef2354509297cfcd15202cc3eda

data/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 ## Unreleased
 
+## 1.3.4
+
+- Fixed handling "binary" format in optional multipart file uploads
+- Cache the resolved OAD. This especially makes things run faster in tests.
+
+## 1.3.3 (yanked)
+
 ## 1.3.2
 
 ### Changed

data/lib/openapi_first/definition/operation.rb

@@ -95,7 +95,7 @@ module OpenapiFirst
       # Returns a unique name for this operation. Used for generating error messages.
       # @visibility private
       def name
-        @name ||= "#{method.upcase} #{path} (#{operation_id})"
+        @name ||= "#{method.upcase} #{path}"
       end
 
       # Returns the path parameters of the operation.

data/lib/openapi_first/failure.rb

@@ -71,8 +71,8 @@ module OpenapiFirst
     private
 
     def generate_message
-      messages = errors&.take(4)&.map(&:error)
-      messages << "... (#{errors.size} errors total)" if errors && errors.size > 4
+      messages = errors&.take(3)&.map(&:error)
+      messages << "... (#{errors.size} errors total)" if errors && errors.size > 3
       messages&.join('. ')
     end
   end

data/lib/openapi_first/json_refs.rb

@@ -32,112 +32,122 @@ require 'hana'
 require 'json'
 require 'yaml'
 
-module JsonRefs
-  class << self
-    def dereference(doc)
-      file_cache = {}
-      Dereferencer.new(Dir.pwd, doc, file_cache).call
-    end
+module OpenapiFirst
+  class FileNotFoundError < StandardError; end
+
+  module JsonRefs
+    class << self
+      def dereference(doc)
+        file_cache = {}
+        Dereferencer.new(Dir.pwd, doc, file_cache).call
+      end
 
-    def load(filename)
-      doc_dir = File.dirname(filename)
-      doc = Loader.handle(filename)
-      file_cache = {}
-      Dereferencer.new(doc_dir, doc, file_cache).call
+      def load(filename)
+        doc_dir = File.dirname(filename)
+        doc = Loader.handle(filename)
+        file_cache = {}
+        Dereferencer.new(filename, doc_dir, doc, file_cache).call
+      end
     end
-  end
 
-  module LocalRef
-    module_function
+    module LocalRef
+      module_function
 
-    def call(path:, doc:)
-      Hana::Pointer.new(path[1..]).eval(doc)
+      def call(path:, doc:)
+        Hana::Pointer.new(path[1..]).eval(doc)
+      end
     end
-  end
 
-  module Loader
-    module_function
+    module Loader
+      module_function
 
-    def handle(filename)
-      body = File.read(filename)
-      return JSON.parse(body) if File.extname(filename) == '.json'
+      def handle(filename)
+        body = File.read(filename)
+        return JSON.parse(body) if File.extname(filename) == '.json'
 
-      YAML.unsafe_load(body)
+        YAML.unsafe_load(body)
+      end
     end
-  end
 
-  class Dereferencer
-    def initialize(doc_dir, doc, file_cache)
-      @doc = doc
-      @doc_dir = doc_dir
-      @file_cache = file_cache
-    end
+    class Dereferencer
+      def initialize(filename, doc_dir, doc, file_cache)
+        @filename = filename
+        @doc = doc
+        @doc_dir = doc_dir
+        @file_cache = file_cache
+      end
 
-    def call(doc = @doc, keys = [])
-      if doc.is_a?(Array)
-        doc.each_with_index do |value, idx|
-          call(value, keys + [idx])
-        end
-      elsif doc.is_a?(Hash)
-        if doc.key?('$ref')
-          dereference(keys, doc['$ref'])
-        else
-          doc.each do |key, value|
-            call(value, keys + [key])
+      def call(doc = @doc, keys = [])
+        if doc.is_a?(Array)
+          doc.each_with_index do |value, idx|
+            call(value, keys + [idx])
+          end
+        elsif doc.is_a?(Hash)
+          if doc.key?('$ref')
+            dereference(keys, doc['$ref'])
+          else
+            doc.each do |key, value|
+              call(value, keys + [key])
+            end
           end
         end
+        doc
       end
-      doc
-    end
 
-    private
+      private
 
-    attr_reader :doc_dir
+      attr_reader :doc_dir
 
-    def dereference(paths, referenced_path)
-      key = paths.pop
-      target = paths.inject(@doc) do |obj, k|
-        obj[k]
+      def dereference(paths, referenced_path)
+        key = paths.pop
+        target = paths.inject(@doc) do |obj, k|
+          obj[k]
+        end
+        value = follow_referenced_value(referenced_path)
+        target[key] = value
       end
-      value = follow_referenced_value(referenced_path)
-      target[key] = value
-    end
 
-    def follow_referenced_value(referenced_path)
-      value = referenced_value(referenced_path)
-      return referenced_value(value['$ref']) if value.is_a?(Hash) && value.key?('$ref')
+      def follow_referenced_value(referenced_path)
+        value = referenced_value(referenced_path)
+        return referenced_value(value['$ref']) if value.is_a?(Hash) && value.key?('$ref')
 
-      value
-    end
+        value
+      end
 
-    def referenced_value(referenced_path)
-      filepath, pointer = referenced_path.split('#')
-      pointer&.prepend('#')
-      return dereference_local(pointer) if filepath.empty?
+      def referenced_value(referenced_path)
+        filepath, pointer = referenced_path.split('#')
+        pointer&.prepend('#')
+        return dereference_local(pointer) if filepath.empty?
 
-      dereferenced_file = dereference_file(filepath)
-      return dereferenced_file if pointer.nil?
+        dereferenced_file = dereference_file(filepath)
+        return dereferenced_file if pointer.nil?
 
-      LocalRef.call(
-        path: pointer,
-        doc: dereferenced_file
-      )
-    end
+        LocalRef.call(
+          path: pointer,
+          doc: dereferenced_file
+        )
+      end
 
-    def dereference_local(referenced_path)
-      LocalRef.call(path: referenced_path, doc: @doc)
-    end
+      def dereference_local(referenced_path)
+        LocalRef.call(path: referenced_path, doc: @doc)
+      end
 
-    def dereference_file(referenced_path)
-      referenced_path = File.expand_path(referenced_path, doc_dir) unless File.absolute_path?(referenced_path)
-      @file_cache[referenced_path] ||= load_referenced_file(referenced_path)
-    end
+      def dereference_file(referenced_path)
+        referenced_path = File.expand_path(referenced_path, doc_dir) unless File.absolute_path?(referenced_path)
+        @file_cache[referenced_path] ||= load_referenced_file(referenced_path)
+      end
 
-    def load_referenced_file(absolute_path)
-      directory = File.dirname(absolute_path)
+      def load_referenced_file(absolute_path)
+        directory = File.dirname(absolute_path)
 
-      referenced_doc = Loader.handle(absolute_path)
-      Dereferencer.new(directory, referenced_doc, @file_cache).call
+        unless File.exist?(absolute_path)
+          raise FileNotFoundError,
+                "Problem while loading file referenced in #{@filename}: File not found #{absolute_path}"
+        end
+
+        referenced_doc = Loader.handle(absolute_path)
+        Dereferencer.new(@filename, directory, referenced_doc, @file_cache).call
+      end
     end
   end
 end

data/lib/openapi_first/runtime_response.rb

@@ -22,8 +22,10 @@ module OpenapiFirst
     # @attr_reader [String] content_type The content_type of the Rack::Response.
     def_delegators :@rack_response, :status, :content_type
 
-    # @attr_reader [String] name The name of the operation. Used for generating error messages.
-    def_delegators :@operation, :name # @visibility private
+    # @return [String] name The name of the operation. Used for generating error messages.
+    def name
+      "#{@operation.name} response status: #{status}"
+    end
 
     # Checks if the response is valid. Runs the validation unless it has been run before.
     # @return [Boolean]

data/lib/openapi_first/schema.rb

@@ -42,7 +42,7 @@ module OpenapiFirst
     def binary_format(data, property, property_schema, _parent)
       return unless property_schema.is_a?(Hash) && property_schema['format'] == 'binary'
 
-      data[property] = data[property][:tempfile].read
+      data[property] = data.dig(property, :tempfile)&.read if data[property]
     end
   end
 end

data/lib/openapi_first/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OpenapiFirst
-  VERSION = '1.3.2'
+  VERSION = '1.3.4'
 end

data/lib/openapi_first.rb

@@ -49,7 +49,8 @@ module OpenapiFirst
   # @!visibility private
   module Bundle
     def self.resolve(spec_path)
-      JsonRefs.load(spec_path)
+      @file_cache ||= {}
+      @file_cache[File.expand_path(spec_path).to_sym] ||= JsonRefs.load(spec_path)
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openapi_first
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.4
 platform: ruby
 authors:
 - Andreas Haller
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-04 00:00:00.000000000 Z
+date: 2024-03-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hana