salesflip-jobboersen_integration 0.1.1 → 0.2.0

data/lib/app/lib/account_extras.rb

@@ -3,8 +3,8 @@ module AccountExtras
     base.class_eval do
       field :tjb_id
       
-      after_create :create_in_tjb
-      after_update :update_in_tjb, :unless => :do_not_update
+      after_create :create_in_tjb, :if => :prospect_or_customer?
+      after_update :update_in_tjb, :unless => :do_not_update, :if => :prospect_or_customer?
       
       attr_accessor :do_not_update
     end
@@ -17,19 +17,29 @@ module AccountExtras
         "/administration/accounts.xml?auth_token=#{self.user.tjb_auth_token}",
           self.to_xml(:only => [:name]))
       attributes = Hash.from_xml(response.body)['account']
-      self.update_attributes :tjb_id => attributes['_id'],
-        :do_not_update => true
+      self.update_attributes :tjb_id => attributes['_id'], :do_not_update => true
     rescue
       nil
     end
     
     def update_in_tjb
-      connection.put("/administration/accounts/#{self.tjb_id}.xml?auth_token=#{self.user.tjb_auth_token}",
-        self.to_xml(:only => [:name]))
+      if self.tjb_id
+        connection.put(
+          "/administration/accounts/#{self.tjb_id}.xml?auth_token=#{self.user.tjb_auth_token}",
+          self.to_xml(:only => [:name]))
+      else
+        create_in_tjb
+      end
+    rescue
+      nil
     end
     
     def connection
       ActiveResource::Connection.new(JobboersenIntegration.jobboersen_uri)
     end
+    
+    def prospect_or_customer?
+      %w(Prospect Customer).include?(self.account_type)
+    end
   end
 end

data/lib/app/lib/contact_extras.rb

@@ -4,7 +4,7 @@ module ContactExtras
       field :tjb_id
       
       after_create :create_in_tjb
-      after_update :update_in_tjb, :unless => :do_not_put
+      after_update :update_in_tjb, :unless => :do_not_update
       
       attr_accessor :do_not_update
     end
@@ -23,8 +23,13 @@ module ContactExtras
     end
     
     def update_in_tjb
-      connection.put(
-        "/administration/contacts/#{self.tjb_id}.xml?auth_token=#{self.user.tjb_auth_token}", tjb_xml)
+      if self.tjb_id
+        connection.put(
+          "/administration/contacts/#{self.tjb_id}.xml?auth_token=#{self.user.tjb_auth_token}",
+          tjb_xml)
+      else
+        create_in_tjb
+      end
     rescue
       nil
     end

data/lib/app/lib/user_extras.rb

@@ -20,17 +20,20 @@ module UserExtras
       if response.code == '200'
         attributes = Hash.from_xml(response.body)['administrator']
         self.update_attributes :tjb_id => attributes['_id'],
-          :tjb_auth_token => attributes['authentication_token'],
-          :do_not_update => true
+          :tjb_auth_token => attributes['authentication_token'], :do_not_update => true
       end
     rescue
       nil
     end
     
     def update_in_tjb
-      connection.put(
-        "/administration/administrators/#{self.tjb_id}.xml?auth_token=#{self.tjb_auth_token}",
-        self.to_xml(:only => :email))
+      if self.tjb_id
+        connection.put(
+          "/administration/administrators/#{self.tjb_id}.xml?auth_token=#{self.tjb_auth_token}",
+          self.to_xml(:only => :email))
+      else
+        create_in_tjb
+      end
     rescue
       nil
     end

data/lib/jobboersen_integration/engine.rb

@@ -19,7 +19,7 @@ module JobboersenIntegration
       Salesflip::Plugin.register(:jobboersen_integration) do
         name        'JobBoersen Integration'
         author      'Matt Beedle'
-        version     '1.1'
+        version     '0.2.0'
         description 'Everything to seemlessly integrate 1000jobboersen.de'
       end
       

data/test/models/account.rb

@@ -2,6 +2,7 @@ class Account
   include Mongoid::Document
   
   field :name
+  field :account_type
   
   referenced_in   :user
   references_many :contacts

data/test/unit/account_test.rb

@@ -15,20 +15,76 @@ class AccountTest < ActiveSupport::TestCase
     assert Account.instance_methods.include?('do_not_update=')
   end
   
