maestrano-connector-rails 0.2.16 → 0.2.17

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

@@ -14,19 +14,59 @@ module Maestrano::Connector::Rails
     end
 
     def external_entity_name
-      if self.external?
-        self.entity_name
+      if external?
+        entity_name
       else
         raise "Forbidden call"
       end
     end
 
     def connec_entity_name
-      if self.external?
+      if external?
         raise "Forbidden call"
       else
-        self.entity_name
+        entity_name
       end
     end
+
+    def names_hash
+      if external?
+        {external_entity: entity_name.downcase}
+      else
+        {connec_entity: entity_name.downcase}
+      end
+    end
+
+    def create_idmap_from_external_entity(entity, connec_entity_name, organization)
+      if external?
+        h = names_hash.merge({
+          external_id: get_id_from_external_entity_hash(entity),
+          name: object_name_from_external_entity_hash(entity),
+          connec_entity: connec_entity_name.downcase,
+          organization_id: organization.id
+        })
+        Maestrano::Connector::Rails::IdMap.create(h)
+      else
+        raise 'Forbidden call'
+      end
+    end
+
+    def create_idmap_from_connec_entity(entity, external_entity_name, organization)
+      if external?
+        raise 'Forbidden call'
+      else
+        h = names_hash.merge({
+          connec_id: entity['id'],
+          name: object_name_from_connec_entity_hash(entity),
+          external_entity: external_entity_name.downcase,
+          organization_id: organization.id
+        })
+        Maestrano::Connector::Rails::IdMap.create(h)
+      end
+    end
+
+    def map_external_entity_with_idmap(external_entity, connec_entity_name, idmap, organization)
+      {entity: map_to(connec_entity_name, external_entity, organization), idmap: idmap}
+    end
   end
 end

data/lib/generators/connector/install_generator.rb

@@ -21,25 +21,6 @@ module Connector
         end
       end
 
