enju_inter_library_loan 0.1.1 → 0.2.0.beta.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d2bcac6011293de59a9d4a656971187377ed76ff
-  data.tar.gz: e6650ad9133144185ad4b185a212a32156498df5
+  metadata.gz: 898e7475ac7f7f654e572612a849329e6acc4a61
+  data.tar.gz: e3143faec0387d62e08bc875bdc8daa9259c3b0b
 SHA512:
-  metadata.gz: 7d9828182b90a77de5bd233b7d53f302730e6807be51edc72dcb1a2c7fca0561593482100f09c6c2795c2b395feaa25de55fec94a155c008650803bab744de6e
-  data.tar.gz: d8e17a2f85c9cebe9ed9d42ff6bdee8c14ca15cbf6578e8ac71b99f5fd3e428575d9bb7a8122ac029238a5c932108e3390739c09381b7cf5968188edfdf5e7b0
+  metadata.gz: 4287306f7f701305a6d95bf5553dff358156ee2c9eeba20da315fde38c0bd061a99da73778485d4a8536556a490fc0ddee0bd32a12e8511add1b8d2d2738a7a8
+  data.tar.gz: c53a4a69eb02d18ae1f9a1ac5fcc4b362f0378887b776c3788bc2aede93840ad55eef74901b91f308c36454d19b6e025780f061ee2cee3be38c83dbe1cbb546d

data/README.rdoc

@@ -1,7 +1,7 @@
 = EnjuInterLibraryLoan
