maestrano-connector-rails 0.2.0 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c188e34977256c1d84f9fd43bd026233acf2833f
-  data.tar.gz: d7cf8ca5aa83e6ab639fa1ca2b71ef7a48614f47
+  metadata.gz: cb050d36d97af9d68812330c80a035c9c91fb28b
+  data.tar.gz: ebe39a369c26144236473fc9a9da62c381c95c62
 SHA512:
-  metadata.gz: 0ce346b75f378364d2d74922f6c0950713ec1b9f8f6dec9753e2194a2b2be0631d6acc8a6dfbd1c96b8d8c15dbfa638e8b58829eb62f98e6d2e2e705cb22e81e
-  data.tar.gz: 4d1c4050ee4ac5a88ed3c722d8ff3663de69ba5a69d105a60263d046d13b661bdce424c427f045ea9136cf6d27b4fd89cd6c7f8edd903eff622614964e189f78
+  metadata.gz: b0680266b20d2ceed8569d483d38788bd42afc935af499b1e94cfd3aa9534b066deca706d98ee60355b35321cd153cc6d30e238b3f517e2bff1ac73d4a0284ab
+  data.tar.gz: 1bdc3afa7d0e70a3b8752d1a0ba7650074249ecd1b17d7d2cd02ebf3efdf7f836f4ce90820a995064c4d3635f7ee51b070c792e669a75fb153102a7e1c19e8db

data/DEVELOPER.md

@@ -4,7 +4,7 @@ Release the gem using jeweler
 
 ``` bash
 gem install jeweler
-bundle exec rake version:bump:minor
+bundle exec rake version:bump:patch
 bundle exec rake gemspec:release
 bundle exec rake release
 ```

data/README.md

