Add status field to info query since it is used in sync rake task

spree-contrib · Oct 13, 2016 · 1410b2a · 1410b2a
1 parent 8a271c3
commit 1410b2a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/lib/spree/chimpy/interface/list.rb b/lib/spree/chimpy/interface/list.rb
@@ -75,7 +75,7 @@ def info(email)
         #maximum of 50 emails allowed to be passed in
         begin
           response = api_member_call(email)
-            .retrieve(params: { "fields" => "email_address,merge_fields"})
+            .retrieve(params: { "fields" => "email_address,merge_fields,status"})
 
           response = response.symbolize_keys
           response.merge(:email => response[:email_address])

diff --git a/spec/lib/list_interface_spec.rb b/spec/lib/list_interface_spec.rb
@@ -74,7 +74,7 @@
   context "member info" do
     it "find when no errors" do
       expect(member_api).to receive(:retrieve).with(
-        { params: { "fields" => "email_address,merge_fields" } }
+        { params: { "fields" => "email_address,merge_fields,status" } }
       ).and_return(info_response)
       expect(interface.info(email)).to include(
         email_address: email,
@@ -83,7 +83,7 @@
     end
     it "adds legacy field email for backwards compatibility" do
       expect(member_api).to receive(:retrieve).with(
-        { params: { "fields" => "email_address,merge_fields" } }
+        { params: { "fields" => "email_address,merge_fields,status" } }
       ).and_return(info_response)
       expect(interface.info(email)).to include(email: email)
     end