-  should 'should update the tjb_id when the account has been created in 1000jobboersen' do
-    FakeWeb.register_uri(:post,
-      'http://1000jobboersen.de/administration/accounts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp',
-      :body => File.read('test/support/account_create_success.xml'))
-    account = Account.create! :name => 'test account', :user => @user
-    assert_equal '4bcdbfea24d8f22230000011', account.tjb_id
+  context 'creating a new account' do
+    should 'should update the tjb_id when the account has been created in 1000jobboersen' do
+      FakeWeb.register_uri(:post,
+        'http://1000jobboersen.de/administration/accounts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp',
+        :body => File.read('test/support/account_create_success.xml'))
+      account = Account.create! :name => 'test account', :user => @user, :account_type => 'Prospect'
+      assert_equal '4bcdbfea24d8f22230000011', account.tjb_id
+    end
+    
+    should 'not attempt to create in 1000jobboersen if the account is not type: "Prospect" or "Customer"' do
+      FakeWeb.register_uri(:post,
+        'http://1000jobboersen.de/administration/accounts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp',
+        :body => File.read('test/support/account_create_success.xml'))
+      %w(Competitor Investor).each do |type|
+        account = Account.create! :name => 'test account', :user => @user, :account_type => type
+        assert account.tjb_id.blank?
+      end
+    end
+  
+    should 'not blow up if a 500 response is returned' do
+      FakeWeb.register_uri(:post,
+        'http://1000jobboersen.de/administration/accounts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp',
+        :status => '500')
+      account = Account.create! :name => 'test account', :user => @user, :account_type => 'Prospect'
+      assert account.id
+      assert !account.new_record?
+    end
   end
   
-  should 'not blow up if a 500 response is returned' do
-    FakeWeb.register_uri(:post,
-      'http://1000jobboersen.de/administration/accounts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp',
-      :status => '500')
-    account = Account.create :name => 'test account', :user => @user
-    assert account.id
-    assert !account.new_record?
+  context 'updating an account' do
+    context 'when the account exists in 1000jobboersen' do
+      setup do
+        FakeWeb.register_uri(:post,
+          'http://1000jobboersen.de/administration/accounts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp',
+          :body => File.read('test/support/account_create_success.xml'))
+      end
+    
+      should 'not blow up when an error response is returned' do
+        account = Account.create! :name => 'test account', :user => @user
+        FakeWeb.register_uri(:put,
+          "http://1000jobboersen.de/administration/accounts/#{account.tjb_id}.xml?auth_token=#{@user.tjb_auth_token}",
+          :status => '500')
+        account = Account.find(account.id)
+        account.update_attributes! :name => 'test'
+        assert account.valid?
+      end
+    end
+    
+    context 'when the account does not exist in 1000jobboersen' do
+      setup do
+        FakeWeb.register_uri(:post,
+          'http://1000jobboersen.de/administration/accounts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp',
+          :body => File.read('test/support/account_create_success.xml'))
+          FakeWeb.register_uri(:put,
+            "http://1000jobboersen.de/administration/accounts/.xml?auth_token=#{@user.tjb_auth_token}",
+            :status => '500')
+        @account = Account.create! :name => 'test account', :account_type => 'Investor', :user => @user
+      end
+      
+      should 'create the account in 1000jobboersen' do
+        assert @account.tjb_id.nil?
+        @account.update_attributes :account_type => 'Prospect'
+        assert_equal '4bcdbfea24d8f22230000011', @account.tjb_id
+      end
+      
+      should 'not attempt to create account in tjb unless account_type is "Prospect" or "Customer"' do
+        assert @account.tjb_id.nil?
+        @account.update_attributes :account_type => 'some other type'
+        assert @account.tjb_id.nil?
+      end
+    end
   end
 end

data/test/unit/contact_test.rb

@@ -15,33 +15,75 @@ class ContactTest < ActiveSupport::TestCase
     assert Contact.instance_methods.include?('do_not_update=')
   end
   
