bpl-institution-management 0.0.2

checksums.yaml

@@ -0,0 +1,7 @@
+---
+SHA1:
+  metadata.gz: 4b12ce3c38e5393d54480a92615e7b373b59c738
+  data.tar.gz: 40e1d905234af541aa6120a089b139dbcebc3128
+SHA512:
+  metadata.gz: a4502986ef49b1f0d058d79615d33a969abc3a938aa4015a70c800f3292a8168ae4f05ed4d15cab6338b6c4a4b41a6a5a502472e743c9f4bb911b6ebb740324f
+  data.tar.gz: e056a3936d2f7732ed8e00c77372213f45af7ca5b815cf7a2fb94dc0a97578b4eae4e42b52bcc9cd3a2aa8b067acb8662cef4321eb41af0996b941d1c225f85a

data/.gitignore

@@ -0,0 +1,70 @@
+# Generated by default by Github #
+##################################
+*.rbc
+*.sassc
+.sass-cache
+capybara-*.html
+.rspec
+/.bundle
+/vendor/bundle
+/log/*
+/tmp/*
+/db/*.sqlite3
+/public/system/*
+/coverage/
+/spec/tmp/*
+**.orig
+rerun.txt
+pickle-email-*.html
+
+# Specific configuration files #
+################################
+hydra-ldap.yml
+database.yml
+fedora.yml
+solr.yml
+role_map_cucumber.yml
+role_map_development.yml
+role_map_production.yml
+role_map_test.yml
+ark.yml
+omniauth-polaris.yml
+bpl_config.yml
+
+# Files to ignore by default in Git #
+#####################################
+*.pdf
+*.db
+*.jar
+*.gz
+*.tar
+*.zip
+*.dll
+*.sqlite
+*.sql
+
+# Specific Files to ignore #
+############################
+Thumbs.db
+Icon?
+.DS_Store?
+.DS_Store
+ehthumbs.db
+.idea # Rubymine config files
+/.idea/*
+Gemfile.lock
+
+# Directories to ignore #
+#########################
+/public/data/*
+/public/batch_uploads/*
+/jetty/*
+/solr_conf/*
+/fedora_conf/*
+/hydra-jetty/*
+/*jetty*
+
+# Ignore edited Linux files #
+#############################
+
+*~

data/Gemfile

@@ -0,0 +1,4 @@
+source 'https://rubygems.org'
+
+# Specify your gem's dependencies in hydra-batch-edit.gemspec
+gemspec

data/README.md

@@ -0,0 +1,19 @@
+An engine gem to provide a RDBMS backed list of roles and their associated user.  This replaces the hydra default role mapper.
+
+
+##Installing:
+
+* Add: ```gem 'bpl-institution-management'``` to your Gemfile and then ```bundle install```
+* ```rails generate roles```
+* ```rake db:migrate```
+* Add the following [cancan](https://github.com/ryanb/cancan) abilities:
+
+```ruby
+  # app/models/ability.rb
+  TBA
+```
+
+##Testing:
+
+* Install a system javascript runtime or uncomment therubyracer in spec/support/Gemfile
+* Run ```rake generate spec``` to generate a test rails app at spec/internal and test it

data/Rakefile

@@ -0,0 +1,56 @@
+#!/usr/bin/env rake
+require "bundler/gem_tasks"
+require 'rspec/core/rake_task'
+
+ENV["RAILS_ROOT"] ||= 'spec/internal'
+
+desc 'Default: run specs.'
+task :default => :spec
+
+
+task :spec => [:generate] do |t|
+  focused_spec = ENV['SPEC'] ? " SPEC=#{File.join(GEM_ROOT, ENV['SPEC'])}" : ''
+  within_test_app do
+    system "rake myspec#{focused_spec}"
+    abort "Error running bpl-institution-management" unless $?.success?
+  end
+end
+
+
+
+desc "Create the test rails app"
+task :generate do
+  unless File.exists?('spec/internal/Rakefile')
+    puts "Generating rails app"
+    `rails new spec/internal`
+    puts "Copying gemfile"
+    `cp spec/support/Gemfile spec/internal`
+    puts "Copying generator"
+    `cp -r spec/support/lib/generators spec/internal/lib`
+
+    within_test_app do
+      puts "Bundle install"
+      puts `bundle install`
+      puts "running generator"
+      puts `rails generate test_app`
+
+      puts "running migrations"
+      puts `rake db:migrate db:test:prepare`
+    end
+  end
+  puts "Running specs"
+end
+
+desc "Clean out the test rails app"
+task :clean do
+  puts "Removing sample rails app"
+  `rm -rf spec/internal`
+end
+
+def within_test_app
+  FileUtils.cd('spec/internal')
+  Bundler.with_clean_env do
+    yield
+  end
+  FileUtils.cd('../..')
+end

data/app/controllers/concerns/bpl/institution_management/institutions_behavior.rb

@@ -0,0 +1,87 @@
+module Bpl
+  module InstitutionManagement
+    module InstitutionsBehavior
+      extend ActiveSupport::Concern
+
+      included do
+        load_and_authorize_resource 
+      end
+
+      def index
+      end
+
+      def show
+        redirect_to institution_management.edit_institution_path(@institution) if can? :edit, @institution
+      end
+
+      def new
+      end
+
+      def edit
+      end
+
+      def create
+
+        @institution = Institution.new(institution_params)
+        #@institution.name = params[:institution][:name]
+
+        institution = Bplmodels::Institution.find(:label_ssim=>@institution.name).first
+        if institution == nil
+          @localid = @institution.name
+          @localid_type = "Physical Location From UI"
+
+          response = Typhoeus::Request.post(ARK_CONFIG_GLOBAL['url'] + "/arks.json", :params => {:ark=>{:namespace_ark => ARK_CONFIG_GLOBAL['namespace_commonwealth_ark'], :namespace_id=>ARK_CONFIG_GLOBAL['namespace_commonwealth_pid'], :url_base => ARK_CONFIG_GLOBAL['ark_commonwealth_base'], :model_type => Bplmodels::Institution.name, :local_original_identifier=>@localid, :local_original_identifier_type=>@localid_type}})
+
+          as_json = JSON.parse(response.body)
+
+          institution = Bplmodels::Institution.new(:pid=>as_json["pid"])
+          institution.label =  @institution.name
+          institution.descMetadata.insert_title(@institution.name, nil)
+
+          institution.save!
+        end
+
+        @institution.pid = institution.pid
+
+        if (@institution.save)
+          redirect_to institution_management.edit_institution_path(@institution), notice: 'Institution was successfully created.'
+        else
+          render action: "new"
+        end
+      end
+
+      def update
+        @institution = Institution.find(params[:id])
+
+        institution = Bplmodels::Institution.find(:label_ssim=>@institution.name).first
+        institution.label = @institution.name
+        institution.save!
+
+        if (@institution.update_attributes(institution_params))
+          redirect_to institution_management.edit_institution_path(@institution), notice: 'Institution was successfully updated.'
+        else
+          render action: "edit"
+        end
+      end
+
+      def destroy
+        institution = Bplmodels::Institution.find(:label_ssim=>@institution.name).first
+        institution.label = @institution.name
+        institution.delete
+
+        if (@institution.destroy)
+          redirect_to institution_management.institutions_path, notice: 'Institution was successfully deleted.'
+        else
+          redirect_to institution_management.institutions_path
+        end
+      end
+
+      private
+
+      def institution_params
+          params.require(:institution).permit(:name, :pid)
+      end
+
+    end
+  end
+end

data/app/controllers/concerns/bpl/institution_management/user_institutions_behavior.rb

@@ -0,0 +1,30 @@
+module Bpl
+  module InstitutionManagement
+    module UserInstitutionsBehavior
+      extend ActiveSupport::Concern
+
+      included do
+        load_and_authorize_resource :institution
+      end
+
+      def create
+        authorize! :add_user, @institution
+        u = ::User.find_by_user_key(params[:user_key])
+        if u
+          u.institutions << @institution
+          u.save!
+          redirect_to institution_management.institution_path(@institution)
+        else
+          redirect_to institution_management.institution_path(@institution), :flash=> {:error=>"Unable to find the user #{params[:user_key]}"}
+        end
+      end
+
+      def destroy
+        authorize! :remove_user, @institution
+        @institution.users.delete(::User.find(params[:id]))
+        redirect_to institution_management.institution_path(@institution)
+      end
+    end
+  end
+end
+

data/app/controllers/institutions_controller.rb

@@ -0,0 +1,4 @@
+class InstitutionsController < ApplicationController
+  include Bpl::InstitutionManagement::InstitutionsBehavior
+end
+

data/app/controllers/user_institutions_controller.rb

@@ -0,0 +1,13 @@
+class UserInstitutionsController < ApplicationController
+
+  #FIXME: Done to satisfy CanCan. See: http://stackoverflow.com/questions/12756469/cancan-load-and-authorize-resource-triggers-forbidden-attributes or http://stackoverflow.com/questions/20150322/rails-4-strong-params-forbiddenattributeserror
+  before_filter :fixCanCan, only: [:create, :edit]
+
+  def fixCanCan
+    params[:institution] = institution_params
+  end
+
+  include Bpl::InstitutionManagement::UserInstitutionsBehavior
+end
+
+

data/app/models/concerns/bpl/institution_management/user_institutions.rb

@@ -0,0 +1,79 @@
+module Bpl
+  module InstitutionManagement
+    module UserInstitutions
+      extend ActiveSupport::Concern
+      included do
+        has_and_belongs_to_many :institutions
+      end
+
+
+      def institution_objects
+        Bplmodels::Institution.find(self.list_institution_pids)
+      end
+
+      def list_institution_pids
+        if superuser?
+          #return all pids if superuser
+          Institution.pluck(:pid)
+        else
+          #return only associated pids if normal user
+          institutions.pluck(:pid)
+        end
+
+      end
+
+      def list_institution_names
+        if superuser?
+          #return all pids if superuser
+          Institution.pluck(:name)
+        else
+          #return only associated pids if normal user
+          institutions.pluck(:name)
+        end
+
+      end
+
+      def list_collection_pids
+        collection_list = []
+        if superuser?
+          #return all pids if superuser
+          Bplmodels::Collection.find_in_batches("has_model_ssim"=>"info:fedora/afmodel:Bplmodels_Collection") do |collection_group|
+            collection_group.each { |solr_objects|
+              pid = solr_objects['id']
+              collection_list << pid
+            }
+          end
+
+        else
+          #return only associated pids if normal user
+          self.institution_objects.each do |institution|
+            institution.collections.each do |collection|
+              collection_list << collection.pid
+            end
+          end
+        end
+        collection_list
+      end
+
+      def list_collection_names
+        collection_list = []
+        if superuser?
+          #return all pids if superuser
+
+          Bplmodels::Collection.all.each do |collection|
+            collection_list << collection.label
+          end
+
+        else
+          #return only associated pids if normal user
+          self.institution_objects.collections.each do |collection|
+            collection_list << collection.label
+          end
+
+        end
+        collection_list
+      end
+      
+    end
+  end
+end

data/app/models/institution.rb

@@ -0,0 +1,13 @@
+class Institution < ActiveRecord::Base
+  has_and_belongs_to_many :users
+
+  #attr_accessible :pid, :name
+
+  validates :name,
+    uniqueness: true,
+    format: { with: /\A[a-zA-Z0-9._ -]+\z/,
+      :message => "Only letters, numbers, hyphens, underscores, spaces and periods are allowed"}
+
+end
+
+

data/app/views/institutions/edit.html.erb

@@ -0,0 +1,28 @@
+<h2>Institution:</h2>
+<%= bootstrap_form_for @institution, :url=>institution_management.institution_path(@institution) do |f| %>
+  <%= f.text_field :name, :label=> 'Institution name' %>
+  <%= f.actions do %>
+    <%= f.submit "Update" %>
+  <% end %>
+<% end %>
+<% if can? :destroy, Institution %>
+  <%= button_to "Delete", institution_management.institution_path(@institution), :method=>:delete, :class=>'btn btn-danger' %>
+<% end %>
+<h3>Accounts:</h3>
+<ul>
+  <% @institution.users.each do |user| %>
+    <li><%= user.user_key %>
+      <% if can? :remove_user, Institution %>
+        <%= button_to "Remove User", institution_management.institution_user_path(@institution, user), :method=>:delete, :class=>'btn btn-danger' %>
+      <% end %>
+    </li>
+  <% end %>
+</ul>
+<h3>Add a new account:</h3>
+<%= bootstrap_form_tag institution_management.institution_users_path(@institution) do %>
+  <%= bootstrap_text_field_tag 'user_key', '', :label=>'User' %>
+  <%= bootstrap_actions do %>
+    <%= bootstrap_submit_tag "Add" %>
+    <%= bootstrap_cancel_tag %>
+  <% end %>
+<% end %>

data/app/views/institutions/index.html.erb

@@ -0,0 +1,11 @@
+<h2>Institution</h2>
+<ul>
+  <% @institutions.each do |institution| %>
+    <li><%=link_to institution.name, institution_management.institution_path(institution) %></li>
+  <% end %>
+</ul>
+
+<% if can? :create, Institution %>
+  <%= button_to "Create a new institution", institution_management.new_institution_path, method: :get, class: 'btn btn-primary' %>
+<% end %>
+

data/app/views/institutions/new.html.erb

@@ -0,0 +1,5 @@
+<%= bootstrap_form_for @institution, :url=>institution_management.institutions_path do |f| %>
+  <%= f.text_field :name, :label=> 'Institution name' %>
+  <%= f.actions %>
+<% end %>
+

data/app/views/institutions/show.html.erb

@@ -0,0 +1,19 @@
+<h2>Institution: <%= @institution.name %></h2>
+<h3>Accounts:</h3>
+<ul>
+  <% @institution.users.each do |user| %>
+    <li><%= user.user_key %>
+      <% if can? :remove_user, Institution %>
+        <%= button_to "Remove User", institution_management.institution_user_path(@institution, user), :method=>:delete, :class=>'btn btn-danger' %>
+      <% end %>
+    </li>
+  <% end %>
+</ul>
+<h3>Add a new account:</h3>
+<%= bootstrap_form_tag institution_management.institution_users_path(@institution) do %>
+  <%= bootstrap_text_field_tag 'user_key', '', :label=>'User' %>
+  <%= bootstrap_actions do %>
+    <%= bootstrap_submit_tag "Add" %>
+    <%= bootstrap_cancel_tag %>
+  <% end %>
+<% end %>

data/bpl-institution-management.gemspec

@@ -0,0 +1,24 @@
+# -*- encoding: utf-8 -*-
+require File.expand_path('../lib/bpl/institution_management/version', __FILE__)
+
+Gem::Specification.new do |gem|
+  gem.authors       = ["Steven Anderson"]
+  gem.email         = ["sanderson@bpl.org"]
+  gem.description   = %q{Rails engine to do user institutions in an RDBMS for hydra-head}
+  gem.summary       = %q{Rails engine to do user institutions in an RDBMS for hydra-head}
+  gem.homepage      = "https://github.com/boston-library/bpl-institution-management"
+
+  gem.files         = `git ls-files`.split($\)
+  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
+  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
+  gem.name          = "bpl-institution-management"
+  gem.require_paths = ["lib"]
+  gem.version       = Bpl::InstitutionManagement::VERSION
+
+  gem.add_dependency 'cancan'
+  gem.add_dependency 'bootstrap_forms'
+
+  gem.add_development_dependency 'rake'
+  gem.add_development_dependency 'rails'
+  gem.add_development_dependency 'rspec-rails'
+end

data/config/routes.rb

@@ -0,0 +1,7 @@
+Bpl::InstitutionManagement::Engine.routes.draw do
+  # Generic file routes
+  resources :institutions do
+    resources :users, :only=>[:create, :destroy], :controller => "user_institutions"
+  end
+end
+

data/lib/bpl-institution-management.rb

@@ -0,0 +1,2 @@
+require "bpl/institution_management"
+require 'bootstrap_forms'

data/lib/bpl/institution_management.rb

@@ -0,0 +1,11 @@
+module Bpl
+  module InstitutionManagement
+    class Engine < ::Rails::Engine
+      engine_name 'institution_management'
+
+      # Rails 4 should do this automatically:
+      config.paths.add "app/controllers/concerns", eager_load: true
+      config.paths.add "app/models/concerns", eager_load: true
+    end
+  end
+end

data/lib/bpl/institution_management/version.rb

@@ -0,0 +1,5 @@
+module Bpl
+  module InstitutionManagement
+    VERSION = "0.0.2"
+  end
+end

data/lib/generators/institutions/institutions_generator.rb

@@ -0,0 +1,71 @@
+# -*- encoding : utf-8 -*-
+require 'rails/generators'
+require 'rails/generators/migration'     
+
+class InstitutionsGenerator < Rails::Generators::Base
+  include Rails::Generators::Migration
+
+  source_root File.expand_path('../templates', __FILE__)
+  
+  argument     :model_name, :type => :string , :default => "user"
+  desc """
+This generator makes the following changes to your application:
+ 1. Creates several database migrations if they do not exist in /db/migrate
+ 2. Adds user behavior to the user model
+ 2. Adds routes
+       """ 
+
+  # Implement the required interface for Rails::Generators::Migration.
+  # taken from http://github.com/rails/rails/blob/master/activerecord/lib/generators/active_record.rb
+  def self.next_migration_number(path)
+    unless @prev_migration_nr
+      @prev_migration_nr = Time.now.utc.strftime("%Y%m%d%H%M%S").to_i
+    else
+      @prev_migration_nr += 1
+    end
+    @prev_migration_nr.to_s
+  end
+
+  # Setup the database migrations
+  def copy_migrations
+    # Can't get this any more DRY, because we need this order.
+    %w{user_institutions.rb}.each do |f|
+      better_migration_template f
+    end
+  end
+
+  # Add behaviors to the user model
+  def inject_user_roles_behavior
+    file_path = "app/models/#{model_name.underscore}.rb"
+    if File.exists?(file_path) 
+      code = "\n # Connects this user object to Institution-management behaviors. " +
+        "\n include Bpl::InstitutionManagement::UserInstitutions\n"
+      inject_into_file file_path, code, { :after => /include Hydra::User/ }
+    else
+      puts "     \e[31mFailure\e[0m  bpl-institution-management requires a user object. This generators assumes that the model is defined in the file #{file_path}, which does not exist.  If you used a different name, please re-run the generator and provide that name as an argument. Such as \b  rails -g roles client"
+    end    
+  end
+
+
+  # The engine routes have to come after the devise routes so that /users/sign_in will work
+  def inject_routes
+    routing_code = "mount Bpl::InstitutionManagement::Engine => '/'"
+    sentinel = /devise_for :users/
+    inject_into_file 'config/routes.rb', "\n  #{routing_code}\n", { :after => sentinel, :verbose => false }
+    
+  end
+
+  private  
+  
+  def better_migration_template (file)
+    begin
+      migration_template "migrations/#{file}", "db/migrate/#{file}"
+      sleep 1 # ensure scripts have different time stamps
+    rescue
+      puts "  \e[1m\e[34mMigrations\e[0m  " + $!.message
+    end
+  end
+
+end  
+
+

data/lib/generators/institutions/templates/migrations/user_institutions.rb

@@ -0,0 +1,19 @@
+class UserInstitutions < ActiveRecord::Migration
+  def up
+    create_table :institutions do |t|
+      t.string :name
+      t.string :pid
+    end
+    create_table :institutions_users, :id => false do |t|
+      t.references :institution
+      t.references :user
+    end
+    add_index :institutions_users, [:institution_id, :user_id]
+    add_index :institutions_users, [:user_id, :institution_id]
+  end
+
+  def down
+    drop_table :institutions_users
+    drop_table :institutions
+  end
+end

data/spec/.gitignore

@@ -0,0 +1 @@
+/internal

data/spec/controllers/institutions_controller_spec.rb

@@ -0,0 +1,117 @@
+require 'spec_helper'
+
+describe InstitutionController do
+  let(:ability) do
+    ability = Object.new
+    ability.extend(CanCan::Ability)
+    controller.stub(:current_ability).and_return(ability)
+    ability
+  end
+
+  let(:institution) do
+    Institution.create(name: 'foo')
+  end
+
+
+  describe "with a user who cannot edit institutions" do
+    it "should not be able to view institution index" do
+      lambda { get :index }.should raise_error CanCan::AccessDenied
+    end
+    it "should not be able to view institution" do
+      lambda { get :show, id: institution }.should raise_error CanCan::AccessDenied
+    end
+    it "should not be able to view new institution form" do
+      lambda { get :new }.should raise_error CanCan::AccessDenied
+    end
+    it "should not be able to create a institution" do
+      lambda { post :create, :institution=>{name: 'my_institution'}}.should raise_error CanCan::AccessDenied
+    end
+    it "should not be able to update a institution" do
+      lambda { put :update, id: institution}.should raise_error CanCan::AccessDenied
+    end
+    it "should not be able to remove a institution" do
+      lambda { delete :destroy, id: institution}.should raise_error CanCan::AccessDenied
+    end
+  end
+
+  describe "with a user who can read institutions" do
+    before do
+      ability.can :read, Institution
+    end
+    it "should be able to see the list of institutions" do
+      get :index
+      response.should be_successful
+      assigns[:institutions].should == [institution]
+    end
+
+    it "should be able to see a single institution" do
+      get :show, id: institution
+      response.should be_successful
+      assigns[:institution].should == institution
+    end
+  end
+
+  describe "with a user who can only update institution 'foo'" do
+    it "should be redirected to edit" do
+      ability.can :read, Institution
+      ability.can :update, Institution, id: institution.id
+      get :show, id: institution
+      response.should redirect_to @routes.url_helpers.edit_institution_path(assigns[:institution])
+    end
+  end
+  
+  describe "with a user who can create institutions" do
+    before do
+      ability.can :create, Institution
+    end
+    it "should be able to make a new institution" do
+      get :new
+      response.should be_successful
+      assigns[:institution].should be_kind_of Institution
+    end
+
+    it "should be able to create a new institution" do
+      post :create, :institution=>{name: 'my_institution'} 
+      response.should redirect_to @routes.url_helpers.edit_institution_path(assigns[:institution])
+      assigns[:institution].should_not be_new_record
+      assigns[:institution].name.should == 'my_institution'
+    end
+    it "should not create institution with an error" do
+      post :create, :institution=>{name: 'my institution'} 
+      assigns[:institution].name.should == 'my institution'
+      assigns[:institution].errors[:name].should == ['Only letters, numbers, hyphens, underscores and periods are allowed']
+      response.should be_successful
+    end
+  end
+
+  describe "with a user who can update institutions" do
+    before do
+      ability.can :update, Institution
+    end
+
+    it "should be able to update a institution" do
+      put :update, id: institution, :institution=>{name: 'my_institution'} 
+      response.should redirect_to @routes.url_helpers.edit_institution_path(assigns[:institution])
+      assigns[:institution].should_not be_new_record
+      assigns[:institution].name.should == 'my_institution'
+    end
+    it "should not update institution with an error" do
+      put :update,  id: institution, :institution=>{name: 'my institution'} 
+      assigns[:institution].name.should == 'my institution'
+      assigns[:institution].errors[:name].should == ['Only letters, numbers, hyphens, underscores and periods are allowed']
+      response.should be_successful
+    end
+  end
+
+  describe "with a user who can remove institutions" do
+    before do
+      ability.can :destroy, Institution
+    end
+
+    it "should be able to destroy a institution" do
+      delete :destroy, id: institution 
+      response.should redirect_to @routes.url_helpers.institutions_path
+    end
+  end
+
+end

data/spec/controllers/user_institutions_controller_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+describe UserInstitutionsController do
+  let(:ability) do
+    ability = Object.new
+    ability.extend(CanCan::Ability)
+    controller.stub(:current_ability).and_return(ability)
+    ability
+  end
+
+  let(:institution) do
+    Institution.create(name: 'foo')
+  end
+
+  describe "with a user who cannot edit users" do
+    it "should not be able to add a user" do
+      lambda { post :create, institution_id: institution, user_key: 'foo@example.com'}.should raise_error CanCan::AccessDenied
+    end
+    it "should not be able to remove a user" do
+      lambda { delete :destroy, institution_id: institution, id: 7}.should raise_error CanCan::AccessDenied
+    end
+  end
+
+  describe "with a user who can edit users" do
+    before do
+      ability.can :read, Institution
+    end
+    describe "adding users" do
+      before do
+        ability.can :add_user, Institution
+      end
+      it "should not be able to add a user that doesn't exist" do
+        User.should_receive(:find_by_user_key).with('foo@example.com').and_return(nil)
+        post :create, institution_id: institution, user_key: 'foo@example.com'
+        flash[:error].should == "Unable to find the user foo@example.com"
+      end
+      it "should be able to add a user" do
+        u = User.create!(email: 'foo@example.com', password: 'password', password_confirmation: 'password')
+        post :create, institution_id: institution, user_key: 'foo@example.com'
+        institution.reload.users.should == [u]
+      end
+    end
+    describe "removing users" do
+      before do
+        ability.can :remove_user, Institution
+      end
+      let (:user) do
+        u = User.new(email: 'foo@example.com', password: 'password', password_confirmation: 'password')
+        u.institutions = [institution]
+        u.save!
+        u
+      end
+      it "should be able to remove a user" do
+        user.institutions.should == [institution]
+        delete :destroy, institution_id: institution, id: user.id
+        institution.reload.users.should == []
+      end
+    end
+  end
+end
+

data/spec/lib/user_institutions_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe Bpl::InstitutionManagement::UserInstitutions do
+  subject do
+    User.create!(email: 'fred@example.com', password: 'password')
+  end
+
+  it "should have admin?" do
+    subject.should_not be_admin
+  end
+
+  it "should have institutions" do
+    subject.institutions.should == []
+    subject.institutions << Institution.create!(name: 'librarian')
+    subject.institutions.first.name.should == 'librarian'
+
+  end
+
+  it "should have groups" do
+    subject.institutions.should == []
+    subject.institutions << Institution.create!(name: 'librarian')
+    subject.save!
+    subject.groups.should include('registered', 'librarian')
+  end
+
+end

data/spec/models/institution_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Institution do
+  it "should require a name" do
+    subject.should_not be_valid
+    subject.name = 'foo'
+    subject.should be_valid
+  end
+
+  it "should not allow space in the name" do
+    subject.name = 'foo bar'
+    subject.should_not be_valid
+  end
+
+  it "should not allow comma in the name" do
+    subject.name = 'foo,bar'
+    subject.should_not be_valid
+  end
+
+  it "should not allow ampersand in the name" do
+    subject.name = 'foo&bar'
+    subject.should_not be_valid
+  end
+
+  it "should not allow less-than in the name" do
+    subject.name = 'foo<bar'
+    subject.should_not be_valid
+  end
+
+  it "should validate uniqueness" do
+    subject.name ='foo'
+    subject.save!
+    Institution.new(name: 'foo').should_not be_valid
+  end
+end

data/spec/routing/institution_management_routes_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+describe "Routes for institution_management" do
+  before(:each) do
+    @routes = Bpl::InstitutionManagement::Engine.routes
+    # so we have to do this instead:
+    # engine routes broke in rspec rails 2.12.1, so we had to add this:
+    assertion_instance.instance_variable_set(:@routes, @routes)
+  end
+  it "should route index" do 
+    { :get => '/institutions' }.should route_to( :controller => "institutions", :action => "index")
+  end
+  it "should create institutions" do 
+    { :post => '/institutions' }.should route_to( :controller => "institutions", :action => "create")
+  end
+  it "should show institutions" do 
+    { :get => '/institutions/7' }.should route_to( :controller => "institutions", :action => "show", :id => '7')
+  end
+  it "should add users" do 
+    { :post => '/institutions/7/users' }.should route_to( :controller => "user_institutions", :institution_id=>'7', :action => "create")
+  end
+  it "should remove users" do 
+    { :delete => '/institutions/7/users/5' }.should route_to( :controller => "user_institutions", :institution_id=>'7', :id=>'5', :action => "destroy")
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,14 @@
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+ENV["RAILS_ENV"] ||= 'test'
+
+require File.expand_path("config/environment", ENV['RAILS_ROOT'] || File.expand_path("../internal", __FILE__))
+require 'rspec/rails'
+
+
+RSpec.configure do |config|
+  config.use_transactional_fixtures = true
+  config.before(:each, :type=>"controller") { @routes = Bpl::InstitutionManagement::Engine.routes }
+  config.include Devise::TestHelpers, :type => :controller
+  
+end

data/spec/support/Gemfile

@@ -0,0 +1,28 @@
+source 'https://rubygems.org'
+
+gem 'rails'
+
+# Bundle edge Rails instead:
+# gem 'rails', :git => 'git://github.com/rails/rails.git'
+
+gem 'sqlite3'
+gem 'blacklight'
+gem 'hydra-head'
+
+
+# Gems used only for assets and not required
+# in production environments by default.
+group :assets do
+  gem 'sass-rails',   '~> 3.2.3'
+  gem 'coffee-rails', '~> 3.2.1'
+
+  # See https://github.com/sstephenson/execjs#readme for more supported runtimes
+  #gem 'therubyracer', :platforms => :ruby
+
+  gem 'uglifier', '>= 1.0.3'
+end
+
+gem 'jquery-rails'
+
+gem 'hydra-role-management', :path=>'../../'
+gem 'rspec-rails'

data/spec/support/app/models/sample.rb

@@ -0,0 +1,37 @@
+class Sample
+  # This is a stub model for testing.
+
+  cattr_accessor :objects
+  self.objects = {}
+
+  def self.create(params={})
+    obj = Sample.new
+    obj.save
+    obj
+  end
+
+  def save()
+    @pid ||= "sample:#{(rand * 1000).to_i}"
+    self.class.objects[@pid] = self
+  end
+
+  def update_attributes(attributes)
+    attributes.each do |k, v|
+      instance_variable_set "@#{k.to_s}".to_sym, v
+      
+      self.class.send :attr_accessor, k
+    end
+  end
+
+  def self.find(pid)
+    objects[pid]
+  end
+
+  def pid
+    @pid
+  end
+  
+  def destroy
+    self.class.objects.delete(@pid)
+  end
+end

data/spec/support/app/models/solr_document.rb

@@ -0,0 +1,5 @@
+# -*- encoding : utf-8 -*-
+class SolrDocument 
+
+  include Blacklight::Solr::Document
+end

data/spec/support/config/initializers/hydra_config.rb

@@ -0,0 +1,28 @@
+# The following lines determine which user attributes your hydrangea app will use
+# This configuration allows you to use the out of the box ActiveRecord associations between users and user_attributes
+# It also allows you to specify your own user attributes
+# The easiest way to override these methods would be to create your own module to include in User
+# For example you could create a module for your local LDAP instance called MyLocalLDAPUserAttributes:
+#   User.send(:include, MyLocalLDAPAttributes)
+# As long as your module includes methods for full_name, affiliation, and photo the personalization_helper should function correctly
+#
+
+# windows doesn't properly require hydra-head (from the gemfile), so we need to require it explicitly here:
+require 'hydra/head' unless defined? Hydra
+
+if Hydra.respond_to?(:configure)
+  Hydra.configure(:shared) do |config|
+    # This specifies the solr field names of permissions-related fields.
+    # You only need to change these values if you've indexed permissions by some means other than the Hydra's built-in tooling.
+    # If you change these, you must also update the permissions request handler in your solrconfig.xml to return those values
+    indexer = Solrizer::Descriptor.new(:string, :stored, :indexed, :multivalued)
+    config[:permissions] = {
+      :discover => {:group =>ActiveFedora::SolrService.solr_name("discover_access_group", indexer), :individual=>ActiveFedora::SolrService.solr_name("discover_access_person", indexer)},
+      :read => {:group =>ActiveFedora::SolrService.solr_name("read_access_group", indexer), :individual=>ActiveFedora::SolrService.solr_name("read_access_person", indexer)},
+      :edit => {:group =>ActiveFedora::SolrService.solr_name("edit_access_group", indexer), :individual=>ActiveFedora::SolrService.solr_name("edit_access_person", indexer)},
+      :owner => ActiveFedora::SolrService.solr_name("depositor", indexer),
+      :embargo_release_date => ActiveFedora::SolrService.solr_name("embargo_release_date", Solrizer::Descriptor.new(:date, :stored, :indexed))
+    }
+    
+  end
+end

data/spec/support/lib/generators/test_app_generator.rb

@@ -0,0 +1,47 @@
+require 'rails/generators'
+
+class TestAppGenerator < Rails::Generators::Base
+  source_root File.expand_path("../../../../support", __FILE__)
+
+  # # Inject call to Hydra::BatchEdit.add_routes in config/routes.rb
+  # def inject_routes
+  #   insert_into_file "config/routes.rb", :after => '.draw do' do
+  #     "\n  # Add BatchEdit routes."
+  #     "\n  Hydra::BatchEdit.add_routes(self)"
+  #   end
+  # end
+
+
+  def run_blacklight_generator
+    say_status("warning", "GENERATING BL", :yellow)       
+
+    generate 'blacklight', '--devise'
+  end
+
+  def run_hydra_head_generator
+    say_status("warning", "GENERATING HH", :yellow)       
+
+    generate 'hydra:head', '-f'
+  end
+
+  def run_roles_generator
+    say_status("warning", "GENERATING ROLES", :yellow)       
+
+    generate 'roles', '-f'
+
+  end
+
+  # def copy_test_models
+  #   copy_file "app/models/sample.rb"
+  #   copy_file "app/models/solr_document.rb"
+  #   copy_file "db/migrate/20111101221803_create_searches.rb"
+  # end
+
+  def copy_rspec_rake_task
+    copy_file "lib/tasks/rspec.rake"
+  end
+
+  def copy_hydra_config
+    copy_file "config/initializers/hydra_config.rb"
+  end
+end

data/spec/support/lib/tasks/rspec.rake

@@ -0,0 +1,9 @@
+require 'rspec/core/rake_task'
+desc "run the hydra-batch-edit gem spec"
+gem_home = File.expand_path('../../../../..', __FILE__)
+RSpec::Core::RakeTask.new(:myspec) do |t|
+  t.pattern = gem_home + '/spec/**/*_spec.rb'
+  t.rspec_opts = "--colour"
+  t.ruby_opts = "-I#{gem_home}/spec"
+end
+