@@ -1,5 +1,7 @@
+[ ![Codeship Status for maestrano/maestrano-connector-rails](https://codeship.com/projects/e7990b70-b04d-0133-c4e7-5e399acd7f73/status?branch=master)](https://codeship.com/projects/132645)
+
 <p align="center">
-  <img src="https://raw.github.com/berardpi/maestrano-connector-rails/master/maestrano.png" alt="Maestrano Logo">
+  <img src="https://raw.github.com/maestrano/maestrano-connector-rails/master/maestrano.png" alt="Maestrano Logo">
   <br/>
   <br/>
 </p>
@@ -29,7 +31,7 @@ Before integrating with us you will need an Connec!™ API ID and Connec!™ API
 ## Getting Started
 Create a new rails application using the connector template
 ```console
-rails new <project_name> -m https://raw.githubusercontent.com/Berardpi/maestrano-connector-template/master/maestrano-connector-template.rb
+rails new <project_name> -m https://raw.githubusercontent.com/Berardpi/maestrano-connector-rails/master/template/maestrano-connector-template.rb
 ```
 
 If and only if you have an error in the template's rails generate step, you'll need to re-run the following command in your project folder:
@@ -135,4 +137,4 @@ rails g connector:complex_entity
 
 This will generate some example files demonstrating a 1 to 2 correspondance between Connec!™ person and external contact and lead data models.
 
-The complex entities workflow uses two methods to pre-process data which you have to implements for each complex entity (see contact_and_lead.rb). They are called before the mapping step, and you can use them to perform any data model specific operations.
+The complex entities workflow uses two methods to pre-process data which you have to implements for each complex entity (see contact_and_lead.rb). They are called before the mapping step, and you can use them to perform any data model specific operations.

data/VERSION

@@ -1 +1 @@
-0.2.0
+0.2.2

data/app/controllers/maestrano/connec_controller.rb

@@ -0,0 +1,62 @@
+class Maestrano::ConnecController < Maestrano::Rails::WebHookController
+
+  def notifications
+    Rails.logger.debug "Received notification from Connec!: #{params}"
+
+    begin
+      params['notification'].each do |entity_name, entities|
+        if entity_instance_hash = find_entity_instance(entity_name)
+          entity_instance = entity_instance_hash[:instance]
+
+          entities.each do |entity|
+            if (organization = Maestrano::Connector::Rails::Organization.find_by(uid: entity[:group_id], tenant: params[:tenant])) && organization.oauth_uid
+              Maestrano::Connector::Rails::ConnectorLogger.log('info', organization, "Received entity from Connec! webhook: Entity=#{entity_name}, Data=#{entity}")
+              if organization.sync_enabled && organization.synchronized_entities[entity_instance_hash[:name].to_sym]
+                external_client = Maestrano::Connector::Rails::External.get_client(organization)
+
+                # Build expected input for consolidate_and_map_data
+                if entity_instance_hash[:is_complex]
+                  entity = Hash[ *entity_instance.connec_entities_names.collect{|name| name == entity_name.singularize ? [name, [entity]] : [ name, []]}.flatten(1) ]
+                  entity_instance.consolidate_and_map_data(entity, Hash[ *entity_instance.external_entities_names.collect{|name| [ name, []]}.flatten(1) ], organization, {})
+                else
+                  entity = [entity]
+                  entity_instance.consolidate_and_map_data(entity, [], organization, {})
+                end
+
+                entity_instance.push_entities_to_external(external_client, entity, organization)
+              end
+
+            else
+              Rails.logger.warn "Received notification from Connec! for unknown group or group without oauth: #{entity['group_id']} (tenant: #{params[:tenant]})"
+            end
+          end
+        else
+          Rails.logger.warn "Received notification from Connec! for unknow entity: #{entity_name}"
+        end
+      end
+    rescue => e
+      Rails.logger.warn("error processing notification #{e.message} - #{e.backtrace.join("\n")}")
+    end
+
+    head 200, content_type: "application/json"
+  end
+
+
+
+  private
+    def find_entity_instance(entity_name)
+      if Maestrano::Connector::Rails::Entity.entities_list.include?(entity_name.singularize)
+        return {instance: "Entities::#{entity_name.singularize.titleize.split.join}".constantize.new, is_complex: false, name: entity_name.singularize}
+      else
+        Maestrano::Connector::Rails::Entity.entities_list.each do |entity_name_from_list|
+          instance = "Entities::#{entity_name_from_list.singularize.titleize.split.join}".constantize.new
+          if instance.methods.include?('connec_entities_names'.to_sym)
+            return {instance: instance, is_complex: true, name: entity_name_from_list} if instance.connec_entities_names.include?(entity_name.singularize)
+          elsif instance.methods.include?('connec_entity_name'.to_sym)
+            return {instance: instance, is_complex: false, name: entity_name_from_list} if instance.connec_entity_name == entity_name.singularize
+          end
+        end
+      end
+      nil
+    end
+end

data/app/jobs/maestrano/connector/rails/synchronization_job.rb

@@ -8,6 +8,7 @@ module Maestrano::Connector::Rails
     #  * :full_sync => true  synchronization is performed without date filtering
     #  * :connec_preemption => true|false : preemption is always|never given to connec in case of conflict (if not set, the most recently updated entity is kept)
     def perform(organization, opts)
+      return unless organization.sync_enabled
       ConnectorLogger.log('info', organization, "Start synchronization, opts=#{opts}")
       current_synchronization = Synchronization.create_running(organization)
 

data/app/models/maestrano/connector/rails/id_map.rb

@@ -1,6 +1,5 @@
 module Maestrano::Connector::Rails
   class IdMap < ActiveRecord::Base
-  	self.table_name = "maestrano_connector_rails_id_maps"
 
     belongs_to :organization
   end

data/app/models/maestrano/connector/rails/organization.rb

@@ -1,6 +1,5 @@
 module Maestrano::Connector::Rails
   class Organization < ActiveRecord::Base
-    self.table_name = "maestrano_connector_rails_organizations"
 
     # Enable Maestrano for this group
     maestrano_group_via :provider, :uid, :tenant do |group, maestrano|

data/app/models/maestrano/connector/rails/synchronization.rb

@@ -1,6 +1,5 @@
 module Maestrano::Connector::Rails
   class Synchronization < ActiveRecord::Base
-    self.table_name = "maestrano_connector_rails_synchronizations"
 
     #===================================
     # Associations

data/app/models/maestrano/connector/rails/user.rb

@@ -1,6 +1,5 @@
 module Maestrano::Connector::Rails
   class User < ActiveRecord::Base
-    self.table_name = "maestrano_connector_rails_users"
 
     # Enable Maestrano for this user
     maestrano_user_via :provider, :uid, :tenant do |user, maestrano|

data/app/models/maestrano/connector/rails/user_organization_rel.rb

@@ -1,6 +1,5 @@
 module Maestrano::Connector::Rails
   class UserOrganizationRel < ActiveRecord::Base
-  	self.table_name = "maestrano_connector_rails_user_organization_rels"
 
     #===================================
     # Associations

data/config/routes.rb

@@ -1,28 +1,9 @@
 Maestrano::Connector::Rails::Engine.routes.draw do
   maestrano_routes
 
-  #--------------------------------------------------------------------
-  # Maintainability issue
-  # Should use maestrano_routes but it creates a scoping error with the metadata routes
-  # get '/maestrano/metadata', to: '/maestrano/rails/metadata#index'
-  # get '/maestrano/metadata/:tenant', to: '/maestrano/rails/metadata#index', as: 'tenant'
-
-  # namespace :maestrano do
-  #   namespace :auth do
-  #     resources :saml, only:[] do
-  #       get 'init', on: :collection
-  #       get 'init/:tenant', on: :collection, to: 'saml#init', as: 'tenant'
-  #       post 'consume', on: :collection
-  #     end
-  #   end
-
-  #   namespace :account do
-  #     resources :groups, only: [:destroy] do
-  #       resources :users, only: [:destroy], controller: 'group_users'
-  #     end
-  #   end
-  # end
-  #--------------------------------------------------------------------
+  namespace :maestrano do
+    post 'connec/notifications/:tenant' => 'connec#notifications'
+  end
 
   match 'signout', to: 'sessions#destroy', as: 'signout', via: [:get, :post]
 end

data/db/migrate/20151122162100_create_users.rb

@@ -0,0 +1,16 @@
+class CreateUsers < ActiveRecord::Migration
+  def change
+    create_table :users do |t|
+      t.string :provider
+      t.string :uid
+      t.string :first_name
+      t.string :last_name
+      t.string :email
+      t.string :tenant
+
+      t.timestamps null: false
+    end
+
+    add_index :users, [:uid, :tenant], name: 'user_uid_index'
+  end
+end

data/db/migrate/{20151122162414_create_maestrano_connector_rails_organizations.rb → 20151122162414_create_organizations.rb}

@@ -1,6 +1,6 @@
-class CreateMaestranoConnectorRailsOrganizations < ActiveRecord::Migration
+class CreateOrganizations < ActiveRecord::Migration
   def change
-    create_table :maestrano_connector_rails_organizations do |t|
+    create_table :organizations do |t|
       t.string :provider
       t.string :uid
       t.string :name
@@ -16,6 +16,6 @@ class CreateMaestranoConnectorRailsOrganizations < ActiveRecord::Migration
 
       t.timestamps null: false
     end
-    add_index :maestrano_connector_rails_organizations, [:uid, :tenant], name: 'orga_uid_index'
+    add_index :organizations, [:uid, :tenant], name: 'orga_uid_index'
   end
 end

data/db/migrate/20151122162613_create_user_organization_rels.rb

@@ -0,0 +1,12 @@
+class CreateUserOrganizationRels < ActiveRecord::Migration
+  def change
+    create_table :user_organization_rels do |t|
+      t.integer :user_id
+      t.integer :organization_id
+
+      t.timestamps
+    end
+    add_index :user_organization_rels, :organization_id, name: 'rels_orga_index'
+    add_index :user_organization_rels, :user_id, name: 'rels_user_index'
+  end
+end

data/db/migrate/20151122163325_create_synchronizations.rb

@@ -0,0 +1,13 @@
+class CreateSynchronizations < ActiveRecord::Migration
+  def change
+    create_table :synchronizations do |t|
+      t.integer :organization_id
+      t.string  :status
+      t.text    :message
+      t.boolean :partial, default: false
+
+      t.timestamps null: false
+    end
+    add_index :synchronizations, :organization_id, name: 'synchronization_orga_id_index'
+  end
+end

data/db/migrate/20151122163449_create_id_maps.rb

@@ -0,0 +1,18 @@
+class CreateIdMaps < ActiveRecord::Migration
+  def change
+    create_table :id_maps do |t|
+      t.string :connec_id
+      t.string :connec_entity
+      t.string :external_id
+      t.string :external_entity
+      t.integer :organization_id
+      t.datetime :last_push_to_connec
+      t.datetime :last_push_to_external
+
+      t.timestamps null: false
+    end
+    add_index :id_maps, [:connec_id, :connec_entity, :organization_id], name: 'idmap_connec_index'
+    add_index :id_maps, [:external_id, :external_entity, :organization_id], name: 'idmap_external_index'
+    add_index :id_maps, :organization_id, name: 'idmap_organization_index'
+  end
+end

data/db/migrate/20160205132857_add_sync_enabled_to_organizations.rb

@@ -0,0 +1,5 @@
+class AddSyncEnabledToOrganizations < ActiveRecord::Migration
+  def change
+    add_column :organizations, :sync_enabled, :boolean, default: false
+  end
+end

data/lib/generators/connector/install_generator.rb

@@ -30,7 +30,8 @@ module Connector
           "get 'home/index' => 'home#index'",
           "get 'admin/index' => 'admin#index'",
           "put 'admin/update' => 'admin#update'",
-          "post 'admin/synchronize' => 'admin#synchronize'"
+          "post 'admin/synchronize' => 'admin#synchronize'",
+          "put 'admin/toggle_sync' => 'admin#toggle_sync'"
         ]
 
         in_root do

data/lib/generators/connector/templates/admin_controller.rb

@@ -41,6 +41,16 @@ class AdminController < ApplicationController
     redirect_to root_path
   end
 
+  def toggle_sync
+    if is_admin
+      current_organization = Maestrano::Connector::Rails::Organization.first
+      current_organization.update(sync_enabled: !current_organization.sync_enabled)
+      flash[:notice] = current_organization.sync_enabled ? 'Synchronization enabled' : 'Synchronization disabled'
+    end
+
+    redirect_to admin_index_path
+  end
+
   private
     def is_admin
       current_user && current_organization && is_admin?(current_user, current_organization)

data/lib/generators/connector/templates/admin_index.html.erb

@@ -26,14 +26,18 @@ and functions available on the admin panel -->
       </li>
     </ul>
 
+    <h3>Enable or disable synchronizations</h3>
+    <%= "Synchronizations are currently #{@organization.sync_enabled ? 'enabled' : 'disabled'}." %>
+    <%= link_to "#{@organization.sync_enabled ? 'Disable' : 'Enable'} them", admin_toggle_sync_path, method: :put %>
+
     <h3>Synchronized entities</h3>
-    <p>You can customize which entities are synchronized by the connector.</p>
+    <p>You can customize which entities are synchronized by the connector</p>
     <%= form_tag admin_update_path(id: @organization.id), method: :put do %>
       <% @organization.synchronized_entities.each do |k, v| %>
           <label for="<%= k %>">#{k.to_s.humanize}</label>
           <input checked="<%= v %>" id="<%= k %>" name="<%= k %>" type="checkbox"></input>
           <% if v  && @organization.oauth_uid %>
-            <%= link_to "Force a synchronization for #{k.to_s.pluralize} only", admin_synchronize_path(opts: {only_entities: [k.to_s]}), method: :post %>
+            <%= link_to "Force a synchronization for #{k.to_s.humanize.pluralize} only", admin_synchronize_path(opts: {only_entities: [k.to_s]}), method: :post %>
           <% end %>
       <% end %>
       <%= submit_tag :Update%>

data/lib/generators/connector/templates/home_index.html.erb

@@ -18,6 +18,12 @@ can be displayed on the home page -->
       Linked to external app
     <% end %>
 
+    <% if !@organization.sync_enabled %>
+      Synchronizations are currently disabled. Go to the
+      <%= link_to " admin panel", admin_index_path %>
+      to enable them.
+    <% end %>
+
     <h2>Last synchronization</h2>
     <% if @synchronizations.first %>
       <%= "#{@synchronizations.first.updated_at} #{@synchronizations.first.status}" %>

data/maestrano-connector-rails.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: maestrano-connector-rails 0.2.0 ruby lib
+# stub: maestrano-connector-rails 0.2.2 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "maestrano-connector-rails"
-  s.version = "0.2.0"
+  s.version = "0.2.2"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Pierre Berard"]
-  s.date = "2016-02-05"
+  s.date = "2016-02-08"
   s.description = "Maestrano is the next generation marketplace for SME applications. See https://maestrano.com for details."
   s.email = "pierre.berard@maestrano.com"
   s.executables = ["rails"]
@@ -32,6 +32,7 @@ Gem::Specification.new do |s|
     "app/controllers/maestrano/account/groups_controller.rb",
     "app/controllers/maestrano/application_controller.rb",
     "app/controllers/maestrano/auth/saml_controller.rb",
+    "app/controllers/maestrano/connec_controller.rb",
     "app/controllers/maestrano/sessions_controller.rb",
     "app/helpers/maestrano/connector/rails/session_helper.rb",
     "app/jobs/maestrano/connector/rails/synchronization_job.rb",
@@ -49,11 +50,12 @@ Gem::Specification.new do |s|
     "app/models/maestrano/connector/rails/user_organization_rel.rb",
     "bin/rails",
     "config/routes.rb",
-    "db/migrate/20151122162100_create_maestrano_connector_rails_users.rb",
-    "db/migrate/20151122162414_create_maestrano_connector_rails_organizations.rb",
-    "db/migrate/20151122162613_create_maestrano_connector_rails_user_organization_rels.rb",
-    "db/migrate/20151122163325_create_maestrano_connector_rails_synchronizations.rb",
-    "db/migrate/20151122163449_create_maestrano_connector_rails_id_maps.rb",
+    "db/migrate/20151122162100_create_users.rb",
+    "db/migrate/20151122162414_create_organizations.rb",
+    "db/migrate/20151122162613_create_user_organization_rels.rb",
+    "db/migrate/20151122163325_create_synchronizations.rb",
+    "db/migrate/20151122163449_create_id_maps.rb",
+    "db/migrate/20160205132857_add_sync_enabled_to_organizations.rb",
     "lib/generators/connector/USAGE",
     "lib/generators/connector/complex_entity_generator.rb",
     "lib/generators/connector/install_generator.rb",
@@ -77,6 +79,7 @@ Gem::Specification.new do |s|
     "maestrano-connector-rails.gemspec",
     "maestrano.png",
     "pkg/maestrano-connector-rails-0.0.1.gem",
+    "spec/controllers/connec_controller_spec.rb",
     "spec/dummy/README.md",
     "spec/dummy/Rakefile",
     "spec/dummy/app/assets/images/.keep",
@@ -122,6 +125,12 @@ Gem::Specification.new do |s|
     "spec/dummy/config/routes.rb",
     "spec/dummy/config/secrets.yml",
     "spec/dummy/db/development.sqlite3",
+    "spec/dummy/db/migrate/20160204223447_create_users.maestrano_connector_rails_engine.rb",
+    "spec/dummy/db/migrate/20160204223448_create_organizations.maestrano_connector_rails_engine.rb",
+    "spec/dummy/db/migrate/20160204223449_create_user_organization_rels.maestrano_connector_rails_engine.rb",
+    "spec/dummy/db/migrate/20160204223450_create_synchronizations.maestrano_connector_rails_engine.rb",
+    "spec/dummy/db/migrate/20160204223451_create_id_maps.maestrano_connector_rails_engine.rb",
+    "spec/dummy/db/migrate/20160205133024_add_sync_enabled_to_organizations.maestrano_connector_rails_engine.rb",
     "spec/dummy/db/schema.rb",
     "spec/dummy/db/test.sqlite3",
     "spec/dummy/lib/assets/.keep",
@@ -142,7 +151,12 @@ Gem::Specification.new do |s|
     "spec/models/synchronization_spec.rb",
     "spec/models/user_organization_rel_spec.rb",
     "spec/models/user_spec.rb",
-    "spec/spec_helper.rb"
+    "spec/routing/connec_routing_spec.rb",
+    "spec/spec_helper.rb",
+    "template/factories.rb",
+    "template/gitignore",
+    "template/maestrano-connector-template.rb",
+    "template/spec_helper.rb"
   ]
   s.homepage = "http://github.com/maestrano/maestrano-connector-rails"
   s.licenses = ["MIT"]