-  should 'only send first_name, last_name, email, password and password_confirmation in xml' do
-    contact = Contact.new :first_name => 'Matt', :last_name => 'Beedle',
-      :email => 'mattbeedle@googlemail.com', :password => 'berlin5000',
-      :password_confirmation => 'berlin5000', :another_field => 'asfewaf'
-    assert_match(/first\-name/, contact.tjb_xml)
-    assert_match(/last\-name/, contact.tjb_xml)
-    assert_match(/email/, contact.tjb_xml)
-    assert_match(/password/, contact.tjb_xml)
-    assert_match(/password\-confirmation/, contact.tjb_xml)
-    assert_no_match(/another\-field/, contact.tjb_xml)
-  end
+  context 'creating a new contact (user)' do
+    should 'only send first_name, last_name, email, password and password_confirmation in xml' do
+      contact = Contact.new :first_name => 'Matt', :last_name => 'Beedle',
+        :email => 'mattbeedle@googlemail.com', :password => 'berlin5000',
+        :password_confirmation => 'berlin5000', :another_field => 'asfewaf'
+      assert_match(/first\-name/, contact.tjb_xml)
+      assert_match(/last\-name/, contact.tjb_xml)
+      assert_match(/email/, contact.tjb_xml)
+      assert_match(/password/, contact.tjb_xml)
+      assert_match(/password\-confirmation/, contact.tjb_xml)
+      assert_no_match(/another\-field/, contact.tjb_xml)
+    end
+  
+    should 'set the tjb_id when the contact has been created in 1000jobboersen' do
+      FakeWeb.register_uri(:post,
+        "http://1000jobboersen.de/administration/contacts.xml?auth_token=#{@user.tjb_auth_token}",
+        :body => File.read('test/support/contact_create_success.xml'))
+      contact = Contact.create! :first_name => 'Matt', :last_name => 'Beedle',
+        :email => 'mattbeedle@googlemail.com', :user => @user
+      assert_equal '4b8ead6a24d8f247be001073', contact.tjb_id
+    end
   
-  should 'set the tjb_id when the contact has been created in 1000jobboersen' do
-    FakeWeb.register_uri(:post,
-      "http://1000jobboersen.de/administration/contacts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp",
-      :body => File.read('test/support/contact_create_success.xml'))
-    contact = Contact.create! :first_name => 'Matt', :last_name => 'Beedle',
-      :email => 'mattbeedle@googlemail.com', :user => @user
-    assert_equal '4b8ead6a24d8f247be001073', contact.tjb_id
+    should 'not blow up when 1000jobboersen returns an error response' do
+      FakeWeb.register_uri(:post,
+        "http://1000jobboersen.de/administration/contacts.xml?auth_token=#{@user.tjb_auth_token}",
+        :status => '500')
+      contact = Contact.create! :first_name => 'Matt', :last_name => 'Beedle',
+        :email => 'mattbeedle@googlemail.com', :user => @user
+      assert !contact.new_record?
+    end
   end
   
-  should 'not blow up when 1000jobboersen returns an error response' do
-    FakeWeb.register_uri(:post,
-      "http://1000jobboersen.de/administration/contacts.xml?auth_token=La9Hbm8SnPJN-F-zUhlp",
-      :status => '500')
-    contact = Contact.create! :first_name => 'Matt', :last_name => 'Beedle',
-      :email => 'mattbeedle@googlemail.com', :user => @user
-    assert !contact.new_record?
+  context 'updating a contact' do
+    context 'when the contact exists in 1000jobboersen' do
+      setup do
+        FakeWeb.register_uri(:post,
+          "http://1000jobboersen.de/administration/contacts.xml?auth_token=#{@user.tjb_auth_token}",
+          :body => File.read('test/support/contact_create_success.xml'))
+      end
+    
+      should 'not blow up when an error response is returned' do
+        contact = Contact.create! :first_name => 'Matt', :last_name => 'Beedle',
+          :email => 'mattbeedle@googlemail.com', :user => @user
+        FakeWeb.register_uri(:put,
+          "http://1000jobboersen.de/administration/users/#{contact.tjb_id}.xml?auth_token=#{@user.tjb_auth_token}",
+          :status => '500')
+        contact = Contact.find(contact.id)
+        contact.update_attributes :first_name => 'test'
+        assert contact.valid?
+      end
+    end
+    
+    context 'when the contact does not exist in 1000jobboersen' do
+      setup do
+        FakeWeb.register_uri(:post,
+          "http://1000jobboersen.de/administration/contacts.xml?auth_token=#{@user.tjb_auth_token}",
+          :status => '500')
+        @contact = Contact.create! :first_name => 'Matt', :last_name => 'Beedle',
+            :email => 'mattbeedle@googlemail.com', :user => @user
+      end
+      
+      should 'create the contact (user) in 1000jobboersen' do
+        FakeWeb.register_uri(:post,
+          "http://1000jobboersen.de/administration/contacts.xml?auth_token=#{@user.tjb_auth_token}",
+          :body => File.read('test/support/contact_create_success.xml'))
+        assert @contact.tjb_id.blank?
+        @contact.update_attributes :first_name => 'someone'
+        assert_equal '4b8ead6a24d8f247be001073', @contact.tjb_id
+      end
+    end
   end
 end

