link to holding location and sort based on code

.gitignore

@@ -2,7 +2,7 @@
 log/*.log
 pkg/
 Gemfile.lock
-spec/internal/
+.internal_test_app/
 
 # Ignore coverage
 coverage/

Gemfile

@@ -5,17 +5,28 @@ source 'https://rubygems.org'
 # development dependencies will be added by default to the :development group.
 gemspec
 
-file = File.expand_path("Gemfile", ENV['ENGINE_CART_DESTINATION'] || ENV['RAILS_ROOT'] || File.expand_path("../spec/internal", __FILE__))
-if File.exists?(file)
-  puts "Loading #{file} ..." if $DEBUG # `ruby -d` or `bundle -v`
-  instance_eval File.read(file)
+# BEGIN ENGINE_CART BLOCK
+# engine_cart: 0.8.0
+# engine_cart stanza: 0.8.0
+# the below comes from engine_cart, a gem used to test this Rails engine gem in the context of a Rails app.
+file = File.expand_path("Gemfile", ENV['ENGINE_CART_DESTINATION'] || ENV['RAILS_ROOT'] || File.expand_path(".internal_test_app", File.dirname(__FILE__)))
+if File.exist?(file)
+  begin
+    eval_gemfile file
+  rescue Bundler::GemfileError => e
+    Bundler.ui.warn '[EngineCart] Skipping Rails application dependencies:'
+    Bundler.ui.warn e.message
+  end
 else
+  Bundler.ui.warn "[EngineCart] Unable to find test application dependencies in #{file}, using placeholder dependencies"
+
   gem 'rails', ENV['RAILS_VERSION'] if ENV['RAILS_VERSION']
 
-  if ENV['RAILS_VERSION'] and ENV['RAILS_VERSION'] =~ /^4.2/
+  if ENV['RAILS_VERSION'].nil? || ENV['RAILS_VERSION'] =~ /^4.2/
     gem 'responders', "~> 2.0"
     gem 'sass-rails', ">= 5.0"
   else
     gem 'sass-rails', "< 5.0"
   end
 end
+# END ENGINE_CART BLOCK

app/controllers/locations/holding_locations_controller.rb

@@ -6,7 +6,7 @@ class HoldingLocationsController < ApplicationController
 
     # GET /holding_locations
     def index
-      @holding_locations = HoldingLocation.all
+      @holding_locations = HoldingLocation.order(code: :asc)
     end
 
     # GET /holding_locations/1

app/views/locations/holding_locations/index.html.erb

@@ -3,8 +3,8 @@
 <table class="table">
   <thead>
     <tr>
-      <th>Label</th>
       <th>Code</th>
+      <th>Label</th>
       <th>Library</th>
       <th>Hours Location</th>
       <th>Aeon Location</th>
@@ -19,8 +19,8 @@
   <tbody>
     <% @holding_locations.each do |holding_location| %>
       <tr>
-        <td><%= link_to holding_location.label, holding_location %></td>
-        <td><%= holding_location.code %></td>
+        <td><%= link_to holding_location.code, holding_location %></td>
+        <td><%= holding_location.label %></td>
         <td><%= link_to holding_location.library.code, holding_location.library %></td>
         <td><%= hours_render holding_location %></td>
         <td><%= holding_location.aeon_location %></td>

lib/locations/version.rb

@@ -1,3 +1,3 @@
 module Locations
-  VERSION = "0.1.1"
+  VERSION = "0.1.2"
 end

spec/features/locations/location_views_associated_links_spec.rb

@@ -22,7 +22,7 @@
   scenario 'Link to holding location from holding location label rather than extra show link' do
     holding_location
     visit holding_locations_path
-    click_link holding_location.label
+    click_link holding_location.code
   end
 
   scenario 'Link to delivery location from delivery location label rather than extra show link' do

spec/requests/locations/holding_locations_json_view_spec.rb

@@ -14,7 +14,7 @@ module Locations
       it "/holding_locations looks as we'd expect" do
         2.times { FactoryGirl.create(:holding_location) }
         expected = []
-        HoldingLocation.all.each do |holding_location|
+        HoldingLocation.order(code: :asc).each do |holding_location|
           attrs = {
             label: holding_location.label,
             code: holding_location.code,

spec/spec_helper.rb

@@ -1,6 +1,6 @@
 ENV['RAILS_ENV'] ||= 'test'
 
-require File.expand_path('../../spec/internal/config/environment', __FILE__)
+require File.expand_path('../../.internal_test_app/config/environment', __FILE__)
 require 'factory_girl_rails'
 require 'rspec/rails'
 require 'engine_cart'