gojek#108 Allow async backends to return whatever they want so that, …

…for example, the thread backend allows for Thread#join to be called

lib/wrest.rb

@@ -6,8 +6,6 @@
 # is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 # See the License for the specific language governing permissions and limitations under the License.
 
-require 'rubygems'
-
 require 'net/http'
 require 'net/https'
 require 'forwardable'
@@ -84,7 +82,3 @@ def self.use_curl!
 
 # Load Wrest::Resource
 # require "#{Wrest::Root}/wrest/resource"
-
-# if (ENV['RAILS_ENV'] == 'test' || (Kernel.const_defined?(:RAILS_ENV) && (RAILS_ENV == 'test')))
-#   require "#{Wrest::Root}/wrest/test"
-# end

lib/wrest/multipart.rb

@@ -50,7 +50,6 @@ def post_multipart(parameters = {}, headers = {}, &block)
     # This implementation of asynchronous post_multipart is currently in alpha. Hence, it should not be used in production.
     def post_multipart_async(parameters = {}, headers = {}, &block)
       (@options[:asynchronous_backend] || Wrest::AsyncRequest.default_backend).execute(Http::PostMultipart.new(self, parameters, headers, block ? @options.merge(:callback_block => block) : @options))
-      nil
     end
 
     # Makes a multipart/form-data encoded PUT request to this URI. This is a convenience API
@@ -66,7 +65,6 @@ def put_multipart(parameters = {}, headers = {}, &block)
     # This implementation of asynchronous put_multipart is currently in alpha. Hence, it should not be used in production.
     def put_multipart_async(parameters = {}, headers = {}, &block)
       (@options[:asynchronous_backend] || Wrest::AsyncRequest.default_backend).execute(Http::PutMultipart.new(self, parameters, headers, block ? @options.merge(:callback_block => block) : @options))
-      nil
     end
   end
 

lib/wrest/uri.rb

@@ -150,7 +150,6 @@ def get(parameters = {}, headers = {}, &block)
     # This implementation of asynchronous get is currently in alpha. Hence, it should not be used in production.
     def get_async(parameters = {}, headers = {}, &block)
       @asynchronous_backend.execute(build_get(parameters, headers, &block))
-      nil
     end
 
     # Make a PUT request to this URI. This is a convenience API
@@ -170,7 +169,6 @@ def put(body = '', headers = {}, parameters = {}, &block)
     # This implementation of asynchronous put is currently in alpha. Hence, it should not be used in production.
     def put_async(body = '', headers = {}, parameters = {}, &block)
       @asynchronous_backend.execute(build_put(body, headers, parameters, &block))
-      nil
     end
 
     # Makes a POST request to this URI. This is a convenience API
@@ -192,7 +190,6 @@ def post(body = '', headers = {}, parameters = {}, &block)
     # This implementation of asynchronous post is currently in alpha. Hence, it should not be used in production.
     def post_async(body = '', headers = {}, parameters = {}, &block)
       @asynchronous_backend.execute(build_post(body, headers, parameters, &block))
-      nil
     end
 
     # Makes a POST request to this URI. This is a convenience API
@@ -218,7 +215,6 @@ def post_form(parameters = {}, headers = {}, &block)
     # This implementation of asynchronous post_form is currently in alpha. Hence, it should not be used in production.
     def post_form_async(parameters = {}, headers = {}, &block)
       @asynchronous_backend.execute(build_post_form(parameters, headers, &block))
-      nil
     end
 
     # Makes a DELETE request to this URI. This is a convenience API
@@ -238,7 +234,6 @@ def delete(parameters = {}, headers = {}, &block)
     # This implementation of asynchronous delete is currently in alpha. Hence, it should not be used in production.
     def delete_async(parameters = {}, headers = {}, &block)
       @asynchronous_backend.execute(build_delete(parameters, headers, &block))
-      nil
     end
 
     # Makes an OPTIONS request to this URI. This is a convenience API