-      def include_routes
-        sentinel = 'Rails.application.routes.draw do'
-        code_lines = [
-          "mount Maestrano::Connector::Rails::Engine, at: '/'\n",
-          "root 'home#index'",
-          "get 'home/index' => 'home#index'",
-          "get 'admin/index' => 'admin#index'",
-          "put 'admin/update' => 'admin#update'",
-          "post 'admin/synchronize' => 'admin#synchronize'",
-          "put 'admin/toggle_sync' => 'admin#toggle_sync'"
-        ]
-
-        in_root do
-          gsub_file 'config/routes.rb', /(#{Regexp.escape(sentinel)})/mi do |match|
-            "#{match}\n  #{code_lines.join("\n  ")}\n"
-          end
-        end
-      end
-
       def copy_entity
         copy_file 'entity.rb', 'app/models/maestrano/connector/rails/entity.rb'
       end
@@ -52,14 +33,29 @@ module Connector
         copy_file 'example_entity.rb', 'app/models/entities/example_entitiy.rb'
       end
 
-      def copy_home
+      def copy_controllers_and_views
         copy_file 'home_controller.rb', 'app/controllers/home_controller.rb'
-        copy_file 'home_index.html.erb', 'app/views/home/index.html.erb'
+        copy_file 'home_controller_spec.rb', 'spec/controllers/home_controller_spec.rb'
+        copy_file 'home_index.haml', 'app/views/home/index.html.haml'
+
+        copy_file 'synchronizations_controller.rb', 'app/controllers/synchronizations_controller.rb'
+        copy_file 'synchronizations_controller_spec.rb', 'spec/controllers/synchronizations_controller_spec.rb'
+        copy_file 'synchronizations_index.haml', 'app/views/synchronizations/index.html.haml'
+
+        copy_file 'shared_entities_controller.rb', 'app/controllers/shared_entities_controller.rb'
+        copy_file 'shared_entities_controller_spec.rb', 'spec/controllers/shared_entities_controller_spec.rb'
+        copy_file 'shared_entities_index.haml', 'app/views/shared_entities/index.html.haml'
+
+        copy_file 'layouts.haml', 'app/views/layouts/application.html.haml'
       end
 
-      def copy_admin_view
-        copy_file 'admin_controller.rb', 'app/controllers/admin_controller.rb'
-        copy_file 'admin_index.html.erb', 'app/views/admin/index.html.erb'
+      def copy_stylesheets
+        copy_file 'stylesheets/application.sass', 'app/assets/stylesheets/application.sass'
+        copy_file 'stylesheets/banners.sass', 'app/assets/stylesheets/banners.sass'
+        copy_file 'stylesheets/home.sass', 'app/assets/stylesheets/home.sass'
+        copy_file 'stylesheets/layout.sass', 'app/assets/stylesheets/layout.sass'
+        copy_file 'stylesheets/spacers.sass', 'app/assets/stylesheets/spacers.sass'
+        copy_file 'stylesheets/variables.sass', 'app/assets/stylesheets/variables.sass'
       end
 
       def copy_oauth_controller

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

@@ -1,17 +1,40 @@
-# TODO
-# This controller is given as an example of a possible home
-# Home funcitons :
-# * Display generic information about the connector (mostly for not connected users)
-# * Link the connector to a Maestrano organization
-# * Acces the last synchronization and a synchronization history (with their status)
 class HomeController < ApplicationController
   def index
-    if current_user
-      @organization = current_organization
+    @organization = current_organization
+  end
+
+  def update
+    organization = Maestrano::Connector::Rails::Organization.find_by_id(params[:id])
 
-      if @organization
-        @synchronizations = Maestrano::Connector::Rails::Synchronization.where(organization_id: @organization.id).order(updated_at: :desc).limit(40)
+    if organization && is_admin?(current_user, organization)
+      organization.synchronized_entities.keys.each do |entity|
+        if !!params["#{entity}"]
+          organization.synchronized_entities[entity] = true
+        else
+          organization.synchronized_entities[entity] = false
+        end
       end
+      organization.save
+    end
+
+    redirect_to(:back)
+  end
+
+  def synchronize
+    if is_admin
+      Maestrano::Connector::Rails::SynchronizationJob.perform_later(current_organization, params['opts'] || {})
+      flash[:info] = 'Synchronization requested'
     end
+
+    redirect_to(:back)
+  end
+
+  def toggle_sync
+    if is_admin
+      current_organization.update(sync_enabled: !current_organization.sync_enabled)
+      flash[:info] = current_organization.sync_enabled ? 'Synchronization enabled' : 'Synchronization disabled'
+    end
+
+    redirect_to(:back)
   end
 end

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

@@ -0,0 +1,141 @@
+require 'spec_helper'
+
+describe HomeController, :type => :controller do
+  let(:back_path) { home_index_path }
+  before(:each) do
+    request.env["HTTP_REFERER"] = back_path
+  end
+
+  describe 'index' do
+    subject { get :index }
+
+    it { expect(subject).to be_success }
+  end
+
+  describe 'update' do
+    let(:organization) { create(:organization, synchronized_entities: {'people' => false, 'item' => true}) }
+    subject { put :update, id: organization.id, 'people' => true, 'item' => false, 'lala' => true }
+
+
+    context 'when user is not admin' do
+      before {
+        allow_any_instance_of(Maestrano::Connector::Rails::SessionHelper).to receive(:is_admin?).and_return(false)
+      }
+
+      it { expect(subject).to redirect_to back_path }
+    end
+
+    context 'when user is admin' do
+      before {
+        allow_any_instance_of(Maestrano::Connector::Rails::SessionHelper).to receive(:is_admin?).and_return(true)
+      }
+
+      it { expect(subject).to redirect_to back_path }
+
+      it 'updates organization' do
+        subject
+        organization.reload
+        expect(organization.synchronized_entities).to eq({'people' => true, 'item' => false})
+      end
+    end
+  end
+
+  describe 'synchronize' do
+    subject { post :synchronize }
+    let(:organization) { create(:organization) }
+    before {
+      allow_any_instance_of(Maestrano::Connector::Rails::SessionHelper).to receive(:current_organization).and_return(organization)
+    }
+
+    context 'when user is not admin' do
+      before {
+        allow_any_instance_of(ApplicationHelper).to receive(:is_admin).and_return(false)
+      }
+
+      it { expect(subject).to redirect_to back_path }
+
+      it 'does nothing' do
+        expect(Maestrano::Connector::Rails::SynchronizationJob).to_not receive(:perform_later)
+        subject
+      end
+    end
+
+    context 'when user is admin' do
+      before {
+        allow_any_instance_of(ApplicationHelper).to receive(:is_admin).and_return(true)
+      }
+
+      it { expect(subject).to redirect_to back_path }
+
+      context 'with opts' do
+        subject { post :synchronize, opts: 'opts' }
+
+        it 'calls perform_later with opts' do
+          expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization, 'opts')
+          subject
+        end
+      end
+
+      context 'without opts' do
+        subject { post :synchronize}
+
+        it 'calls perform_later with empty opts hash' do
+          expect(Maestrano::Connector::Rails::SynchronizationJob).to receive(:perform_later).with(organization, {})
+          subject
+        end
+      end
+    end
+
+    describe 'toggle_sync' do
+      subject { put :toggle_sync }
+      let(:organization) { create(:organization, sync_enabled: true) }
+      before {
+        allow_any_instance_of(Maestrano::Connector::Rails::SessionHelper).to receive(:current_organization).and_return(organization)
+      }
+
+      context 'when user is not an admin' do
+        before {
+          allow_any_instance_of(ApplicationHelper).to receive(:is_admin).and_return(false)
+        }
+        it { expect(subject).to redirect_to back_path }
+
+        it 'does nothing' do
+          expect{ subject }.to_not change{ organization.sync_enabled }
+        end
+      end
+
+      context 'when user is admin' do
+        before {
+          allow_any_instance_of(ApplicationHelper).to receive(:is_admin).and_return(true)
+        }
+        it { expect(subject).to redirect_to back_path }
+
+        it 'change sync_enabled from true to false' do
+          expect{ subject }.to change{ organization.sync_enabled }.from(true).to(false)
+        end
+
+        it 'change sync_enabled from false to true' do
+          organization.update(sync_enabled: false)
+          expect{ subject }.to change{ organization.sync_enabled }.from(false).to(true)
+        end
+      end
+    end
+  end
+
+  describe 'redirect_to_external' do
+    subject { get :redirect_to_external }
+
+    context 'when organization has a redirect url' do
+      let(:organization) {  create(:organization, instance_url: 'url') }
+      before {
+        allow_any_instance_of(Maestrano::Connector::Rails::SessionHelper).to receive(:current_organization).and_return(organization)
+      }
+
+      it {expect(subject).to redirect_to('url')}
+    end
+
+    context 'otherwise' do
+      it {expect(subject).to redirect_to('https://login.salesforce.com')}
+    end
+  end
+end