-{<img src="https://secure.travis-ci.org/next-l/enju_inter_library_loan.svg?branch=1.1" alt="Build Status" />}[http://travis-ci.org/next-l/enju_inter_library_loan]
-{<img src="https://coveralls.io/repos/next-l/enju_inter_library_loan/badge.svg?branch=1.1&service=github" alt="Coverage Status" />}[https://coveralls.io/github/next-l/enju_inter_library_loan?branch=1.1]
-{<img src="https://hakiri.io/github/next-l/enju_inter_library_loan/1.1.svg" alt="security" />}[https://hakiri.io/github/next-l/enju_inter_library_loan/1.1]
+{<img src="https://travis-ci.org/next-l/enju_inter_library_loan.svg?branch=1.2" alt="Build Status" />}[https://travis-ci.org/next-l/enju_inter_library_loan]
+{<img src="https://coveralls.io/repos/next-l/enju_inter_library_loan/badge.svg?branch=1.2&service=github" alt="Coverage Status" />}[https://coveralls.io/github/next-l/enju_inter_library_loan?branch=1.2]
+{<img src="https://hakiri.io/github/next-l/enju_inter_library_loan/1.2.svg" alt="security" />}[https://hakiri.io/github/next-l/enju_inter_library_loan/1.2]
 
 This project rocks and uses MIT-LICENSE.
 

data/app/controllers/inter_library_loans_controller.rb

@@ -1,7 +1,8 @@
 class InterLibraryLoansController < ApplicationController
-  load_and_authorize_resource
-  before_filter :get_item
-  before_filter :store_page, :only => :index
+  before_action :store_page, only: :index
+  before_action :set_inter_library_loan, only: [:show, :edit, :update, :destroy]
+  before_action :check_policy, only: [:index, :new, :create]
+  before_action :get_item
 
   # GET /inter_library_loans
   # GET /inter_library_loans.json
@@ -120,6 +121,15 @@ class InterLibraryLoansController < ApplicationController
   end
 
   private
+  def set_inter_library_loan
+    @inter_library_loan = InterLibraryLoan.find(params[:id])
+    authorize @inter_library_loan
+  end
+
+  def check_policy
+    authorize InterLibraryLoan
+  end
+
   def inter_library_loan_params
     params.require(:inter_library_loan).permit(:item_id, :borrowing_library_id)
   end

data/app/models/inter_library_loan_transition.rb

@@ -17,4 +17,5 @@ end
 #  inter_library_loan_id :integer
 #  created_at            :datetime
 #  updated_at            :datetime
+#  most_recent           :boolean
 #

data/app/policies/inter_library_loan_policy.rb

@@ -0,0 +1,21 @@
+class InterLibraryLoanPolicy < ApplicationPolicy
+  def index?
+    user.try(:has_role?, 'Librarian')
+  end
+
+  def show?
+    user.try(:has_role?, 'Librarian')
+  end
+
+  def create?
+    user.try(:has_role?, 'Librarian')
+  end
+
+  def update?
+    user.try(:has_role?, 'Librarian')
+  end
+
+  def destroy?
+    user.try(:has_role?, 'Librarian')
+  end
+end

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

@@ -26,13 +26,17 @@
       <%= l(inter_library_loan.shipped_at) if inter_library_loan.shipped_at -%><br />
       <%= l(inter_library_loan.received_at) if inter_library_loan.received_at -%>
     </td>
-    <%- if can? :destroy, inter_library_loan -%>
-      <td>
+    <td>
+      <%- if policy(inter_library_loan).show? -%>
         <%= link_to t('page.show'), inter_library_loan -%>
+      <%- end -%>
+      <%- if policy(inter_library_loan).update? -%>
         <%= link_to t('page.edit'), edit_inter_library_loan_path(inter_library_loan) -%>
+      <%- end -%>
+      <%- if policy(inter_library_loan).destroy? -%>
         <%= link_to t('page.destroy'), inter_library_loan, :data => {:confirm => t('page.are_you_sure')}, :method => :delete -%>
-      </td>
-    <%- end -%>
+      <%- end -%>
+    </td>
   </tr>
 <%- end -%>
 </table>
@@ -43,7 +47,7 @@
 
 <div id="submenu" class="ui-corner-all ui-widget-content">
   <ul>
-    <%- if can? :create, InterLibraryLoan -%>
+    <%- if policy(InterLibraryLoan).create? -%>
       <li><%= link_to t('page.new', :model => t('activerecord.models.inter_library_loan')), new_inter_library_loan_path -%></li>
     <%- end -%>
   </ul>

data/db/migrate/20160703190932_add_most_recent_to_inter_library_loan_transitions.rb

@@ -0,0 +1,9 @@
+class AddMostRecentToInterLibraryLoanTransitions < ActiveRecord::Migration
+  def up
+    add_column :inter_library_loan_transitions, :most_recent, :boolean, null: true
+  end
+
+  def down
+    remove_column :inter_library_loan_transitions, :most_recent
+  end
+end

data/lib/enju_inter_library_loan/engine.rb

@@ -1,4 +1,4 @@
-require 'enju_leaf'
+require 'enju_circulation'
 
 module EnjuInterLibraryLoan
   class Engine < ::Rails::Engine

data/lib/enju_inter_library_loan/version.rb

@@ -1,3 +1,3 @@
 module EnjuInterLibraryLoan
-  VERSION = "0.1.1"
+  VERSION = "0.2.0.beta.2"
 end

data/spec/controllers/inter_library_loans_controller_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require 'rails_helper'
 require 'sunspot/rails/spec_helper'
 
 describe InterLibraryLoansController do
@@ -35,14 +35,14 @@ describe InterLibraryLoansController do
 
       it "assigns empty as @inter_library_loans" do
         get :index
-        assigns(:inter_library_loans).should be_empty
+        assigns(:inter_library_loans).should be_nil
       end
     end
 
     describe "When not logged in" do
       it "assigns empty as @inter_library_loans" do
         get :index
-        assigns(:inter_library_loans).should be_empty
+        assigns(:inter_library_loans).should be_nil
       end
     end
   end
@@ -123,7 +123,7 @@ describe InterLibraryLoansController do
 
       it "should not assign the requested inter_library_loan as @inter_library_loan" do
         get :new
-        assigns(:inter_library_loan).should_not be_valid
+        assigns(:inter_library_loan).should be_nil
         response.should be_forbidden
       end
     end
@@ -131,7 +131,7 @@ describe InterLibraryLoansController do
     describe "When not logged in" do
       it "should not assign the requested inter_library_loan as @inter_library_loan" do
         get :new
-        assigns(:inter_library_loan).should_not be_valid
+        assigns(:inter_library_loan).should be_nil
         response.should redirect_to(new_user_session_url)
       end
     end

data/spec/dummy/app/controllers/application_controller.rb

@@ -1,7 +1,8 @@
 class ApplicationController < ActionController::Base
   protect_from_forgery
+  include EnjuLibrary::Controller
+  include EnjuBiblio::Controller
+  after_action :verify_authorized
 
-  enju_leaf
-  enju_biblio
-  enju_library
+  include Pundit
 end

data/spec/dummy/app/models/user.rb

@@ -6,7 +6,7 @@ class User < ActiveRecord::Base
 
   # Setup accessible (or protected) attributes for your model
 
-  enju_leaf_user_model
-  enju_circulation_user_model
-  enju_message_user_model
+  include EnjuSeed::EnjuUser
+  include EnjuMessage::EnjuUser
+  include EnjuCirculation::EnjuUser
 end

data/spec/dummy/config/application.rb

@@ -2,8 +2,12 @@ require File.expand_path('../boot', __FILE__)
 
 require 'rails/all'
 
-Bundler.require
-require "enju_inter_library_loan"
+Bundler.require(*Rails.groups)
+require 'enju_inter_library_loan'
+require 'enju_leaf'
+require 'enju_circulation'
+require 'globalize'
+require 'globalize-accessors'
 
 module Dummy
   class Application < Rails::Application
@@ -11,16 +15,6 @@ module Dummy
     # Application configuration should go into files in config/initializers
     # -- all .rb files in that directory are automatically loaded.
 
-    # Custom directories with classes and modules you want to be autoloadable.
-    # config.autoload_paths += %W(#{config.root}/extras)
-
-    # Only load the plugins named here, in the order given (default is alphabetical).
-    # :all can be used as a placeholder for all plugins not explicitly named.
-    # config.plugins = [ :exception_notification, :ssl_requirement, :all ]
-
-    # Activate observers that should always be running.
-    # config.active_record.observers = :cacher, :garbage_collector, :forum_observer
-
     # Set Time.zone default to the specified zone and make Active Record auto-convert to this zone.
     # Run "rake -D time" for a list of tasks for finding time zone names. Default is UTC.
     # config.time_zone = 'Central Time (US & Canada)'
@@ -29,19 +23,8 @@ module Dummy
     # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s]
     # config.i18n.default_locale = :de
 
-    # Configure the default encoding used in templates for Ruby 1.9.
-    config.encoding = "utf-8"
-
-    # Configure sensitive parameters which will be filtered from the log file.
-    config.filter_parameters += [:password]
-
-    # Enable the asset pipeline
-    config.assets.enabled = true
-
-    # Version of your assets, change this if you want to expire all your assets
-    config.assets.version = '1.0'
+    # Do not swallow errors in after_commit/after_rollback callbacks.
+    config.active_record.raise_in_transactional_callbacks = true
   end
 end
 
-require 'enju_leaf'
-require 'enju_circulation'

data/spec/dummy/config/environments/development.rb

@@ -1,30 +1,41 @@
-Dummy::Application.configure do
-  # Settings specified here will take precedence over those in config/application.rb
+Rails.application.configure do
+  # Settings specified here will take precedence over those in config/application.rb.
 
   # In the development environment your application's code is reloaded on
-  # every request.  This slows down response time but is perfect for development
+  # every request. This slows down response time but is perfect for development
   # since you don't have to restart the web server when you make code changes.
   config.cache_classes = false
 
-  # Log error messages when you accidentally call methods on nil.
-  config.whiny_nils = true
+  # Do not eager load code on boot.
+  config.eager_load = false
 
-  # Show full error reports and disable caching
+  # Show full error reports and disable caching.
   config.consider_all_requests_local       = true
   config.action_controller.perform_caching = false
 
-  # Don't care if the mailer can't send
+  # Don't care if the mailer can't send.
   config.action_mailer.raise_delivery_errors = false
 
-  # Print deprecation notices to the Rails logger
+  # Print deprecation notices to the Rails logger.
   config.active_support.deprecation = :log
 
-  # Only use best-standards-support built into browsers
-  config.action_dispatch.best_standards_support = :builtin
+  # Raise an error on page load if there are pending migrations.
+  config.active_record.migration_error = :page_load
 
-  # Do not compress assets
-  config.assets.compress = false
-
-  # Expands the lines which load the assets
+  # Debug mode disables concatenation and preprocessing of assets.
+  # This option may cause significant delays in view rendering with a large
+  # number of complex assets.
   config.assets.debug = true
+
+  # Asset digests allow you to set far-future HTTP expiration dates on all assets,
+  # yet still be able to expire them through the digest params.
+  config.assets.digest = true
+
+  # Adds additional error checking when serving assets at runtime.
+  # Checks for improperly declared sprockets dependencies.
+  # Raises helpful error messages.
+  config.assets.raise_runtime_errors = true
+
+  # Raises error for missing translations
+  # config.action_view.raise_on_missing_translations = true
 end

data/spec/dummy/config/environments/production.rb

@@ -1,60 +1,79 @@
-Dummy::Application.configure do
-  # Settings specified here will take precedence over those in config/application.rb
+Rails.application.configure do
+  # Settings specified here will take precedence over those in config/application.rb.
 
-  # Code is not reloaded between requests
+  # Code is not reloaded between requests.
   config.cache_classes = true
 
-  # Full error reports are disabled and caching is turned on
+  # Eager load code on boot. This eager loads most of Rails and
+  # your application in memory, allowing both threaded web servers
+  # and those relying on copy on write to perform better.
+  # Rake tasks automatically ignore this option for performance.
+  config.eager_load = true
+
+  # Full error reports are disabled and caching is turned on.
   config.consider_all_requests_local       = false
   config.action_controller.perform_caching = true
 
-  # Disable Rails's static asset server (Apache or nginx will already do this)
-  config.serve_static_assets = false
+  # Enable Rack::Cache to put a simple HTTP cache in front of your application
+  # Add `rack-cache` to your Gemfile before enabling this.
+  # For large-scale production use, consider using a caching reverse proxy like
+  # NGINX, varnish or squid.
+  # config.action_dispatch.rack_cache = true
+
+  # Disable serving static files from the `/public` folder by default since
+  # Apache or NGINX already handles this.
+  config.serve_static_files = ENV['RAILS_SERVE_STATIC_FILES'].present?
 
-  # Compress JavaScripts and CSS
-  config.assets.compress = true
+  # Compress JavaScripts and CSS.
+  config.assets.js_compressor = :uglifier
+  # config.assets.css_compressor = :sass
 
-  # Don't fallback to assets pipeline if a precompiled asset is missed
+  # Do not fallback to assets pipeline if a precompiled asset is missed.
   config.assets.compile = false
 
-  # Generate digests for assets URLs
+  # Asset digests allow you to set far-future HTTP expiration dates on all assets,
+  # yet still be able to expire them through the digest params.
   config.assets.digest = true
 
-  # Defaults to Rails.root.join("public/assets")
-  # config.assets.manifest = YOUR_PATH
+  # `config.assets.precompile` and `config.assets.version` have moved to config/initializers/assets.rb
 
-  # Specifies the header that your server uses for sending files
-  # config.action_dispatch.x_sendfile_header = "X-Sendfile" # for apache
-  # config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for nginx
+  # Specifies the header that your server uses for sending files.
+  # config.action_dispatch.x_sendfile_header = 'X-Sendfile' # for Apache
+  # config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for NGINX
 
   # Force all access to the app over SSL, use Strict-Transport-Security, and use secure cookies.
   # config.force_ssl = true
 
-  # See everything in the log (default is :info)
-  # config.log_level = :debug
+  # Use the lowest log level to ensure availability of diagnostic information
+  # when problems arise.
+  config.log_level = :debug
 
-  # Use a different logger for distributed setups
-  # config.logger = SyslogLogger.new
+  # Prepend all log lines with the following tags.
+  # config.log_tags = [ :subdomain, :uuid ]
 
-  # Use a different cache store in production
-  # config.cache_store = :mem_cache_store
+  # Use a different logger for distributed setups.
+  # config.logger = ActiveSupport::TaggedLogging.new(SyslogLogger.new)
 
-  # Enable serving of images, stylesheets, and JavaScripts from an asset server
-  # config.action_controller.asset_host = "http://assets.example.com"
+  # Use a different cache store in production.
+  # config.cache_store = :mem_cache_store
 
-  # Precompile additional assets (application.js, application.css, and all non-JS/CSS are already added)
-  # config.assets.precompile += %w( search.js )
+  # Enable serving of images, stylesheets, and JavaScripts from an asset server.
+  # config.action_controller.asset_host = 'http://assets.example.com'
 
-  # Disable delivery errors, bad email addresses will be ignored
+  # Ignore bad email addresses and do not raise email delivery errors.
+  # Set this to true and configure the email server for immediate delivery to raise delivery errors.
   # config.action_mailer.raise_delivery_errors = false
 
-  # Enable threaded mode
-  # config.threadsafe!
-
   # Enable locale fallbacks for I18n (makes lookups for any locale fall back to
-  # the I18n.default_locale when a translation can not be found)
+  # the I18n.default_locale when a translation cannot be found).
   config.i18n.fallbacks = true
 
-  # Send deprecation notices to registered listeners
+  # Send deprecation notices to registered listeners.
   config.active_support.deprecation = :notify
+
+  # Use default logging formatter so that PID and timestamp are not suppressed.
+  config.log_formatter = ::Logger::Formatter.new
+
+  # Do not dump schema after migrations.
+  config.active_record.dump_schema_after_migration = false
 end

data/spec/dummy/config/environments/test.rb

@@ -1,39 +1,44 @@
-Dummy::Application.configure do
-  # Settings specified here will take precedence over those in config/application.rb
+Rails.application.configure do
+  # Settings specified here will take precedence over those in config/application.rb.
 
   # The test environment is used exclusively to run your application's
-  # test suite.  You never need to work with it otherwise.  Remember that
+  # test suite. You never need to work with it otherwise. Remember that
   # your test database is "scratch space" for the test suite and is wiped
-  # and recreated between test runs.  Don't rely on the data there!
+  # and recreated between test runs. Don't rely on the data there!
   config.cache_classes = true
 
-  # Configure static asset server for tests with Cache-Control for performance
-  config.serve_static_assets = true
-  config.static_cache_control = "public, max-age=3600"
+  # Do not eager load code on boot. This avoids loading your whole application
+  # just for the purpose of running a single test. If you are using a tool that
+  # preloads Rails for running tests, you may have to set it to true.
+  config.eager_load = false
 
-  # Log error messages when you accidentally call methods on nil
-  config.whiny_nils = true
+  # Configure static file server for tests with Cache-Control for performance.
+  config.serve_static_files   = true
+  config.static_cache_control = 'public, max-age=3600'
 
-  # Show full error reports and disable caching
+  # Show full error reports and disable caching.
   config.consider_all_requests_local       = true
   config.action_controller.perform_caching = false
 
-  # Raise exceptions instead of rendering exception templates
+  # Raise exceptions instead of rendering exception templates.
   config.action_dispatch.show_exceptions = false
 
-  # Disable request forgery protection in test environment
-  config.action_controller.allow_forgery_protection    = false
+  # Disable request forgery protection in test environment.
+  config.action_controller.allow_forgery_protection = false
 
   # Tell Action Mailer not to deliver emails to the real world.
   # The :test delivery method accumulates sent emails in the
   # ActionMailer::Base.deliveries array.
   config.action_mailer.delivery_method = :test
 
-  # Use SQL instead of Active Record's schema dumper when creating the test database.
-  # This is necessary if your schema can't be completely dumped by the schema dumper,
-  # like if you have constraints or database-specific column types
-  # config.active_record.schema_format = :sql
+  # Randomize the order test cases are executed.
+  config.active_support.test_order = :random
 
-  # Print deprecation notices to the stderr
+  # Print deprecation notices to the stderr.
   config.active_support.deprecation = :stderr
+
+  # Raises error for missing translations
+  # config.action_view.raise_on_missing_translations = true
+
+  config.action_mailer.default_url_options = {:host => 'localhost:3000'}
 end

data/spec/dummy/db/migrate/149_create_message_templates.rb

@@ -0,0 +1,18 @@
+class CreateMessageTemplates < ActiveRecord::Migration
+  def self.up
+    create_table :message_templates do |t|
+      t.string :status, :null => false
+      t.text :title, :null => false
+      t.text :body, :null => false
+      t.integer :position
+      t.string :locale, :default => I18n.default_locale.to_s
+
+      t.timestamps
+    end
+    add_index :message_templates, :status, :unique => true
+  end
+
+  def self.down
+    drop_table :message_templates
+  end
+end

data/spec/dummy/db/migrate/154_create_messages.rb

@@ -0,0 +1,23 @@
+class CreateMessages < ActiveRecord::Migration
+  def self.up
+    create_table :messages, :force => true do |t|
+      t.datetime :read_at
+      t.integer  :receiver_id, :sender_id
+      t.string   :subject, :null => false
+      t.text     :body
+      t.integer :message_request_id
+      t.integer :parent_id
+
+      t.timestamps
+    end
+
+    add_index :messages, :sender_id
+    add_index :messages, :receiver_id
+    add_index :messages, :message_request_id
+    add_index :messages, :parent_id
+  end
+
+  def self.down
+    drop_table :messages
+  end
+end

data/spec/dummy/db/migrate/20080819181903_create_message_requests.rb

@@ -0,0 +1,18 @@
+class CreateMessageRequests < ActiveRecord::Migration
+  def self.up
+    create_table :message_requests do |t|
+      t.integer :sender_id
+      t.integer :receiver_id
+      t.integer :message_template_id
+      t.datetime :sent_at
+      t.datetime :deleted_at
+      t.text :body
+
+      t.timestamps
+    end
+  end
+
+  def self.down
+    drop_table :message_requests
+  end
+end

data/spec/dummy/db/migrate/20110913115320_add_lft_and_rgt_to_message.rb

@@ -0,0 +1,11 @@
+class AddLftAndRgtToMessage < ActiveRecord::Migration
+  def self.up
+    add_column :messages, :lft, :integer
+    add_column :messages, :rgt, :integer
+  end
+
+  def self.down
+    remove_column :messages, :rgt
+    remove_column :messages, :lft
+  end
+end

data/spec/dummy/db/migrate/20120125050502_add_depth_to_message.rb

@@ -0,0 +1,6 @@
+class AddDepthToMessage < ActiveRecord::Migration
+  def change
+    add_column :messages, :depth, :integer
+
+  end
+end

data/spec/dummy/db/migrate/20140110122216_create_user_import_files.rb

@@ -6,7 +6,7 @@ class CreateUserImportFiles < ActiveRecord::Migration
       t.datetime :executed_at
       t.string :user_import_file_name
       t.string :user_import_content_type
-      t.string :user_import_file_size
+      t.integer :user_import_file_size
       t.datetime :user_import_updated_at
       t.string :user_import_fingerprint
       t.string :edit_mode

data/spec/dummy/db/migrate/20140518111006_create_message_transitions.rb

@@ -0,0 +1,18 @@
+class CreateMessageTransitions < ActiveRecord::Migration
+  def change
+    create_table :message_transitions do |t|
+      t.string :to_state
+      if ActiveRecord::Base.configurations[Rails.env]["adapter"].try(:match, /mysql/)
+        t.text :metadata
+      else
+        t.text :metadata, default: "{}"
+      end
+      t.integer :sort_key
+      t.integer :message_id
+      t.timestamps
+    end
+
+    add_index :message_transitions, :message_id
+    add_index :message_transitions, [:sort_key, :message_id], unique: true
+  end
+end

data/spec/dummy/db/migrate/20140518135713_create_message_request_transitions.rb

@@ -0,0 +1,18 @@
+class CreateMessageRequestTransitions < ActiveRecord::Migration
+  def change
+    create_table :message_request_transitions do |t|
+      t.string :to_state
+      if ActiveRecord::Base.configurations[Rails.env]["adapter"].try(:match, /mysql/)
+        t.text :metadata
+      else
+        t.text :metadata, default: "{}"
+      end
+      t.integer :sort_key
+      t.integer :message_request_id
+      t.timestamps
+    end
+
+    add_index :message_request_transitions, :message_request_id
+    add_index :message_request_transitions, [:sort_key, :message_request_id], unique: true, name: "index_message_request_transitions_on_sort_key_and_request_id"
+  end
+end

data/spec/dummy/db/migrate/20140821151023_create_colors.rb

@@ -0,0 +1,14 @@
+class CreateColors < ActiveRecord::Migration
+  def change
+    create_table :colors do |t|
+      t.integer :library_group_id
+      t.string :property
+      t.string :code
+      t.integer :position
+
+      t.timestamps
+    end
+
+    add_index :colors, :library_group_id
+  end
+end

data/spec/dummy/db/migrate/20150124152756_add_foreign_key_to_shelves_referencing_libraries.rb

@@ -0,0 +1,6 @@
+class AddForeignKeyToShelvesReferencingLibraries < ActiveRecord::Migration
+  def change
+    add_foreign_key :shelves, :libraries
+    add_foreign_key :items, :shelves
+  end
+end