Fix more rubocop offenses

Author: sferik

.rubocop/metrics.yml

@@ -16,6 +16,8 @@ Metrics/ClassLength:
     - array
     - heredoc
     - method_call
+  Exclude:
+    - 'test/support/**/*.rb'
 
 Metrics/MethodLength:
   CountAsOne:

Gemfile

@@ -12,7 +12,6 @@ group :development do
 end
 
 group :test do
-  gem "certificate_authority", "~> 1.0", require: false
   gem "logger"
 
   gem "backports"

lib/http/base64.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 module HTTP
+  # Strict Base64 encoding utilities
   module Base64
     module_function
 

lib/http/chainable.rb

@@ -5,6 +5,7 @@
 require "http/headers"
 
 module HTTP
+  # HTTP verb methods and client configuration DSL
   module Chainable
     include HTTP::Base64
 

lib/http/chainable/helpers.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 module HTTP
+  # HTTP verb methods and client configuration DSL
   module Chainable
     # Mapping of proxy argument positions to hash keys and expected types
     PROXY_ARG_MAP = [

lib/http/client.rb

@@ -18,6 +18,7 @@ class Client
     include Chainable
     include RequestBuilder
 
+    # Pattern matching HTTP or HTTPS URI schemes
     HTTP_OR_HTTPS_RE = %r{^https?://}i
 
     # Initialize a new HTTP Client

lib/http/connection.rb

@@ -13,6 +13,7 @@ class Connection
 
     # Allowed values for CONNECTION header
     KEEP_ALIVE = "Keep-Alive"
+    # Connection: close header value
     CLOSE      = "close"
 
     # Attempt to read this much data
@@ -113,10 +114,7 @@ def readpartial(size = BUFFER_SIZE)
       return chunk if chunk
 
       eof = read_more(size) == :eof
-      if eof && !@parser.finished? && body_framed?
-        close
-        raise ConnectionError, "response body ended prematurely"
-      end
+      check_premature_eof(eof)
 
       finished = eof || @parser.finished?
       chunk    = @parser.read(size)
@@ -221,6 +219,27 @@ def init_state(options)
       @parser               = Response::Parser.new
     end
 
+    # Check for premature end-of-file and raise if detected
+    #
+    # @example
+    #   check_premature_eof(:eof)
+    #
+    # @return [void]
+    # @api private
+    def check_premature_eof(eof)
+      return unless eof && !@parser.finished? && body_framed?
+
+      close
+      raise ConnectionError, "response body ended prematurely"
+    end
+
+    # Check if the response body has a known framing mechanism
+    #
+    # @example
+    #   body_framed?
+    #
+    # @return [Boolean]
+    # @api private
     def body_framed?
       @parser.headers.include?(Headers::TRANSFER_ENCODING) ||
         @parser.headers.include?(Headers::CONTENT_LENGTH)

lib/http/content_type.rb

@@ -1,8 +1,11 @@
 # frozen_string_literal: true
 
 module HTTP
+  # Parsed representation of a Content-Type header
   class ContentType
+    # Pattern for extracting MIME type from Content-Type header
     MIME_TYPE_RE = %r{^([^/]+/[^;]+)(?:$|;)}
+    # Pattern for extracting charset from Content-Type header
     CHARSET_RE   = /;\s*charset=([^;]+)/i
 
     # MIME type of the content

lib/http/errors.rb

@@ -9,7 +9,9 @@ class ConnectionError < Error; end
 
   # Types of Connection errors
   class ResponseHeaderError < ConnectionError; end
+  # Error raised when reading from a socket fails
   class SocketReadError < ConnectionError; end
+  # Error raised when writing to a socket fails
   class SocketWriteError < ConnectionError; end
 
   # Generic Request error

lib/http/feature.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 module HTTP
+  # Base class for HTTP client features (middleware)
   class Feature
     # Wraps an HTTP request
     #