data/lib/generators/connector/templates/home_index.haml

@@ -0,0 +1,103 @@
+/ TODO
+/ Replace ApplicationName
+.home
+  .banners
+    .row
+      .col-md-10.col-md-offset-2
+        %h2 ApplicationName Connector
+        %p
+          Link your Maestrano account to ApplicationName to get your business in synch. Check the status of your connection on this screen.
+
+  .container
+    - if current_user
+      - unless is_admin
+        .row
+          .col-md-12.alert.alert-warning
+            Only administrators can modify the application settings
+
+      .row.link-step
+        .col-md-1.text-center.link-step-number
+          %span.badge.link-step-badge
+            1
+        .col-md-3.text-center
+          =image_tag "logos/default-white.png", class: 'logo'
+        .col-md-6.link-step-description
+          %h
+            Your Maestrano company <strong>#{@organization.name} (#{@organization.uid})</strong> is currently linked
+        .col-md-2.text-center.link-step-action
+          = link_to "Disconnect", Maestrano::Connector::Rails::Engine.routes.url_helpers.maestrano_signout_path, class: "btn btn-warning btn-lg #{is_admin ? '' : 'disabled'}"
+
+      .spacer1
+
+      .row.link-step
+        .col-md-1.text-center.link-step-number
+          %span.badge.link-step-badge
+            2
+        .col-md-3.text-center
+          =image_tag "logos/external.png", class: 'logo'
+        .col-md-6.link-step-description
+          %h
+            - if @organization.oauth_uid
+              Your ApplicationName account <strong>#{@organization.oauth_name} (#{@organization.oauth_uid})</strong> is currently linked
+            - else
+              Your ApplicationName account is not linked
+        .col-md-2.text-center.link-step-action
+          - if @organization.oauth_uid
+            / TODO
+            = link_to "Disconnect", root_path, class: "btn btn-warning btn-lg #{is_admin ? '' : 'disabled'}"
+          - else
+            / TODO
+            = link_to "Link to ApplicationName", root_path, class: "btn btn-warning btn-lg #{is_admin ? '' : 'disabled'}"
+
+      .spacer1
+
+      .row.link-step
+        .col-md-1.text-center.link-step-number
+          %span.badge.link-step-badge
+            3
+        .col-md-3.text-center
+          =image_tag "logos/sync.png", class: 'logo'
+        .col-md-6.link-step-description
+          %h
+            - if @organization.sync_enabled
+              Synchronizations are enabled
+            - else
+              Synchronizations are disabled
+        .col-md-2.text-center.link-step-action
+          = link_to "Toggle #{@organization.sync_enabled ? 'off' : 'on'}", home_toggle_sync_path, method: :put, class: "btn btn-warning btn-lg #{is_admin ? '' : 'disabled'}"
+
+    - else
+      .row
+        .col-md-4.col-md-offset-4.center
+          = link_to "Link your Maestrano account", Maestrano::Connector::Rails::Engine.routes.url_helpers.default_maestrano_auth_saml_index_path(tenant: :default), class: 'btn btn-warning'
+
+    - if @organization
+      .spacer2
+      .row
+        .col-md-12
+          %h1
+            Synchronized entities
+            %small You can customize which entities are synchronized by the connector
+
+      .spacer1
+      .row
+        .col-md-10.col-md-offset-1
+          = form_tag home_update_path(id: @organization.id), method: :put do
+            - @organization.synchronized_entities.each do |k, v|
+              .row.sync-entity
+                .col-md-1.link-step-action
+                  %input{type: "checkbox", id: "#{k}", name: "#{k}", checked: v}
+                .col-md-7{style: 'padding-top: 5px;'}
+                  %label{:for => "#{k}"} #{k.to_s.humanize}
+                -if is_admin
+                  .col-md-4.text-right
+                    - if v && @organization.oauth_uid
+                      - if @organization.sync_enabled
+                        = link_to "Force a synchronization for #{k.to_s.humanize.pluralize} only", home_synchronize_path(opts: {only_entities: [k.to_s]}), method: :post, class: "btn btn-warning btn-sm #{is_admin && @organization.sync_enabled ? '' : 'disabled'}"
+                      - else
+                        .btn.btn-warning.btn-sm.disabled= "Force a synchronization for #{k.to_s.humanize.pluralize} only"
+            -if is_admin
+              .spacer1
+              .row
+                .col-md-4.col-md-offset-8.text-right
+                  =submit_tag :Save, class: "btn btn-primary"

