hubspot-api-ruby 0.9.0 → 0.11.0

data/spec/lib/hubspot/connection_spec.rb

@@ -1,18 +1,12 @@
 describe Hubspot::Connection do
-  before do
-    Hubspot.configure hapikey: 'fake'
-  end
-
   describe ".get_json" do
     it "returns the parsed response from the GET request" do
       path = "/some/path"
       body = { key: "value" }
 
-      stub_request(:get, "https://api.hubapi.com/some/path?hapikey=fake").
-        to_return(status: 200, body: JSON.generate(body))
+      stub_request(:get, "https://api.hubapi.com/some/path").to_return(status: 200, body: JSON.generate(body))
 
       result = Hubspot::Connection.get_json(path, {})
-
       expect(result).to eq({ "key" => "value" })
     end
   end
@@ -22,11 +16,9 @@ describe Hubspot::Connection do
       path = "/some/path"
       body = { id: 1, name: "ABC" }
 
-      stub_request(:post, "https://api.hubapi.com/some/path?hapikey=fake&name=ABC").
-        to_return(status: 200, body: JSON.generate(body))
+      stub_request(:post, "https://api.hubapi.com/some/path?name=ABC").to_return(status: 200, body: JSON.generate(body))
 
       result = Hubspot::Connection.post_json(path, params: { name: "ABC" })
-
       expect(result).to eq({ "id" => 1, "name" => "ABC" })
     end
   end
@@ -35,11 +27,9 @@ describe Hubspot::Connection do
     it "returns the response from the DELETE request" do
       path = "/some/path"
 
-      stub_request(:delete, "https://api.hubapi.com/some/path?hapikey=fake").
-        to_return(status: 204, body: JSON.generate({}))
+      stub_request(:delete, "https://api.hubapi.com/some/path").to_return(status: 204, body: JSON.generate({}))
 
       result = Hubspot::Connection.delete_json(path, {})
-
       expect(result.code).to eq(204)
     end
   end
@@ -49,20 +39,18 @@ describe Hubspot::Connection do
       path = "/some/path"
       update_options = { params: {}, body: {} }
 
-      stub_request(:put, "https://api.hubapi.com/some/path?hapikey=fake").
-        to_return(status: 200, body: JSON.generate(vid: 123))
+      stub_request(:put, "https://api.hubapi.com/some/path").to_return(status: 200, body: JSON.generate(vid: 123))
 
       response = Hubspot::Connection.put_json(path, update_options)
 
       assert_requested(
         :put,
-        "https://api.hubapi.com/some/path?hapikey=fake",
+        "https://api.hubapi.com/some/path",
          {
            body: "{}",
            headers: { "Content-Type" => "application/json" },
          }
       )
-
       expect(response).to eq({ "vid" => 123 })
     end
 
@@ -72,13 +60,13 @@ describe Hubspot::Connection do
 
       logger = stub_logger
 
-      stub_request(:put, "https://api.hubapi.com/some/path?hapikey=fake").
-        to_return(status: 200, body: JSON.generate("response body"))
+      stub_request(:put, "https://api.hubapi.com/some/path").to_return(status: 200,
+                                                                       body: JSON.generate("response body"))
 
       Hubspot::Connection.put_json(path, update_options)
 
       expect(logger).to have_received(:info).with(<<~MSG)
-        Hubspot: https://api.hubapi.com/some/path?hapikey=fake.
+        Hubspot: https://api.hubapi.com/some/path.
         Body: {}.
         Response: 200 "response body"
       MSG
@@ -88,8 +76,7 @@ describe Hubspot::Connection do
       path = "/some/path"
       update_options = { params: {}, body: {} }
 