data/spec/controllers/connec_controller_spec.rb

@@ -0,0 +1,144 @@
+require 'spec_helper'
+
+describe Maestrano::ConnecController, type: :controller do
+  routes { Maestrano::Connector::Rails::Engine.routes }
+
+  describe 'notifications' do
+    let(:notif) { {} }
+    let(:group_id) { 'cld_333' }
+    let(:entity) { {group_id: group_id, last_name: 'Georges', first_name: 'Teddy'} }
+    subject { post :notifications, tenant: 'default', notification: notif }
+
+    context 'without authentication' do
+      before {
+        controller.class.before_filter :authenticate_maestrano!
+      }
+
+      it 'respond with unauthorized' do
+        subject
+        expect(response.status).to eq(401)
+      end
+    end
+
+    context 'with authentication' do
+      before {
+        controller.class.skip_before_filter :authenticate_maestrano!
+      }
+
+      it 'is a success' do
+        subject
+        expect(response.status).to eq(200)
+      end
+
+      context "with an unknown entity" do
+        let(:notif) { {people: [entity]} }
+        before {
+          allow(Maestrano::Connector::Rails::Entity).to receive(:entities_list).and_return(%w())
+        }
+
+        it 'logs a warning' do
+          expect(Rails.logger).to receive(:warn).with('Received notification from Connec! for unknow entity: people')
+          subject
+        end
+      end
+
+      context "with a known complex entity" do
+        let(:notif) { {lead: [entity]} }
+
+        before {
+          allow(Maestrano::Connector::Rails::Entity).to receive(:entities_list).and_return(%w(contact_and_lead))
+          class Entities::ContactAndLead < Maestrano::Connector::Rails::ComplexEntity
+            def connec_entities_names
+              %w(lead)
+            end
+          end
+          module Entities::SubEntities end;
+          class Entities::SubEntities::Lead < Maestrano::Connector::Rails::SubEntityBase
+          end
+        }
+
+        it 'looks for an organization' do
+          expect(Maestrano::Connector::Rails::Organization).to receive(:find_by).with(uid: group_id, tenant: 'default')
+          subject
+        end
+
+        context 'when syncing' do
+          before {
+            allow_any_instance_of(Entities::ContactAndLead).to receive(:external_entities_names).and_return(%w())
+          }
+          let!(:organization) { create(:organization, uid: group_id, oauth_uid: 'lala', sync_enabled: true, synchronized_entities: {contact_and_lead: true}) }
+
+          it 'process the data and push them' do
+            expect_any_instance_of(Entities::ContactAndLead).to receive(:consolidate_and_map_data).with({"lead"=>[entity]}, {}, organization, {})
+            expect_any_instance_of(Entities::ContactAndLead).to receive(:push_entities_to_external)
+            subject
+          end
+        end
+      end
+
+      context "with a known non complex entity" do
+        let(:notif) { {people: [entity]} }
+
+        before {
+          allow(Maestrano::Connector::Rails::Entity).to receive(:entities_list).and_return(%w(person))
+          class Entities::Person < Maestrano::Connector::Rails::Entity
+          end
+        }
+
+        it 'looks for an organization' do
+          expect(Maestrano::Connector::Rails::Organization).to receive(:find_by).with(uid: group_id, tenant: 'default')
+          subject
+        end
+
+        context 'with an invalid organization' do
+          context 'with no organization' do
+            it 'logs a warning' do
+              expect(Rails.logger).to receive(:warn).with("Received notification from Connec! for unknown group or group without oauth: #{group_id} (tenant: default)")
+              subject
+            end
+          end
+
+          context 'with an organization with no oauth' do
+            let!(:organization) { create(:organization, uid: group_id, oauth_uid: nil) }
+
+            it 'logs a warning' do
+              expect(Rails.logger).to receive(:warn).with("Received notification from Connec! for unknown group or group without oauth: #{group_id} (tenant: default)")
+              subject
+            end
+          end
+        end
+
+        context 'with a valid organization' do
+          context 'with sync disabled' do
+            let!(:organization) { create(:organization, uid: group_id, oauth_uid: 'lala', sync_enabled: false, synchronized_entities: {person: true}) }
+
+            it 'does nothing' do
+              expect(Maestrano::Connector::Rails::External).to_not receive(:get_client).with(organization)
+              subject
+            end
+          end
+
+          context 'with sync disabled for this entity' do
+            let!(:organization) { create(:organization, uid: group_id, oauth_uid: 'lala', sync_enabled: true, synchronized_entities: {person: false}) }
+
+            it 'does nothing' do
+              expect(Maestrano::Connector::Rails::External).to_not receive(:get_client).with(organization)
+              subject
+            end
+          end
+
+          context "when syncing" do
+            let!(:organization) { create(:organization, uid: group_id, oauth_uid: 'lala', sync_enabled: true, synchronized_entities: {person: true}) }
+
+            it 'process the data and push them' do
+              expect_any_instance_of(Entities::Person).to receive(:consolidate_and_map_data).with([entity], [], organization, {})
+              expect_any_instance_of(Entities::Person).to receive(:push_entities_to_external)
+              subject
+            end
+          end
+        end
+
+      end
+    end
+  end
+end