data/lib/generators/connector/templates/layouts.haml

@@ -0,0 +1,45 @@
+/ TODO
+/ Replace ApplicationName
+!!!
+%html
+  %head
+    %title ApplicationName Connector for Connec!™
+    = stylesheet_link_tag    'application', media: 'all', 'data-turbolinks-track' => true
+    = javascript_include_tag 'application', 'data-turbolinks-track' => true
+    = csrf_meta_tags
+  %body
+    .navbar.navbar-fixed-top.navbar-inverse
+      .container-fluid
+        .collapse.navbar-collapse#maestrano-navbar-collapse
+          %ul.nav.navbar-nav
+            =image_tag "logos/default.png", class: 'external-logo'
+
+          %ul.nav.navbar-nav
+            %li=link_to "Status", root_path
+            %li=link_to "History", synchronizations_index_path
+            -if is_admin
+              %li=link_to "Mapping", shared_entities_index_path
+
+          -if current_user
+            %ul.nav.navbar-nav.navbar-right
+              %li.dropdown
+                %a{:href=>"#", :class=>"dropdown-toggle", :'data-toggle'=>"dropdown", :role=>"button"}
+                  Hi #{current_user.first_name} #{current_user.last_name}
+
+    .padding-navbar
+
+    .wrapper{style: 'margin-bottom: 15px; '}
+      - if flash.any?
+        - flash.each do |flash_key, message|
+          .alert{:class => "alert-#{flash_key}"}
+            %strong= flash.discard(flash_key)
+
+      = yield
+
+    .footer
+      .container
+        .row
+          .col-xs-12
+            .copyright
+              © Copyright #{Time.now.year} Maestrano Pty Ltd (ABN: 80 152 564 424).
+              All rights reserved. Various trademarks held by their respective owners.