as_json should ignore all arguments. Add default to_json. (#54)

Author: ioquatix

lib/protocol/http/body/readable.rb

@@ -93,7 +93,7 @@ def join
 					end
 				end
 
-				def as_json
+				def as_json(...)
 					{
 						class: self.class.name,
 						length: self.length,
@@ -102,6 +102,10 @@ def as_json
 						empty: self.empty?
 					}
 				end
+				
+				def to_json(...)
+					as_json.to_json(...)
+				end
 			end
 		end
 	end

lib/protocol/http/body/wrapper.rb

@@ -51,13 +51,17 @@ def read
 					@body.read
 				end
 
-				def as_json
+				def as_json(...)
 					{
 						class: self.class.name,
 						body: @body&.as_json
 					}
 				end
 
+				def to_json(...)
+					as_json.to_json(...)
+				end
+				
 				def inspect
 					@body.inspect
 				end

lib/protocol/http/request.rb

@@ -73,7 +73,7 @@ def idempotent?
 				@method != Methods::POST && (@body.nil? || @body.empty?)
 			end
 
-			def as_json
+			def as_json(...)
 				{
 					scheme: @scheme,
 					authority: @authority,
@@ -86,6 +86,10 @@ def as_json
 				}
 			end
 
+			def to_json(...)
+				as_json.to_json(...)
+			end
+			
 			def to_s
 				"#{@scheme}://#{@authority}: #{@method} #{@path} #{@version}"
 			end

lib/protocol/http/response.rb

@@ -104,7 +104,7 @@ def self.for_exception(exception)
 				Response[500, Headers['content-type' => 'text/plain'], ["#{exception.class}: #{exception.message}"]]
 			end
 
-			def as_json
+			def as_json(...)
 				{
 					version: @version,
 					status: @status,
@@ -114,6 +114,10 @@ def as_json
 				}
 			end
 
+			def to_json(...)
+				as_json.to_json(...)
+			end
+			
 			def to_s
 				"#{@status} #{@version}"
 			end

test/protocol/http/body/readable.rb

@@ -42,4 +42,20 @@
 			expect(body.join).to be_nil
 		end
 	end
+	
+	with "#as_json" do
+		it "generates a JSON representation" do
+			expect(body.as_json).to have_keys(
+				class: be == subject.name,
+				length: be_nil,
+				stream: be == false,
+				ready: be == false,
+				empty: be == false,
+			)
+		end
+		
+		it "generates a JSON string" do
+			expect(JSON.dump(body)).to be == body.to_json
+		end
+	end
 end

test/protocol/http/body/wrapper.rb

@@ -71,5 +71,9 @@
 				body: be == source.as_json
 			)
 		end
+		
+		it "generates a JSON string" do
+			expect(JSON.dump(body)).to be == body.to_json
+		end
 	end
 end

test/protocol/http/request.rb

@@ -5,6 +5,8 @@
 
 require 'protocol/http/request'
 
+require 'json'
+
 describe Protocol::HTTP::Request do
 	let(:headers) {Protocol::HTTP::Headers.new}
 	let(:body) {nil}
@@ -38,6 +40,10 @@
 					protocol: nil
 				}
 			end
+			
+			it "generates a JSON string" do
+				expect(JSON.dump(request)).to be == request.to_json
+			end
 		end
 
 		it "should not be HEAD" do

test/protocol/http/response.rb

@@ -113,6 +113,10 @@
 					protocol: be == nil,
 				)
 			end
+			
+			it "generates a JSON string" do
+				expect(JSON.dump(response)).to be == response.to_json
+			end
 		end
 
 		it_behaves_like InformationalResponse
@@ -182,6 +186,10 @@
 					protocol: be == nil,
 				)
 			end
+			
+			it "generates a JSON string" do
+				expect(JSON.dump(response)).to be == response.to_json
+			end
 		end
 
 		it_behaves_like SuccessfulResponse