data/spec/dummy/db/migrate/20160204223447_create_users.maestrano_connector_rails_engine.rb

@@ -0,0 +1,17 @@
+# This migration comes from maestrano_connector_rails_engine (originally 20151122162100)
+class CreateUsers < ActiveRecord::Migration
+  def change
+    create_table :users do |t|
+      t.string :provider
+      t.string :uid
+      t.string :first_name
+      t.string :last_name
+      t.string :email
+      t.string :tenant
+
+      t.timestamps null: false
+    end
+
+    add_index :users, [:uid, :tenant], name: 'user_uid_index'
+  end
+end

data/spec/dummy/db/migrate/20160204223448_create_organizations.maestrano_connector_rails_engine.rb

@@ -0,0 +1,22 @@
+# This migration comes from maestrano_connector_rails_engine (originally 20151122162414)
+class CreateOrganizations < ActiveRecord::Migration
+  def change
+    create_table :organizations do |t|
+      t.string :provider
+      t.string :uid
+      t.string :name
+      t.string :tenant
+
+      t.string :oauth_provider
+      t.string :oauth_uid
+      t.string :oauth_token
+      t.string :refresh_token
+      t.string :instance_url
+
+      t.string :synchronized_entities
+
+      t.timestamps null: false
+    end
+    add_index :organizations, [:uid, :tenant], name: 'orga_uid_index'
+  end
+end

