ruby-pardot 1.3.2 → 1.4.0

data/spec/pardot/objects/opportunities_spec.rb

@@ -1,66 +1,63 @@
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+# frozen_string_literal: true
+
+require File.expand_path("#{File.dirname(__FILE__)}/../../spec_helper")
 
 describe Pardot::Objects::Opportunities do
-  
-  before do
-    @client = create_client
-  end
-  
-  describe "query" do
-    
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>\n<rsp stat="ok" version="1.0">
-        <result>
-          <total_results>2</total_results>
-          <opportunity>
-            <name>Jim</name>
-            <type>Great</type>
-          </opportunity>
-          <opportunity>
-            <name>Sue</name>
-            <type>Good</type>
-          </opportunity>
-        </result>
-      </rsp>)
-    end
-    
-    before do
-      @client = create_client
-    end
-    
-    it "should take in some arguments" do
-      fake_get "/api/opportunity/version/3/do/query?id_greater_than=200&format=simple", sample_results
-      
-      expect(@client.opportunities.query(:id_greater_than => 200)).to eq({"total_results" => 2, 
-        "opportunity"=>[
-          {"type"=>"Great", "name"=>"Jim"}, 
-          {"type"=>"Good", "name"=>"Sue"}
-        ]})
-      assert_authorization_header
-    end
-    
-  end
-  
-  describe "create_by_email" do
-    
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>
-      <rsp stat="ok" version="1.0">
-        <opportunity>
-          <name>Jim</name>
-          <type>Good</type>
-        </opportunity>
-      </rsp>)
-    end
-    
-    it "should return the prospect" do
-      fake_post "/api/opportunity/version/3/do/create/prospect_email/user@test.com?type=Good&format=simple&name=Jim", sample_results
-      
-      expect(@client.opportunities.create_by_email("user@test.com", :name => "Jim", :type => "Good")).to eq({"name"=>"Jim", "type"=>"Good"})
+  create_auth_managers.each do |auth_manager|
+    context auth_manager.test_name_suffix do
+      let(:client) { auth_manager.create_client }
+
+      describe 'query' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>\n<rsp stat="ok" version="1.0">
+            <result>
+              <total_results>2</total_results>
+              <opportunity>
+                <name>Jim</name>
+                <type>Great</type>
+              </opportunity>
+              <opportunity>
+                <name>Sue</name>
+                <type>Good</type>
+              </opportunity>
+            </result>
+          </rsp>)
+        end
+
+        it 'should take in some arguments' do
+          fake_get '/api/opportunity/version/3/do/query?id_greater_than=200&format=simple', sample_results
+
+          expected = {
+            'total_results' => 2,
+            'opportunity' => [
+              { 'type' => 'Great', 'name' => 'Jim' },
+              { 'type' => 'Good', 'name' => 'Sue' }
+            ]
+          }
+          expect(client.opportunities.query(id_greater_than: 200)).to eq(expected)
+          assert_authorization_header auth_manager
+        end
+      end
+
+      describe 'create_by_email' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>
+          <rsp stat="ok" version="1.0">
+            <opportunity>
+              <name>Jim</name>
+              <type>Good</type>
+            </opportunity>
+          </rsp>)
+        end
+
+        it 'should return the prospect' do
+          fake_post '/api/opportunity/version/3/do/create/prospect_email/user@test.com?type=Good&format=simple&name=Jim', sample_results
+
+          expect(client.opportunities.create_by_email('user@test.com', name: 'Jim', type: 'Good')).to eq({ 'name' => 'Jim', 'type' => 'Good' })
 
-      assert_authorization_header
+          assert_authorization_header auth_manager
+        end
+      end
     end
-    
   end
-  
-end
+end

data/spec/pardot/objects/prospect_accounts_spec.rb

@@ -1,80 +1,77 @@
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+# frozen_string_literal: true
 
