Filter for known HTTP methods on Path (#50)

lib/openapi_contracts/doc/path.rb

@@ -2,7 +2,8 @@ module OpenapiContracts
   class Doc::Path
     def initialize(schema)
       @schema = schema
-      @methods = @schema.to_h do |method, _|
+
+      @methods = (known_http_methods & @schema.keys).to_h do |method|
         [method, Doc::Method.new(@schema.navigate(method))]
       end
     end
@@ -14,5 +15,12 @@ def methods
     def with_method(method)
       @methods[method]
     end
+
+    private
+
+    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
+    end
   end
 end

lib/openapi_contracts/doc/schema.rb

@@ -26,7 +26,7 @@ def fragment
       pointer.map { |p| p.gsub('/', '~1') }.join('/').then { |s| "#/#{s}" }
     end
 
-    delegate :dig, :fetch, :key?, :[], :to_h, to: :as_h
+    delegate :dig, :fetch, :keys, :key?, :[], :to_h, to: :as_h
 
     def at_pointer(pointer)
       self.class.new(raw, pointer)

spec/fixtures/openapi/openapi.yaml

@@ -16,6 +16,10 @@ info:
 tags:
   - name: Auth
     description: Authentication
+  - name: Comments
+    description: Comments
+  - name: Messages
+    description: Messages
   - name: User
     description: Operations about user
 servers:
@@ -45,5 +49,7 @@ paths:
                 $ref: components/schemas/Numbers.yaml#/All
   /messages/{id}:
     $ref: 'paths/message.yaml'
+  /comments/{id}:
+    $ref: 'paths/comment.yaml'
   /user:
     $ref: 'paths/user.yaml'

spec/fixtures/openapi/paths/comment.yaml

@@ -0,0 +1,25 @@
+parameters:
+- name: id
+  in: path
+  description: Id of the comment.
+  required: true
+  schema:
+    type: string
+get:
+  tags:
+    - Comment
+  summary: Get Comment
+  description: Get Comment
+  operationId: get_comment
+  responses:
+    '200':
+      description: OK
+patch:
+  tags:
+    - Comment
+  summary: Update Comment
+  description: Patch Comment
+  operationId: patch_comment
+  responses:
+    '204':
+      description: OK

spec/openapi_contracts/doc_spec.rb

@@ -30,6 +30,6 @@
 
     it { is_expected.to all be_a(OpenapiContracts::Doc::Response) }
 
-    it { is_expected.to have_attributes(count: 8) }
+    it { is_expected.to have_attributes(count: 10) }
   end
 end