Merge pull request #63 from af83/4189-duplicate-route

4189 duplicate route

6 files changed, 133 insertions, 20 deletions

diff --git a/spec/controllers/routes_controller_spec.rb b/spec/controllers/routes_controller_spec.rb
index 000b799db..336f20945 100644
--- a/spec/controllers/routes_controller_spec.rb
+++ b/spec/controllers/routes_controller_spec.rb
@@ -1,7 +1,9 @@
-RSpec.describe RoutesController, :type => :controller do
+Route = Chouette::Route
+
+RSpec.describe RoutesController, type: :controller do
   login_user
 
-  let!(:route) { create(:route) }
+  let(:route) { create(:route) }
 
   it { is_expected.to be_kind_of(ChouetteController) }
 
@@ -10,6 +12,7 @@ RSpec.describe RoutesController, :type => :controller do
       # expect(response).to redirect_to( referential_line_path(referential,route.line) )
     end
   end
+
   shared_examples_for "line and referential linked" do
     it "assigns route.line as @line" do
       expect(assigns[:line]).to eq(route.line)
@@ -19,6 +22,7 @@ RSpec.describe RoutesController, :type => :controller do
       expect(assigns[:referential]).to eq(referential)
     end
   end
+
   shared_examples_for "route, line and referential linked" do
     it "assigns route as @route" do
       expect(assigns[:route]).to eq(route)
@@ -28,8 +32,8 @@ RSpec.describe RoutesController, :type => :controller do
 
   describe "GET /index" do
     before(:each) do
-      get :index, :line_id => route.line_id,
-          :referential_id => referential.id
+      get :index, line_id: route.line_id,
+          referential_id: referential.id
     end
 
     it_behaves_like "line and referential linked"
@@ -38,9 +42,9 @@ RSpec.describe RoutesController, :type => :controller do
 
   describe "POST /create" do
     before(:each) do
-      post :create, :line_id => route.line_id,
-          :referential_id => referential.id,
-          :route => { :name => "changed"}
+      post :create, line_id: route.line_id,
+          referential_id: referential.id,
+          route: { name: "changed"}
 
     end
     it_behaves_like "line and referential linked"
@@ -49,9 +53,9 @@ RSpec.describe RoutesController, :type => :controller do
 
   describe "PUT /update" do
     before(:each) do
-      put :update, :id => route.id, :line_id => route.line_id,
-          :referential_id => referential.id,
-          :route => route.attributes
+      put :update, id: route.id, line_id: route.line_id,
+          referential_id: referential.id,
+          route: route.attributes
     end
 
     it_behaves_like "route, line and referential linked"
@@ -60,9 +64,9 @@ RSpec.describe RoutesController, :type => :controller do
 
   describe "GET /show" do
     before(:each) do
-      get :show, :id => route.id,
-          :line_id => route.line_id,
-          :referential_id => referential.id
+      get :show, id: route.id,
+          line_id: route.line_id,
+          referential_id: referential.id
     end
 
     it_behaves_like "route, line and referential linked"
@@ -71,11 +75,22 @@ RSpec.describe RoutesController, :type => :controller do
       expect(assigns[:map]).to be_an_instance_of(RouteMap)
       expect(assigns[:map].route).to eq(route)
     end
-
-    #it "assigns route.stop_points.paginate(:page => nil) as @stop_points" do
-    #  expect(assigns[:stop_points]).to eq(route.stop_points.paginate(:page => nil))
-    #end
   end
 
-end
 
