2 files changed, 44 insertions, 2 deletions
diff --git a/spec/controllers/exports_controller_spec.rb b/spec/controllers/exports_controller_spec.rb
index 9d8dde4ff..e2b89fc26 100644
--- a/spec/controllers/exports_controller_spec.rb
+++ b/spec/controllers/exports_controller_spec.rb
@@ -85,7 +85,7 @@ RSpec.describe ExportsController, :type => :controller do
     context "with the token" do
       it 'should be successful' do
         post :upload, workbench_id: workbench.id, id: export.id, token: export.token_upload
-        expect(response).to be_redirect
+        expect(response).to be_success
       end
     end
 
diff --git a/spec/controllers/routes_controller_spec.rb b/spec/controllers/routes_controller_spec.rb
index e4dc6bc23..b7cb66b46 100644
--- a/spec/controllers/routes_controller_spec.rb
+++ b/spec/controllers/routes_controller_spec.rb
@@ -42,11 +42,14 @@ RSpec.describe RoutesController, type: :controller do
     before(:each) do
       post :create, line_id: route.line_id,
           referential_id: referential.id,
-          route: { name: "changed"}
+          route: { name: "changed", published_name: "published_name"}
 
     end
     it_behaves_like "line and referential linked"
     it_behaves_like "redirected to referential_line_path(referential,line)"
+    it "sets metadata" do
+      expect(Chouette::Route.last.metadata.creator_username).to eq @user.username
+    end
   end
 
   describe "PUT /update" do
@@ -58,6 +61,9 @@ RSpec.describe RoutesController, type: :controller do
 
     it_behaves_like "route, line and referential linked"
     it_behaves_like "redirected to referential_line_path(referential,line)"
+    it "sets metadata" do
+      expect(Chouette::Route.last.metadata.modifier_username).to eq @user.username
+    end
   end
 
   describe "GET /show" do
@@ -83,6 +89,42 @@ RSpec.describe RoutesController, type: :controller do
 
       expect(Chouette::Route.last.name).to eq(I18n.t('activerecord.copy', name: route.name))
       expect(Chouette::Route.last.published_name).to eq(route.published_name)
+      expect(Chouette::Route.last.stop_area_ids).to eq route.stop_area_ids
+    end
+
+    context "when opposite = true" do
+      it "creates a new route on the opposite way " do
+        expect do
+          post :duplicate,
+            referential_id: route.line.line_referential_id,
+            line_id: route.line_id,
+            id: route.id,
+            opposite: TRUE
+        end.to change { Chouette::Route.count }.by(1)
+
+        expect(Chouette::Route.last.name).to eq(I18n.t('routes.opposite', name: route.name))
+        expect(Chouette::Route.last.published_name).to eq(Chouette::Route.last.name)
+        expect(Chouette::Route.last.opposite_route).to eq(route)
+        expect(Chouette::Route.last.stop_area_ids).to eq route.stop_area_ids.reverse
+      end
+    end
+
+    context "on a duplicated route" do
+      let!(:duplicated){ route.duplicate }
+      it "creates a new route on the opposite way " do
+        expect do
+          post :duplicate,
+            referential_id: duplicated.line.line_referential_id,
+            line_id: duplicated.line_id,
+            id: duplicated.id,
+            opposite: TRUE
+        end.to change { Chouette::Route.count }.by(1)
+
+        expect(Chouette::Route.last.name).to eq(I18n.t('routes.opposite', name: duplicated.name))
+        expect(Chouette::Route.last.published_name).to eq(Chouette::Route.last.name)
+        expect(Chouette::Route.last.opposite_route).to eq(duplicated)
+        expect(Chouette::Route.last.stop_area_ids).to eq duplicated.stop_area_ids.reverse
+      end
     end
   end
 end