Refs #2428 #2427 : User permissions

1 files changed, 13 insertions, 13 deletions

diff --git a/spec/features/routes_spec.rb b/spec/features/routes_spec.rb
index bc2088712..e82987811 100644
--- a/spec/features/routes_spec.rb
+++ b/spec/features/routes_spec.rb
@@ -9,6 +9,7 @@ describe "Routes", :type => :feature do
   let!(:route2) { create(:route, :line => line) }
   #let!(:stop_areas) { Array.new(4) { create(:stop_area) } }
   let!(:stop_points) { Array.new(4) { create(:stop_point, :route => route) } }
+  let!(:journey_pattern) { create(:journey_pattern, route: route) }
 
   describe "from lines page to a line page" do
     it "display line's routes" do
@@ -55,48 +56,49 @@ describe "Routes", :type => :feature do
   end
 
   describe 'show' do
+    before(:each) { visit referential_line_route_path(referential, line, route) }
+
     context 'user has permission to edit journey patterns' do
       it 'shows edit links for journey patterns' do
-        visit referential_line_route_path(referential, line, route)
         expect(page).to have_content(I18n.t('actions.edit'))
       end
     end
 
     context 'user does not have permission to edit journey patterns' do
       it 'does not show edit links for journey patterns' do
-        @user.update_attribute(:permissions, ['journey_patterns.create', 'journey_patterns.destroy'])
+        @user.update_attribute(:permissions, [])
         visit referential_line_route_path(referential, line, route)
-        expect(page).not_to have_content(I18n.t('actions.edit'))
+        expect(page).not_to have_link(I18n.t('actions.edit'), href: edit_referential_line_route_journey_pattern_path(referential, line, route, journey_pattern))
       end
     end
 
     context 'user has permission to destroy journey patterns' do
       it 'shows destroy links for journey patterns' do
-        visit referential_line_route_path(referential, line, route)
         expect(page).to have_content(I18n.t('actions.destroy'))
       end
     end
 
-    context 'user does not have permission to edit journey patterns' do
+    context 'user does not have permission to destroy journey patterns' do
       it 'does not show destroy links for journey patterns' do
-        @user.update_attribute(:permissions, ['journey_patterns.create', 'journey_patterns.edit'])
+        @user.update_attribute(:permissions, [])
         visit referential_line_route_path(referential, line, route)
-        expect(page).not_to have_content(I18n.t('actions.destroy'))
+        expect(page).not_to have_link(I18n.t('actions.destroy'), href: referential_line_route_journey_pattern_path(referential, line, route, journey_pattern))
       end
     end
   end
 
   describe 'referential line show' do
+    before(:each) { visit referential_line_path(referential, line) }
+
     context 'user has permission to edit routes' do
       it 'shows edit buttons for routes' do
-        visit referential_line_path(referential, line)
         expect(page).to have_css('span.fa.fa-pencil')
       end
     end
 
     context 'user does not have permission to edit routes' do
       it 'does not show edit buttons for routes' do
-        @user.update_attribute(:permissions, ['routes.create', 'routes.destroy'])
+        @user.update_attribute(:permissions, [])
         visit referential_line_path(referential, line)
         expect(page).not_to have_css('span.fa.fa-pencil')
       end
@@ -104,14 +106,13 @@ describe "Routes", :type => :feature do
 
     context 'user has permission to create routes' do
       it 'shows link to a create route page' do
-        visit referential_line_path(referential, line)
         expect(page).to have_content(I18n.t('routes.actions.new'))
       end
     end
 
     context 'user does not have permission to create routes' do
       it 'does not show link to a create route page' do
-        @user.update_attribute(:permissions, ['routes.edit', 'routes.destroy'])
+        @user.update_attribute(:permissions, [])
         visit referential_line_path(referential, line)
         expect(page).not_to have_content(I18n.t('routes.actions.new'))
       end
@@ -119,14 +120,13 @@ describe "Routes", :type => :feature do
 
     context 'user has permission to destroy routes' do
       it 'shows destroy buttons for routes' do
-        visit referential_line_path(referential, line)
         expect(page).to have_css('span.fa.fa-trash-o')
       end
     end
 
     context 'user does not have permission to destroy routes' do
       it 'does not show destroy buttons for routes' do
-        @user.update_attribute(:permissions, ['routes.edit', 'routes.create'])
+        @user.update_attribute(:permissions, [])
         visit referential_line_path(referential, line)
         expect(page).not_to have_css('span.fa.fa-trash-o')
       end