metadata

@@ -0,0 +1,161 @@
+--- !ruby/object:Gem::Specification
+name: bpl-institution-management
+version: !ruby/object:Gem::Version
+  version: 0.0.2
+platform: ruby
+authors:
+- Steven Anderson
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2015-06-09 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: cancan
+  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_forms
+  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: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+description: Rails engine to do user institutions in an RDBMS for hydra-head
+email:
+- sanderson@bpl.org
+executables: []
+extensions: []
+extra_rdoc_files: []
+files:
+- .gitignore
+- Gemfile
+- README.md
+- Rakefile
+- app/controllers/concerns/bpl/institution_management/institutions_behavior.rb
+- app/controllers/concerns/bpl/institution_management/user_institutions_behavior.rb
+- app/controllers/institutions_controller.rb
+- app/controllers/user_institutions_controller.rb
+- app/models/concerns/bpl/institution_management/user_institutions.rb
+- app/models/institution.rb
+- app/views/institutions/edit.html.erb
+- app/views/institutions/index.html.erb
+- app/views/institutions/new.html.erb
+- app/views/institutions/show.html.erb
+- bpl-institution-management.gemspec
+- config/routes.rb
+- lib/bpl-institution-management.rb
+- lib/bpl/institution_management.rb
+- lib/bpl/institution_management/version.rb
+- lib/generators/institutions/institutions_generator.rb
+- lib/generators/institutions/templates/migrations/user_institutions.rb
+- spec/.gitignore
+- spec/controllers/institutions_controller_spec.rb
+- spec/controllers/user_institutions_controller_spec.rb
+- spec/lib/user_institutions_spec.rb
+- spec/models/institution_spec.rb
+- spec/routing/institution_management_routes_spec.rb
+- spec/spec_helper.rb
+- spec/support/Gemfile
+- spec/support/app/models/sample.rb
+- spec/support/app/models/solr_document.rb
+- spec/support/config/initializers/hydra_config.rb
+- spec/support/lib/generators/test_app_generator.rb
+- spec/support/lib/tasks/rspec.rake
+homepage: https://github.com/boston-library/bpl-institution-management
+licenses: []
+metadata: {}
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
+  - - '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements: []
+rubyforge_project: 
+rubygems_version: 2.0.3
+signing_key: 
+specification_version: 4
+summary: Rails engine to do user institutions in an RDBMS for hydra-head
+test_files:
+- spec/.gitignore
+- spec/controllers/institutions_controller_spec.rb
+- spec/controllers/user_institutions_controller_spec.rb
+- spec/lib/user_institutions_spec.rb
+- spec/models/institution_spec.rb
+- spec/routing/institution_management_routes_spec.rb
+- spec/spec_helper.rb
+- spec/support/Gemfile
+- spec/support/app/models/sample.rb
+- spec/support/app/models/solr_document.rb
+- spec/support/config/initializers/hydra_config.rb
+- spec/support/lib/generators/test_app_generator.rb
+- spec/support/lib/tasks/rspec.rake
+has_rdoc: