Refs #2428 #2427 : User permissions

1 files changed, 6 insertions, 7 deletions

diff --git a/spec/features/journey_pattern_spec.rb b/spec/features/journey_pattern_spec.rb
index 380241099..1efed040b 100644
--- a/spec/features/journey_pattern_spec.rb
+++ b/spec/features/journey_pattern_spec.rb
@@ -9,16 +9,17 @@ describe "JourneyPatterns", :type => :feature do
   let!(:journey_pattern) { create(:journey_pattern, :route => route) }
 
   describe 'show' do
+    before(:each) { visit referential_line_route_journey_pattern_path(referential, line, route, journey_pattern) }
+
     context 'user has permission to create journey patterns' do
       it 'shows the create link for journey pattern' do
-        visit referential_line_route_journey_pattern_path(referential, line, route, journey_pattern)
         expect(page).to have_content(I18n.t('journey_patterns.actions.new'))
       end
     end
 
     context 'user does not have permission to create journey patterns' do
       it 'does not show the create link for journey pattern' do
-        @user.update_attribute(:permissions, ['journey_patterns.edit', 'journey_patterns.destroy'])
+        @user.update_attribute(:permissions, [])
         visit referential_line_route_journey_pattern_path(referential, line, route, journey_pattern)
         expect(page).not_to have_content(I18n.t('journey_patterns.actions.new'))
       end
@@ -26,14 +27,13 @@ describe "JourneyPatterns", :type => :feature do
 
     context 'user has permission to edit journey patterns' do
       it 'shows the edit link for journey pattern' do
-        visit referential_line_route_journey_pattern_path(referential, line, route, journey_pattern)
         expect(page).to have_content(I18n.t('journey_patterns.actions.edit'))
       end
     end
 
     context 'user does not have permission to edit journey patterns' do
       it 'does not show the edit link for journey pattern' do
-        @user.update_attribute(:permissions, ['journey_patterns.create', 'journey_patterns.destroy'])
+        @user.update_attribute(:permissions, [])
         visit referential_line_route_journey_pattern_path(referential, line, route, journey_pattern)
         expect(page).not_to have_content(I18n.t('journey_patterns.actions.edit'))
       end
@@ -41,14 +41,13 @@ describe "JourneyPatterns", :type => :feature do
 
     context 'user has permission to destroy journey patterns' do
       it 'shows the destroy link for journey pattern' do
-        visit referential_line_route_journey_pattern_path(referential, line, route, journey_pattern)
         expect(page).to have_content(I18n.t('journey_patterns.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 the destroy link for journey pattern' do
-        @user.update_attribute(:permissions, ['journey_patterns.create', 'journey_patterns.edit'])
+        @user.update_attribute(:permissions, [])
         visit referential_line_route_journey_pattern_path(referential, line, route, journey_pattern)
         expect(page).not_to have_content(I18n.t('journey_patterns.actions.destroy'))
       end