-describe Pardot::Objects::ProspectAccounts do
-
-  before do
-    @client = create_client
-  end
-
-  describe "query" do
-
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>
-      <rsp stat="ok" version="1.0">
-        <result>
-          <total_results>2</total_results>
-          <prospectAccount>
-            <name>Spaceships R Us</name>
-          </prospectAccount>
-          <prospectAccount>
-            <name>Monsters Inc</name>
-          </prospectAccount>
-        </result>
-      </rsp>)
-    end
-
-    it "should take in some arguments and respond with valid items" do
-      fake_get "/api/prospectAccount/version/3/do/query?assigned=true&format=simple", sample_results
-
-      expect(@client.prospect_accounts.query(:assigned => true)).to eq({'total_results' => 2,
-        'prospectAccount'=>[
-          {'name'=>'Spaceships R Us'},
-          {'name'=>'Monsters Inc'}
-        ]})
-      assert_authorization_header
-    end
-
-  end
-
-  describe 'read' do
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>
-      <rsp stat="ok" version="1.0">
-        <prospectAccount>
-          <id>1234</id>
-          <name>SupaDupaPanda</name>
-        </prospectAccount>
-        </rsp>)
-    end
+require File.expand_path("#{File.dirname(__FILE__)}/../../spec_helper")
 
-    it 'should return a valid account' do
-      fake_post '/api/prospectAccount/version/3/do/read/id/1234?assigned=true&format=simple', sample_results
-
-      expect(@client.prospect_accounts.read('1234', :assigned => true)).to eq({'id' => '1234', 'name' => 'SupaDupaPanda' })
-      assert_authorization_header
-    end
-
-  end
-
-
-  describe 'create' do
-
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>
-      <rsp stat="ok" version="1.0">
-        <prospectAccount>
-          <name>SuperPanda</name>
-        </prospectAccount>
-      </rsp>)
-    end
-
-    it 'should return the prospect account' do
-      fake_post '/api/prospectAccount/version/3/do/create?format=simple&name=SuperPanda', sample_results
-
-      expect(@client.prospect_accounts.create(:name => 'SuperPanda')).to eq({"name"=>"SuperPanda"})
-      assert_authorization_header
+describe Pardot::Objects::ProspectAccounts do
+  create_auth_managers.each do |auth_manager|
+    context auth_manager.test_name_suffix do
+      let(:client) { auth_manager.create_client }
+
+      describe 'query' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>
+          <rsp stat="ok" version="1.0">
+            <result>
+              <total_results>2</total_results>
+              <prospectAccount>
+                <name>Spaceships R Us</name>
+              </prospectAccount>
+              <prospectAccount>
+                <name>Monsters Inc</name>
+              </prospectAccount>
+            </result>
+          </rsp>)
+        end
+
+        it 'should take in some arguments and respond with valid items' do
+          fake_get '/api/prospectAccount/version/3/do/query?assigned=true&format=simple', sample_results
+
+          expect(client.prospect_accounts.query(assigned: true)).to eq({ 'total_results' => 2,
+                                                                         'prospectAccount' => [
+                                                                           { 'name' => 'Spaceships R Us' },
+                                                                           { 'name' => 'Monsters Inc' }
+                                                                         ] })
+          assert_authorization_header auth_manager
+        end
+      end
+
+      describe 'read' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>
+          <rsp stat="ok" version="1.0">
+            <prospectAccount>
+              <id>1234</id>
+              <name>SupaDupaPanda</name>
+            </prospectAccount>
+            </rsp>)
+        end
+
+        it 'should return a valid account' do
+          fake_post '/api/prospectAccount/version/3/do/read/id/1234?assigned=true&format=simple', sample_results
+
+          expect(client.prospect_accounts.read('1234',
+                                               assigned: true)).to eq({ 'id' => '1234', 'name' => 'SupaDupaPanda' })
+          assert_authorization_header auth_manager
+        end
+      end
+
+      describe 'create' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>
+          <rsp stat="ok" version="1.0">
+            <prospectAccount>
+              <name>SuperPanda</name>
+            </prospectAccount>
+          </rsp>)
+        end
+
+        it 'should return the prospect account' do
+          fake_post '/api/prospectAccount/version/3/do/create?format=simple&name=SuperPanda', sample_results
+
+          expect(client.prospect_accounts.create(name: 'SuperPanda')).to eq({ 'name' => 'SuperPanda' })
+          assert_authorization_header auth_manager
+        end
+      end
     end
