Introduce Payload::Unspecified to solve:

godfat/rest-core#28
godfat · Mar 9, 2018 · fd822fa · fd822fa
1 parent b62bb3e
commit fd822fa
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/lib/rest-builder/client.rb b/lib/rest-builder/client.rb
@@ -2,6 +2,7 @@
 require 'thread'
 require 'weakref'
 
+require 'rest-builder/payload'
 require 'rest-builder/promise'
 require 'rest-builder/middleware'
 require 'rest-builder/engine/dry'
@@ -200,7 +201,7 @@ def default_env
       {REQUEST_METHOD  => :get,
        REQUEST_PATH    => '/' ,
        REQUEST_QUERY   => {}  ,
-       REQUEST_PAYLOAD => {}  ,
+       REQUEST_PAYLOAD => Payload::Unspecified.new,
        REQUEST_HEADERS => {}  ,
        FAIL            => []  ,
        LOG             => []  ,

diff --git a/lib/rest-builder/payload.rb b/lib/rest-builder/payload.rb
@@ -15,6 +15,8 @@
 
 module RestBuilder
   class Payload
+    Unspecified = Class.new(Hash)
+
     def self.generate_with_headers payload, headers
       h = if p = generate(payload)
             p.headers.merge(headers)

diff --git a/test/test_client.rb b/test/test_client.rb
@@ -207,4 +207,13 @@ def default_timeout # module defaults could be overriden
       should.eq "#{url}?a=b"
     simple.wait
   end
+
+  would 'give unspecified payload like a hash' do
+    client = RestBuilder::Builder.client{ run RestBuilder::Identity }.new
+    payload = client.get(url, {},
+      RestBuilder::RESPONSE_KEY => RestBuilder::REQUEST_PAYLOAD)
+
+    payload.should.kind_of?(Hash)
+    payload.should.kind_of?(RestBuilder::Payload::Unspecified)
+  end
 end