Remove redundant Accept input validation

Author: samuel-williams-shopify

lib/protocol/http/header/accept.rb

@@ -82,7 +82,7 @@ def self.parse(value)
 				#
 				# @parameter value [String] a raw header value containing one or more media types separated by commas.
 				def << value
-					self.concat(split_values(value))
+					self.concat(value.scan(SEPARATOR).map(&:strip))
 				end
 
 				# Converts the parsed header value into a raw header value.
@@ -109,12 +109,6 @@ def media_ranges
 
 				private
 
-				def split_values(value)
-					raise TypeError, "Expected a String, got: #{value.class}" unless value.is_a?(String)
-					
-					value.scan(SEPARATOR).map(&:strip)
-				end
-				
 				def parse_media_range(value)
 					if match = value.match(MEDIA_RANGE)
 						type = match[:type]

test/protocol/http/header/accept.rb

@@ -136,23 +136,13 @@
 			expect(subject.parse("").media_ranges).to be == []
 		end
 
-		it "rejects nil" do
-			expect{subject.parse(nil)}.to raise_exception(TypeError)
-		end
-		
 		it "ignores empty list members" do
 			expect(subject.parse(", text/html,").media_ranges).to have_attributes(
 				length: be == 1,
 			)
 		end
 	end
 
-	with "#<<" do
-		it "rejects nil" do
-			expect{subject.new << nil}.to raise_exception(TypeError)
-		end
-	end
-	
 	with "text/html;q=0, text/plain;q=0.123, application/json;q=1.000" do
 		it "accepts standard quality factors" do
 			expect(media_ranges.collect(&:quality_factor)).to be == [1.0, 0.123, 0.0]