data/test/unit/user_test.rb

@@ -17,19 +17,56 @@ class UserTest < ActiveSupport::TestCase
   should 'have tjb_auth_token field' do
     assert User.fields.map(&:first).include?('tjb_auth_token')
   end
-    
-  should 'update the tjb_id when the administrator account has been created in 1000jobboersen' do
-    FakeWeb.register_uri(:post, 'http://1000jobboersen.de/administration/administrators.xml',
-      :body => File.read('test/support/administrator_create_success.xml'))
-    user = User.create! :email => 'test@test.com'
-    assert_equal '4bcdbfea24d8f22230000011', user.tjb_id
-    assert_equal 'La9Hbm8SnPJN-F-zUhlp', user.tjb_auth_token
+  
+  context 'creating a new user' do
+    should 'update the tjb_id when the administrator account has been created in 1000jobboersen' do
+      FakeWeb.register_uri(:post, 'http://1000jobboersen.de/administration/administrators.xml',
+        :body => File.read('test/support/administrator_create_success.xml'))
+      user = User.create! :email => 'test@test.com'
+      assert_equal '4bcdbfea24d8f22230000011', user.tjb_id
+      assert_equal 'La9Hbm8SnPJN-F-zUhlp', user.tjb_auth_token
+    end
+  
+    should 'not blow up when an error response is returned' do
+      FakeWeb.register_uri(:post, 'http://1000jobboersen.de/administration/administrators.xml',
+        :status => '500')
+      user = User.create! :email => 'test@test.com'
+      assert !user.new_record?
+    end
   end
   
-  should 'not blow up when an error response is returned' do
-    FakeWeb.register_uri(:post, 'http://1000jobboersen.de/administration/administrators.xml',
-      :status => '500')
-    user = User.create! :email => 'test@test.com'
-    assert !user.new_record?
+  context 'updating a user (administrator)' do
+    context 'when the administrator exists in 1000jobboersen' do
+      setup do
+        FakeWeb.register_uri(:post, 'http://1000jobboersen.de/administration/administrators.xml',
+          :body => File.read('test/support/administrator_create_success.xml'))
+        @user = User.create! :email => 'test@test.com'
+      end
+      
+      should 'not blow up when an error response is returned' do
+        FakeWeb.register_uri(:put,
+          "http://1000jobboersen.de/administration/administrators/#{@user.id}.xml?auth_token=#{@user.tjb_auth_token}",
+          :status => '500')
+        @user = User.find(@user.id)
+        @user.update_attributes :email => 'another_test@test.com'
+        assert @user.valid?
+      end
+    end
+    
+    context 'when the administrator does not exist in 1000jobboersen' do
+      setup do
+        FakeWeb.register_uri(:post, 'http://1000jobboersen.de/administration/administrators.xml',
+          :status => '500')
+        @user = User.create! :email => 'test@test.com'
+      end
+      
+      should 'attempt to create the administrator in 1000jobboersen' do
+        @user = User.find(@user.id)
+        FakeWeb.register_uri(:post, 'http://1000jobboersen.de/administration/administrators.xml',
+          :body => File.read('test/support/administrator_create_success.xml'))
+        @user.update_attributes! :email => 'another_test@test.com'
+        assert_equal '4bcdbfea24d8f22230000011', @user.tjb_id
+      end
+    end
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: salesflip-jobboersen_integration
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 23
   prerelease: false
   segments: 
   - 0
-  - 1
-  - 1
-  version: 0.1.1
+  - 2
+  - 0
+  version: 0.2.0
 platform: ruby
 authors: []