From da60766c207ac372e2ebd1d42be7c7b9c3ee4957 Mon Sep 17 00:00:00 2001 From: Alban Peignier Date: Thu, 28 Dec 2017 08:50:56 +0100 Subject: Use current_organisation as begin_of_association_chain in Lines/Companies/Networks/GroupOfLinesController. Refs #5102 --- spec/controllers/lines_controller_spec.rb | 4 ++-- spec/factories/line_referentials.rb | 9 +++++++++ 2 files changed, 11 insertions(+), 2 deletions(-) (limited to 'spec') diff --git a/spec/controllers/lines_controller_spec.rb b/spec/controllers/lines_controller_spec.rb index ce5adbbdd..65fe88b96 100644 --- a/spec/controllers/lines_controller_spec.rb +++ b/spec/controllers/lines_controller_spec.rb @@ -1,14 +1,14 @@ RSpec.describe LinesController, :type => :controller do login_user - let(:line_referential) { create :line_referential } + let(:line_referential) { create :line_referential, member: @user.organisation } let(:line) { create :line, line_referential: line_referential } describe 'PUT deactivate' do let(:request){ put :deactivate, id: line.id, line_referential_id: line_referential.id } it 'should redirect to 403' do - expect(request).to redirect_to "/403" + expect(request).to redirect_to "/403" end with_permission "lines.change_status" do diff --git a/spec/factories/line_referentials.rb b/spec/factories/line_referentials.rb index e9e6dce5a..8c2aad646 100644 --- a/spec/factories/line_referentials.rb +++ b/spec/factories/line_referentials.rb @@ -2,5 +2,14 @@ FactoryGirl.define do factory :line_referential do sequence(:name) { |n| "Line Referential #{n}" } objectid_format 'stif_codifligne' + + transient do + member nil + end + + after(:create) do |line_referential, evaluator| + line_referential.add_member evaluator.member if evaluator.member + line_referential.save + end end end -- cgit v1.2.3 From 25e039b3de2b45526b233c3ef4d95bb4c9f40a18 Mon Sep 17 00:00:00 2001 From: Alban Peignier Date: Thu, 28 Dec 2017 09:27:16 +0100 Subject: Associate current organisation with tested LineReferential in features specs. Refs #5102 --- spec/features/companies_spec.rb | 2 +- spec/features/group_of_lines_permissions_spec.rb | 4 ++-- spec/features/group_of_lines_spec.rb | 2 +- spec/features/lines_spec.rb | 3 ++- spec/features/networks_spec.rb | 2 +- 5 files changed, 7 insertions(+), 6 deletions(-) (limited to 'spec') diff --git a/spec/features/companies_spec.rb b/spec/features/companies_spec.rb index 1b9dae56f..4e778b3a0 100644 --- a/spec/features/companies_spec.rb +++ b/spec/features/companies_spec.rb @@ -4,7 +4,7 @@ require 'spec_helper' describe "Companies", :type => :feature do login_user - let(:line_referential) { create :line_referential } + let(:line_referential) { create :line_referential, member: @user.organisation } let!(:companies) { Array.new(2) { create :company, line_referential: line_referential } } subject { companies.first } diff --git a/spec/features/group_of_lines_permissions_spec.rb b/spec/features/group_of_lines_permissions_spec.rb index 5c03481ec..33d78c0dd 100644 --- a/spec/features/group_of_lines_permissions_spec.rb +++ b/spec/features/group_of_lines_permissions_spec.rb @@ -22,7 +22,7 @@ describe "Group of lines", :type => :feature do context 'on show view' do let( :path ){ line_referential_group_of_line_path(line_referential, group_of_line, delete: true) } - context 'if permissions present → ' do + context 'if permissions present → ' do let( :permission ){ true } it 'shows the appropriate buttons' do @@ -30,7 +30,7 @@ describe "Group of lines", :type => :feature do expect( page ).to have_link('Supprimer', href: expected_url) end end - context 'if permissions absent → ' do + context 'if permissions absent → ' do let( :permission ){ false } it 'shows the appropriate buttons' do diff --git a/spec/features/group_of_lines_spec.rb b/spec/features/group_of_lines_spec.rb index 59101ccd5..8b88e6e9e 100644 --- a/spec/features/group_of_lines_spec.rb +++ b/spec/features/group_of_lines_spec.rb @@ -10,7 +10,7 @@ describe "Group of lines", :type => :feature do let!(:group_of_lines) { Array.new(2) { create(:group_of_line, line_referential: line_referential) } } subject { group_of_lines.first } - let(:line_referential) { create :line_referential } + let(:line_referential) { create :line_referential, member: @user.organisation } before :each do subject.lines << line diff --git a/spec/features/lines_spec.rb b/spec/features/lines_spec.rb index 2a442bd2f..f142b7da9 100644 --- a/spec/features/lines_spec.rb +++ b/spec/features/lines_spec.rb @@ -1,7 +1,8 @@ +# coding: utf-8 describe "Lines", type: :feature do login_user - let(:line_referential) { create :line_referential } + let(:line_referential) { create :line_referential, member: @user.organisation } let!(:network) { create(:network) } let!(:company) { create(:company) } let!(:lines) { Array.new(2) { create :line_with_stop_areas, network: network, company: company, line_referential: line_referential } } diff --git a/spec/features/networks_spec.rb b/spec/features/networks_spec.rb index 75070e7fa..1a822e7c9 100644 --- a/spec/features/networks_spec.rb +++ b/spec/features/networks_spec.rb @@ -4,7 +4,7 @@ require 'spec_helper' describe "Networks", :type => :feature do login_user - let(:line_referential) { create :line_referential } + let(:line_referential) { create :line_referential, member: @user.organisation } let!(:networks) { Array.new(2) { create(:network, line_referential: line_referential) } } subject { networks.first } -- cgit v1.2.3