-
   end
-
 end

data/spec/pardot/objects/prospects_spec.rb

@@ -1,62 +1,62 @@
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+# frozen_string_literal: true
+
+require File.expand_path("#{File.dirname(__FILE__)}/../../spec_helper")
 
 describe Pardot::Objects::Prospects do
-  
-  before do
-    @client = create_client
-  end
-  
-  describe "query" do
-    
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>
-      <rsp stat="ok" version="1.0">
-        <result>
-          <total_results>2</total_results>
-          <prospect>
-            <first_name>Jim</first_name>
-            <last_name>Smith</last_name>
-          </prospect>
-          <prospect>
-            <first_name>Sue</first_name>
-            <last_name>Green</last_name>
-          </prospect>
-        </result>
-      </rsp>)
-    end
-    
-    it "should take in some arguments" do
-      fake_get "/api/prospect/version/3/do/query?assigned=true&format=simple", sample_results
-      
-      expect(@client.prospects.query(:assigned => true)).to eq({"total_results" => 2, 
-        "prospect"=>[
-          {"last_name"=>"Smith", "first_name"=>"Jim"}, 
-          {"last_name"=>"Green", "first_name"=>"Sue"}
-        ]})
-      assert_authorization_header
-    end
-    
-  end
-  
-  describe "create" do
-    
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>
-      <rsp stat="ok" version="1.0">
-        <prospect>
-          <first_name>Jim</first_name>
-          <last_name>Smith</last_name>
-        </prospect>
-      </rsp>)
-    end
-    
-    it "should return the prospect" do
-      fake_post "/api/prospect/version/3/do/create/email/user%40test.com?first_name=Jim&format=simple", sample_results
+  create_auth_managers.each do |auth_manager|
+    context auth_manager.test_name_suffix do
+      let(:client) { auth_manager.create_client }
+
+      describe 'query' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>
+          <rsp stat="ok" version="1.0">
+            <result>
+              <total_results>2</total_results>
+              <prospect>
+                <first_name>Jim</first_name>
+                <last_name>Smith</last_name>
+              </prospect>
+              <prospect>
+                <first_name>Sue</first_name>
+                <last_name>Green</last_name>
+              </prospect>
+            </result>
+          </rsp>)
+        end
+
+        it 'should take in some arguments' do
+          fake_get '/api/prospect/version/3/do/query?assigned=true&format=simple', sample_results
+
+          expect(client.prospects.query(assigned: true)).to eq({ 'total_results' => 2,
+                                                                 'prospect' => [
+                                                                   { 'last_name' => 'Smith', 'first_name' => 'Jim' },
+                                                                   { 'last_name' => 'Green', 'first_name' => 'Sue' }
+                                                                 ] })
+          assert_authorization_header auth_manager
+        end
+      end
+
+      describe 'create' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>
+          <rsp stat="ok" version="1.0">
+            <prospect>
+              <first_name>Jim</first_name>
+              <last_name>Smith</last_name>
+            </prospect>
+          </rsp>)
+        end
+
+        it 'should return the prospect' do
+          fake_post '/api/prospect/version/3/do/create/email/user%40test.com?first_name=Jim&format=simple',
+                    sample_results
 
-      expect(@client.prospects.create("user@test.com", :first_name => "Jim")).to eq({"last_name"=>"Smith", "first_name"=>"Jim"})
-      assert_authorization_header
+          expect(client.prospects.create('user@test.com',
+                                         first_name: 'Jim')).to eq({ 'last_name' => 'Smith', 'first_name' => 'Jim' })
+          assert_authorization_header auth_manager
+        end
+      end
     end
