Restore coverage validation

Author: ioquatix

lib/protocol/http/body/deflate.rb

@@ -110,10 +110,15 @@ def self.for(body, window_size = GZIP, level = DEFAULT_LEVEL)
 				#
 				# @returns [String | Nil] the compressed chunk or `nil` if the stream is closed.
 				def read
-					return if @stream.finished?
+					if @stream.finished?
+						return
+					end
 
 					# The stream might have been closed while waiting for the chunk to come in.
-					while chunk = super
+					loop do
+						chunk = super
+						break unless chunk
+
 						unless chunk.empty?
 							@input_length += chunk.bytesize
 

lib/protocol/http/body/stream.rb

@@ -86,7 +86,10 @@ def read(length = nil, buffer = nil)
 								return buffer
 							end
 						else
-							while chunk = read_next
+							loop do
+								chunk = read_next
+								break unless chunk
+
 								buffer << chunk
 							end
 
@@ -202,7 +205,11 @@ def read_until(pattern, offset = 0, chomp: false)
 						@buffer ||= read_next
 						return nil if @buffer.nil?
 
-						until index = @buffer.index(pattern, offset)
+						index = nil
+						loop do
+							index = @buffer.index(pattern, offset)
+							break if index
+
 							offset = @buffer.bytesize - split_offset
 
 							offset = 0 if offset < 0

lib/protocol/http/header/accept_charset.rb

@@ -17,11 +17,19 @@ class AcceptCharset < Split
 				# https://tools.ietf.org/html/rfc7231#section-5.3.3
 				CHARSET = /\A(?<name>#{TOKEN})(;q=(?<q>#{QVALUE}))?\z/
 
+				# A parsed character set entry with an optional quality factor.
 				Charset = Struct.new(:name, :q) do
+					# The quality factor for this character set.
+					#
+					# @returns [Float] the parsed quality factor, defaulting to `1.0`.
 					def quality_factor
 						(q || 1.0).to_f
 					end
 
+					# Compare character sets by descending quality factor.
+					#
+					# @parameter other [Charset] the other character set to compare.
+					# @returns [Integer] the comparison result.
 					def <=> other
 						other.quality_factor <=> self.quality_factor
 					end

lib/protocol/http/header/accept_encoding.rb

@@ -20,11 +20,19 @@ class AcceptEncoding < Split
 				# https://tools.ietf.org/html/rfc7231#section-5.3.4
 				ENCODING = /\A(?<name>#{TOKEN})(;q=(?<q>#{QVALUE}))?\z/
 
+				# A parsed content encoding entry with an optional quality factor.
 				Encoding = Struct.new(:name, :q) do
+					# The quality factor for this encoding.
+					#
+					# @returns [Float] the parsed quality factor, defaulting to `1.0`.
 					def quality_factor
 						(q || 1.0).to_f
 					end
 
+					# Compare encodings by descending quality factor.
+					#
+					# @parameter other [Encoding] the other encoding to compare.
+					# @returns [Integer] the comparison result.
 					def <=> other
 						other.quality_factor <=> self.quality_factor
 					end

lib/protocol/http/header/accept_language.rb

@@ -23,11 +23,19 @@ class AcceptLanguage < Split
 				# https://greenbytes.de/tech/webdav/rfc7231.html#quality.values
 				LANGUAGE = /\A(?<name>#{NAME})(\s*;\s*q=(?<q>#{QVALUE}))?\z/
 
+				# A parsed language entry with an optional quality factor.
 				Language = Struct.new(:name, :q) do
+					# The quality factor for this language.
+					#
+					# @returns [Float] the parsed quality factor, defaulting to `1.0`.
 					def quality_factor
 						(q || 1.0).to_f
 					end
 
+					# Compare languages by descending quality factor.
+					#
+					# @parameter other [Language] the other language to compare.
+					# @returns [Integer] the comparison result.
 					def <=> other
 						other.quality_factor <=> self.quality_factor
 					end

lib/protocol/http/header/te.rb

@@ -45,14 +45,24 @@ class TE < Split
 
 				# A single transfer coding entry with optional quality factor
 				TransferCoding = Struct.new(:name, :q) do
+					# The quality factor for this transfer coding.
+					#
+					# @returns [Float] the parsed quality factor, defaulting to `1.0`.
 					def quality_factor
 						(q || 1.0).to_f
 					end
 
+					# Compare transfer codings by descending quality factor.
+					#
+					# @parameter other [TransferCoding] the other transfer coding to compare.
+					# @returns [Integer] the comparison result.
 					def <=> other
 						other.quality_factor <=> self.quality_factor
 					end
 
+					# Convert the transfer coding entry to a header value.
+					#
+					# @returns [String] the formatted transfer coding with quality factor when present.
 					def to_s
 						if q && q != 1.0
 							"#{name};q=#{q}"
@@ -143,4 +153,3 @@ def self.trailer?
 		end
 	end
 end
-