openapi_first 2.2.0 → 2.2.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6f86609c16829b7f0f94d837167a44e4cf93b38a392ac34b93a9ff9d7a794fc
-  data.tar.gz: 1524298280aa89e6cbae16ce80259ed1d0928b4018747eb6c3385ebed0e8db8b
+  metadata.gz: 9458fb5b815dcc64722a1829dc2082f60a92b36ce7af9fa8529c041544b190d4
+  data.tar.gz: 5f5982e1e71f5f1a150860be23291863ff3c431b3bce332899b01150e2c9fc11
 SHA512:
-  metadata.gz: a0d30f3482838dd995f44b5ede610ada74ff1a5d2c2e29a1b3fa109da98cbc5cd42bf8616b4f7e6c79b558d1c16b77ca1eb85b4fc82bcbd9e40c6dcf75aea927
-  data.tar.gz: 7f87349cd6fa98cef19f929db3deeb1c94a9b2eed759f23f592ba145c315726cfe8ff3f5fb3f50444b566a5d95b94370439015fd995554ecececc7029adff9ec
+  metadata.gz: c92c5cbda9bcd884299a265fcc713c0370e0766670e2b3199af9169dfe29855ed78ae5ad1a70fd98326a4f764403e0b9429132b97ee9f3b887d5d9af772acd69
+  data.tar.gz: '05083e92a954621b54667d8f99c133b2489e70eaa2b7332885c867053c81667a0eba0779ad210c150c8402814dff6d144044349271e8fe06808bafe0e5ad94ca'

data/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 ## Unreleased
 
+## 2.2.1
+
+- Fix issue with $ref resolving paths poiting outside directories `$ref: '../a/b.yaml'` (https://github.com/ahx/openapi_first/issues/313)
+- Remove warning about missing assertions when using assert_api_conform (https://github.com/ahx/openapi_first/issues/313)
+
 ## 2.2.0
 
 - Fix support for discriminator in response bodies if no mapping is defined (https://github.com/ahx/openapi_first/issues/285)

data/lib/openapi_first/json.rb

@@ -22,5 +22,4 @@ begin
   end
 rescue LoadError
   require 'json'
-  puts 'openapi_first uses the default json gem'
 end

data/lib/openapi_first/ref_resolver.rb

@@ -40,7 +40,7 @@ module OpenapiFirst
       def initialize(value, context: value, dir: nil)
         @value = value
         @context = context
-        @dir = dir
+        @dir = (dir && File.absolute_path(dir)) || Dir.pwd
       end
 
       # The value of this node
@@ -108,10 +108,11 @@ module OpenapiFirst
 
       def schema(options = {})
         ref_resolver = JSONSchemer::CachedResolver.new do |uri|
-          FileLoader.load(File.join(dir, uri.path))
+          FileLoader.load(uri.path)
         end
-        root = JSONSchemer::Schema.new(context, ref_resolver:, **options)
-        JSONSchemer::Schema.new(value, nil, root, **options)
+        base_uri = URI::File.build({ path: "#{dir}/" })
+        root = JSONSchemer::Schema.new(context, base_uri:, ref_resolver:, **options)
+        JSONSchemer::Schema.new(value, nil, root, base_uri:, **options)
       end
     end
 

data/lib/openapi_first/test/methods.rb

@@ -1,20 +1,19 @@
 # frozen_string_literal: true
 
+require_relative 'minitest_helpers'
+require_relative 'plain_helpers'
+
 module OpenapiFirst
+  # Test integration
   module Test
     # Methods to use in integration tests
     module Methods
-      def assert_api_conform(status: nil, api: :default)
-        api = OpenapiFirst::Test[api]
-        request = respond_to?(:last_request) ? last_request : @request
-        response = respond_to?(:last_response) ? last_response : @response
-        if status && status != response.status
-          raise OpenapiFirst::Error,
-                "Expected status #{status}, but got #{response.status} " \
-                "from #{request.request_method.upcase} #{request.path}."
+      def self.included(base)
+        if Test.minitest?(base)
+          base.include(OpenapiFirst::Test::MinitestHelpers)
+        else
+          base.include(OpenapiFirst::Test::PlainHelpers)
         end
-        api.validate_request(request, raise_error: true)
-        api.validate_response(request, response, raise_error: true)
       end
     end
   end

data/lib/openapi_first/test/minitest_helpers.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+module OpenapiFirst
+  module Test
+    # Assertion methods for Minitest
+    module MinitestHelpers
+      # :nocov:
+      def assert_api_conform(status: nil, api: :default)
+        api = OpenapiFirst::Test[api]
+        request = respond_to?(:last_request) ? last_request : @request
+        response = respond_to?(:last_response) ? last_response : @response
+
+        if status
+          assert_equal status, response.status,
+                       "Expected status #{status}, but got #{response.status} " \
+                       "from #{request.request_method.upcase} #{request.path}."
+        end
+
+        validated_request = api.validate_request(request, raise_error: false)
+        validated_response = api.validate_response(request, response, raise_error: false)
+
+        assert validated_request.valid?, validated_request.error&.exception_message
+        assert validated_response.valid?, validated_response.error&.exception_message
+      end
+      # :nocov:
+    end
+  end
+end

data/lib/openapi_first/test/plain_helpers.rb

@@ -0,0 +1,26 @@
+# frozen_string_literal: true
+
+module OpenapiFirst
+  module Test
+    # Assertion methods to use when no known test framework was found
+    # These methods just raise an exception if an error was found
+    module PlainHelpers
+      def assert_api_conform(status: nil, api: :default)
+        api = OpenapiFirst::Test[api]
+        # :nocov:
+        request = respond_to?(:last_request) ? last_request : @request
+        response = respond_to?(:last_response) ? last_response : @response
+        # :nocov:
+
+        if status && status != response.status
+          raise OpenapiFirst::Error,
+                "Expected status #{status}, but got #{response.status} " \
+                "from #{request.request_method.upcase} #{request.path}."
+        end
+
+        api.validate_request(request, raise_error: true)
+        api.validate_response(request, response, raise_error: true)
+      end
+    end
+  end
+end

data/lib/openapi_first/test.rb

@@ -5,6 +5,12 @@ require_relative 'test/methods'
 module OpenapiFirst
   # Test integration
   module Test
+    def self.minitest?(base)
+      base.include?(::Minitest::Assertions)
+    rescue NameError
+      false
+    end
+
     class NotRegisteredError < StandardError; end
 
     DEFINITIONS = {} # rubocop:disable Style/MutableConstant

data/lib/openapi_first/version.rb

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

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: openapi_first
 version: !ruby/object:Gem::Version
-  version: 2.2.0
+  version: 2.2.1
 platform: ruby
 authors:
 - Andreas Haller
 bindir: bin
 cert_chain: []
-date: 2025-01-11 00:00:00.000000000 Z
+date: 2025-01-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hana
@@ -124,6 +124,8 @@ files:
 - lib/openapi_first/schema/validation_result.rb
 - lib/openapi_first/test.rb
 - lib/openapi_first/test/methods.rb
+- lib/openapi_first/test/minitest_helpers.rb
+- lib/openapi_first/test/plain_helpers.rb
 - lib/openapi_first/validated_request.rb
 - lib/openapi_first/validated_response.rb
 - lib/openapi_first/validators/request_body.rb