openapi_contracts 0.7.1 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e6c02afb8cd5c3ad005ff169eb4c1294b8de98dd97c4cd643609a8f3a5cc66d5
-  data.tar.gz: e8808416c903da9662789d67da35fa6b2f44e9ba7659b7baa675ef5cdabbcd8a
+  metadata.gz: 25a37c81bfa9cb9d1d26f26a25b7c81f0675a153c199223639bdd48c76537cb6
+  data.tar.gz: 48529ed541f4ad72568d222d2a57e30ef7f1a7b5e59fbb447961a58d57eafe26
 SHA512:
-  metadata.gz: 199fa68a93326277251d413ff5e6c3d2b4718b3345a74cb79f227252a4e741f4cb179f05eaa304b2807b29e64f5510a0c9f5bec817c9373337c68a7f8a1ba8d1
-  data.tar.gz: 81c16071342cf8203380e85ced5bf52974231371d269c72ea4d182b472148b4e801c12cc3ca699237643559c1a52dd935f9b984f73c974e51002061da8b3dfa3
+  metadata.gz: 1cc4ee62cea4e015834ae6a493bc771f13670b553cfdc1085b7dac79ea5a347583786ec4cd1db31c9054d88a010dd5d779c64f13df4388a3f55a3eae07ef577a
+  data.tar.gz: 9991124beb074d92107b1026e6ffd529cca988d8154eac6c6cc92aa5103df5de11cf917d1809545e26810de07959202f295a5d23a1a15077e639d2212bbc6286

data/README.md

@@ -47,8 +47,9 @@ it 'responds with 200 and matches the doc' do
 
 The `match_openapi_doc($doc)` method allows passing options as a 2nd argument.
 This allows overriding the default request.path lookup in case this does not find
-the correct response definition in your schema. This is especially important with
-dynamic paths.
+the correct response definition in your schema. This can be usefull when there is
+dynamic parameters in the path and the matcher fails to resolve the request path to
+an endpoint in the openapi specification.
 
 Example:
 
@@ -67,7 +68,9 @@ raise result.errors.merge("/n") unless result.valid?
 
 ### How it works
 
-It uses the `request.path`, `request.method`, `status` and `headers` on the test subject (which must be the response) to find the response schema in the OpenAPI document. Then it does the following checks:
+It uses the `request.path`, `request.method`, `status` and `headers` on the test subject
+(which must be the response) to find the response schema in the OpenAPI document.
+Then it does the following checks:
 
 * The response is documented
 * Required headers are present

data/lib/openapi_contracts/doc/parameter.rb

@@ -0,0 +1,86 @@
+module OpenapiContracts
+  class Doc::Parameter
+    attr_reader :schema
+
+    def initialize(options)
+      @name = options[:name]
+      @in = options[:in]
+      @required = options[:required]
+      @schema = options[:schema]
+    end
+
+    def matches?(value)
+      case schema['type']
+      when 'integer'
+        integer_parameter_matches?(value)
+      when 'number'
+        number_parameter_matches?(value)
+      when 'string'
+        string_parameter_matches?(value)
+      else
+        # Not yet implemented
+        false
+      end
+    end
+
+    private
+
+    def integer_parameter_matches?(value)
+      return false unless /^-?\d+$/.match?(value)
+
+      parsed = value.to_i
+      return false unless minimum_number_matches?(parsed)
+      return false unless maximum_number_matches?(parsed)
+
+      true
+    end
+
+    def number_parameter_matches?(value)
+      return false unless /^-?(\d+\.)?\d+$/.match?(value)
+
+      parsed = value.to_f
+      return false unless minimum_number_matches?(parsed)
+      return false unless maximum_number_matches?(parsed)
+
+      true
+    end
+
+    def minimum_number_matches?(value)
+      if (min = schema['minimum'])
+        if schema['exclusiveMinimum']
+          return false if value <= min
+        elsif value < min
+          return false
+        end
+      end
+      true
+    end
+
+    def maximum_number_matches?(value)
+      if (max = schema['maximum'])
+        if schema['exclusiveMaximum']
+          return false if value >= max
+        elsif value > max
+          return false
+        end
+      end
+      true
+    end
+
+    def string_parameter_matches?(value)
+      if (pat = schema['pattern'])
+        Regexp.new(pat).match?(value)
+      else
+        if (min = schema['minLength']) && (value.length < min)
+          return false
+        end
+
+        if (max = schema['maxLength']) && (value.length > max)
+          return false
+        end
+
+        true
+      end
+    end
+  end
+end

