rename params dup variable to params_copy. add test. remove version bump.

Author: danarnold

lib/emailable/client.rb

@@ -17,9 +17,9 @@ def initialize
     end
 
     def request(method, endpoint, params = {})
-      req_params = params.dup
-      api_key = req_params.delete(:api_key)
-      access_token = req_params.delete(:access_token)
+      params_copy = params.dup
+      api_key = params_copy.delete(:api_key)
+      access_token = params_copy.delete(:access_token)
 
       uri = URI("#{@base_url}/#{endpoint}")
       headers = {
@@ -31,12 +31,14 @@ def request(method, endpoint, params = {})
         tries ||= 3
         http_response =
           if method == :get
-            uri.query = URI.encode_www_form(req_params) unless req_params.empty?
+            unless params_copy.empty?
+              uri.query = URI.encode_www_form(params_copy)
+            end
             request = Net::HTTP::Get.new(uri, headers)
             @connection.request(request)
           elsif method == :post
             request = Net::HTTP::Post.new(uri, headers)
-            request.body = req_params.to_json
+            request.body = params_copy.to_json
             @connection.request(request)
           end
 

lib/emailable/version.rb

@@ -1,3 +1,3 @@
 module Emailable
-  VERSION = '4.2.2'
+  VERSION = '4.2.1'
 end

test/authentication_test.rb

@@ -30,4 +30,12 @@ def test_request_time_authentication_takes_priority_over_global
     refute_nil Emailable.verify(@email, api_key: @api_key).domain
   end
 
+  def test_does_not_modify_params
+    params = { api_key: @api_key, email: @email }
+    Emailable.verify(params[:email], params)
+
+    assert_equal @api_key, params[:api_key]
+    assert_equal @email, params[:email]
+  end
+
 end