maestrano-connector-rails 0.2.16 → 0.2.17

data/spec/models/sub_entity_base_spec.rb

@@ -48,4 +48,63 @@ describe Maestrano::Connector::Rails::SubEntityBase do
       it { expect{ subject.connec_entity_name }.to raise_error('Not implemented') }
     end
   end
+
+  describe 'names_hash' do
+    let(:bool) { true }
+    before {
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:external?).and_return(bool)
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:entity_name).and_return('Name')
+    }
+
+    context 'when external' do
+      it { expect(subject.names_hash).to eql({external_entity: 'name'}) }
+    end
+    context 'when not external' do
+      let(:bool) { false }
+      it { expect(subject.names_hash).to eql({connec_entity: 'name'}) }
+    end
+  end
+
+  describe 'create_idmap_from_external_entity' do
+    let(:organization) { create(:organization) }
+    let(:bool) { true }
+    before {
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:external?).and_return(bool)
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:entity_name).and_return('Name')
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:get_id_from_external_entity_hash).and_return('id')
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:object_name_from_external_entity_hash).and_return('object name')
+    }
+
+    context 'when external' do
+      it {
+        expect(Maestrano::Connector::Rails::IdMap).to receive(:create).with({:external_entity=>"name", :external_id=>"id", :name=>"object name", :connec_entity=>"lala", :organization_id=>1})
+        subject.create_idmap_from_external_entity({}, 'lala', organization)
+      }
+    end
+    context 'when not external' do
+      let(:bool) { false }
+      it { expect{ subject.create_idmap_from_external_entity({}, '', organization) }.to raise_error('Forbidden call') }
+    end
+  end
+
+  describe 'create_idmap_from_connec_entity' do
+    let(:organization) { create(:organization) }
+    let(:bool) { true }
+    before {
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:external?).and_return(bool)
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:entity_name).and_return('Name')
+      allow_any_instance_of(Maestrano::Connector::Rails::SubEntityBase).to receive(:object_name_from_connec_entity_hash).and_return('object name')
+    }
+
+    context 'when external' do
+      it { expect{ subject.create_idmap_from_connec_entity({}, '', organization) }.to raise_error('Forbidden call') }
+    end
+    context 'when not external' do
+      let(:bool) { false }
+      it {
+        expect(Maestrano::Connector::Rails::IdMap).to receive(:create).with({:connec_entity=>"name", :connec_id=>"lili", :name=>"object name", :external_entity=>"lala", :organization_id=>1})
+        subject.create_idmap_from_connec_entity({'id' => 'lili'}, 'lala', organization)
+      }
+    end
+  end
 end

data/template/maestrano-connector-template.rb

@@ -3,7 +3,7 @@ def 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}"
+        git :clone => "--quiet https://github.com/berardpi/maestrano-connector-rails/tree/include-frontend #{tempdir}"
 
         "#{tempdir}/template"
       else
@@ -49,7 +49,6 @@ gem 'figaro'
 gem 'tzinfo-data', platforms: [:mingw, :mswin, :jruby]
 
 gem 'maestrano-connector-rails'
-gem 'delayed_job_active_record'
 
 
 gem_group :test do
@@ -70,8 +69,11 @@ copy_file 'gitignore', '.gitignore'
 after_bundle do
   remove_dir 'app/mailers'
   remove_dir 'test'
+  remove_file 'app/views/layouts/application.html.erb'
+  remove_file 'app/assets/stylesheets/application.css'
   copy_file 'spec_helper.rb', 'spec/spec_helper.rb'
   copy_file 'factories.rb', 'spec/factories.rb'
+  copy_file 'routes.rb', 'config/routes.rb'
 
   application do <<-RUBY
     config.generators do |g|
@@ -85,7 +87,6 @@ after_bundle do
   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

data/template/routes.rb

@@ -0,0 +1,14 @@
+Rails.application.routes.draw do
+  mount Maestrano::Connector::Rails::Engine, at: '/'
+
+  root 'home#index'
+  get 'home/index' => 'home#index'
+  get 'home/redirect_to_external' => 'home#redirect_to_external'
+  get 'home/index' => 'home#index'
+  put 'home/update' => 'home#update'
+  post 'home/synchronize' => 'home#synchronize'
+  put 'home/toggle_sync' => 'home#toggle_sync'
+
+  get 'synchronizations/index' => 'synchronizations#index'
+  get 'shared_entities/index' => 'shared_entities#index'
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: maestrano-connector-rails
 version: !ruby/object:Gem::Version
-  version: 0.2.16
+  version: 0.2.17
 platform: ruby
 authors:
 - Pierre Berard
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-03-07 00:00:00.000000000 Z
+date: 2016-03-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: maestrano-rails
@@ -52,6 +52,48 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: haml-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: bootstrap-sass
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: autoprefixer-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: shoulda
   requirement: !ruby/object:Gem::Requirement
@@ -230,8 +272,6 @@ files:
 - lib/generators/connector/USAGE
 - lib/generators/connector/complex_entity_generator.rb
 - lib/generators/connector/install_generator.rb
-- lib/generators/connector/templates/admin_controller.rb
-- lib/generators/connector/templates/admin_index.html.erb
 - lib/generators/connector/templates/complex_entity_example/contact.rb
 - lib/generators/connector/templates/complex_entity_example/contact_and_lead.rb
 - lib/generators/connector/templates/complex_entity_example/contact_mapper.rb
@@ -243,13 +283,28 @@ files:
 - lib/generators/connector/templates/example_entity.rb
 - lib/generators/connector/templates/external.rb
 - lib/generators/connector/templates/home_controller.rb