-      stub_request(:put, "https://api.hubapi.com/some/path?hapikey=fake").
-        to_return(status: 401)
+      stub_request(:put, "https://api.hubapi.com/some/path").to_return(status: 401)
 
       expect {
         Hubspot::Connection.put_json(path, update_options)
@@ -99,80 +86,85 @@ describe Hubspot::Connection do
 
   context 'private methods' do
     describe ".generate_url" do
-      let(:path){ "/test/:email/profile" }
-      let(:params){{email: "test"}}
-      let(:options){{}}
-      subject{ Hubspot::Connection.send(:generate_url, path, params, options) }
-      before{ Hubspot.configure(hapikey: "demo", portal_id: "62515") }
+      let(:path) { "/test/:email/profile" }
+      let(:params) { { email: "test" } }
+      let(:options) { {} }
+      subject { Hubspot::Connection.send(:generate_url, path, params, options) }
 
       it "doesn't modify params" do
-        expect{ subject }.to_not change{params}
+        expect { subject }.to_not change{params}
       end
 
       context "with a portal_id param" do
-        let(:path){ "/test/:portal_id/profile" }
-        let(:params){{}}
-        it{ should == "https://api.hubapi.com/test/62515/profile?hapikey=demo" }
+        let(:path) { "/test/:portal_id/profile" }
+        let(:params) { {} }
+
+        before do
+          Hubspot.configure(access_token: ENV.fetch("HUBSPOT_ACCESS_TOKEN"), portal_id: ENV.fetch("HUBSPOT_PORTAL_ID"))
+        end
+
+        it { should == "https://api.hubapi.com/test/#{ENV.fetch('HUBSPOT_PORTAL_ID')}/profile" }
       end
 
       context "when configure hasn't been called" do
-        before{ Hubspot::Config.reset! }
+        before { Hubspot::Config.reset! }
         it "raises a config exception" do
-          expect{ subject }.to raise_error Hubspot::ConfigurationError
+          expect { subject }.to raise_error Hubspot::ConfigurationError
         end
       end
 
       context "with interpolations but no params" do
-        let(:params){{}}
+        let(:params) { {} }
+
         it "raises an interpolation exception" do
           expect{ subject }.to raise_error Hubspot::MissingInterpolation
         end
       end
 
       context "with an interpolated param" do
-        let(:params){ {email: "email@address.com"} }
-        it{ should == "https://api.hubapi.com/test/email%40address.com/profile?hapikey=demo" }
+        let(:params) { { email: "email@address.com" } }
+        it { should == "https://api.hubapi.com/test/email%40address.com/profile" }
       end
 
       context "with multiple interpolated params" do
-        let(:path){ "/test/:email/:id/profile" }
-        let(:params){{email: "email@address.com", id: 1234}}
-        it{ should == "https://api.hubapi.com/test/email%40address.com/1234/profile?hapikey=demo" }
+        let(:path) { "/test/:email/:id/profile" }
+        let(:params) { { email: "email@address.com", id: 1234 } }
+        it { should == "https://api.hubapi.com/test/email%40address.com/1234/profile" }
       end
 
       context "with query params" do
-        let(:params){{email: "email@address.com", id: 1234}}
-        it{ should == "https://api.hubapi.com/test/email%40address.com/profile?id=1234&hapikey=demo" }
+        let(:params) { { email: "email@address.com", id: 1234 } }
+        it { should == "https://api.hubapi.com/test/email%40address.com/profile?id=1234" }
 
         context "containing a time" do
           let(:start_time) { Time.now }
-          let(:params){{email: "email@address.com", id: 1234, start: start_time}}
-          it{ should == "https://api.hubapi.com/test/email%40address.com/profile?id=1234&start=#{start_time.to_i * 1000}&hapikey=demo" }
+          let(:params) { { email: "email@address.com", id: 1234, start: start_time } }
+          it { should == "https://api.hubapi.com/test/email%40address.com/profile?id=1234&start=#{start_time.to_i * 1000}" }
         end
 
         context "containing a range" do
           let(:start_time) { Time.now }
           let(:end_time) { Time.now + 1.year }
-          let(:params){{email: "email@address.com", id: 1234, created__range: start_time..end_time }}
-          it{ should == "https://api.hubapi.com/test/email%40address.com/profile?id=1234&created__range=#{start_time.to_i * 1000}&created__range=#{end_time.to_i * 1000}&hapikey=demo" }
+          let(:params) { { email: "email@address.com", id: 1234, created__range: start_time..end_time  } }
+          it { should == "https://api.hubapi.com/test/email%40address.com/profile?id=1234&created__range=#{start_time.to_i * 1000}&created__range=#{end_time.to_i * 1000}" }
         end
 
         context "containing an array of strings" do
-          let(:path){ "/test/emails" }
-          let(:params){{batch_email: %w(email1@example.com email2@example.com)}}
-          it{ should == "https://api.hubapi.com/test/emails?email=email1%40example.com&email=email2%40example.com&hapikey=demo" }
+          let(:path) { "/test/emails" }
+          let(:params) { { batch_email: %w(email1@example.com email2@example.com) } }
+          it { should == "https://api.hubapi.com/test/emails?email=email1%40example.com&email=email2%40example.com" }
         end
       end
 
       context "with options" do
-        let(:options){ {base_url: "https://cool.com", hapikey: false} }
-        it{ should == "https://cool.com/test/test/profile"}
+        let(:options) { { base_url: "https://cool.com", access_token: false } }
+        it { should == "https://cool.com/test/test/profile" }
       end
 
       context "passing Array as parameters for batch mode, key is prefixed with batch_" do
         let(:path) { Hubspot::ContactList::LIST_BATCH_PATH }
         let(:params) { { batch_list_id: [1,2,3] } }
-        it{ should == "https://api.hubapi.com/contacts/v1/lists/batch?listId=1&listId=2&listId=3&hapikey=demo" }
+        it { should == "https://api.hubapi.com/contacts/v1/lists/batch?listId=1&listId=2&listId=3" }
       end
     end
   end
@@ -191,10 +183,7 @@ describe Hubspot::EventConnection do
     let(:headers) { nil }
 
     subject { described_class.trigger(path, options) }
-    before do
-      Hubspot.configure(hapikey: 'demo', portal_id: '62515')
-      allow(described_class).to receive(:get).and_return(true)
-    end
+    before { allow(described_class).to receive(:get).and_return(true) }
 
     it 'calls get with a custom url' do
       subject

data/spec/lib/hubspot/contact_list_spec.rb

@@ -1,7 +1,18 @@
 describe Hubspot::ContactList do
+  # uncomment if you need to create test data in your panel.
+  # note that sandboxes have a limit of 5 dynamic lists
+  # before(:all) do
+  #   VCR.use_cassette("create_all_lists") do
+  #     25.times { Hubspot::ContactList.create!(name: SecureRandom.hex) }
+  #     3.times { Hubspot::ContactList.create!(name: SecureRandom.hex, dynamic: true, filters: [[{ operator: "EQ", value: "@hubspot", property: "twitterhandle", type: "string"}]]) }
+  #   end
+  # end
+
   let(:example_contact_list_hash) do
     VCR.use_cassette("contact_list_example") do
-      HTTParty.get("https://api.hubapi.com/contacts/v1/lists/124897?hapikey=demo").parsed_response
+      headers = { Authorization: "Bearer #{ENV.fetch('HUBSPOT_ACCESS_TOKEN')}" }
+      response = HTTParty.get("https://api.hubapi.com/contacts/v1/lists/static?count=1", headers: headers).parsed_response
+      response['lists'].first
     end
   end
 
@@ -9,12 +20,6 @@ describe Hubspot::ContactList do
     Hubspot::ContactList.create!(name: "static list #{SecureRandom.hex}")
   end
 
-  let(:example_contact_hash) do
-    VCR.use_cassette("contact_example") do
-      HTTParty.get("https://api.hubapi.com/contacts/v1/contact/email/testingapis@hubspot.com/profile?hapikey=demo").parsed_response
-    end
-  end
-
   shared_examples "count and offset" do |params|
     it 'returns only the number of objects specified by count' do
       result = instance_exec(count: 2, &params[:block])
@@ -38,7 +43,7 @@ describe Hubspot::ContactList do
     its(:id) { should be_an(Integer) }
     its(:portal_id) { should be_a(Integer) }
     its(:name) { should_not be_empty }
-    its(:dynamic) { should be true }
+    its(:dynamic) { should be false }
     its(:properties) { should be_a(Hash) }
   end
 
@@ -57,12 +62,6 @@ describe Hubspot::ContactList do
       end
     end
 
-    it 'returns by default 20 contact lists' do
-      expect(list.contacts.count).to eql 20
-      contact = list.contacts.first
-      expect(contact).to be_a(Hubspot::Contact)
-    end
-
     it 'returns by default 20 contact lists with paging data' do
       contact_data = list.contacts({bypass_cache: true, paged: true})
       contacts = contact_data['contacts']
@@ -73,15 +72,10 @@ describe Hubspot::ContactList do
       expect(contacts.count).to eql 20
       contact = contacts.first
       expect(contact).to be_a(Hubspot::Contact)
-    end
-
-    it 'add default properties to the contacts returned' do
-      contact = list.contacts.first
       expect(contact.email).to_not be_empty
     end
 
     it_behaves_like 'count and offset', {block: ->(r) { Hubspot::ContactList.find(list.id).contacts(r) }}
-
   end
 
   describe '.create' do
@@ -119,13 +113,6 @@ describe Hubspot::ContactList do
   end
 
   describe '.all' do
-    before(:all) do
-      VCR.use_cassette("create_all_lists") do
-        25.times { Hubspot::ContactList.create!(name: SecureRandom.hex) }
-        25.times { Hubspot::ContactList.create!(name: SecureRandom.hex, dynamic: true, filters: [[{ operator: "EQ", value: "@hubspot", property: "twitterhandle", type: "string"}]]) }
-      end
-    end
-
     context 'all list types' do
       cassette 'find_all_lists'
 
@@ -146,7 +133,7 @@ describe Hubspot::ContactList do
 
       it 'returns by defaut all the static contact lists' do
         lists = Hubspot::ContactList.all(static: true)
-        expect(lists.count).to be > 20
+        expect(lists.count).to be > 2
 
         list = lists.first
         expect(list).to be_a(Hubspot::ContactList)
@@ -159,7 +146,7 @@ describe Hubspot::ContactList do
 
       it 'returns by defaut all the dynamic contact lists' do
         lists = Hubspot::ContactList.all(dynamic: true)
-        expect(lists.count).to be > 20
+        expect(lists.count).to be > 2
 
         list = lists.first
         expect(list).to be_a(Hubspot::ContactList)
@@ -173,7 +160,7 @@ describe Hubspot::ContactList do
       cassette "contact_list_find"
       subject { Hubspot::ContactList.find(id) }
 
-      let(:list) { Hubspot::ContactList.create!(name: SecureRandom.hex) }
+      let(:list) { Hubspot::ContactList.new(example_contact_list_hash) }
 
       context 'when the contact list is found' do
         let(:id) { list.id.to_i }
@@ -188,7 +175,7 @@ describe Hubspot::ContactList do
 
       context 'Wrong parameter type given' do
         it 'raises an error' do
-          expect { Hubspot::ContactList.find(static_list) }.to raise_error(Hubspot::InvalidParams)
+          expect { Hubspot::ContactList.find({ foo: :bar }) }.to raise_error(Hubspot::InvalidParams)
         end
       end
 
@@ -287,12 +274,13 @@ describe Hubspot::ContactList do
 
     context 'static list' do
       it 'returns true if removes all contacts in batch mode' do
-        contacts = static_list.contacts(count: 2)
-        expect(static_list.remove([contacts.first, contacts.last])).to be true
+        list = Hubspot::ContactList.new(example_contact_list_hash)
+        contacts = list.contacts(count: 2)
+        expect(list.remove([contacts.first, contacts.last])).to be true
       end
 
       it 'returns false if the contact cannot be removed' do
-        contact_not_present_in_list = Hubspot::Contact.new(example_contact_hash)
+        contact_not_present_in_list = Hubspot::Contact.new(1234)
         expect(static_list.remove(contact_not_present_in_list)).to be false
       end
     end

data/spec/lib/hubspot/contact_properties_spec.rb

@@ -38,6 +38,26 @@ describe Hubspot::ContactProperties do
       end
     end
 
+    describe '.find' do
+      context 'existing property' do
+        cassette 'contact_properties/existing_property'
+
+        it 'should return a contact property by name if it exists' do
+          response = Hubspot::ContactProperties.find('full_name')
+          expect(response['name']).to eq 'full_name'
+          expect(response['label']).to eq 'Full name'
+        end
+      end
+
+      context 'non-existent property' do
+        cassette 'contact_properties/non_existent_property'
+
+        it 'should return an error for a missing property' do
+          expect{ Hubspot::ContactProperties.find('this_does_not_exist') }.to raise_error(Hubspot::NotFoundError)
+        end
+      end
+    end
+
     let(:params) { {
       'name'                          => 'my_new_property',
       'label'                         => 'This is my new property',
@@ -141,6 +161,26 @@ describe Hubspot::ContactProperties do
       end
     end
 
+    describe '.find_group' do
+      context 'existing group' do
+        cassette 'contact_properties/existing_group'
+
+        it 'should return a contact property by name if it exists' do
+          response = Hubspot::ContactProperties.find_group('contactinformation')
+          expect(response['name']).to eq 'contactinformation'
+          expect(response['displayName']).to eq 'Contact information'
+        end
+      end
+
+      context 'non-existent group' do
+        cassette 'contact_properties/non_existent_group'
+
+        it 'should return an error for a missing group' do
+          expect{ Hubspot::ContactProperties.find_group('this_does_not_exist') }.to raise_error(Hubspot::NotFoundError)
+        end
+      end
+    end
+
     let(:params) { { 'name' => 'ff_group1', 'displayName' => 'Test Group One', 'displayOrder' => 100, 'badParam' => 99 } }
 
     describe '.create_group!' do

data/spec/lib/hubspot/contact_spec.rb

@@ -209,7 +209,7 @@ RSpec.describe Hubspot::Contact do
 
       let!(:contact1) { create :contact }
 
-      subject { contact1.merge(1) }
+      subject { contact1.merge(contact1.id) }
 
       it 'raises an error' do
         expect {

data/spec/lib/hubspot/custom_event_spec.rb

@@ -1,7 +1,8 @@
 RSpec.describe Hubspot::CustomEvent do
-  let(:portal_id) { '62515' }
-  let(:sent_portal_id) { portal_id }
-  before { Hubspot.configure(hapikey: 'demo', custom_event_prefix: 'foobar') }
+  before do
+    Hubspot.configure(access_token: ENV.fetch("HUBSPOT_ACCESS_TOKEN"), portal_id: ENV.fetch("HUBSPOT_PORTAL_ID"),
+                      custom_event_prefix: 'foobar')
+  end
 
   describe '.trigger' do
     let(:event_name) { 'my_awesome_event' }
@@ -9,7 +10,7 @@ RSpec.describe Hubspot::CustomEvent do
     let(:properties) { { prop_foo: 'bar' } }
     let(:options) { {} }
     let(:base_url) { 'https://api.hubapi.com' }
-    let(:url) { "#{base_url}/events/v3/send?hapikey=demo" }
+    let(:url) { "#{base_url}/events/v3/send" }
 
     subject { described_class.trigger(event_name, email, properties, options) }
 

data/spec/lib/hubspot/deal_properties_spec.rb

@@ -37,6 +37,26 @@ describe Hubspot::DealProperties do
         end
       end
     end
+    
+    describe '.find' do
+      context 'existing property' do
+        cassette 'deal_properties/existing_property'
+
+        it 'should return a deal property by name if it exists' do
+          response = Hubspot::DealProperties.find('hs_acv')
+          expect(response['name']).to eq 'hs_acv'
+          expect(response['label']).to eq 'Annual contract value'
+        end
+      end
+
+      context 'non-existent property' do
+        cassette 'deal_properties/non_existent_property'
+
+        it 'should return an error for a missing property' do
+          expect{ Hubspot::DealProperties.find('this_does_not_exist') }.to raise_error(Hubspot::NotFoundError)
+        end
+      end
+    end
 
     let(:params) { {
       'name'                          => "my_new_property_#{SecureRandom.hex}",
@@ -140,6 +160,26 @@ describe Hubspot::DealProperties do
       end
     end
 
+    describe '.find_group' do
+      context 'existing group' do
+        cassette 'deal_properties/existing_group'
+
+        it 'should return a deal property by name if it exists' do
+          response = Hubspot::DealProperties.find_group('deal_revenue')
+          expect(response['name']).to eq 'deal_revenue'
+          expect(response['displayName']).to eq 'Deal revenue'
+        end
+      end
+
+      context 'non-existent group' do
+        cassette 'deal_properties/non_existent_group'
+
+        it 'should return an error for a missing group' do
+          expect{ Hubspot::DealProperties.find_group('this_does_not_exist') }.to raise_error(Hubspot::NotFoundError)
+        end
+      end
+    end
+
     let(:params) { { 'name' => 'ff_group1', 'displayName' => 'Test Group One', 'displayOrder' => 100, 'badParam' => 99 } }
 
     describe '.create_group!' do

data/spec/lib/hubspot/engagement_spec.rb

@@ -1,22 +1,14 @@
 describe Hubspot::Engagement do
-
   let(:contact) { Hubspot::Contact.create("#{SecureRandom.hex}@hubspot.com") }
   let(:engagement) { Hubspot::EngagementNote.create!(contact.id, "foo") }
-  let(:example_engagement_hash) do
-    VCR.use_cassette("engagement_example") do
-      HTTParty.get("https://api.hubapi.com/engagements/v1/engagements/3981023?hapikey=demo").parsed_response
-    end
-  end
-  let(:example_associated_engagement_hash) do
-    VCR.use_cassette("engagement_associated_example") do
-      HTTParty.get("https://api.hubapi.com/engagements/v1/engagements/58699206?hapikey=demo").parsed_response
-    end
-  end
 
   # http://developers.hubspot.com/docs/methods/contacts/get_contact
 
   describe "#initialize" do
     subject{ Hubspot::Engagement.new(example_engagement_hash) }
+
+    let(:example_engagement_hash) { { 'engagement' => { 'id' => 3981023, 'portalId' => 62515, 'associations' => {} } } }
+
     it  { should be_an_instance_of Hubspot::Engagement }
     its (:id) { should == 3981023 }
   end

data/spec/lib/hubspot/event_spec.rb

@@ -1,7 +1,8 @@
 describe Hubspot::Event do
-  let(:portal_id) { '62515' }
+  let(:portal_id) { ENV.fetch("HUBSPOT_PORTAL_ID") }
   let(:sent_portal_id) { portal_id }
-  before { Hubspot.configure(hapikey: 'demo', portal_id: portal_id) }
+
+  before { Hubspot.configure(access_token: ENV.fetch("HUBSPOT_ACCESS_TOKEN"), portal_id: portal_id) }
 
   describe '.trigger' do
     let(:event_id) { '000000001625' }

data/spec/lib/hubspot/file_spec.rb

@@ -1,12 +1,23 @@
-describe Hubspot do
-  describe Hubspot::File do
-    before { Hubspot.configure(hapikey: "demo") }
-    describe ".find_by_id" do
-      it "should fetch specific file" do
-        VCR.use_cassette("file_find") do
-          file = Hubspot::File.find_by_id(5323360053)
-          file.id.should eq(5323360053)
-        end
+describe Hubspot::File do
+  let(:example_file_hash) do
+    VCR.use_cassette('file_example') do
+      headers = { Authorization: "Bearer #{ENV.fetch('HUBSPOT_ACCESS_TOKEN')}" }
+      body = {
+        file: File.open(File::NULL, 'r'),
+        options: { access: 'PRIVATE', ttl: 'P1M', overwrite: false }.to_json,
+        folderPath: '/'
+      }
+      url = 'https://api.hubapi.com/filemanager/api/v3/files/upload'
+      HTTParty.post(url, body: body, headers: headers).parsed_response
+    end
+  end
+
+  describe ".find_by_id" do
+    it "should fetch specific file" do
+      VCR.use_cassette("file_find") do
+        id = example_file_hash['objects'].first['id']
+        file = Hubspot::File.find_by_id(id)
+        file.id.should eq(id)
       end
     end
   end

data/spec/lib/hubspot/form_spec.rb

@@ -1,7 +1,9 @@
 describe Hubspot::Form do
   let(:example_form_hash) do
     VCR.use_cassette('form_example') do
-      HTTParty.get("https://api.hubapi.com#{Hubspot::Form::FORMS_PATH}/038e819c-5262-4c13-8550-8cabe38c4309/?hapikey=demo").parsed_response
+      guid = Hubspot::Form.all.first.guid
+      headers = { Authorization: "Bearer #{ENV.fetch('HUBSPOT_ACCESS_TOKEN')}" }
+      HTTParty.get("https://api.hubapi.com#{Hubspot::Form::FORMS_PATH}/#{guid}", headers: headers).parsed_response
     end
   end
 
@@ -29,8 +31,6 @@ describe Hubspot::Form do
   end
 
   describe '.all' do
-    before { Hubspot.configure(hapikey: 'demo') }
-
     cassette 'find_all_forms'
 
     it 'returns all forms' do
@@ -42,8 +42,6 @@ describe Hubspot::Form do
   end
 
   describe '.find' do
-    before { Hubspot.configure(hapikey: 'demo') }
-
     cassette 'form_find'
     subject { Hubspot::Form.find(Hubspot::Form.all.first.guid) }
 
@@ -79,7 +77,6 @@ describe Hubspot::Form do
   end
 
   describe '#fields' do
-    before { Hubspot.configure(hapikey: 'demo') }
     context 'returning all the fields' do
       cassette 'fields_among_form'
 
@@ -119,11 +116,11 @@ describe Hubspot::Form do
   describe '#submit' do
     cassette 'form_submit_data'
 
-    let(:form) { Hubspot::Form.create!(create_params) }
+    let(:form) { Hubspot::Form.all.first }
 
     context 'with a valid portal id' do
       before do
-        Hubspot.configure(hapikey: ENV.fetch("HUBSPOT_HAPI_KEY"), portal_id: ENV.fetch("HUBSPOT_PORTAL_ID"))
+        Hubspot.configure(access_token: ENV.fetch("HUBSPOT_ACCESS_TOKEN"), portal_id: ENV.fetch("HUBSPOT_PORTAL_ID"))
       end
 
       it 'returns true if the form submission is successful' do
@@ -135,7 +132,7 @@ describe Hubspot::Form do
 
     context 'with an invalid portal id' do
       before do
-        Hubspot.configure(hapikey: ENV.fetch("HUBSPOT_HAPI_KEY"), portal_id: "xxx")
+        Hubspot.configure(access_token: ENV.fetch("HUBSPOT_ACCESS_TOKEN"), portal_id: "xxx")
       end
 
       it 'returns false in case of errors' do
@@ -149,7 +146,7 @@ describe Hubspot::Form do
       let(:f) { Hubspot::Form.new('guid' => form.guid) }
 
       before do
-        Hubspot.configure(hapikey: ENV.fetch("HUBSPOT_HAPI_KEY"), portal_id: ENV.fetch("HUBSPOT_PORTAL_ID"))
+        Hubspot.configure(access_token: ENV.fetch("HUBSPOT_ACCESS_TOKEN"), portal_id: ENV.fetch("HUBSPOT_PORTAL_ID"))
       end
 
       it 'returns true if the form submission is successful' do
@@ -170,8 +167,8 @@ describe Hubspot::Form do
     let(:params) { { name: new_name, redirect: redirect } }
     subject { form.update!(params) }
 
-    it { should be_an_instance_of Hubspot::Form }
     it 'updates properties' do
+      should be_an_instance_of Hubspot::Form
       subject.properties['name'].should start_with('updated form name ')
       subject.properties['redirect'].should be == redirect
     end

data/spec/lib/hubspot/meeting_spec.rb

@@ -1,6 +1,12 @@
 RSpec.describe Hubspot::Meeting do
 
-  let(:hubspot_owner_id) { 123 }
+  let(:hubspot_owner_id) do
+    VCR.use_cassette('meeting_owner') do
+      headers = { Authorization: "Bearer #{ENV.fetch('HUBSPOT_ACCESS_TOKEN')}" }
+      owners = HTTParty.get('https://api.hubapi.com/owners/v2/owners', headers: headers).parsed_response
+      owners.first['ownerId']
+    end
+  end
   let(:hs_meeting_title) { 'hs_meeting_title' }
   let(:hs_meeting_body) { 'hs_meeting_body' }
   let(:hs_meeting_start_time) { DateTime.strptime('2022-05-03T10:00:00+01:00', '%Y-%m-%dT%H:%M:%S%z') }