+  describe "POST /duplicate" do
+    let!( :route_prime ){ route }
+
+    it "creates a new route" do
+      expect do
+        post :duplicate,
+          referential_id: route.line.line_referential_id,
+          line_id: route.line_id,
+          id: route.id
+      end.to change { Route.count }.by(1)
+
+      expect(Route.last.name).to eq(route.name)
+      expect(Route.last.published_name).to eq(route.published_name)
+    end
+  end
+end
diff --git a/spec/models/chouette/route/route_duplication_spec.rb b/spec/models/chouette/route/route_duplication_spec.rb
new file mode 100644
index 000000000..6645b909f
--- /dev/null
+++ b/spec/models/chouette/route/route_duplication_spec.rb
@@ -0,0 +1,52 @@
+# From Chouette import what we need ™
+Route     = Chouette::Route
+StopArea  = Chouette::StopArea
+StopPoint = Chouette::StopPoint
+
+RSpec.describe Route do
+
+  let!( :route ){ create :route }
+
+  context '#duplicate' do 
+    describe 'properties' do
+      it 'same attribute values' do
+        route.duplicate
+        expect( values_for_create(Route.last, except: %w{objectid}) ).to eq( values_for_create( route, except: %w{objectid} ) )
+      end
+      it 'and others cannot' do
+        expect{ route.duplicate name: 'YAN', line_id: 42  }.to raise_error(ArgumentError)
+      end
+      it 'same associated stop_areeas' do
+        expect( route.duplicate.stop_areas.pluck(:id) ).to eq(route.stop_areas.pluck(:id))
+      end
+    end
+
+    describe 'side_effects' do
+      it {
+        expect{ route.duplicate }.to change{Route.count}.by(1)
+      }
+      it 'duplicates its stop points' do
+        expect{ route.duplicate }.to change{StopPoint.count}.by(route.stop_points.count)
+      end
+      it 'does bot duplicate the stop areas' do
+        expect{ route.duplicate }.not_to change{StopArea.count}
+      end
+    end
+
+    describe 'is idempotent, concerning' do
+      let( :first_duplicate ){ route.duplicate  }
+      let( :second_duplicate ){ first_duplicate.reload.duplicate }
+
+      it 'the required attributes' do
+        expect( values_for_create(first_duplicate, except: %w{objectid}) ).to eq( values_for_create( second_duplicate, except: %w{objectid} ) )
+      end 
+
+      it 'the stop areas' do
+        expect( first_duplicate.stop_areas.pluck(:id) ).to eq( route.stop_areas.pluck(:id) )
+        expect( second_duplicate.stop_areas.pluck(:id) ).to eq( first_duplicate.stop_areas.pluck(:id) )
+      end
+
+    end
+  end
+
+end
diff --git a/spec/models/chouette/stop_point_spec.rb b/spec/models/chouette/stop_point_spec.rb
index 5eae8caf0..329e76a75 100644
--- a/spec/models/chouette/stop_point_spec.rb
+++ b/spec/models/chouette/stop_point_spec.rb
@@ -1,6 +1,7 @@
-require 'spec_helper'
+# From Chouette import what we need ™
+StopPoint = Chouette::StopPoint
 
-describe Chouette::StopPoint, :type => :model do
+describe StopPoint, :type => :model do
   let!(:vehicle_journey) { create(:vehicle_journey)}
   subject { Chouette::Route.find( vehicle_journey.route_id).stop_points.first }
 
@@ -38,4 +39,18 @@ describe Chouette::StopPoint, :type => :model do
       expect(jpsp_stop_point_ids(@vehicle.journey_pattern_id)).not_to include(@stop_point.id)
     end
   end
+
+  describe '#duplicate' do
+    let!( :new_route ){ create :route }
+
+    it 'creates a new instance' do
+      expect{ subject.duplicate(for_route: new_route) }.to change{ StopPoint.count }.by(1)
+    end
+    it 'new instance has a new route' do
+      expect(subject.duplicate(for_route: new_route).route).to eq(new_route)
+    end
+    it 'and old stop_area' do
+      expect(subject.duplicate(for_route: new_route).stop_area).to eq(subject.stop_area)
+    end
+  end
 end
diff --git a/spec/policies/route_policy_spec.rb b/spec/policies/route_policy_spec.rb
index 243d85acb..d7edceaef 100644
--- a/spec/policies/route_policy_spec.rb
+++ b/spec/policies/route_policy_spec.rb
@@ -6,6 +6,10 @@ RSpec.describe RoutePolicy, type: :policy do
     it_behaves_like 'permitted policy and same organisation', 'routes.create', archived: true
   end
 
+  permissions :duplicate? do
+    it_behaves_like 'permitted policy and same organisation', 'routes.create', archived: true
+  end
+
   permissions :destroy? do
     it_behaves_like 'permitted policy and same organisation', 'routes.destroy', archived: true
   end
diff --git a/spec/routing/routes_routing_spec.rb b/spec/routing/routes_routing_spec.rb
new file mode 100644
index 000000000..311de9f39
--- /dev/null
+++ b/spec/routing/routes_routing_spec.rb
@@ -0,0 +1,12 @@
+RSpec.describe "routes for Routes", type: :routing do
+  context "routes /referentials/:id/lines/:id/routes/:id/duplicate" do
+
+    let( :controller ){ {controller: 'routes', referential_id: ':referential_id', line_id: ':line_id', id: ':id'}  }
+
+    it 'with method post to #post_duplicate' do
+      expect(
+        post: '/referentials/:referential_id/lines/:line_id/routes/:id/duplicate'
+      ).to route_to controller.merge(action: 'duplicate')
+    end
+  end
+end
diff --git a/spec/support/helpers/model_compare_helpers.rb b/spec/support/helpers/model_compare_helpers.rb
new file mode 100644
index 000000000..a10892af0
--- /dev/null
+++ b/spec/support/helpers/model_compare_helpers.rb
@@ -0,0 +1,15 @@
+module Support::ModelCompareHelpers
+  
+  def values_for_create obj, **overrides
+    except = overrides.delete(:except) || []
+    keys = obj.attributes.keys - except - %w{id created_at updated_at}
+    overrides.inject(obj.attributes.slice(*keys)){ |atts, (k,v)|
+      atts.merge k.to_s => v
+    }
+  end
+
+end
+
+RSpec.configure do | rspec |
+  rspec.include Support::ModelCompareHelpers, type: :model
+end