-    
   end
-  
 end

data/spec/pardot/objects/users_spec.rb

@@ -1,65 +1,62 @@
-require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+# frozen_string_literal: true
+
+require File.expand_path("#{File.dirname(__FILE__)}/../../spec_helper")
 
 describe Pardot::Objects::Users do
-  
-  before do
-    @client = create_client
-  end
-  
-  describe "query" do
-    
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>\n<rsp stat="ok" version="1.0">
-        <result>
-          <total_results>2</total_results>
-          <user>
-            <email>user@test.com</email>
-            <first_name>Jim</first_name>
-          </user>
-          <user>
-            <email>user@example.com</email>
-            <first_name>Sue</first_name>
-          </user>
-        </result>
-      </rsp>)
-    end
-    
-    before do
-      @client = create_client
-    end
-    
-    it "should take in some arguments" do
-      fake_get "/api/user/version/3/do/query?id_greater_than=200&format=simple", sample_results
-      
-      expect(@client.users.query(:id_greater_than => 200)).to eq({"total_results" => 2, 
-        "user"=>[
-          {"email"=>"user@test.com", "first_name"=>"Jim"}, 
-          {"email"=>"user@example.com", "first_name"=>"Sue"}
-        ]})
-      assert_authorization_header
-    end
-    
-  end
-  
-  describe "read_by_email" do
-    
-    def sample_results
-      %(<?xml version="1.0" encoding="UTF-8"?>
-      <rsp stat="ok" version="1.0">
-        <user>
-          <email>user@example.com</email>
-          <first_name>Sue</first_name>
-        </user>
-      </rsp>)
-    end
-    
-    it "should return the prospect" do
-      fake_post "/api/user/version/3/do/read/email/user@test.com?format=simple", sample_results
-      
-      expect(@client.users.read_by_email("user@test.com")).to eq({"email"=>"user@example.com", "first_name"=>"Sue"})
-      assert_authorization_header
+  create_auth_managers.each do |auth_manager|
+    context auth_manager.test_name_suffix do
+      let(:client) { auth_manager.create_client }
+
+      describe 'query' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>\n<rsp stat="ok" version="1.0">
+            <result>
+              <total_results>2</total_results>
+              <user>
+                <email>user@test.com</email>
+                <first_name>Jim</first_name>
+              </user>
+              <user>
+                <email>user@example.com</email>
+                <first_name>Sue</first_name>
+              </user>
+            </result>
+          </rsp>)
+        end
+
+        it 'should take in some arguments' do
+          fake_get '/api/user/version/3/do/query?id_greater_than=200&format=simple', sample_results
+
+          expect(client.users.query(id_greater_than: 200)).to eq({ 'total_results' => 2,
+                                                                   'user' => [
+                                                                     { 'email' => 'user@test.com',
+                                                                       'first_name' => 'Jim' },
+                                                                     { 'email' => 'user@example.com',
+                                                                       'first_name' => 'Sue' }
+                                                                   ] })
+          assert_authorization_header auth_manager
+        end
+      end
+
+      describe 'read_by_email' do
+        def sample_results
+          %(<?xml version="1.0" encoding="UTF-8"?>
+          <rsp stat="ok" version="1.0">
+            <user>
+              <email>user@example.com</email>
+              <first_name>Sue</first_name>
+            </user>
+          </rsp>)
+        end
+
+        it 'should return the prospect' do
+          fake_post '/api/user/version/3/do/read/email/user@test.com?format=simple', sample_results
+
+          expect(client.users.read_by_email('user@test.com')).to eq({ 'email' => 'user@example.com',
+                                                                      'first_name' => 'Sue' })
+          assert_authorization_header auth_manager
+        end
+      end
     end
-    
   end
-  
-end
+end