data/spec/dummy/db/migrate/20160204223449_create_user_organization_rels.maestrano_connector_rails_engine.rb

@@ -0,0 +1,13 @@
+# This migration comes from maestrano_connector_rails_engine (originally 20151122162613)
+class CreateUserOrganizationRels < ActiveRecord::Migration
+  def change
+    create_table :user_organization_rels do |t|
+      t.integer :user_id
+      t.integer :organization_id
+
+      t.timestamps
+    end
+    add_index :user_organization_rels, :organization_id, name: 'rels_orga_index'
+    add_index :user_organization_rels, :user_id, name: 'rels_user_index'
+  end
+end

data/spec/dummy/db/migrate/20160204223450_create_synchronizations.maestrano_connector_rails_engine.rb

@@ -0,0 +1,14 @@
+# This migration comes from maestrano_connector_rails_engine (originally 20151122163325)
+class CreateSynchronizations < ActiveRecord::Migration
+  def change
+    create_table :synchronizations do |t|
+      t.integer :organization_id
+      t.string  :status
+      t.text    :message
+      t.boolean :partial, default: false
+
+      t.timestamps null: false
+    end
+    add_index :synchronizations, :organization_id, name: 'synchronization_orga_id_index'
+  end
+end

data/spec/dummy/db/migrate/20160204223451_create_id_maps.maestrano_connector_rails_engine.rb

@@ -0,0 +1,19 @@
+# This migration comes from maestrano_connector_rails_engine (originally 20151122163449)
+class CreateIdMaps < ActiveRecord::Migration
+  def change
+    create_table :id_maps do |t|
+      t.string :connec_id
+      t.string :connec_entity
+      t.string :external_id
+      t.string :external_entity
+      t.integer :organization_id
+      t.datetime :last_push_to_connec
+      t.datetime :last_push_to_external
+
+      t.timestamps null: false
+    end
+    add_index :id_maps, [:connec_id, :connec_entity, :organization_id], name: 'idmap_connec_index'
+    add_index :id_maps, [:external_id, :external_entity, :organization_id], name: 'idmap_external_index'
+    add_index :id_maps, :organization_id, name: 'idmap_organization_index'
+  end
+end

data/spec/dummy/db/migrate/20160205133024_add_sync_enabled_to_organizations.maestrano_connector_rails_engine.rb

@@ -0,0 +1,6 @@
+# This migration comes from maestrano_connector_rails_engine (originally 20160205132857)
+class AddSyncEnabledToOrganizations < ActiveRecord::Migration
+  def change
+    add_column :organizations, :sync_enabled, :boolean, default: false
+  end
+end

data/spec/dummy/db/schema.rb

@@ -11,9 +11,9 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20151122163449) do
+ActiveRecord::Schema.define(version: 20160205133024) do
 
-  create_table "maestrano_connector_rails_id_maps", force: :cascade do |t|
+  create_table "id_maps", force: :cascade do |t|
     t.string   "connec_id"
     t.string   "connec_entity"
     t.string   "external_id"