data/lib/openapi_contracts/doc/path.rb

@@ -1,6 +1,7 @@
 module OpenapiContracts
   class Doc::Path
-    def initialize(schema)
+    def initialize(path, schema)
+      @path = path
       @schema = schema
 
       @methods = (known_http_methods & @schema.keys).to_h do |method|
@@ -8,16 +9,50 @@ module OpenapiContracts
       end
     end
 
+    def dynamic?
+      @path.include?('{')
+    end
+
+    def matches?(path)
+      @path == path || regexp_path.match(path) do |m|
+        m.named_captures.each do |k, v|
+          return false unless parameter_matches?(k, v)
+        end
+        true
+      end
+    end
+
     def methods
       @methods.each_value
     end
 
+    def static?
+      !dynamic?
+    end
+
     def with_method(method)
       @methods[method]
     end
 
     private
 
+    def parameter_matches?(name, value)
+      parameter = @schema['parameters']
+        &.find { |p| p['name'] == name && p['in'] == 'path' }
+        &.then { |s| Doc::Parameter.new(s.with_indifferent_access) }
+
+      return false unless parameter
+
+      parameter.matches?(value)
+    end
+
+    def regexp_path
+      re = /\{(\S+)\}/
+      @path.gsub(re) { |placeholder|
+        placeholder.match(re) { |m| "(?<#{m[1]}>[^/]*)" }
+      }.then { |str| Regexp.new(str) }
+    end
+
     def known_http_methods
       # https://developer.mozilla.org/en-US/docs/Web/HTTP/Methods
       %w(get head post put delete connect options trace patch).freeze

data/lib/openapi_contracts/doc.rb

@@ -4,6 +4,7 @@ module OpenapiContracts
     autoload :FileParser, 'openapi_contracts/doc/file_parser'
     autoload :Method,     'openapi_contracts/doc/method'
     autoload :Parser,     'openapi_contracts/doc/parser'
+    autoload :Parameter,  'openapi_contracts/doc/parameter'
     autoload :Path,       'openapi_contracts/doc/path'
     autoload :Response,   'openapi_contracts/doc/response'
     autoload :Schema,     'openapi_contracts/doc/schema'
@@ -17,8 +18,9 @@ module OpenapiContracts
     def initialize(schema)
       @schema = Schema.new(schema)
       @paths = @schema['paths'].to_h do |path, _|
-        [path, Path.new(@schema.at_pointer(['paths', path]))]
+        [path, Path.new(path, @schema.at_pointer(['paths', path]))]
       end
+      @dynamic_paths = paths.select(&:dynamic?)
     end
 
     # Returns an Enumerator over all paths
@@ -42,7 +44,11 @@ module OpenapiContracts
     end
 
     def with_path(path)
-      @paths[path]
+      if @paths.key?(path)
+        @paths[path]
+      else
+        @dynamic_paths.find { |p| p.matches?(path) }
+      end
     end
   end
 end

data/lib/openapi_contracts/validators/body.rb

@@ -23,9 +23,9 @@ module OpenapiContracts::Validators
 
     def error_to_message(error)
       if error.key?('details')
-        error['details'].to_a.map do |(key, val)|
+        error['details'].to_a.map { |(key, val)|
           "#{key.humanize}: #{val} at #{error['data_pointer']}"
-        end.to_sentence
+        }.to_sentence
       else
         "#{error['data'].inspect} at #{error['data_pointer']} does not match the schema"
       end

data/lib/openapi_contracts.rb

@@ -1,5 +1,6 @@
 require 'active_support'
 require 'active_support/core_ext/array'
+require 'active_support/core_ext/hash'
 require 'active_support/core_ext/class'
 require 'active_support/core_ext/module'
 require 'active_support/core_ext/string'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openapi_contracts
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.8.0
 platform: ruby
 authors:
 - mkon
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-06-15 00:00:00.000000000 Z
+date: 2023-07-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -141,6 +141,7 @@ files:
 - lib/openapi_contracts/doc/file_parser.rb
 - lib/openapi_contracts/doc/header.rb
 - lib/openapi_contracts/doc/method.rb
+- lib/openapi_contracts/doc/parameter.rb
 - lib/openapi_contracts/doc/parser.rb
 - lib/openapi_contracts/doc/path.rb
 - lib/openapi_contracts/doc/response.rb