-- lib/generators/connector/templates/home_index.html.erb
+- lib/generators/connector/templates/home_controller_spec.rb
+- lib/generators/connector/templates/home_index.haml
+- lib/generators/connector/templates/layouts.haml
 - lib/generators/connector/templates/oauth_controller.rb
+- lib/generators/connector/templates/shared_entities_controller.rb
+- lib/generators/connector/templates/shared_entities_controller_spec.rb
+- lib/generators/connector/templates/shared_entities_index.haml
+- lib/generators/connector/templates/stylesheets/application.sass
+- lib/generators/connector/templates/stylesheets/banners.sass
+- lib/generators/connector/templates/stylesheets/home.sass
+- lib/generators/connector/templates/stylesheets/layout.sass
+- lib/generators/connector/templates/stylesheets/spacers.sass
+- lib/generators/connector/templates/stylesheets/variables.sass
+- lib/generators/connector/templates/sychronizations_controller_spec.rb
+- lib/generators/connector/templates/synchronizations_controller.rb
+- lib/generators/connector/templates/synchronizations_index.haml
 - lib/maestrano-connector-rails.rb
 - lib/maestrano/connector/rails.rb
 - maestrano-connector-rails.gemspec
 - maestrano.png
 - pkg/maestrano-connector-rails-0.0.1.gem
+- pkg/maestrano-connector-rails-0.2.16.gem
 - pkg/maestrano-connector-rails-0.2.4.gem
 - spec/controllers/connec_controller_spec.rb
 - spec/dummy/README.md
@@ -324,6 +379,7 @@ files:
 - template/factories.rb
 - template/gitignore
 - template/maestrano-connector-template.rb
+- template/routes.rb
 - template/spec_helper.rb
 homepage: http://github.com/maestrano/maestrano-connector-rails
 licenses:

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

@@ -1,58 +0,0 @@
-# TODO
-# This controller is given as an example of a possible admin implementation
-# The admin functions should be restricted to admin users
-# Admin funcitons :
-#  * Link account to external application
-#  * Disconnect account from external application
-#  * Launch a manual syncrhonization for all entities or a sub-part of them
-#  * Chose which entities are synchronized by the connector
-#  * Access a list of the organization's idmaps
-class AdminController < ApplicationController
-
-  def index
-    if is_admin
-      @organization = current_organization
-      @idmaps = Maestrano::Connector::Rails::IdMap.where(organization_id: @organization.id).order(:connec_entity)
-    end
-  end
-
-  def update
-    organization = Maestrano::Connector::Rails::Organization.find_by_id(params[:id])
-
-    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 admin_index_path
-  end
-
-  def synchronize
-    if is_admin
-      Maestrano::Connector::Rails::SynchronizationJob.perform_later(current_organization, params['opts'] || {})
-    end
-
-    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)
-    end
-end

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

@@ -1,55 +0,0 @@
-<!-- TODO -->
-<!-- This template is given as an example of the informations
-and functions available on the admin panel -->
-<!-- This view is designed according to the admin controller supplied -->
-<div class='admin'>
-  <% unless @organization %>
-    <p> You need to be Admin or Super Admin in your company to access this panel.</p>
-  <% else %>
-    <ul>
-      <li>
-        Maestrano account
-        <%= link_to "Disconnect your Maestrano account", maestrano_connector_rails.signout_path %>
-      </li>
-
-      <li>
-        External account
-        <% if @organization.oauth_uid %>
-          <%= link_to "Disconnect #{@organization.name} (#{@organization.uid})", 'some_path' %>
-        <% else %>
-          <%= link_to "Connect #{@organization.name} (#{@organization.uid})", 'some_other_path' %>
-        <% end %>
-      </li>
-
-      <li>
-        <%= link_to "Force a synchronization", admin_synchronize_path, method: :post %>
-      </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>
-    <%= 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.humanize.pluralize} only", admin_synchronize_path(opts: {only_entities: [k.to_s]}), method: :post %>
-          <% end %>
-      <% end %>
-      <%= submit_tag :Update%>
-    <% end %>
-
-    <h3>IdMap</h3>
-    <% @idmaps.each do |idmap| %>
-      <%= idmap.connec_entity ? idmap.connec_entity.humanize : nil %>
-      <%= idmap.connec_id %>
-      <%= idmap.external_entity ? idmap.external_entity.humanize : nil %>
-      <%= idmap.external_id %>
-    <% end %>
-
-  <% end %>
-</div>

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

@@ -1,48 +0,0 @@
-<!-- TODO -->
-<!-- This template is given as an example of the informations that
-can be displayed on the home page -->
-<!-- This view is designed according to the home controller supplied -->
-<div class='home'>
-  <%= link_to 'Admin panel', admin_index_path %>
-  <% unless current_user %>
-    <%= link_to "Link your Maestrano account", maestrano_connector_rails.init_maestrano_auth_saml_index_path(tenant: :default) %>
-    <p>Some generic information about the connector</p>
-
-  <% else %>
-    <% unless @organization.oauth_uid %>
-      Not linked to external app
-      <% if is_admin?(current_user, @organization) %>
-        <%= link_to "Link this company to ...", 'some_path' %>
-      <% end %>
-    <% else %>
-      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}" %>
-      <% if @synchronizations.first.is_error? %>
-        <%= @synchronizations.first.message %>
-      <% elsif @synchronizations.first.is_success? && @synchronizations.first.partial? %>
-        Partial synchronization
-      <% end %>
-    <% end %>
-
-    <h2>Synchronizations history</h2>
-    <% @synchronizations.each do |sync| %>
-      <%= sync.updated_at %>
-      <%= sync.status %>
-      <% if sync.is_error %>
-        <%= sync.message %>
-      <% elsif sync.is_success? && sync.partial? %>
-        Partial synchronization
-      <% end %>
-    <% end %>
-  <% end %>
-</div>