Introduce explicit support for informational responses. (#42)

* Add a little bit more documentation.

Author: ioquatix

lib/protocol/http/response.rb

@@ -29,42 +29,63 @@ def hijack?
 				false
 			end
 
+			# Whether the status is 100 (continue).
 			def continue?
 				@status == 100
 			end
 
+			# Whether the status is considered informational.
+			def informational?
+				@status and @status >= 100 && @status < 200
+			end
+			
+			# Whether the status is considered final. Note that 101 is considered final.
+			def final?
+				# 101 is effectively a final status.
+				@status and @status >= 200 || @status == 101
+			end
+			
+			# Whether the status is 200 (ok).
 			def ok?
 				@status == 200
 			end
 
+			# Whether the status is considered successful.
 			def success?
 				@status and @status >= 200 && @status < 300
 			end
 
+			# Whether the status is 206 (partial content).
 			def partial?
 				@status == 206
 			end
 
+			# Whether the status is considered a redirection.
 			def redirection?
 				@status and @status >= 300 && @status < 400
 			end
 
+			# Whether the status is 304 (not modified).
 			def not_modified?
 				@status == 304
 			end
 
+			# Whether the status is 307 (temporary redirect) and should preserve the method of the request when following the redirect.
 			def preserve_method?
 				@status == 307 || @status == 308
 			end
 
+			# Whether the status is considered a failure.
 			def failure?
 				@status and @status >= 400 && @status < 600
 			end
 
+			# Whether the status is 400 (bad request).
 			def bad_request?
 				@status == 400
 			end
 
+			# Whether the status is 500 (internal server error).
 			def internal_server_error?
 				@status == 500
 			end

test/protocol/http/response.rb

@@ -10,21 +10,47 @@
 	let(:headers) {Protocol::HTTP::Headers.new}
 	let(:body) {nil}
 
+	InformationalResponse = Sus::Shared("informational response") do
+		it "should be informational" do
+			expect(response).to be(:informational?)
+		end
+		
+		it "should not be a failure" do
+			expect(response).not.to be(:failure?)
+		end
+	end
+	
 	SuccessfulResponse = Sus::Shared("successful response") do
 		it "should be successful" do
 			expect(response).to be(:success?)
 		end
 
+		it "should be final" do
+			expect(response).to be(:final?)
+		end
+		
+		it "should not be informational" do
+			expect(response).not.to be(:informational?)
+		end
+		
 		it "should not be a failure" do
 			expect(response).not.to be(:failure?)
 		end
 	end
 
 	RedirectionResponse = Sus::Shared("redirection response") do
+		it "should be final" do
+			expect(response).to be(:final?)
+		end
+		
 		it "should be a redirection" do
 			expect(response).to be(:redirection?)
 		end
 
+		it "should not be informational" do
+			expect(response).not.to be(:informational?)
+		end
+		
 		it "should not be a failure" do
 			expect(response).not.to be(:failure?)
 		end
@@ -35,6 +61,14 @@
 			expect(response).not.to be(:success?)
 		end
 
+		it "should be final" do
+			expect(response).to be(:final?)
+		end
+		
+		it "should not be informational" do
+			expect(response).not.to be(:informational?)
+		end
+		
 		it "should be a failure" do
 			expect(response).to be(:failure?)
 		end
@@ -51,7 +85,34 @@
 			expect(response).not.to be(:preserve_method?)
 		end
 	end
-			
+	
+	with "100 Continue" do
+		let(:response) {subject.new("HTTP/1.1", 100, headers)}
+		
+		it "should have attributes" do
+			expect(response).to have_attributes(
+				version: be == "HTTP/1.1",
+				status: be == 100,
+				headers: be == headers,
+				body: be == nil,
+				protocol: be == nil
+			)
+		end
+		
+		it_behaves_like InformationalResponse
+		
+		it "should be a continue" do
+			expect(response).to be(:continue?)
+		end
+		
+		it "should have a String representation" do
+			expect(response.to_s).to be == "100 HTTP/1.1"
+		end
+		
+		it "should have an Array representation" do
+			expect(response.to_ary).to be == [100, headers, nil]
+		end
+	end
 
 	with "301 Moved Permanently" do
 		let(:response) {subject.new("HTTP/1.1", 301, headers, body)}