@@ -25,11 +25,11 @@ ActiveRecord::Schema.define(version: 20151122163449) do
     t.datetime "updated_at",            null: false
   end
 
-  add_index "maestrano_connector_rails_id_maps", ["connec_id", "connec_entity", "organization_id"], name: "idmap_connec_index"
-  add_index "maestrano_connector_rails_id_maps", ["external_id", "external_entity", "organization_id"], name: "idmap_external_index"
-  add_index "maestrano_connector_rails_id_maps", ["organization_id"], name: "idmap_organization_index"
+  add_index "id_maps", ["connec_id", "connec_entity", "organization_id"], name: "idmap_connec_index"
+  add_index "id_maps", ["external_id", "external_entity", "organization_id"], name: "idmap_external_index"
+  add_index "id_maps", ["organization_id"], name: "idmap_organization_index"
 
-  create_table "maestrano_connector_rails_organizations", force: :cascade do |t|
+  create_table "organizations", force: :cascade do |t|
     t.string   "provider"
     t.string   "uid"
     t.string   "name"
@@ -40,13 +40,14 @@ ActiveRecord::Schema.define(version: 20151122163449) do
     t.string   "refresh_token"
     t.string   "instance_url"
     t.string   "synchronized_entities"
-    t.datetime "created_at",            null: false
-    t.datetime "updated_at",            null: false
+    t.datetime "created_at",                            null: false
+    t.datetime "updated_at",                            null: false
+    t.boolean  "sync_enabled",          default: false
   end
 
-  add_index "maestrano_connector_rails_organizations", ["uid", "tenant"], name: "orga_uid_index"
+  add_index "organizations", ["uid", "tenant"], name: "orga_uid_index"
 
-  create_table "maestrano_connector_rails_synchronizations", force: :cascade do |t|
+  create_table "synchronizations", force: :cascade do |t|
     t.integer  "organization_id"
     t.string   "status"
     t.text     "message"
@@ -55,19 +56,19 @@ ActiveRecord::Schema.define(version: 20151122163449) do
     t.datetime "updated_at",                      null: false
   end
 
-  add_index "maestrano_connector_rails_synchronizations", ["organization_id"], name: "synchronization_orga_id_index"
+  add_index "synchronizations", ["organization_id"], name: "synchronization_orga_id_index"
 
-  create_table "maestrano_connector_rails_user_organization_rels", force: :cascade do |t|
+  create_table "user_organization_rels", force: :cascade do |t|
     t.integer  "user_id"
     t.integer  "organization_id"
     t.datetime "created_at"
     t.datetime "updated_at"
   end
 
-  add_index "maestrano_connector_rails_user_organization_rels", ["organization_id"], name: "rels_orga_index"
-  add_index "maestrano_connector_rails_user_organization_rels", ["user_id"], name: "rels_user_index"
+  add_index "user_organization_rels", ["organization_id"], name: "rels_orga_index"
+  add_index "user_organization_rels", ["user_id"], name: "rels_user_index"
 
-  create_table "maestrano_connector_rails_users", force: :cascade do |t|
+  create_table "users", force: :cascade do |t|
     t.string   "provider"
     t.string   "uid"
     t.string   "first_name"
@@ -78,6 +79,6 @@ ActiveRecord::Schema.define(version: 20151122163449) do
     t.datetime "updated_at", null: false
   end
 
-  add_index "maestrano_connector_rails_users", ["uid", "tenant"], name: "user_uid_index"
+  add_index "users", ["uid", "tenant"], name: "user_uid_index"
 
 end

data/spec/jobs/syncrhonization_job_spec.rb

@@ -3,32 +3,47 @@ require 'spec_helper'
 describe Maestrano::Connector::Rails::SynchronizationJob do
   let(:organization) { create(:organization) }
   subject { Maestrano::Connector::Rails::SynchronizationJob.perform_now(organization, {}) }
-  
+
   describe 'perform' do
-    it 'creates a synchronization' do
-      expect{ subject }.to change{ Maestrano::Connector::Rails::Synchronization.count }.by(1)
+    context 'with sync_enabled set to false' do
+      it 'does not creates a syncrhonization' do
+        expect{ subject }.to change{ Maestrano::Connector::Rails::Synchronization.count }.by(0)
+      end
+
+      it 'does not calls sync entity' do
+        expect_any_instance_of(Maestrano::Connector::Rails::SynchronizationJob).to_not receive(:sync_entity)
+        subject
+      end
     end
 
-    it 'calls sync entity on all the organization synchronized entities set to true' do
-      organization.synchronized_entities[organization.synchronized_entities.keys.first] = false
-      expect_any_instance_of(Maestrano::Connector::Rails::SynchronizationJob).to receive(:sync_entity).exactly(organization.synchronized_entities.count - 1).times
+    context 'with sync_enabled set to true' do
+      before {organization.update(sync_enabled: true)}
 
-      subject
-    end
+      it 'creates a synchronization' do
+        expect{ subject }.to change{ Maestrano::Connector::Rails::Synchronization.count }.by(1)
+      end
 
-    context 'with options' do
-      context 'with only_entities' do
-        subject { Maestrano::Connector::Rails::SynchronizationJob.perform_now(organization, {only_entities: %w(people price)}) }
+      it 'calls sync entity on all the organization synchronized entities set to true' do
+        organization.synchronized_entities[organization.synchronized_entities.keys.first] = false
+        expect_any_instance_of(Maestrano::Connector::Rails::SynchronizationJob).to receive(:sync_entity).exactly(organization.synchronized_entities.count - 1).times
 
-        it 'calls sync entity on the specified entities' do
-          expect_any_instance_of(Maestrano::Connector::Rails::SynchronizationJob).to receive(:sync_entity).twice
+        subject
+      end
 
-          subject
-        end
+      context 'with options' do
+        context 'with only_entities' do
+          subject { Maestrano::Connector::Rails::SynchronizationJob.perform_now(organization, {only_entities: %w(people price)}) }
+
+          it 'calls sync entity on the specified entities' do
+            expect_any_instance_of(Maestrano::Connector::Rails::SynchronizationJob).to receive(:sync_entity).twice
+
+            subject
+          end
 
