Change the http_request provider to use the new Chef::REST API

Author: ultraninja

chef/lib/chef/provider/http_request.rb

@@ -34,6 +34,7 @@ def action_get
         body = @rest.run_request(
           :GET, 
           @rest.create_url("#{@new_resource.url}?message=#{message}"),
+          {},
           false,
           10,
           false
@@ -49,6 +50,7 @@ def action_put
         body = @rest.run_request(
           :PUT,
           @rest.create_url("#{@new_resource.url}"),
+          {},
           message,
           10,
           false
@@ -64,6 +66,7 @@ def action_post
         body = @rest.run_request(
           :POST,
           @rest.create_url("#{@new_resource.url}"),
+          {},
           message,
           10,
           false
@@ -78,6 +81,7 @@ def action_delete
         body = @rest.run_request(
           :DELETE,
           @rest.create_url("#{@new_resource.url}"),
+          {},
           false,
           10,
           false

chef/spec/unit/provider/http_request_spec.rb

@@ -83,7 +83,7 @@
   end
 
   it "should run a GET request" do
-    @rest.should_receive(:run_request).with(:GET, @rest.create_url, false, 10, false)
+    @rest.should_receive(:run_request).with(:GET, @rest.create_url, {}, false, 10, false)
     @provider.action_get
   end
 
@@ -117,13 +117,13 @@
   end
 
   it "should run a PUT request with the message as the payload" do
-    @rest.should_receive(:run_request).with(:PUT, @rest.create_url, @new_resource.message, 10, false)
+    @rest.should_receive(:run_request).with(:PUT, @rest.create_url, {}, @new_resource.message, 10, false)
     @provider.action_put
   end
 
   it "should inflate a message block at runtime" do
     @new_resource.stub!(:message).and_return(lambda { "return" })
-    @rest.should_receive(:run_request).with(:PUT, @rest.create_url, "return", 10, false)    
+    @rest.should_receive(:run_request).with(:PUT, @rest.create_url, {}, "return", 10, false)    
     @provider.action_put
   end
 
@@ -157,13 +157,13 @@
   end
 
   it "should run a PUT request with the message as the payload" do
-    @rest.should_receive(:run_request).with(:POST, @rest.create_url, @new_resource.message, 10, false)
+    @rest.should_receive(:run_request).with(:POST, @rest.create_url, {}, @new_resource.message, 10, false)
     @provider.action_post
   end
 
   it "should inflate a message block at runtime" do
     @new_resource.stub!(:message).and_return(lambda { "return" })
-    @rest.should_receive(:run_request).with(:POST, @rest.create_url, "return", 10, false)    
+    @rest.should_receive(:run_request).with(:POST, @rest.create_url, {}, "return", 10, false)    
     @provider.action_post
   end
 
@@ -197,7 +197,7 @@
   end
 
   it "should run a DELETE request" do
-    @rest.should_receive(:run_request).with(:DELETE, @rest.create_url, false, 10, false)
+    @rest.should_receive(:run_request).with(:DELETE, @rest.create_url, {}, false, 10, false)
     @provider.action_delete
   end