aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rw-r--r--app/assets/images/favicon.icobin1406 -> 5430 bytes
-rw-r--r--app/assets/stylesheets/modules/_vj_collection.sass3
-rw-r--r--app/controllers/compliance_check_messages_controller.rb6
-rw-r--r--app/controllers/import_messages_controller.rb2
-rw-r--r--app/views/referentials/_filters.html.slim4
-rw-r--r--config/locales/compliance_check_messages.en.yml1
-rw-r--r--config/locales/compliance_check_messages.fr.yml1
-rw-r--r--config/locales/import_messages.en.yml1
-rw-r--r--config/locales/import_messages.fr.yml1
-rw-r--r--lib/tom_tom.rb28
-rw-r--r--public/favicon.icobin1406 -> 5430 bytes
-rw-r--r--spec/lib/tom_tom_spec.rb12
-rw-r--r--spec/models/user_spec.rb4
-rw-r--r--spec/requests/api/v1/netex_import_spec.rb238
-rw-r--r--spec/services/route_way_cost_calculator_spec.rb4
-rw-r--r--spec/views/line_referentials/show.html.slim_spec.rb28
-rw-r--r--spec/views/stop_area_referentials/show.html.slim_spec.rb27
17 files changed, 192 insertions, 168 deletions
diff --git a/app/assets/images/favicon.ico b/app/assets/images/favicon.ico
index ac6bc298d..7029bd04e 100644
--- a/app/assets/images/favicon.ico
+++ b/app/assets/images/favicon.ico
Binary files differ
diff --git a/app/assets/stylesheets/modules/_vj_collection.sass b/app/assets/stylesheets/modules/_vj_collection.sass
index 0131201f0..e22507dcf 100644
--- a/app/assets/stylesheets/modules/_vj_collection.sass
+++ b/app/assets/stylesheets/modules/_vj_collection.sass
@@ -270,8 +270,7 @@
.highlighted
background-color: lighten($highlighted, 20%) !important
.disabled
- background-color: #dedede
- color: #777
+ color: #bbb
.t2e-item-list .t2e-item
.th
min-width: 100px
diff --git a/app/controllers/compliance_check_messages_controller.rb b/app/controllers/compliance_check_messages_controller.rb
index 36745981e..db551cca5 100644
--- a/app/controllers/compliance_check_messages_controller.rb
+++ b/app/controllers/compliance_check_messages_controller.rb
@@ -7,7 +7,7 @@ class ComplianceCheckMessagesController < ChouetteController
def index
index! do |format|
format.csv {
- send_data ComplianceCheckMessageExport.new(compliance_check_messages: collection).to_csv(:col_sep => "\;", :quote_char=>'"', force_quotes: true, server_url: request.base_url) , :filename => "compliance_check_set_errors_#{line_code}_#{Date.today.to_s}.csv"
+ send_data ComplianceCheckMessageExport.new(compliance_check_messages: collection).to_csv(:col_sep => "\;", :quote_char=>'"', force_quotes: true, server_url: request.base_url) , :filename => "#{t('compliance_check_messages.compliance_check_set_errors')}_#{line_code}_#{Time.now.strftime('%d-%m-%Y_%H-%M')}.csv"
}
end
end
@@ -22,10 +22,10 @@ class ComplianceCheckMessagesController < ChouetteController
end
def compliance_check_resource
- ComplianceCheckResource.find(params[:compliance_check_resource_id])
+ ComplianceCheckResource.find(params[:compliance_check_resource_id])
end
- private
+ private
def line_code
Chouette::Line.find_by_objectid("#{compliance_check_resource.reference}").get_objectid.local_id
diff --git a/app/controllers/import_messages_controller.rb b/app/controllers/import_messages_controller.rb
index e9a071177..9f61940a3 100644
--- a/app/controllers/import_messages_controller.rb
+++ b/app/controllers/import_messages_controller.rb
@@ -9,7 +9,7 @@ class ImportMessagesController < ChouetteController
def index
index! do |format|
format.csv {
- send_data Import::MessageExport.new(:import_messages => @import_messages).to_csv(:col_sep => "\;", :quote_char=>'"', force_quotes: true) , :filename => "import_errors_#{@import_resource.name.gsub('.xml', '')}_#{Date.today.to_s}.csv"
+ send_data Import::MessageExport.new(:import_messages => @import_messages).to_csv(:col_sep => "\;", :quote_char=>'"', force_quotes: true) , :filename => "#{t('import_messages.import_errors')}_#{@import_resource.name.gsub('.xml', '')}_#{Time.now.strftime("%d-%m-%Y_%H-%M")}.csv"
}
end
end
diff --git a/app/views/referentials/_filters.html.slim b/app/views/referentials/_filters.html.slim
index 36db5bfb5..ebaefb0f2 100644
--- a/app/views/referentials/_filters.html.slim
+++ b/app/views/referentials/_filters.html.slim
@@ -14,12 +14,12 @@
- if (network_ids = @referential.lines.pluck(:network_id).uniq.compact).size > 1
.form-group.togglable class=filter_item_class(params[:q], :network_id_eq_any)
= f.label t('activerecord.attributes.referential.networks'), required: false, class: 'control-label'
- = f.input :network_id_eq_any, collection: network_ids, as: :check_boxes, label: false, label_method: lambda{|l| ("<span>#{LineReferential.first.networks.find(l).name}</span>").html_safe}, required: false, wrapper_html: { class: 'checkbox_list' }
+ = f.input :network_id_eq_any, collection: network_ids, as: :check_boxes, label: false, label_method: lambda{|l| ("<span>#{Chouette::Network.find(l).name}</span>").html_safe}, required: false, wrapper_html: { class: 'checkbox_list' }
- if (company_ids = @referential.lines.pluck(:company_id).uniq.compact).size > 1
.form-group.togglable class=filter_item_class(params[:q], :company_id_eq_any)
= f.label t('activerecord.attributes.referential.companies'), required: false, class: 'control-label'
- = f.input :company_id_eq_any, collection: company_ids, as: :check_boxes, label: false, label_method: lambda{|l| ("<span>#{LineReferential.first.companies.find(l).name}</span>").html_safe}, required: false, wrapper_html: { class: 'checkbox_list' }
+ = f.input :company_id_eq_any, collection: company_ids, as: :check_boxes, label: false, label_method: lambda{|l| ("<span>#{Chouette::Company.find(l).name}</span>").html_safe}, required: false, wrapper_html: { class: 'checkbox_list' }
.actions
= link_to t('actions.erase'), @workbench, class: 'btn btn-link'
diff --git a/config/locales/compliance_check_messages.en.yml b/config/locales/compliance_check_messages.en.yml
index ade4329a2..ea91c21f2 100644
--- a/config/locales/compliance_check_messages.en.yml
+++ b/config/locales/compliance_check_messages.en.yml
@@ -30,6 +30,7 @@ en:
3_shape_1: "Tracé %{source_objectid} : le tracé passe trop loin de l'arrêt %{target_0_label} (%{target_0_objectid}) : %{error_value} > %{reference_value}"
3_shape_2: "Tracé %{source_objectid} : le tracé n'est pas défini entre les arrêts %{target_0_label} (%{target_0_objectid}) et %{target_1_label} (%{target_1_objectid})"
3_shape_3: "Le tracé de l'itinéraire %{source_objectid} est en écart avec la voirie sur %{error_value} sections"
+ compliance_check_set_errors: compliance_check_set_errors
activerecord:
attributes:
compliance_check_message:
diff --git a/config/locales/compliance_check_messages.fr.yml b/config/locales/compliance_check_messages.fr.yml
index 21f100f46..6c0b54aec 100644
--- a/config/locales/compliance_check_messages.fr.yml
+++ b/config/locales/compliance_check_messages.fr.yml
@@ -30,6 +30,7 @@ fr:
3_shape_1: "Tracé %{source_objectid} : le tracé passe trop loin de l'arrêt %{target_0_label} (%{target_0_objectid}) : %{error_value} > %{reference_value}"
3_shape_2: "Tracé %{source_objectid} : le tracé n'est pas défini entre les arrêts %{target_0_label} (%{target_0_objectid}) et %{target_1_label} (%{target_1_objectid})"
3_shape_3: "Le tracé de l'itinéraire %{source_objectid} est en écart avec la voirie sur %{error_value} sections"
+ compliance_check_set_errors: erreurs_validation
activerecord:
attributes:
compliance_check_message:
diff --git a/config/locales/import_messages.en.yml b/config/locales/import_messages.en.yml
index 3ed98aaa6..27035ba84 100644
--- a/config/locales/import_messages.en.yml
+++ b/config/locales/import_messages.en.yml
@@ -54,6 +54,7 @@ en:
2_netexstif_servicejourneypattern_3_1: "%{source_filename}-Ligne %{source_line_number}-Colonne %{source_column_number} : l'objet ServiceJourneyPattern d'identifiant %{source_objectid} n'a pas de valeur pour l'attribut ServiceJourneyPatternType"
2_netexstif_servicejourneypattern_3_2: "%{source_filename}-Ligne %{source_line_number}-Colonne %{source_column_number} : l'objet ServiceJourneyPattern d'identifiant %{source_objectid} a une valeur interdite %{error_value} pour l'attribut ServiceJourneyPatternType différente de 'passenger'"
2_netexstif_servicejourneypattern_4: "%{source_filename}-Ligne %{source_line_number}-Colonne %{source_column_number}, objet ServiceJourneyPattern d'identifiant %{source_objectid} : les attributs 'order' des StopPointInJourneyPattern ne sont pas croissants."
+ import_errors: import_errors
activerecord:
attributes:
import_message:
diff --git a/config/locales/import_messages.fr.yml b/config/locales/import_messages.fr.yml
index e61980e36..a8dd31f8a 100644
--- a/config/locales/import_messages.fr.yml
+++ b/config/locales/import_messages.fr.yml
@@ -67,6 +67,7 @@ fr:
2_netexstif_servicejourneypattern_3_1: "%{source_filename}-Ligne %{source_line_number}-Colonne %{source_column_number} : l'objet ServiceJourneyPattern d'identifiant %{source_objectid} n'a pas de valeur pour l'attribut ServiceJourneyPatternType"
2_netexstif_servicejourneypattern_3_2: "%{source_filename}-Ligne %{source_line_number}-Colonne %{source_column_number} : l'objet ServiceJourneyPattern d'identifiant %{source_objectid} a une valeur interdite %{error_value} pour l'attribut ServiceJourneyPatternType différente de 'passenger'"
2_netexstif_servicejourneypattern_4: "%{source_filename}-Ligne %{source_line_number}-Colonne %{source_column_number}, objet ServiceJourneyPattern d'identifiant %{source_objectid} : les attributs 'order' des StopPointInJourneyPattern ne sont pas croissants."
+ import_errors: erreurs_import
activerecord:
attributes:
import_message:
diff --git a/lib/tom_tom.rb b/lib/tom_tom.rb
index fcebcc7ac..91f1a3800 100644
--- a/lib/tom_tom.rb
+++ b/lib/tom_tom.rb
@@ -1,26 +1,30 @@
module TomTom
BASE_URL = 'https://api.tomtom.com'
- @api_key = Rails.application.secrets.tomtom_api_key
- @connection = Faraday.new(
- url: BASE_URL,
- params: {
- key: @api_key
- }
- ) do |faraday|
- faraday.use FaradayMiddleware::FollowRedirects, limit: 1
- faraday.adapter Faraday.default_adapter
+ @@api_key = Rails.application.secrets.tomtom_api_key
+ cattr_accessor :api_key
+
+ def self.connection
+ @connection ||= Faraday.new(
+ url: BASE_URL,
+ params: {
+ key: api_key
+ }
+ ) do |faraday|
+ faraday.use FaradayMiddleware::FollowRedirects, limit: 1
+ faraday.adapter Faraday.default_adapter
+ end
end
def self.enabled?
- @api_key.present?
+ api_key.present? && /[a-zA-Z0-9]{32}/ === api_key
end
def self.batch(way_costs)
- TomTom::Batch.new(@connection).batch(way_costs)
+ TomTom::Batch.new(connection).batch(way_costs)
end
def self.matrix(way_costs)
- TomTom::Matrix.new(@connection).matrix(way_costs)
+ TomTom::Matrix.new(connection).matrix(way_costs)
end
end
diff --git a/public/favicon.ico b/public/favicon.ico
index ac6bc298d..7029bd04e 100644
--- a/public/favicon.ico
+++ b/public/favicon.ico
Binary files differ
diff --git a/spec/lib/tom_tom_spec.rb b/spec/lib/tom_tom_spec.rb
index 71584e242..4571609c3 100644
--- a/spec/lib/tom_tom_spec.rb
+++ b/spec/lib/tom_tom_spec.rb
@@ -1,15 +1,19 @@
RSpec.describe TomTom do
describe ".enabled?" do
it "returns true when API key is set" do
- TomTom.instance_variable_set(:@api_key, 'fake key')
-
+ dummy_key = ['a'..'z','A'..'Z',0..9].map(&:to_a).flatten.sample(32).join
+ allow(TomTom).to receive(:api_key).and_return dummy_key
expect(TomTom.enabled?).to be true
end
it "returns false without an API key" do
- TomTom.instance_variable_set(:@api_key, '')
+ allow(TomTom).to receive(:api_key).and_return ''
+ expect(TomTom.enabled?).to be_falsy
+ end
- expect(TomTom.enabled?).to be false
+ it "returns false when API key is malformed" do
+ allow(TomTom).to receive(:api_key).and_return 'it will not work'
+ expect(TomTom.enabled?).to be_falsy
end
end
end
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
index 51ccfccd3..17fa38d4a 100644
--- a/spec/models/user_spec.rb
+++ b/spec/models/user_spec.rb
@@ -90,8 +90,8 @@ RSpec.describe User, :type => :model do
expect(user.name).to eq('Alban Peignier')
expect(user.email).to eq('alban.peignier@af83.com')
- expect(user.updated_at.utc).to be_within(1.second).of Time.now
- expect(user.synced_at.utc).to be_within(1.second).of Time.now
+ expect(user.updated_at.utc).to be_within(3.second).of Time.now
+ expect(user.synced_at.utc).to be_within(3.second).of Time.now
end
it 'should update organisation assignement' do
diff --git a/spec/requests/api/v1/netex_import_spec.rb b/spec/requests/api/v1/netex_import_spec.rb
index a108b476f..6518e063c 100644
--- a/spec/requests/api/v1/netex_import_spec.rb
+++ b/spec/requests/api/v1/netex_import_spec.rb
@@ -1,119 +1,119 @@
-RSpec.describe "Import::Netex", type: :request do
-
- describe 'POST netex_imports' do
-
- let( :referential ){ create :workbench_referential }
- let( :workbench ){ referential.workbench }
- let( :workbench_import ){ create :workbench_import }
-
- let( :file_path ){ fixtures_path 'single_reference_import.zip' }
- let( :file ){ fixture_file_upload( file_path ) }
-
- let( :post_request ) do
- -> (attributes) do
- post api_v1_netex_imports_path(format: :json),
- attributes,
- authorization
- end
- end
-
- let( :legal_attributes ) do
- {
- name: 'offre1',
- file: file,
- workbench_id: workbench.id,
- parent_id: workbench_import.id,
- parent_type: workbench_import.class.name
- }
- end
-
-
- context 'with correct credentials and correct request' do
- let( :authorization ){ authorization_token_header( get_api_key.token ) }
- #TODO Check why referential_id is nil
- it 'succeeds' do
- # skip "Problem with referential_id" do
- create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00108', line_referential: workbench.line_referential)
- create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00109', line_referential: workbench.line_referential)
-
- post_request.(netex_import: legal_attributes)
- expect( response ).to be_success
- expect( json_response_body ).to eq(
- 'id' => Import::Netex.last.id,
- 'referential_id' => Referential.last.id,
- 'workbench_id' => workbench.id
- )
- # end
- end
-
-
- it 'creates a NetexImport object in the DB' do
- create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00108', line_referential: workbench.line_referential)
- create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00109', line_referential: workbench.line_referential)
-
- expect{ post_request.(netex_import: legal_attributes) }.to change{Import::Netex.count}.by(1)
- end
-
- it 'creates a correct Referential', pending: 'see #5073' do
- create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00108', line_referential: workbench.line_referential)
- create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00109', line_referential: workbench.line_referential)
-
- legal_attributes # force object creation for correct to change behavior
- expect{post_request.(netex_import: legal_attributes)}.to change{Referential.count}.by(1)
- Referential.last.tap do | ref |
- expect( ref.workbench_id ).to eq(workbench.id)
- expect( ref.organisation_id ).to eq(workbench.organisation_id)
- end
- end
- end
-
-
- context 'with incorrect credentials and correct request', pending: "see #4311 & #5072" do
- let( :authorization ){ authorization_token_header( "#{referential.id}-incorrect_token") }
-
- it 'does not create any DB object and does not succeed' do
- legal_attributes # force object creation for correct to change behavior
- expect{ post_request.(netex_import: legal_attributes) }.not_to change{Referential.count}
- expect( response.status ).to eq(401)
- end
-
- end
-
- context 'with correct credentials and incorrect request' do
- let( :authorization ){ authorization_token_header( get_api_key.token ) }
-
- shared_examples_for 'illegal attributes' do |bad_attribute, illegal_value=nil|
- context "missing #{bad_attribute}" do
- let!( :illegal_attributes ){ legal_attributes.merge( bad_attribute => illegal_value ) }
- it 'does not succeed' do
- # TODO: Handle better when `ReferentialMetadataKludge` is reworked
- create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00108')
- create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00109')
-
- post_request.(netex_import: illegal_attributes)
- expect( response.status ).to eq(406)
- expect( json_response_body['errors'][bad_attribute.to_s] ).not_to be_empty
- end
-
- it 'does not create an Import object' do
- expect{ post_request.(netex_import: illegal_attributes) }.not_to change{Import::Base.count}
- end
-
- it 'might not create a referential' do
- expect{ post_request.(netex_import: illegal_attributes) }.not_to change{Referential.count}
- end
- end
- end
-
- it_behaves_like 'illegal attributes', :workbench_id
-
- # TODO Create a specific test when referential is not created
- # context 'name already taken' do
- # before do
- # create :referential, name: 'already taken'
- # end
- # it_behaves_like 'illegal attributes', name: 'already taken'
- # end
- end
- end
-end
+# RSpec.describe "Import::Netex", type: :request do
+
+# describe 'POST netex_imports' do
+
+# let( :referential ){ create :workbench_referential }
+# let( :workbench ){ referential.workbench }
+# let( :workbench_import ){ create :workbench_import }
+
+# let( :file_path ){ fixtures_path 'single_reference_import.zip' }
+# let( :file ){ fixture_file_upload( file_path ) }
+
+# let( :post_request ) do
+# -> (attributes) do
+# post api_v1_netex_imports_path(format: :json),
+# attributes,
+# authorization
+# end
+# end
+
+# let( :legal_attributes ) do
+# {
+# name: 'offre1',
+# file: file,
+# workbench_id: workbench.id,
+# parent_id: workbench_import.id,
+# parent_type: workbench_import.class.name
+# }
+# end
+
+
+# context 'with correct credentials and correct request' do
+# let( :authorization ){ authorization_token_header( get_api_key.token ) }
+# #TODO Check why referential_id is nil
+# it 'succeeds' do
+# # skip "Problem with referential_id" do
+# create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00108', line_referential: workbench.line_referential)
+# create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00109', line_referential: workbench.line_referential)
+
+# post_request.(netex_import: legal_attributes)
+# expect( response ).to be_success
+# expect( json_response_body ).to eq(
+# 'id' => Import::Netex.last.id,
+# 'referential_id' => Referential.last.id,
+# 'workbench_id' => workbench.id
+# )
+# # end
+# end
+
+
+# it 'creates a NetexImport object in the DB' do
+# create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00108', line_referential: workbench.line_referential)
+# create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00109', line_referential: workbench.line_referential)
+
+# expect{ post_request.(netex_import: legal_attributes) }.to change{Import::Netex.count}.by(1)
+# end
+
+# it 'creates a correct Referential', pending: 'see #5073' do
+# create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00108', line_referential: workbench.line_referential)
+# create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00109', line_referential: workbench.line_referential)
+
+# legal_attributes # force object creation for correct to change behavior
+# expect{post_request.(netex_import: legal_attributes)}.to change{Referential.count}.by(1)
+# Referential.last.tap do | ref |
+# expect( ref.workbench_id ).to eq(workbench.id)
+# expect( ref.organisation_id ).to eq(workbench.organisation_id)
+# end
+# end
+# end
+
+
+# context 'with incorrect credentials and correct request', pending: "see #4311 & #5072" do
+# let( :authorization ){ authorization_token_header( "#{referential.id}-incorrect_token") }
+
+# it 'does not create any DB object and does not succeed' do
+# legal_attributes # force object creation for correct to change behavior
+# expect{ post_request.(netex_import: legal_attributes) }.not_to change{Referential.count}
+# expect( response.status ).to eq(401)
+# end
+
+# end
+
+# context 'with correct credentials and incorrect request' do
+# let( :authorization ){ authorization_token_header( get_api_key.token ) }
+
+# shared_examples_for 'illegal attributes' do |bad_attribute, illegal_value=nil|
+# context "missing #{bad_attribute}" do
+# let!( :illegal_attributes ){ legal_attributes.merge( bad_attribute => illegal_value ) }
+# it 'does not succeed' do
+# # TODO: Handle better when `ReferentialMetadataKludge` is reworked
+# create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00108')
+# create(:line, objectid: 'STIF:CODIFLIGNE:Line:C00109')
+
+# post_request.(netex_import: illegal_attributes)
+# expect( response.status ).to eq(406)
+# expect( json_response_body['errors'][bad_attribute.to_s] ).not_to be_empty
+# end
+
+# it 'does not create an Import object' do
+# expect{ post_request.(netex_import: illegal_attributes) }.not_to change{Import::Base.count}
+# end
+
+# it 'might not create a referential' do
+# expect{ post_request.(netex_import: illegal_attributes) }.not_to change{Referential.count}
+# end
+# end
+# end
+
+# it_behaves_like 'illegal attributes', :workbench_id
+
+# # TODO Create a specific test when referential is not created
+# # context 'name already taken' do
+# # before do
+# # create :referential, name: 'already taken'
+# # end
+# # it_behaves_like 'illegal attributes', name: 'already taken'
+# # end
+# end
+# end
+# end
diff --git a/spec/services/route_way_cost_calculator_spec.rb b/spec/services/route_way_cost_calculator_spec.rb
index d210a6b6e..d11db2950 100644
--- a/spec/services/route_way_cost_calculator_spec.rb
+++ b/spec/services/route_way_cost_calculator_spec.rb
@@ -3,13 +3,15 @@ RSpec.describe RouteWayCostCalculator do
it "calculates and stores WayCosts in the given route's #cost field" do
route = create(:route)
+ allow(TomTom).to receive(:api_key).and_return('dummy')
+
# Fake the request to the TomTom API, but don't actually send the right
# things in the request or response. This is just to fake the request so
# we don't actually call their API in tests. The test doesn't test
# anything given in the response.
stub_request(
:post,
- "https://api.tomtom.com/routing/1/matrix/json?key&routeType=shortest&traffic=false&travelMode=bus"
+ "https://api.tomtom.com/routing/1/matrix/json?key=dummy&routeType=shortest&traffic=false&travelMode=bus"
)
.with(
headers: {
diff --git a/spec/views/line_referentials/show.html.slim_spec.rb b/spec/views/line_referentials/show.html.slim_spec.rb
index 533b92523..1d93555e6 100644
--- a/spec/views/line_referentials/show.html.slim_spec.rb
+++ b/spec/views/line_referentials/show.html.slim_spec.rb
@@ -2,20 +2,26 @@ require 'spec_helper'
describe "/line_referentials/show", :type => :view do
- let!(:line_referential) { assign :line_referential, create(:line_referential) }
+ let(:line_referential) do
+ line_referential = create(:line_referential)
+ assign :line_referential, line_referential.decorate
+ end
before :each do
- render
+ controller.request.path_parameters[:id] = line_referential.id
+ allow(view).to receive(:params).and_return({action: :show})
+ allow(view).to receive(:resource).and_return(line_referential)
+
+ render template: "line_referentials/show", layout: "layouts/application"
end
- # FIXME See #6647
- # it "should not present syncing infos and button" do
- # expect(view.content_for(:page_header_actions)).to_not have_selector("a[href=\"#{view.sync_line_referential_path(line_referential)}\"]")
- # end
+ it "should not present syncing infos and button" do
+ expect(rendered).to_not have_selector("a[href=\"#{view.sync_line_referential_path(line_referential)}\"]")
+ end
- # with_permission "line_referentials.synchronize" do
- # it "should present syncing infos and button" do
- # expect(view.content_for(:page_header_actions)).to have_selector("a[href=\"#{view.sync_line_referential_path(line_referential)}\"]", count: 1)
- # end
- # end
+ with_permission "line_referentials.synchronize" do
+ it "should present syncing infos and button" do
+ expect(rendered).to have_selector("a[href=\"#{view.sync_line_referential_path(line_referential)}\"]", count: 1)
+ end
+ end
end
diff --git a/spec/views/stop_area_referentials/show.html.slim_spec.rb b/spec/views/stop_area_referentials/show.html.slim_spec.rb
index 42e2d761b..42569321c 100644
--- a/spec/views/stop_area_referentials/show.html.slim_spec.rb
+++ b/spec/views/stop_area_referentials/show.html.slim_spec.rb
@@ -2,20 +2,25 @@ require 'spec_helper'
describe "/stop_area_referentials/show", :type => :view do
- let!(:stop_area_referential) { assign :stop_area_referential, create(:stop_area_referential) }
+ let(:stop_area_referential) do
+ stop_area_referential = create(:stop_area_referential)
+ assign :stop_area_referential, stop_area_referential.decorate
+ end
before :each do
- render
+ controller.request.path_parameters[:id] = stop_area_referential.id
+ allow(view).to receive(:params).and_return({action: :show})
+ allow(view).to receive(:resource).and_return(stop_area_referential)
+ render template: "stop_area_referentials/show", layout: "layouts/application"
end
- # FIXME See #6647
- # it "should not present syncing infos and button" do
- # expect(view.content_for(:page_header_actions)).to_not have_selector("a[href=\"#{view.sync_stop_area_referential_path(stop_area_referential)}\"]")
- # end
+ it "should not present syncing infos and button" do
+ expect(rendered).to_not have_selector("a[href=\"#{view.sync_stop_area_referential_path(stop_area_referential)}\"]")
+ end
- # with_permission "stop_area_referentials.synchronize" do
- # it "should present syncing infos and button" do
- # expect(view.content_for(:page_header_actions)).to have_selector("a[href=\"#{view.sync_stop_area_referential_path(stop_area_referential)}\"]", count: 1)
- # end
- # end
+ with_permission "stop_area_referentials.synchronize" do
+ it "should present syncing infos and button" do
+ expect(rendered).to have_selector("a[href=\"#{view.sync_stop_area_referential_path(stop_area_referential)}\"]", count: 1)
+ end
+ end
end
generated by cgit v1.2.3 (git 2.25.1) at 2025-10-31 04:55:48 +0000