-        it 'set the current syncrhonization as partial' do
-          subject
-          expect(Maestrano::Connector::Rails::Synchronization.last.partial).to be(true)
+          it 'set the current syncrhonization as partial' do
+            subject
+            expect(Maestrano::Connector::Rails::Synchronization.last.partial).to be(true)
+          end
         end
       end
     end

data/spec/routing/connec_routing_spec.rb

@@ -0,0 +1,12 @@
+require "spec_helper"
+
+describe Maestrano::ConnecController, type: :controller do
+  routes { Maestrano::Connector::Rails::Engine.routes }
+
+  describe "routing" do
+
+    it "routes to #notifications" do
+      expect(:post => '/maestrano/connec/notifications/default').to route_to("maestrano/connec#notifications", tenant: 'default')
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -4,6 +4,8 @@ require File.expand_path("../dummy/config/environment.rb",  __FILE__)
 require 'rspec/rails'
 require 'factory_girl_rails'
 require 'shoulda/matchers'
+require 'simplecov'
+SimpleCov.start
 
 Rails.backtrace_cleaner.remove_silencers!
 
@@ -16,4 +18,5 @@ RSpec.configure do |config|
   config.infer_base_class_for_anonymous_controllers = false
   config.order = "random"
   config.include FactoryGirl::Syntax::Methods
+  config.include Maestrano::Connector::Rails::Engine.routes.url_helpers
 end

data/template/factories.rb

@@ -0,0 +1,23 @@
+FactoryGirl.define do
+
+  factory :organization, class: Maestrano::Connector::Rails::Organization do
+    name "My company"
+    tenant "default"
+  end
+
+  factory :idmap, class: Maestrano::Connector::Rails::IdMap do
+    connec_id '6798-ada6-te43'
+    connec_entity 'person'
+    external_id '4567ada66'
+    external_entity 'contact'
+    last_push_to_external 2.day.ago
+    last_push_to_connec 1.day.ago
+    association :organization
+  end
+
+  factory :synchronization, class: Maestrano::Connector::Rails::Synchronization do
+    association :organization
+    status 'SUCCESS'
+    partial false
+  end
+end

data/template/gitignore

