connectwise_sdk 0.0.11 → 0.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 59719857b1137c92f2069632546dafb209909cbc
-  data.tar.gz: ebb3dd44f5feaa449268e97013e802bca925e8b0
+  metadata.gz: 7c496eabc57d7fe1e9df44d89ccd55b7cc86f1f6
+  data.tar.gz: a239f7fe73a2973a0609396c58d50aee687bc29d
 SHA512:
-  metadata.gz: 7610d28b69c7f71ff751c43fbd5b48cf94e45a26be07757dd581caf41cb4a67d81a75e6480d8a7cc9eb757e594c510fd9e6caac1908731fd04a5296d0205809e
-  data.tar.gz: 73e74dffb140980811b5750b2780ef39ca6893c2559799a00252b2db8d792e7393e3f73c173266b369a669e2d59ab9e629091a7adc6c046baee148adf6bd703d
+  metadata.gz: 33eb89be5d8001bf092894a3bec732e49e4dd51e293cbcb0dfbf6a573f80e896afa9bcf7545bab4a9577b9363134aee155a64e78acd8fc7bbf6f2d75131c3223
+  data.tar.gz: a32dfd67e4ced046e5ea00a8c750b81361987b85154b7dfe64a7b3dde2b60c62547f72342b1459449fe687aff3f7b4c2a13d783a65bd780fd1d72369a53a6832

data/lib/connectwise/opportunity.rb

@@ -14,8 +14,26 @@ module Connectwise
       @contact = contact
     end
 
+    # TODO - These two id methods are both hacky - should have a contact or company object
+    # Probably need two different objects, one for finds, and another for gets (need to confirm save and get return the same thing)
     def company_id
-      @company
+      if @company.respond_to?(:to_hash)
+        @company.fetch(:company_id) { @company }
+      elsif @company.respond_to?(:company_id)
+        @company.company_id
+      else
+        @company
+      end
+    end
+
+    def contact_id
+      if @contact.respond_to?(:to_hash)
+        @contact.fetch(:contact_rec_id) { @contact }
+      elsif @contact.respond_to?(:id)
+        @contact.id
+      else
+        @contact
+      end
     end
 
     private
@@ -24,8 +42,8 @@ module Connectwise
       attrs = super
       attrs.delete('CompanyId')
       attrs.delete('ContactId')
-      attrs['Company'] = {'CompanyID' => @company.company_id} if @company
-      attrs['Contact'] = {'Id' => @contact.id} if @contact
+      attrs['Company'] = {'CompanyID' => company_id} if @company
+      attrs['Contact'] = {'Id' => contact_id} if @contact
       attrs
     end
 

data/lib/connectwise/version.rb

@@ -1,3 +1,3 @@
 module Connectwise
-  VERSION = "0.0.11"
+  VERSION = "0.1.0"
 end

data/spec/lib/connectwise/opportunity_spec.rb

@@ -7,9 +7,9 @@ describe Connectwise::Opportunity do
   let(:contact_attrs) { {first_name: 'Malcom', last_name: 'Reynolds'} }
   let(:opp_attrs) { {name: 'Something', source: 'EOP', primary_sales_rep: 'Admin1'} }
   let(:full_op) {
-    new_company = Connectwise::Company.new(conn, company_attrs).save
-    new_contact = Connectwise::Contact.new(conn, contact_attrs.merge(company: new_company)).save
-    Connectwise::Opportunity.new(conn, opp_attrs.merge(company: new_company, contact: new_contact)).save
+    @new_company = Connectwise::Company.new(conn, company_attrs).save
+    @new_contact = Connectwise::Contact.new(conn, contact_attrs.merge(company: @new_company)).save
+    Connectwise::Opportunity.new(conn, opp_attrs.merge(company: @new_company, contact: @new_contact)).save
   }
   subject {Connectwise::Opportunity}
 
@@ -35,6 +35,15 @@ describe Connectwise::Opportunity do
     new_opp = full_op
     found_opp = Connectwise::Opportunity.find(conn, new_opp.id)
     expect(found_opp.id).not_to be_nil
+    expect(found_opp.company_id).to eq @new_company.company_id
+    expect(found_opp.contact_id).to eq @new_contact.id
+  end
+
+  it 'can change and resave a found opportunity' do
+    new_opp = full_op
+    found_opp = Connectwise::Opportunity.find(conn, new_opp.id)
+    found_opp.primary_sales_rep = 'Admin2'
+    expect {found_opp.save}.not_to raise_error
   end
 
   it 'fails to find an opportunity' do

data/spec/spec_helper.rb

@@ -7,5 +7,5 @@ def connectwise_credentials
 end
 
 RSpec.configure do |c|
-  c.filter_run_including :focus => true
+  #c.filter_run_including :focus => true
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: connectwise_sdk
 version: !ruby/object:Gem::Version
-  version: 0.0.11
+  version: 0.1.0
 platform: ruby
 authors:
 - Emery A. Miller
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-14 00:00:00.000000000 Z
+date: 2014-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler