ruby-pardot 1.2.0 → 1.4.1

data/spec/pardot/objects/lists_spec.rb

@@ -1,41 +1,38 @@
-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::Lists 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>
-          <list>
-            <name>Asdf List</name>
-          </list>
-          <list>
-            <name>Qwerty List</name>
-          </list>
-        </result>
-      </rsp>)
-    end
-    
-    before do
-      @client = create_client
-    end
-    
-    it "should take in some arguments" do
-      fake_get "/api/list/version/3/do/query?api_key=my_api_key&id_greater_than=200&format=simple&user_key=bar", sample_results
-      
-      @client.lists.query(:id_greater_than => 200).should == {"total_results" => 2, 
-        "list"=>[
-          {"name"=>"Asdf List"}, 
-          {"name"=>"Qwerty List"}
-        ]}
+  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>
+              <list>
+                <name>Asdf List</name>
+              </list>
+              <list>
+                <name>Qwerty List</name>
+              </list>
+            </result>
+          </rsp>)
+        end
+
+        it 'should take in some arguments' do
+          fake_get '/api/list/version/3/do/query?id_greater_than=200&format=simple', sample_results
+
+          expect(client.lists.query(id_greater_than: 200)).to eq({ 'total_results' => 2,
+                                                                   'list' => [
+                                                                     { 'name' => 'Asdf List' },
+                                                                     { 'name' => 'Qwerty List' }
+                                                                   ] })
+          assert_authorization_header auth_manager
+        end
+      end
     end
-    
   end
-  
-end
+end

data/spec/pardot/objects/opportunities_spec.rb

@@ -1,64 +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?api_key=my_api_key&id_greater_than=200&format=simple&user_key=bar", sample_results
-      
-      @client.opportunities.query(:id_greater_than => 200).should == {"total_results" => 2, 
-        "opportunity"=>[
-          {"type"=>"Great", "name"=>"Jim"}, 
-          {"type"=>"Good", "name"=>"Sue"}
-        ]}
-    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&api_key=my_api_key&user_key=bar&format=simple&name=Jim", sample_results
-      
-      @client.opportunities.create_by_email("user@test.com", :name => "Jim", :type => "Good").should == {"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 auth_manager
+        end
+      end
     end
-    
   end
-  
-end
+end

data/spec/pardot/objects/prospect_accounts_spec.rb

@@ -1,78 +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&user_key=bar&api_key=my_api_key", sample_results
-
-      @client.prospect_accounts.query(:assigned => true).should == {'total_results' => 2,
-        'prospectAccount'=>[
-          {'name'=>'Spaceships R Us'},
-          {'name'=>'Monsters Inc'}
-        ]}
-    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&user_key=bar&api_key=my_api_key', sample_results
-
-      @client.prospect_accounts.read('1234', :assigned => true).should == {'id' => '1234', 'name' => 'SupaDupaPanda' }
-    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?api_key=my_api_key&user_key=bar&format=simple&name=SuperPanda', sample_results
-
-      @client.prospect_accounts.create(:name => 'SuperPanda').should == {"name"=>"SuperPanda"}
+require File.expand_path("#{File.dirname(__FILE__)}/../../spec_helper")
 
+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,61 +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&user_key=bar&api_key=my_api_key", sample_results
-      
-      @client.prospects.query(:assigned => true).should == {"total_results" => 2, 
-        "prospect"=>[
-          {"last_name"=>"Smith", "first_name"=>"Jim"}, 
-          {"last_name"=>"Green", "first_name"=>"Sue"}
-        ]}
-    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@test.com?api_key=my_api_key&user_key=bar&format=simple&first_name=Jim", sample_results
-      
-      @client.prospects.create("user@test.com", :first_name => "Jim").should == {"last_name"=>"Smith", "first_name"=>"Jim"}
-      
+  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 auth_manager
+        end
+      end
     end
-    
   end
-  
-end
+end