@@ -0,0 +1,20 @@
+# See https://help.github.com/articles/ignoring-files for more about ignoring files.
+#
+# If you find yourself ignoring temporary files generated by your text editor
+# or operating system, you probably want to add a global ignore instead:
+#   git config --global core.excludesfile '~/.gitignore_global'
+
+# Ignore bundler config.
+/.bundle
+
+# Ignore the default SQLite database.
+/db/*.sqlite3
+/db/*.sqlite3-journal
+
+# Ignore all logfiles and tempfiles.
+/log/*
+!/log/.keep
+/tmp
+
+# Ignore coverage files
+coverage/

data/template/maestrano-connector-template.rb

@@ -0,0 +1,85 @@
+def current_directory
+  @current_directory ||=
+      if __FILE__ =~ %r{\Ahttps?://}
+        tempdir = Dir.mktmpdir("maestrano-connector-rails-")
+        at_exit { FileUtils.remove_entry(tempdir) }
+        git :clone => "--quiet https://github.com/berardpi/maestrano-connector-rails.git #{tempdir}"
+
+        "#{tempdir}/template"
+      else
+        File.expand_path(File.dirname(__FILE__))
+      end
+end
+
+# def current_directory
+#   File.expand_path(File.dirname(__FILE__))
+# end
+
+# Add the current directory to the path Thor uses
+# to look up files
+def source_paths
+  Array(super) + [current_directory]
+end
+
+#
+# Rebuild the Gemfile from scratch
+#
+remove_file 'Gemfile'
+run 'touch Gemfile'
+
+add_source 'https://rubygems.org'
+
+gem 'rails', '4.2.4'
+gem 'turbolinks'
+gem 'jquery-rails'
+gem 'puma'
+gem 'figaro'
+gem 'tzinfo-data', platforms: [:mingw, :mswin, :jruby]
+
+gem 'maestrano-connector-rails'
+gem 'delayed_job_active_record'
+
+gem 'sqlite3'
+
+gem_group :test do
+  gem 'simplecov'
+  gem 'rspec-rails'
+  gem 'factory_girl_rails'
+  gem 'shoulda-matchers'
+  gem 'timecop'
+end
+
+remove_file '.gitignore'
+copy_file 'gitignore', '.gitignore'
+
+
+#
+# Cleanup
+#
+after_bundle do
+  remove_dir 'app/mailers'
+  remove_dir 'test'
+  copy_file 'spec_helper.rb', 'spec/spec_helper.rb'
+  copy_file 'factories.rb', 'spec/factories.rb'
+
+  application do <<-RUBY
+    config.generators do |g|
+      g.test_framework :rspec, fixture: false
+      g.view_specs false
+      g.helper_specs false
+    end
+  RUBY
+  end
+
+  run 'rails g maestrano:initializer'
+  run 'rails g connector:install'
+  run 'bundle exec figaro install'
+  run 'rake railties:install:migrations'
+  run 'rails g delayed_job:active_record'
+  run 'rake db:migrate'
+
+  # Init repo and commit
+  git :init
+  git add: "."
+  git commit: "-a -m 'Initial commit'"
+end

data/template/spec_helper.rb

@@ -0,0 +1,22 @@
+require 'simplecov'
+SimpleCov.start
+
+ENV['RAILS_ENV'] ||= 'test'
+
+require File.expand_path("../../config/environment", __FILE__)
+require 'rspec/rails'
+require 'factory_girl_rails'
+require 'shoulda/matchers'
+
+Rails.backtrace_cleaner.remove_silencers!
+
+# Load support files
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+
+RSpec.configure do |config|
+  config.mock_with :rspec
+  config.use_transactional_fixtures = true
+  config.infer_base_class_for_anonymous_controllers = false
+  config.order = "random"
+  config.include FactoryGirl::Syntax::Methods
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: maestrano-connector-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.2.2
 platform: ruby
 authors:
 - Pierre Berard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-05 00:00:00.000000000 Z
+date: 2016-02-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: maestrano-rails
@@ -200,6 +200,7 @@ files:
 - app/controllers/maestrano/account/groups_controller.rb
 - app/controllers/maestrano/application_controller.rb
 - app/controllers/maestrano/auth/saml_controller.rb
+- app/controllers/maestrano/connec_controller.rb
 - app/controllers/maestrano/sessions_controller.rb
 - app/helpers/maestrano/connector/rails/session_helper.rb
 - app/jobs/maestrano/connector/rails/synchronization_job.rb
@@ -217,11 +218,12 @@ files:
 - app/models/maestrano/connector/rails/user_organization_rel.rb
 - bin/rails
 - config/routes.rb
-- db/migrate/20151122162100_create_maestrano_connector_rails_users.rb
-- db/migrate/20151122162414_create_maestrano_connector_rails_organizations.rb
-- db/migrate/20151122162613_create_maestrano_connector_rails_user_organization_rels.rb
-- db/migrate/20151122163325_create_maestrano_connector_rails_synchronizations.rb
-- db/migrate/20151122163449_create_maestrano_connector_rails_id_maps.rb
+- db/migrate/20151122162100_create_users.rb
+- db/migrate/20151122162414_create_organizations.rb
+- db/migrate/20151122162613_create_user_organization_rels.rb
+- db/migrate/20151122163325_create_synchronizations.rb
+- db/migrate/20151122163449_create_id_maps.rb
+- db/migrate/20160205132857_add_sync_enabled_to_organizations.rb
 - lib/generators/connector/USAGE
 - lib/generators/connector/complex_entity_generator.rb
 - lib/generators/connector/install_generator.rb
@@ -245,6 +247,7 @@ files:
 - maestrano-connector-rails.gemspec
 - maestrano.png
 - pkg/maestrano-connector-rails-0.0.1.gem
+- spec/controllers/connec_controller_spec.rb
 - spec/dummy/README.md
 - spec/dummy/Rakefile
 - spec/dummy/app/assets/images/.keep
@@ -290,6 +293,12 @@ files:
 - spec/dummy/config/routes.rb
 - spec/dummy/config/secrets.yml
 - spec/dummy/db/development.sqlite3
+- spec/dummy/db/migrate/20160204223447_create_users.maestrano_connector_rails_engine.rb
+- spec/dummy/db/migrate/20160204223448_create_organizations.maestrano_connector_rails_engine.rb
+- spec/dummy/db/migrate/20160204223449_create_user_organization_rels.maestrano_connector_rails_engine.rb
+- spec/dummy/db/migrate/20160204223450_create_synchronizations.maestrano_connector_rails_engine.rb
+- spec/dummy/db/migrate/20160204223451_create_id_maps.maestrano_connector_rails_engine.rb
+- spec/dummy/db/migrate/20160205133024_add_sync_enabled_to_organizations.maestrano_connector_rails_engine.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/lib/assets/.keep
@@ -310,7 +319,12 @@ files:
 - spec/models/synchronization_spec.rb
 - spec/models/user_organization_rel_spec.rb
 - spec/models/user_spec.rb
+- spec/routing/connec_routing_spec.rb
 - spec/spec_helper.rb
+- template/factories.rb
+- template/gitignore
+- template/maestrano-connector-template.rb
+- template/spec_helper.rb
 homepage: http://github.com/maestrano/maestrano-connector-rails
 licenses:
 - MIT

data/db/migrate/20151122162100_create_maestrano_connector_rails_users.rb

@@ -1,16 +0,0 @@
-class CreateMaestranoConnectorRailsUsers < ActiveRecord::Migration
-  def change
-    create_table :maestrano_connector_rails_users do |t|
-      t.string :provider
-      t.string :uid
-      t.string :first_name
-      t.string :last_name
-      t.string :email
-      t.string :tenant
-
-      t.timestamps null: false
-    end
-
-    add_index :maestrano_connector_rails_users, [:uid, :tenant], name: 'user_uid_index'
-  end
-end

data/db/migrate/20151122162613_create_maestrano_connector_rails_user_organization_rels.rb

@@ -1,12 +0,0 @@
-class CreateMaestranoConnectorRailsUserOrganizationRels < ActiveRecord::Migration
-  def change
-    create_table :maestrano_connector_rails_user_organization_rels do |t|
-      t.integer :user_id
-      t.integer :organization_id
-
-      t.timestamps
-    end
-    add_index :maestrano_connector_rails_user_organization_rels, :organization_id, name: 'rels_orga_index'
-    add_index :maestrano_connector_rails_user_organization_rels, :user_id, name: 'rels_user_index'
-  end
-end

data/db/migrate/20151122163325_create_maestrano_connector_rails_synchronizations.rb

@@ -1,13 +0,0 @@
-class CreateMaestranoConnectorRailsSynchronizations < ActiveRecord::Migration
-  def change
-    create_table :maestrano_connector_rails_synchronizations do |t|
-      t.integer :organization_id
-      t.string  :status
-      t.text    :message
-      t.boolean :partial, default: false
-
-      t.timestamps null: false
-    end
-    add_index :maestrano_connector_rails_synchronizations, :organization_id, name: 'synchronization_orga_id_index'
-  end
-end

data/db/migrate/20151122163449_create_maestrano_connector_rails_id_maps.rb

@@ -1,18 +0,0 @@
-class CreateMaestranoConnectorRailsIdMaps < ActiveRecord::Migration
-  def change
-    create_table :maestrano_connector_rails_id_maps do |t|
-      t.string :connec_id
-      t.string :connec_entity
-      t.string :external_id
-      t.string :external_entity
-      t.integer :organization_id
-      t.datetime :last_push_to_connec
-      t.datetime :last_push_to_external
-
-      t.timestamps null: false
-    end
-    add_index :maestrano_connector_rails_id_maps, [:connec_id, :connec_entity, :organization_id], name: 'idmap_connec_index'
-    add_index :maestrano_connector_rails_id_maps, [:external_id, :external_entity, :organization_id], name: 'idmap_external_index'
-    add_index :maestrano_connector_rails_id_maps, :organization_id, name: 'idmap_organization_index'
-  end
-end